Как правильно прописать вот это:

Если СтрокаТаблицы.ВидРасчета = ПланыВидовРасчета.ОсновныеНачисленияОрганизаций.ДоплатаЗаНочныеЧасы И СтрокаТаблицы.Сотрудник Из СписокСотрудников Тогда

забавно вырвать кусок кода и спрашивать

СписокСотрудников это полученный запросом список значений.

Вот эту часть как грамотно оформить: И СтрокаТаблицы.Сотрудник Из СписокСотрудников Тогда

(0) lol. Синтаксис помощник в руки, в зависимости от типа СписокСотрудников

(0) надо чтобы сотрудник был в списке?

(5) Да, проверка на наличие сотрудника в этом списке.

(6) читай в СП списокЗначений

Для объекта доступен обход коллекции посредством оператора Для каждого … Из … Цикл. При обходе выбираются элементы коллекции.
Возможно обращение к элементу коллекции посредством оператора [...]. В качестве аргумента передается индекс элемента (нумерация с 0).

(8) не, надо НайтиПоЗначению

(9) Если это список значений а не массив, а может это вообще тз с одной колонкой.

(10) Это точно список значений.

Есть идея разделить все это, то есть отделать Если СтрокаТаблицы.ВидРасчета = ПланыВидовРасчета.ОсновныеНачисленияОрганизаций.ДоплатаЗаНочныеЧасы Тогда, а внутри прописать что-то типа Пока СтрокаТаблицы.Сотрудник В СписокСотрудников Цикл и т.д.

(12) ловите наркомана)

Найти юзай
только не строку целиком, а значение из строки

А запросом выбрать сразу то что нужно, не...???

(15) +1

(15) запрос в цикле??? ай=яй-яй :)

(12) Не ну что за люди. Синтакс-помощник лень открыть. И в (9) ответили уже явно. Все равно чего-то надо еще.

(17)просто запрос не?

(17) Какой цикл?? сразу одним запросом все что нужно из таблиц

(20) И глумится дальше на данными как ТС угодно

Что вы к этой строчке прицепились? :) Человек то ли только из клюшек вынырнул, то ли вообще с 1с на "Вы"...
Тут надо кусок кода смотреть, а не одну строчку.

А так-то (9), с учётом (11).

Получилось, правда не без порно:

Если СтрокаТаблицы.ВидРасчета = ПланыВидовРасчета.ОсновныеНачисленияОрганизаций.ДоплатаЗаНочныеЧасы Тогда
            Пока СписокСотрудников.Следующий() Цикл
                Если СтрокаТаблицы.Сотрудник = СписокСотрудников.Сотрудник Тогда
                СтрокаТаблицы.Показатель1 = 999;
                КонецЕсли;
            КонецЦикла;
        КонецЕсли;

Не, все равно пока не то. Поменялось только одно значение, первое.

(24) Ты гонишь либо что "получилось", либо в (11).

Нет у списка значений метода Следующий()

(24) А ты говорил что это список значений. "СписокСотрудников.Следующий()" не работает со списками значений, а работает с выборками. Так что где то ты обманываешь.

(26) Да, походу на выходе из запроса не список значений.

(29) По ходу рано ещё код писать, надо ЖКК полистать сначала.

Потому показывай весь где получается этот твой список, а не только это условие.

Если НЕ РаботаСДиалогами.ЗаписатьДокументОтменивПроведениеПередВыполнениемДействия(мОбъект, мФорма, "рассчитать") Тогда
        Возврат;
    КонецЕсли;
    
    ОбработкаКомментариев = глЗначениеПеременной("глОбработкаСообщений");
    ОбработкаКомментариев.УдалитьСообщения();
    мОбъект.РассчитатьНачисления();
    
    // получить список сотрудников для изменения
    Запрос = Новый Запрос;
    Запрос.Текст =
    "ВЫБРАТЬ
    |    РаботникиОрганизацийСрезПоследних.ГрафикРаботы,
    |    РаботникиОрганизацийСрезПоследних.Сотрудник
    |ИЗ
    |    РегистрСведений.РаботникиОрганизаций.СрезПоследних(, ) КАК РаботникиОрганизацийСрезПоследних
    |ГДЕ
    |    РаботникиОрганизацийСрезПоследних.ГрафикРаботы = &График";
    
    //Запрос.УстановитьПараметр("ДатаНачала",            НачалоМесяца(ПериодРегистрации));
    //Запрос.УстановитьПараметр("ДатаОкончания",        КонецМесяца(ПериодРегистрации));
    //Запрос.УстановитьПараметр("ПериодРегистрации",    ПериодРегистрации);
    //Запрос.УстановитьПараметр("Организация",        Организация);
    //Запрос.УстановитьПараметр("ГоловнаяОрганизация",ОбщегоНазначенияЗК.ГоловнаяОрганизация(Организация));
    //Запрос.УстановитьПараметр("РегламентВалюта",    Константы.ВалютаРегламентированногоУчета.Получить());
    //Запрос.УстановитьПараметр("Год",                Год(ПериодРегистрации));
    //Запрос.УстановитьПараметр("ДатаНач",            НачалоКвартала(ПериодРегистрации));
    //Запрос.УстановитьПараметр("ДатаКон",            КонецКвартала(ПериодРегистрации));
    //Запрос.УстановитьПараметр("ПолЖ",                 Перечисления.ПолФизическихЛиц.Женский);
    Запрос.УстановитьПараметр("График",             Справочники.ГрафикиРаботы.НайтиПоНаименованию("Сменный график (учетный период 3 месяца) - 40ч."));
    СписокСотрудников = Запрос.Выполнить().Выбрать();
        
    // корректируем начисления на процент
    ТаблицаНачислений = Новый ТаблицаЗначений;
    ТаблицаНачислений = мОбъект[мИмяТабличнойЧасти].Выгрузить();
    Для Каждого СтрокаТаблицы Из ТаблицаНачислений Цикл
        СтрокаТаблицы.Результат = Окр(СтрокаТаблицы.Результат / 50 * 10, 2, 1);
        Если СтрокаТаблицы.ВидРасчета = ПланыВидовРасчета.ОсновныеНачисленияОрганизаций.ДоплатаЗаНочныеЧасы Тогда
            Пока СписокСотрудников.Следующий() Цикл
                Если СтрокаТаблицы.Сотрудник = СписокСотрудников.Сотрудник Тогда
                СтрокаТаблицы.Показатель1 = 999;
                КонецЕсли;
            КонецЦикла;
        КонецЕсли;
    КонецЦикла;
    мОбъект[мИмяТабличнойЧасти].Загрузить(ТаблицаНачислений);
    
    ОбработкаКомментариев.ПоказатьСообщения();
    
    ТекущаяСтрока = мФорма.ЭлементыФормы.Начисления.ТекущаяСтрока;
    Если ТекущаяСтрока <> Неопределено Тогда
        мСтарыйРезультат                = ТекущаяСтрока.Результат;
        мБылоОтработаноДней                = ТекущаяСтрока.ОтработаноДней;
        мБылоОтработаноЧасов            = ТекущаяСтрока.ОтработаноЧасов;
        мБылоОтработаноДнейПоПятидневке    = ТекущаяСтрока.ОтработаноДнейПоПятидневке;
        мБылоНормаДней                    = ТекущаяСтрока.НормаДней;
        мБылоНормаЧасов                    = ТекущаяСтрока.НормаЧасов;
        мБылоНормаДнейПоПятидневке        = ТекущаяСтрока.НормаДнейПоПятидневке;
        мБылоОплаченоДнейЧасов            = ТекущаяСтрока.ОплаченоДнейЧасов;
    КонецЕсли;
    
    ТекущаяСтрока = мФорма.ЭлементыФормы.ДополнительныеНачисления.ТекущаяСтрока;
    Если ТекущаяСтрока <> Неопределено Тогда
        мСтарыйРезультатТЧДополнительныеНачисления            = ТекущаяСтрока.Результат;
        мБылоОплаченоДнейЧасовТЧДополнительныеНачисления    = ТекущаяСтрока.ОплаченоДнейЧасов;
    КонецЕсли;
    
    ЗаполнитьРасчетныйЛисток(Истина);
        
    Закрыть();

и вот это
СтрокаТаблицы.Показатель1 = 999;
значения не меняет, если ты конечно не в ТЗ запрос выгрузил

(33) Почему? У одного сотрудника поменялось значение на 999.

Ох, чувствую, щас насчитают кому-то ЗП... (32)
(32) см. СП меотоды выборки "НайтиСледующий" и "Сбросить"

временные перемещения не учтутся в твоем запросе.

Глобальная цель переделать механизм формирования тарифной ставки для доплаты за ночные часы и только для сотрудников с графиком работы "Сменный график (учетный период 3 месяца) - 40ч." на новую формулу: ТарифнаяСтавкаМесячная * 3 / НормаЧасовЗаКвартал (по производственному календарю)

(36) Почему?

(34) увидел код, тут поменяется
"но только один разик" :)

метод Следующий() позиционирует курсор на конец выборки и обратно не возвращает

юзай Найти, будет проще и код читабельнее

(39)+ позиционирует в твоем цикле

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

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

(42)хихи многие не открывают.даже не знаю,что лучше

(32) "НайтиПоНаименованию". Mother of god...

ТС где траву брал?

"Да, походу на выходе из запроса не список значений."(capllary)

Пятница, внезапно, началась в среду :)

А потом говорят 1С не программисты.
Я конечно понимаю, что 1С знать предметную область может и главное и в ней быть богом, но черт ....

(33) Ты код внимательней читай.
(35) Не нужна ему выборка.
(37) В принципе, код почти рабочий, но не очень эффективный и с кучей ошибок новичка.
Лучше не использовать выборку и перебор, а выгрузить сотрудников из результата в ТЗ или массив, и в полученной коллекции искать.

А так надо перед циклом Пока СписокСотрудников.Следующий() Цикл
добавить
СписокСотрудников.Сбросить()

Сейчас у тебя выборка после первого прохода цикла остаётся позиционированной в конце и вложенный цикл Пока СписокСотрудников.Следующий() Цикл выполняется только один раз.

Но лучше весь код переписать, хотя бы из-за того, что выборка тут неэффективна и вот так:
Справочники.ГрафикиРаботы.НайтиПоНаименованию("Сменный график (учетный период 3 месяца) - 40ч.")

делать не надо. При переименовании графика всё полетит к чертям. Хардкодинг - зло.

Да, и (41) прав, тут всё несколько сложнее.

(48) >>Лучше не использовать выборку и перебор, а выгрузить сотрудников из результата в ТЗ или массив, и в полученной коллекции искать.
Чем это лучше метода НайтиСледующий у выборки?

(48) Справочники.ГрафикиРаботы.НайтиПоНаименованию("Сменный график (учетный период 3 месяца) - 40ч.")


Сам никогда так не ставлю, просто у справочника даже кода нет.

(51) на форму вынести реквизит не?

В запрос можно передать ТЗ с номером строки и сотрудником.
Сделать отбор. Мало смысла тащить всех сотрудников.
Тогда обработка упрощается.

(48) СписокСотрудников.Сбросить()  помогло.

Не забудь про важное замечание
> временные перемещения не учтутся в твоем запросе.

Плюс срез надо делать на конкретную дату. А то заплатите лишнее или не доплатите, в общем косяк будет... )))

(55) А еще лучше срезать на дату начисления. Т.к. сотруднику могут и в середине что-то поменять.

(56) *середине месяца

(55) Ок, спасибо. Учтем.

(37) не пойму,а почему нельзя поменять на вид расчета с нужной формулой

(59) Там не формулы, а что-то с чем-то у нас на предприятии куча начислений разных, так по формуле рассчитывается только одно:  Результат = Сумма
Все остальное в коде исправлено.

(59) Потому что к примеру, там нельзя высчитать НормуЧасовЗаКвартал.