Что вы скажите о стиле кода:

Возврат Запрос.Выполнить().Выгрузить()[0].СуммаДолга;

я так конечно не пишу никогда но в общем то

(0) Cамый оптимальный вариант, на мой взгляд

Выборка = Запрос.Выполнить().Выгрузить();
Если Выборка.Количество() > 0 Тогда
  Возврат Выборка[0].СуммаДолга;
Иначе
  Возврат 0;
КонецЕсли;

зачем вы все выборку делаете до того, как проверите есть там что или нет?

Возврат Запрос.Выполнить().Выгрузить().Итог("СуммаДолга")

уже было?

Иногда сам так делаю, но считаю, что надо проверять, чтобы процедура не вывалилась по ошибке.

Если не учитывать (1), то нормально, но, сказать честно, лучше бы удостоверится, что при отсутствии строк в запросе програ отработает

Для небольшого одноразового скрипта или редконужной обработки - вполне допустимы такие выражения. Но с точки зрения продуктного кода, лучше не злоупотреблять откровенно лишней выгрузкой в ТЗ.

Выборка = Запрос.Выполнить().Выбрать();
Возврат ?((Выборка.Следующий()), Выборка.СуммаДолга, 0);

(0) Считаю что пойдет, но только в том случае, если в результате запроса гарантированно будет минимум одна строка. Иначе это просто быдлокод.

Код из нуля считаю нормальным.

А вот пример того кода который мне не нравится:
Попытка
  Возврат Запрос.Выполнить().Выгрузить()[0].СуммаДолга;
Исключение
  Возврат 0;
КонецПопытки

Не люблю когда много выходов из функции.