Уважаемые специалисты, посоветуйте пожалуйста, можно ли данный код(не мной добавленный в конфигурацию) заменить на

Если ТаблицаТоваров.Количество() = 0 Тогда
   Возврат;
КонецЕсли;

Вот сам код:

    ТабТовары = ТаблицаТоваров.Выгрузить();
    Для Каждого Стр Из ТабТовары Цикл
        
        Стр.НомерСтроки = Индекс + 1;
        
        Если Стр.Выбран Тогда
            
            ЕстьТовары = Истина;
            
        КонецЕсли;
        
    КонецЦикла;
    Если Не ЕстьТовары Тогда
        Возврат Ложь;
    КонецЕсли;

а где анализ поля Выбран ??

вряд ли. в таблице товаров ведь флаг есть "выбран", он то и проверяется.

Если ТаблицаТоваров.Количество() = 0 Тогда
   Возврат ЛОЖЬ;
КонецЕсли;

ТАм не только Выбран но и номер строки устанавливается

НайтиСтроки()

(3) действительно, я набил вручную и пропустил.

Но в целом в коде нет никакой вуду магии, кроме анализа на наличие строки в таблице и его можно сократить? До такого:

Если ТаблицаТоваров.Количество() = 0 Тогда
   Возврат Ложь;
КонецЕсли;

(6) тебе уже озвучили правильный ответ в (5)

А так, "Работает - не трожь"(с)

Можешь сократить, добавив свою проверку в начале

Выбран()
Синтаксис:
Выбран()
Назначение:
Определение факта позиционирования объекта типа документ.
Возвращает: 1 - если документ выбран; 0 - если не выбран.


если Документ записан в базе , метод Записать() то будет 1 , а если только создается - открыта форма , но документ не записан будет 0

Метод перекочевал ещё из 7-ки

В коде УТ во первых заполняется колонка Номер строки:
Стр.НомерСтроки = Индекс + 1
и во вторых устанавливается флаг ЕстьТовары по условию Если Стр.Выбран
В предложенном коде:
Если ТаблицаТоваров.Количество() = 0 Тогда
   Возврат;
КонецЕсли;
не выполняется ни одной из этих двух задач.

ЗЫ: вот потом и пишут везде - УТ11 - гуано и криво сделано.

Функция ЕстьВыбранныеТовары()
Возврат ТаблицаТоваров.НайтиСтроки(Новый Структура("Выбран", Истина)).Количество() > 0;
КонецФункции

(0) Я не вижу смысла править код.

Код читабельный: логика кода понятна. Кроме того, косяков тут нет.

Лучше ТаблицаТоваров.Итог("Выбран") > 0

А в (0) просто не хватает Прервать.

Без полного цикла по всем строкам не будет заполняться поле НомерСтроки:
Стр.НомерСтроки = Индекс + 1

Большое спасибо всем за помощь, меня смутил Выбран, оказалось так обозвали пометку. Самый лучший и простой код - в (12).

(16) Одинаковый номер строки всем строкам тоже можно быстрее установить

(16) Кстати здесь это бессмысленный код, ибо не в основную тз запись, а в выгруженную. Которая потом нигде не используется

(0) При чем здесь УТ 11?

(20) это дописка в УТ11. Согласен, что типовой код здесь отсутствует.

(12)+

Функция ЕстьВыбранныеТовары()
Возврат ТаблицаТоваров.Найти("Выбран", Истина) <>
Неопределено;
КонецФункции