FrankieПрекрасно понимаю, что путь к файлу шаблона обязатально вынести в const.
Обязательно - это перебор. Например, часто его стоит вынести в настройки.

FrankieНо не понимаю, зачем поля отчёта (%Date%, %CustomerName% и т.п.) выносить в const.
Во втором случае мне видится константа на константу, названия полей в шаблоне это что-то вроде имён таблиц и колонок в БД, а они-то в коде лежат значениями.
Ну, вообще говоря, "имена колонок и таблиц" тоже могут лежать константами :)

Думаю, здесь нет единой точки зрения, вопрос в целесообразности (и ее понимании конкретным разработчиком).

FrankieЯ к тому, что в код писать значение пути никак нельзя :)
Не факт. Скажем, в следующем коде


лично я криминала не вижу. Его конечно можно переписать, например, так:


но объяснение, чем это вариант лучше оригинала, скорее всего изрядно позабавит :)
FrankieИ я так думаю, а меня отправляют читать книжки по программированию!
Есть хорошая пословица про программиста, лоб которого подвергается повышенной опасности.... Лично я бы в таком случае отловил бы в коде крикунов фрагменты типа


и заставил бы менять на

FrankieПросто чтобы понять что такое LISTREP_NAMEORG надо, как говорится, контрол-клик сделать, а увидев '%NameOrg%', я бы сразу понял о чём речь...
Это соображение на самом деле не значимо, вопрос договоренностей. Стоит "привыкнуть пользоваться константами", как "сразу понял о чем" будет выполняться.

Другой вопрос - зачем эта константа. В первую очередь: спасет ли она, если потребуется поменять %NameOrg% на %NameOrgMy%? Имхо - за код


cледует однозначно отрывать яйца. Изменение значения однозначно влечет необходимость переименования константы. Далее, имя этой константы как минимум не короче оригинала, читаемость тоже не повышается, как в случае длинных текстовых констант. Единственное преимущество: если вдруг эта ячейка будет переименована, и если константа будет переименована вслед за ней, и если она используется в нескольких местах, то "прочие" места перестанут компилиться. С этим преимуществом связан и недостаток: если константа внедрена неудачно (скажем, в двух шаблонах были одинаково названные поля, использована одна константа, теперь мы переименовываем ячейку в одном из шаблонов, а в другом оставляем как есть) - все, мы самым что ни на есть незаметным образом сломали код. Можно быть почти уверенным, что исправленная версия пройдет тестирование (никто не будет запускать глобальное тестирование из-за такой мелочи), дойдет до пользователя и у него отвалится функциональность.

Да, это преимущество есть, вместе с недостатком. Но имхо - изредка нажать Ctrl-F куда выгоднее, нежели постоянно иметь геморрой с константами.