|
Придираются ли на экзамене 1С:Специалист к стилю кода? | ☑ | ||
---|---|---|---|---|
0
DSblizzard
22.09.11
✎
09:00
|
Например, к такому стилю именования переменных:
Др0 = новый ДанныеРасшифровкиКомпоновкиДанных; Км0 = новый КомпоновщикМакетаКомпоновкиДанных; Макет0 = Км0.Выполнить(СКД, Настройки, Др0); Пкд0 = новый ПроцессорКомпоновкиДанных; Пкд0.Инициализировать(Макет); |
|||
1
Axel2009
22.09.11
✎
09:02
|
неуд
|
|||
2
IamAlexy
22.09.11
✎
09:03
|
___ = новый ДанныеРасшифровкиКомпоновкиДанных;
_____ = новый КомпоновщикМакетаКомпоновкиДанных; ______ = _____.Выполнить(СКД, Настройки, ___); ____ = новый ПроцессорКомпоновкиДанных; ____.Инициализировать(Макет); |
|||
3
DSblizzard
22.09.11
✎
09:04
|
(2) Можно и так, главное, чтобы быстро было :)))
|
|||
4
tertva_1C
22.09.11
✎
09:05
|
Пойми что специфика 1С такова, что помимо тебя этот код могут смотреть другие и поверь если в 99% красивом коде встречается твой 1% да еще с такими названиями то согласись это портит все....
|
|||
5
DSblizzard
22.09.11
✎
09:06
|
(4) Я только про экзамен, не факт, что буду так писать на работе.
|
|||
6
Axel2009
22.09.11
✎
09:08
|
(5) факт на 99%
|
|||
7
verba
22.09.11
✎
09:09
|
Смотрят на все, но оценивают не это.
|
|||
8
krbIso
22.09.11
✎
09:09
|
не факт что ты не будешь так писать на работе
|
|||
9
YV
22.09.11
✎
09:09
|
(0) К стилю - нет. К оптимизации - очень. Например мне сделали замечание за то, что результат запроса выгружал в таблицу а не обходил в выборке. Дескать больше ресурсов нужно.
|
|||
10
Axel2009
22.09.11
✎
09:09
|
(2) и ты ошибку скопировал..
|
|||
11
DSblizzard
22.09.11
✎
09:10
|
(10) Да-да, в последней строке.
|
|||
12
DSblizzard
22.09.11
✎
09:16
|
(4) только что спросил про сокращение названий в рабочем коде у программиста, который дал мне задание: "ты можешь делать все, главное, чтобы работало".
|
|||
13
IamAlexy
22.09.11
✎
09:16
|
(10) плевать
|
|||
14
mzelensky
22.09.11
✎
09:20
|
(0) а мне кажется лучше писать кратко (разумеется кратко, но понятно), чем делать переменные-предложения, как это обычно в типовых, типа:
"ТекущаяСтрокаТаблицыЗначенияСотрудники" !!! |
|||
15
DSblizzard
22.09.11
✎
09:23
|
(14) Согласен. К тому же, в других ЯП это считается хорошим стилем. По крайней мере, для именования локальных переменных.
|
|||
16
unregistered
22.09.11
✎
09:23
|
(14) Проблема в том, что в 90% случаев сокращения остаются понятными только, пока ты этот код пишешь.
А когда пересматриваешь его хотя бы месяц спустя, то большая часть сокращений превращаются в тарабарщину. (0) Надеюсь к тебе придерутся. Без обид. Просто бесит иногда ховнокод. |
|||
17
Stim213
22.09.11
✎
09:24
|
ДанныеРасшифровкиКомпоновкиДанныхОтчета = новый ДанныеРасшифровкиКомпоновкиДанных;
КомпоновщикМакетаКомпоновкиДанныхОтчета = новый КомпоновщикМакетаКомпоновкиДанных; МакетОтчета = КомпоновщикМакетаКомпоновкиДанныхОтчета.Выполнить(СКД, Настройки, Др0); ПроцессорКомпоновкиДанныхОтчета = новый ПроцессорКомпоновкиДанных; ПроцессорКомпоновкиДанныхОтчета.Инициализировать(МакетОтчета); |
|||
18
Рэйв
22.09.11
✎
09:25
|
(14)+1
(16)На это дело есть коменты. |
|||
19
mzelensky
22.09.11
✎
09:28
|
(16) если этот код буду читать Я (тот, кто писал), то и через год не составит труда в нем разобраться, т.к. я подписываю переменные в понятном ДЛЯ СЕБЯ ключе, чтобы интуитивно было понятно.
Если же туда садится другйо прогер...ну тут уже зависит от его сообразительности + наличия коментов! |
|||
20
KRV
22.09.11
✎
09:30
|
(17) МакетОтчета = КомпоновщикМакетаКомпоновкиДанныхОтчета.Выполнить(СКД, Настройки, Др0); Гыыы
|
|||
21
Stim213
22.09.11
✎
09:32
|
(20) не сдал..?
|
|||
22
unregistered
22.09.11
✎
09:33
|
(18) >> На это дело есть коменты
Ага. У тебя в модуле 20 переменных. В каментах каждая из них описана. Перед чтением и разбором модуля будешь заучивать расшифровки имен переменных? А если комментировать каждую строку только ради того, чтобы потом понимать, что МакО это МакетОтчета, то может проще сразу дать нормальное имя переменной? PS Конечно необходим разумный подход. Имена типа ПроцессорВыводаРезультатаКомпоновкиДанныхВТабличныйДокументИмениНуралиеваБорисаГеоргиевича наверное тоже будут перегибом :) |
|||
23
DSblizzard
22.09.11
✎
09:33
|
(20) Все измазались :)
|
|||
24
Jolly Roger
22.09.11
✎
09:38
|
(0) еще имена локальных переменных сделать с маленькой буквы и будет нормально...
|
Форум | Правила | Описание | Объявления | Секции | Поиск | Книга знаний | Вики-миста |