Имя: Пароль:
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;

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

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

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

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

Помогите принять правильное решение с учетом работы в коллективе разработчиков?
6 Deni7
 
29.05.13
12:40
(0) Всех ошибок не исправишь (тем более это недочет просто). Работает - и хорошо.

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

Работает, не трогай!
16 noxxx
 
29.05.13
13:03
Работает, никто не жалуется - не трогай

Работает, не трогай!
18 Старик Юзергад
 
29.05.13
13:27
(17) забил, никто не жалуется.

Работает, не трогай!
19 wowik
 
29.05.13
13:47
не нужно.

Работает, не трогай!
20 EugeniaK
 
29.05.13
13:51
(0) Если больше совсем нечем заняться, то можешь переписать.
Но лучше не лезть.

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

Работает, не трогай!
81 DarkWater
 
30.05.13
15:19
Других дел нет?)

Работает, не трогай!
83 MaxisUssr
 
30.05.13
15:22
Уверен, что прав?

Работает, не трогай!
Закон Брукера: Даже маленькая практика стоит большой теории.