Помогите принять правильное решение в коллективной разработке

JOB

Работа

, , , ,

☑

0 Старик Юзергад

29.05.13

✎

12:36

1. Работает, не трогай!	38% (9)
2. Молча поправь	17% (4)
3. Не умничай	17% (4)
4. Свой вариант решения	17% (4)
5. Молча поправь и оставь свою подпись	13% (3)

Всего мнений: 24

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

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

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

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

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

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

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

15 Lazy Stranger

29.05.13

✎

13:00

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

Молча поправь

21 Лефмихалыч

29.05.13

✎

13:57

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

Молча поправь

78 КонецЦикла

30.05.13

✎

01:32

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

Молча поправь

84 pumbaEO

30.05.13

✎

15:28

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

Молча поправь