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

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

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

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

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

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

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

(0) Всех ошибок не исправишь (тем более это недочет просто). Работает - и хорошо.

(0) всё не перепишешь, прояви свой талант в других местах
когда будет потребность доработки этого участка тогда и ..

Работает, никто не жалуется - не трогай

(17) забил, никто не жалуется.

не нужно.

(0) Если больше совсем нечем заняться, то можешь переписать.
Но лучше не лезть.

Ибо нефиг, это коллектив!
Могут обидеться и подумать, что ты выскочка.
А проги очень ранимые на замечания.

Других дел нет?)

Уверен, что прав?