Нашел случайно в конфе такой код, какой человек и с какой целью писал его, не знаю:

промИтогКоличество=0;
промИтогСумма=0;
промИтогСуммаБезНДС=0;

Для каждого промСтрокаТб из Состав Цикл
промИтогКоличество=промИтогКоличество+пСтрокаТб.Количество;
промИтогСумма=промИтогСумма+пСтрокаТб.Сумма;
промИтогСуммаБезНДС=промИтогСуммаБезНДС+пСтрокаТб.СуммаБезНДС;
КонецЦикла;    

ИтогКоличество=промИтогКоличество;
ИтогСумма=промИтогСумма;
ИтогСуммаБезНДС=промИтогСуммаБезНДС;

Это можно написать так:

ИтогКоличество=Состав.Итог("Количество");
ИтогСумма=Состав.Итог("Сумма");
ИтогСуммаБезНДС=Состав.Итог("СуммаБезНДС");

Помогите принять правильное решение с учетом работы в коллективе разработчиков?

если автор неизвестен, я бы, не задумываясь, молча закомментировал бы этот "шедевр" и забыл бы сразу о нем

нашел косяк, исправь.
А по поводу подписи должен быть общий стандарт, как что и когда помечать. У нас так: правишь функцию, помечай всю целиком своей фамилией и неси ответственность за ее дальнейшую работу.

Если знаешь писателя - скажи что ты поправил, без свидетелей и излишних наставлений

Исправил и забыл, нафига поднимать из глубин уличного туалета фекалии?