Всем доброго времени суток!
Прилагаю скрин: https://dropmefiles.com/uAjZc , на котором есть неоптимальный (некорректный) код. Попутно тоже буду приводить замечания по коду, но и буду благодарен полезным комментариям со стороны сообщества. Сейчас данная задача популярна на собеседованиях и хотелось бы максимально изучить данный вопрос и ничего не упустить. Всем заранее спасибо за позитивное и эффективное участие!

Скрин-то сюда прилагаете?

Прилагался, но не пропустило. Сейчас ещё раз попытаемся. JPG не пропускает? На всякий случай ещё здесь: https://dropmefiles.com/uAjZc

(2) Да вставьте вы это поделие просто текстом.

Мне этот код уже не нравится.... :)

(2) мы... вы там всем отделом сейчас собес проходите в рилтайме что ли...))

В (0) и (2) добавлена ссылка на скрин.

// При изменении элемента формы "Контрагент"
&НаКлиенте
Процедура КонтрагентПриИзменении(Элемент)

    // Получим пометку удаления
    ПометкаУдаления = ПолучитьПометкуУдаленияНаСервере(Контрагент);
    Если ПометкаУдаления = Истина Тогда

        ТекстВопроса = "Контрагент помечен на удаление. Очистить?";
        Ответ = Вопрос(ТекстВопроса, РежимДиалогаВопрос.ДаНет);
        Если Ответ = КодВозвратаДиалога.Да Тогда
            Контрагент = Неопределено;
        КонецЕсли;

    КонецЕсли;

    // Если контрагент пустой, заполним сумму договора
    Если НЕ ЗначениеЗаполнено(Контрагент) Тогда

        Для Каждого СтрокаДоговора Из Объект.ТаблицаДоговоров Цикл
            СтрокаДоговора.СуммаДоговора = ПолучитьСуммуДоговора(СтрокаДоговора.ДоговорКонтрагента);
        КонецЦикла;

    КонецЕсли;

КонецПроцедуры // КонтрагентПриИзменении()

&НаСервере
Функция ПолучитьПометкуУдаленияНаСервере(ТекущийКонтрагент)

    Возврат ТекущийКонтрагент.ПометкаУдаления;

КонецФункции // ПолучитьПометкуУдаленияНаСервере()

&НаСервере
Функция ПолучитьСуммуДоговора(ДоговорКонтрагента)

    Запрос = Новый Запрос;
    Запрос.УстановитьПараметр("ДоговорКонтрагента", ДоговорКонтрагента);
    Запрос.Текст = ".........."; 
    /// Выполнение запроса, обработка результата
    Возврат Выборка.СуммаДоговора;

КонецФункции // ПолучитьСуммуДоговора()

(3) (4) )))

Говнокод + ИИ-код

(6) Спасибо, а то был только скрин)

1. ПолучитьПометкуУдаленияНаСервере() - нахер отдельно если там внутри одна строка, да еще без запросов
2. = Вопрос() - все современные конфы давно отказ от модальности/синхронности
3. = ПолучитьСуммуДоговора() - в цикле? и внутри запрос? мдя

(0) Цикл с запросами на каждого контрагента уже плохо.

(8) Полностью согласен, но полезно разобрать по пунктам.

(10) " нахер отдельно если там внутри одна строка"
Вроде очевидно - на клиенте поле недоступно)

(10) +
4. Контекстный вызов без необходимости

(0)(6) Вызов серверной функции ПолучитьСуммуДоговора(ДоговорКонтрагента) с клиента в цикле - неоптимально и некорректно.

Ещё непонятно откуда изменяется Объект.ТаблицаДоговоров.

(6) это вот такое сейчас на собесах дают? Да, обмельчал кандидат....

Ну самое первое - зачем вы запрос в цикл кидаете? Получите все договору контра и с ними работайте.
Сократить можно код, убрав нафиг, в данном случае, не нужные комментарии. Да и проверку "=Истина" зачем?
Вопрос() - ПоказатьВОпрос().

PS. Ого, тут уже написали... А всё то же самое....

(13) Это понятно, почему не объединить?
Зачем делать для одной пометки когда можно свою ЗначениеРеквизитаОбъекта/ЗначенияРеквизитовОбъекта сразу сделать?

(18) Всё верно, я тоже этот вариант фиксировал: использовать функцию БСП ОбщегоНазначения.ЗначениеРеквизитаОбъекта().

(6) Функция ПолучитьСуммуДоговора(ДоговорКонтрагента) - нужно вызывать без контекста.

(19) только это не избавляет от серверной (без контекста конечно) обертки.
Ну и про наличии БСП в задании ни слова, хотя и открыта собственно конфигурация БСП)

(6) Использовать ПоказатьВопрос(<ОписаниеОповещенияОЗавершении>, <ТекстВопроса>, <Кнопки>, <Таймаут>, <КнопкаПоУмолчанию>, <Заголовок>, <КнопкаТаймаута>)

(21) Можно два варианта, я лично, два вариант озвучивал на собеседованиях. И, кстати, всё, что здесь люди сообщили, я тоже озвучивал, но пытливые соискатели ещё чего-то хотели, вот я и решил убедиться, что ничего не упущено.

Если честно то код рабочий, для джуна пойдет на ура
И даже мидл/сеньор сможет отмазаться с таким

Ну типа запрос в цикле, потому что есть готовая функция и один хрен по одному контрагенту в ТЧ будет 1-2 договора, нет смысла оптимизировать

(24) код рабочий ... по одному контрагенту в ТЧ будет 1-2 договора

Тогда ответь на (16). Поменялся контрагент, идёт обработка ТЧ. А что должно было поменять эту ТЧ!?

(24) если этот контрагент некое общее физическое лицо, через которое идут сотни покупок

(25) Это уже вне задачи :)

Если ПометкаУдаления = Истина Тогда
убрать "= Истина"

(28) Вот тут я бы поспорил
Функция же, может и Неопределено вернуть

(0) 15 лет стажа и код картинкой?
Мир тупеет со страшной силой
Пора повышать ставку до пятерки

(0) У вас там гнездо ливингстаров?

По ходу у ИИ нет никаких шансов
Скоро он переварит все ценное и начнет захлебываться человеческой тупостью и тем, что он сам на этой тупости нагенерил

(32) А Ливингстар и Г1С в этом ему помогут. Особенно есть там  библиотека у Г1С с лакомым для нейросетки названием "Библиотека".

(0) Первый вопрос: "Какая задача стоит за этим кодом" Что он делает я понимаю, а зачем он это делает?

(34) этим и определяется: перед вами кодер по ТЗ или сеньор)

(30) а там еще и окажется что у него зп под лям)
Просто он ублажает пользователей по другому)

(24)Сеньор? Да как бы тут такой код обычно и джун пишет первый и последний раз. Потом ему говорят так не писать он и не пишет больше.
1. Все вызовы контекстные, хотя смысла в этом ноль. У сеньора должна быть аллергия на такое.
2. Про запрос в цикле написали, это конечно очевидно. Но типа "там данных немного поэтому не страшно". А какой смысл делать плохо и не делать хорошо. Буквы на клавиатуре печатающие хороший код такие же как и буквы печатающий плохой. Зачем печатать плохой? Иногда, такое имеет смысл, но здесь запрос без цикла не усложнит ровным счетом ничего.
3. Объектная модель вытаскивания пометки на удаление прямое нарушение рекомендаций 1с и тоже не имеет никакого смысла.
4. Серверные вызовы в клиентских обработчках без фонового выполнения фризят интерфейс и тоже не рекомендуются.

(34)Показывает триггерит ли тебя контекстный вызов без необходимости

(29) В данном случае не может

(39) Это если Контрагент - действительно элемент справочника, а не какая-нибудь Структура

(40) Если эта функция может вернуть не только булево, то тогда по определению одно из двух
Либо нужно не писать "= Истина", потому что программа в случае возврата не булево должна упасть с ошибкой "Хули ты от меня хочешь, осел, иди проверяй код"
Либо нужно проверять не только Истина, но и Ложь и все остальные возможные значения, потому что хрен его знает, что там может вернуться, может там целый букет типа Истина, Ложь, "true", "false", 0, 1 и пр. херобора, рожденная в воспаленном мозгу автора

Вообще, самый ожидаемый вариант — это булево
А раз так, то пихать туда что-то еще — преступление, как минимум, в этом случае нужно переобозвать функцию
А то получится, что функция СписокСправочников возвращает картинку в jpeg, на которой список метаданных, относящихся к справочникам

Ответ = да
И при этом контрагент присвоить неопределено...
?

(37) Написал же про готовую функцию
В типовых такое постоянно

(43)+ Есть типовая/готовая функция получения чего-то
Знаем что цикл будет из малого числа повторов (1-3)
Ну так и хрен с ним пускай функция в цикле вызывается
Будет заметно тормозить - перепишем, вероятно на полный отказ от циклов и готовых функций, все на нормальные запросы