Имя: Пароль:
1C
1С v8
Правильное оформление кода
,
0 assasu
 
01.07.19
10:50
1. комментируем новый код 67% (2)
2. комментируем все и пишем по новому 33% (1)
Всего мнений: 3

Доброго дня.

Есть некая, довольно нагруженная, система на базе УПП.
Есть регламентированный отчет этой системы.
внутри отчета следующие типовые строки

...
бла-бла-бла = Выборка.ЗначениеПоказателя;
...

Требуется добавить к значению свое новое слагаемое.
Хочу понять как более правильно оформить код с учетом дальнейшей поддержки и обновления:

1.
...
//мой код
//бла-бла-бла = Выборка.ЗначениеПоказателя;
бла-бла-бла = Выборка.ЗначениеПоказателя + МояДобавка;
//мой код
...

либо же
2.
...
бла-бла-бла = Выборка.ЗначениеПоказателя
//мой код
+ МояДобавка;
//мой код
...
1 Гипервизор
 
01.07.19
10:54
Где вариант сохранить отчет как внешний?
2 assasu
 
01.07.19
10:56
(1) такого варианта нет. политика компании.
3 RomaH
 
naïve
01.07.19
10:59
1

комментируем все и пишем по новому
4 ДенисЧ
 
01.07.19
10:59
2й вариант не подходит по критерию написания пбуличного кода
5 palsergeich
 
01.07.19
11:01
Оба варианта применимы на практике, зависит от внутренних правил.
Если их нет, то вариант комментирование того что было ранее и написание своего - предпочтительнее.
6 assasu
 
01.07.19
11:03
(5) в варианте 1 типовой код попадает в мои скобки // что требует доп. анализа при обновлении.
7 Cyberhawk
 
01.07.19
11:04
(5) Ага, особенно когда одну строчку надо вставить
8 Гипервизор
 
01.07.19
11:04
(2) Политика компании не использовать внешних отчетов, обработок, печатных форм? А вносить все изменения в типовой код?
Хотя да, очень удобно. Каждый раз при обновлении можно превозмогать и совершать подвиги.
9 catena
 
01.07.19
11:04
Вариант два не везде применим без потерь, первый единообразнее.
10 Cyberhawk
 
01.07.19
11:05
(8) Здравая политика, ибо после обновления конфигурации все что внешнее может перестать работать тупо не проходя синтакс-контроль
11 Cyberhawk
 
01.07.19
11:06
Если комментировать надо стараться так, чтобы в сравнении текстов в разрезе методов наглядны были видны изменения, то конечно же

комментируем новый код
12 assasu
 
01.07.19
11:07
(8) вообразите контору в которой число одновременно работающих программистов поболее ,чем в большинстве компаний, число одновременно работающих бухгалтеров.
13 palsergeich
 
01.07.19
11:07
(7) И что в этом страшного?
14 palsergeich
 
01.07.19
11:08
(12) Тогда нимучь мозги, а юзай git
15 lubitelxml
 
01.07.19
11:08
Желательно выносить в отдельные процедуры/функции, проще потом обновлятся

комментируем новый код
16 Cyberhawk
 
01.07.19
11:14
(13) Снижается наглядность вставки в диалоге сравнения текстов модулей