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

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

(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)
Ну так и хрен с ним пускай функция в цикле вызывается
Будет заметно тормозить - перепишем, вероятно на полный отказ от циклов и готовых функций, все на нормальные запросы

(40)Если там, что угодно, у чего нет свойства "ПометкаУдаления" она упадет с ошибкой, а не вернет Неопределено. Судя по контексту, ничего кроме элемента справочника, мы туда передать не можем.

(43)Есть конечно. ЗначениеРеквизитаОбъекта, называется, про рекомендацию ее использования я и говорил. Она решает проблему объектного доступа.

(44)Можно порассуждать, что здесь какая-то типовая функция. И вот мы ее использовали, потому, что данных мало. Оптимизация выполнения ничего, вредит читабельности и поддерживанию кода. Такой аргумент принимается. Если функция была до этого. Но если тебя спрашивают(на собесе в том числе), что не так с этим кодом, показать пальцем на этот участок имеет смысл. Делать ли что либо с ним? Ну вопрос номер два.

(22) Лучше тогда уж:

ВопросАсинх(<ТекстВопроса>, <Кнопки>, <Таймаут>, <КнопкаПоУмолчанию>, <Заголовок>, <КнопкаТаймаута>)

(6) Интересно, а вот если контрагент пустой, то для каких договоров мы заполняем сумму? Для каких-то абстрактных договоров, которые никак не связаны с контрагентом, которого мы проверяем на пустоту?
Или там в таблице договоров находятся договоры с какими-то другими контрагентами?
Кроме этого еще такой вопрос – если у нас в таблица договоров уже  заполнена (а она судя по всему заполнена), почему при ее заполнении сразу не заполнили сумму договора?

(0) Ахаххаха, я вас собеседовал, возможно)

(20) Нет)

(49) Ну и зачем там контекстный вызов?

(50) Для безконтекстного вызова вам придется передавать табличную часть, которая на клиенте имеет тип ДанныеФормыКоллекция, на сервер по значению (ДанныеФормыКоллекция не сериализуются).
Смотрим пункт 3.
https://its.1c.ru/db/v8std/content/636/hdoc
Из-за особенностей работы ДанныеФормыколлекция
Смотрим пункт 2.
https://its.1c.ru/db/v8std/content/628/hdoc
Прежде чем передать табличную часть на сервер по значению вы её полностью получите на клиент. Легко проверить, написав простую обработку.
За правильный ответ тут считался контекстный серверный вызов заполняющий табличную часть за один серверный вызов.
Сеньора вроде собеседовали, если мы. Если бы не ответил, почему не надо обрабатывать табличную часть без контекста, ответ бы засчитывался.

(50) Опять же контекст, передающийся с клиента на север, это измененные элементы формы. При первом запуске ТЧ пустая и контекста на сервер не пойдет.
Стандарт не могу найти сейчас, но он ищется

(34) Пытался выяснить тоже, но толковых ответов не было.
(46) Точно, как вариант.
(47) Аналогичные вопросы задавал на собеседовании, но внятного ответа так и не услышал. Такое впечатление, что бизнес-логика их не интересовала вообще.
Но когда увидел этот "код", то сразу сообщил, что там всё некорректно и полностью надо переделать, но их такой ответ не устроил и начали погружаться...

(48) Мне два или три раза попался этот вопрос на разных собеседованиях. Я знал, что появится виновник торжества)))

(54) По описанию, похоже на нас. Я говорил, когда этот код показывали, что не надо думать про бизнес логику, нас только код и работа платформы интересует.

Походу появились использователи говнометодик при оценке
Извините но кого пытаетесь нанять?

Вторую копию себя?
Которая точно так же совершит те же ошибки?
И загонит контору в те же условия где потребуется найм еще?

(55) Для мыслящего разработчика забыть про бизнес-логику нецелесообразно.

(57) А зачем мыслить то? Копать надо! Тут столько задач в бэклоге - копать не перекопать...
И пофиг что большая часть от глупости появились и совершенно не нужны

(58) К сожалению, так и есть.

(56) По всей видимости кодера, который не будет думать про бизнес-логику :)

(57) Уважаю ваше мнение

(61) Благодарю за понимание.

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

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

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

Где тут табличная часть?

(63) Готовил этот вопрос))) Там же только "ДоговорКонтрагента")))

Откуда столько наглости, кого-то собеседовать. В своем говнокоде не могут разобраться.

(63) Передайте ДанныеФормыКоллекция с клиента на сервер, потом вновь прочтите сообщение на которое отвечаете.

(66) СтрокаДоговора.СуммаДоговора = ПолучитьСуммуДоговора(СтрокаДоговора.ДоговорКонтрагента);


Чебля?

(63) Потом передайте без контекста. Мы же серверный вызов\обращение к бд в цикле не оставляем?

(63) затем что
1 шаг -функцию эту надо из цикла вынести и передавать туда массив договоров, т.о. убрать запросы в цикле
2 убрать контекст

(63) Вот сразу же следующий шаг напрашивается передавать не один договор а массив их
И в ответ получать ТЗ или структуру, где для каждого переданного договора возвращается некая сумма

(69) Там мы же про говнокод из примера, а не как улучшить.

Из задачи непонятно, кстати, что там дальше делать, может в цикле и норм.

(69) Что бы собрать на клиенте все записи табличной части в массив, вам придется получать с сервера то чего на клиенте нет.
https://its.1c.ru/db/v8std/content/628/hdoc
Т.е. вы получите полную табличную часть с сервера и вернете её обратно. Оптимизация норм. Вместо полноценного серверного вызова, при нулевом контексте, т.к. на клиенте вы ТЧ не заполняли.

(51) Фигня, переделывайте.

Если ТЧ - небольшая, а договоры прям говорят об этом, и, при этом, на форме также тусит таблица марок в несколько тысяч - то лучше перебрать ТЧ на клиенте и отправит в виде массива структур на сервер во внеконтекстном.

Либо сделать копию таблицы в ПриЧтенииНаСервере и поместить во временное хранилище, вручную отслеживая изменение таблицы на клиенте и отсылая на Сервер изменения во внеконтекством вызове.

Всё не так уж и просто в реале в вырожденных случаях.

(70) Да это понятно) Ответ был на (49) на ответ из (20)

(72) Дадада. она подтягивается с сервера при скроллинге по 45 записей, знаем. Но иногда лучше это, чем контекстный.

(72) Эмм начать стоит с определения а сколько же строк в ТЧ Объект.ТаблицаДоговоров ?

(76) Никто не мешает задать этот вопрос. Он тут даже ожидается.

(73) О, заруба! Так кто же прав?
Есть ли вся ТЧ на клиенте, или она будет читаться порциями с сервера при обходе на клиенте?

(76) А ещё первый ли это серверный вызов открытия формы и т.д. в вопросе контекста.
Мы на хорошие деньги и условия нанимали, вопросы не сложные. Джуновский код должен разобрать сеньёр, задача вроде выполнимая.

(78) Конечно порциями. Один раз, кэшируясь на клиенте.

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

(78) Глубоко пофиг если честно
В примере кода придумали заумную ситуацию

Когда по идее две обработки лучше НаСервере но между ними воткнут вопрос, который только НаКлиенте

И типа выкручивайся...

(81) Я бы воткнул один контекстный серверный вызов в самом начале, чтобы забрать на клиента все что дальше потребуется
Ибо вот не пофиг сразу суммы договоров посчитать еще до вопроса
Или только после вопроса

(80) тогда "иногда лучше" из (75) должно встречаться исчезающе редко в обычной практике, специфика с десятком больших ТЧ понятнное дело бывает

(34)(55) Окей, понятно. Отличники дрочащие в присядку на олимпиадные задачки...

(85) Не мало людей нормально  по этому коду пообщались, не считаю, что это олимпиада. Напомню, собеседование на ведущего, по платформе то его можно спросить, что бы не обзывались?

(86) Нужно.

Но говоря словами что хотите услышать и о чем.
А не предполагая что вам телепат попался и может ваши мысли прочитать.
Да еще имеет точно такой же специфический опыт.

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

(87) Ваши аргументы понятны, но мы искали под себя.
Итог из 12 кандидатов с 6 говорить было вовсе не о чем, троим готовы были бы предложить мидла, если ставка была бы свободна, троим сделали оффер, один успешно работает сейчас.

(88) Браво)
"мутный код, который нужно как-то что-то зачем-то оптимизировать"
Записал себе.

(86) Это олимпиада не в сложности, а в неприменимости.

Создание таких условий, который появляются в очень редких моментах бизнес-логики и обычно означают взяли "as is" и попытались запихнуть в решение.
Думаю у многих людей нашего проф. направления будет возникать вопрос "Зачем?" Привыкли разбирать все до косточек и предлагать решения "to be", в переводе на русский "Круглое катить, квадратное тащить" это уже вшито в человека с ростом до ведущего.
Понимать нужно специфику коллег. Задавать практические вопросы, а не выдумывать метафизические. Получите больше хороших собеседований.

(88) Да можно сразу заваливать вопросом: "Когда при левом соединении результат больше чем записей слева?"

ЗЫ
Тут есть два ответа, один простой
А вот второй уже сложный и специфика 1С не только SQL

(92) Да большинство не способны ответить, что такое вообще левое соединение
Начинать можно вообще с вопроса типа если вам зарплату повысили на 10 процентов, а потом понизили на 10 процентов, то она уменьшилась, увеличилась или не поменялась
Большинство с гордостью в глазах за скуренный за школой учебник математики ничтоже сумняся в секунду отвечает, что не изменится, а через пару секунд в глазах появляется паника и сомнения

(93) Ты снова начал про свой НДС сверху, или в сумме
Могу аналогично спросить и хрен ответишь не гугля: "НДС это акциз или нет?"

(88) Первая часть сообщения голословная.
(92) Не в ту сторону понесло.

(94) А что плохого в ворпосе?
Нет, не акциз
И вопрос про акциз не показателен при тестировании программиста

(91) У нас хорошие условия набора, один этап собеседования в формате code-rewieve.
То что мы хотим увидеть на собеседование ширину кругозора разработчика внутри своего стека, собеседуя на ведущего, это нормально.
Когда мне нужно будет считать НДС, знать в деталях бухучет etc. я эти вопросы аналитику задам.
А если соискатель на ведущего программиста в коде не видит заложенные бомбы, или того хуже, на код из (0) говорит, что тут все ОК, то на ведущего его брать смысла нет.

(95) Ага ага, голословная
Помнится, на собеседованиях из десяти человек половина с оглушительным треском резалось на одном только этом вопросе
Вид у человека в этот момент обычно максимально жалкий и растерянный, типа что еще за левое соединение, нахрена мне это знать, почему я просто не могу пользоваться конструктором запросов...

(96) Хехе. Так и вопросы про НДС не показательны.
Правильный ответ: "НДС это один из видов акциза по сути"
Если использовать утиную типизацию при сравнении - ну это когда "если выглядит как утка, крякает как утка, то вероятно это и есть утка"©

(97) Да код в (0) это не бомба, это помойка бессмысленная тупая какая-то
Но, если не вникать в то, что этот код делает, а докапываться именно до всякой ерунды типа контекста, асинхронности и пр., то это разговор вообще не про программирование, а про жесткое тупое кодирование и только
Так что при вопросе типа (0) первый же вопрос нужно задать такой, мы сейчас про то, почему такая тупая архитектура или почему такой тупой код?

(99) Про НДС как раз показательны, потому что это минимальная специфика, а так-то чистая математика

(99) Акциз — это когда товар стоит 5 рублей, а ему хреначат охулиард сверху, чтобы меньше покупали или чтобы больше собрать денег
И причем здесь НДС?

(100) Имеете полное право что на вопрос про архитектуру, что про код. Исходя из него дальше беседу продолжим и все пой

(95)
Видел много "сеньоров" и ТА которые левым соединением подбирали из справочника по кодам

А потом удивлялись ошибкам когда в справочниках дубли по этим кодам
Упс мы подбираем для одной записи а соединение вернуло нам 3 - это как так???
Это был простой ответ

А сложный он в дебрях ИТС закопан...

(100) Помойка в коде вполне осмысленная, есть много вариантов сделать правильно, в зависимости от того, что соискатель сделать решит)

(102) НДС - это когда товар стоит 5 рублей, но мы ему сверху хреначим 22%, "чтобы меньше покупали или чтобы больше собрать денег"

(103) Я к тому, что даже в этой ветке все пофиг, про что вопрос в (0), все погнали в сторону кода
Что про большинство говорит, что они не программисты, а просто кодеры

(100) Он и говорит, в собесе указывали на это (55)

Но сути не меняет, попытка поиграться в платформе, которая создана для бизнес-логики в тру-программирование вида: "Вы должны сразу программировать правильно и видеть бомбы заложенные в коде".

Имеет смысл если он пишут решения, ну или зарплата как х2 рынка.

(106) Еще раз, когда сверху необоснованно много денег накручивают, а не пятую часть
Например, сигареты производят по 10 рублей, а продают по 200
Чтобы меньше травились, а те, кто травятся, чтобы хотя бы бюджет наполняли
Что вот ты на рожон лезешь, когда неправ?

(108) Я не то, чтобы сильно много ходил по собеседованиям, но такой детской хероборы ни разу не встречал
Как-то людей за серьезные деньги интересуют более серьезные вопросы

(109) Акциз может быть фиксированным суммой, или % от объема/веса или от стоимости
В последнем случае это и есть НДС, акциз суммой 0%, 10%, 22% на цену товара

И да.
Упрощать расчет НДС считая от общей суммы для много штук товара конечно же можно.
Но не стоит удивляться разным итогам...

Ибо цена 1 штуки товара без НДС = 0 рублей 1 копейка.
Произведена (или импортирована) партия в 1000 штук.
Какой НДС? А если мы по одной штуке будем отгружать?

(111) Ты утомил своей отсебятиной
Загуглил, получил такое, иди читай

Акциз — это косвенный федеральный налог, который включается в стоимость определённых товаров и фактически оплачивается конечным потребителем. Формально его платят производитель или импортёр, но в итоге сумма налога перекладывается на покупателя. В отличие от НДС, акциз не выделяется отдельной строкой в чеках при розничной продаже — он уже «спрятан» в цене товара. sberbank.comtax-monitor.rukurs.alfabank.ru

Основные характеристики акциза

    Плательщики. Организации, индивидуальные предприниматели, а также лица, перемещающие товары через таможенную границу ЕАЭС, при выполнении определённых операций с подакцизными товарами. glavkniga.ru
    Объект налогообложения. Не сами товары, а операции с ними: производство, продажа, передача, использование для изготовления других продуктов. kurs.alfabank.ru
    Ставки. Устанавливаются статьёй 193 Налогового кодекса РФ и регулярно индексируются. Существуют три вида ставок:
        Твёрдые (специфические) — фиксированная сумма в рублях за единицу товара (литр, лошадиная сила, килограмм, тонну). sberbank.comkurs.alfabank.ru
        Адвалорные — процент от стоимости товара (например, для природного газа — 30%). sberbank.comkurs.alfabank.ru
        Комбинированные — сочетание твёрдой ставки и процента от стоимости (применяются, например, для сигарет и папирос). sberbank.comkurs.alfabank.ru
    Налоговый период — календарный месяц. nalog.gov.rukurs.alfabank.ru
    Порядок уплаты. Сумма акциза к уплате исчисляется для каждого вида подакцизного товара в отдельности и затем суммируется. Сроки уплаты зависят от вида товара и операции. nalog.gov.rukurs.alfabank.ru

Для чего используются акцизы

    Фискальная функция. Акцизы — значительный источник дохода для федерального бюджета.
    Регулирующая функция. Государство ограничивает потребление товаров, которые могут наносить вред здоровью (алкоголь, табак) или окружающей среде (топливо). Чем выше акциз, тем дороже продукция и тем меньше её покупают.
    Социальная функция. Акцизы позволяют государству влиять на структуру потребления и поддерживать отечественных производителей. Например, высокие акцизы на импортные товары делают местную продукцию более конкурентоспособной.

(112) Еще раз повторю:
Если нечто выглядит как акциз, крякает считается (и учитывается, и уплачивается) как акциз - вероятно это и есть акциз.

(111) Бессмысленно разговаривать с туловищем про цены, суммы и пр., когда он и СуммаСНДС/120*20 не знает
Как и с тобой, потому что ты вместо разговора по делу начинаешь ДДСничать и всякую пургу гнать

(113) Дело твое
Чем более некомпетентен ты, тем больше я стою как специалист

(115) Так ответь какая сумма НДС то будет в (111) ?

(116) У меня минимальная консультация два часа
Реквизиты для оплаты в личке

(117) Первый начал про НДС и технично сливаешься?

Повторю: Завалить можно любого!
И меня в том числе ))

(118) Почему сливаюсь?
Оплачивай 8 штук и поговорим про твою проблему

(98) Всякое бывает, я тоже много собеседовал и никогда не наслаждался тем, что если чего-то кандидат не ответил и не глумился над ним. Просто непонятно зачем в (88) прозвучала фраза "Авторам типа ТС..."?

(119) Согласен, завалил...

(120) Согласен, в (88) что-то подгорело, был неправ, приношу извинения, поскольку не в курсе ни деталей, ни уровня развития ТС
Просто тригернуло (0) по совокупности, видимо
Код картинкой, вопрос хрен пойми о чем, недостаток информации...

(122) Мысли (цитирую): "вопрос не пойми о чем, недостаток информации..." - полностью разделяю (см. (53)), ещё с самого начала, как увидел этот "код" на собеседовании.
В (9) пояснял, что достался только скрин для разбора.
Всех благодарю за уделённое время и соображения! Давайте уже закроем ветку!

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

(119) ты вроде до 5 собирался поднимать ,а тут уже 8... инфляция

(125) Потому что за два часа

Ветка закрыта по просьбе автора, потому что тупняк.
Вопрос обновления таблицы договоров (16) так и не был рассмотрен.

Код - говно. Автор - тупой бот.