Кого бы убить за это?
Элементы.Дебитор.Заголовок  = ?(Объект.ВидАвансаЗадолженности = Перечисления.ВидыАвансаЗадолженности.ЗадолженностьПоставщику, "Поставщик (кредитор)", "Покупатель (кредитор)");
Элементы.Кредитор.Заголовок = ?(Объект.ВидАвансаЗадолженности = Перечисления.ВидыАвансаЗадолженности.ЗадолженностьПоставщику, "Новый поставщик",     "Новый покупатель");

Без контекста и не пойму что так возбудило.
Код можно было бы сделать более читаемым, если вынести условие в переменную. А так нормально.

(0) и ?

(0) идеальный код,
убейся сам

(1) Смысл проверять 2 раза одно условие?
Объект.ВидАвансаЗадолженности = Перечисления.ВидыАвансаЗадолженности.ЗадолженностьПоставщику

(4) и это всё?..

вы зажрались в хлам

(0)
Вы реально за это кого-то... Убить?

(5) больше ничего не разглядел ) Может ТС раскроет секрет своего недовольства.

(4) Так проверка производится сравнением со значением перечисления, а оно в данных не хранится, только в метаданных.. следовательно, обращения к БД на это не будет.. следовательно, и напрягаться не имеет смысла.
Единственное, что может напрягать - это обращение через точку к реквизиту объекта.. но думаю, что эта процедура вызывается не настолько часто, чтобы это играло какую-то роль)

А то что полю с названием Дебитор дается заголовок и смысл Кредитор всех устраивает?