Служебные строковые константы в метках

[Logger]

Разбираю код в 2009-й аксапте.
то и дело в разных местах встречаю примеры когда при построении запросов, при формировании строки в QueryBuildRange.value() используются метки.

Например вместо запятой ставят "@SYS35667", вместо '%1' тоже какую-то метку.
Это то зачем делать ?
BP поменяли ?

Мне казалось, что в метки надо выносить только текстовые строки, зависимые от языка. Во всех остальных случаях надо использовать константы или макроопределения. Иначе код становится хуже читаем и легко нарушить его работу кривой меткой. Метки должны только на интерфейс влиять, а не на логику работы.

[Maxim Gorbunov]

Вероятнее всего, сделано так из-за обычного непонимания сути BP. AX становится всё более массовым продуктом, армия программистов растёт. Логично, что качество кода (в том числе и стандартного) в связи с этим падает. Привыкайте.

[lev]

Цитата:

Сообщение от Logger

Разбираю код в 2009-й аксапте.
то и дело в разных местах встречаю примеры когда при построении запросов, при формировании строки в QueryBuildRange.value() используются метки.

Например вместо запятой ставят "@SYS35667", вместо '%1' тоже какую-то метку.
Это то зачем делать ?
BP поменяли ?

Мне казалось, что в метки надо выносить только текстовые строки, зависимые от языка. Во всех остальных случаях надо использовать константы или макроопределения. Иначе код становится хуже читаем и легко нарушить его работу кривой меткой. Метки должны только на интерфейс влиять, а не на логику работы.

Жуть какая...
Мне кажется это просто какой то либо не очень опытный, либо с какими то своими взглядами программист, которому когда сказали: "Везде в аксапте используй метки!"... Вот он их и стал везде использовать

[Ivanhoe]

Я бы не исключал возможности использования какого-нить инструмента, который везде заменяет строку на метку
Самим приходилось такое писать, когда клиенту "срочно" понадобилось перейти на разработку с метками.

[oip]

Всего лишь примеры для The daily Axapta WTF. Собственно EVGL тот топик начал с примера на по сути такую же тему. На русском это называется "Научи дурака богу молиться...".

Цитата:

Сообщение от EVGL

Из творчества программистов датской компании Munk IT:

void Munk_NoteMark()
{
    MenuFunction menuFunction;
    Args         args = new Args();
    ;
    menuFunction = new MenuFunction('@NEO258', MenuItemType::Display);
    args.caller(this);
    menuFunction.run(args);
}

@NEO258: DocuView

[Владимир Максимов]

Я думаю, что некоторую "помощь" в появление таких меток вносит именно что новая версия Best Practiсes, которая настоятельно рекомендует использовать двойные кавычки только для обрамления меток, но ни в коем случае не текстовых констант. Точнее, конечно, не сама по себе "рекомендация", а запускаемая проверка на соответствие рекомендациям.

Это надо еще "догадаться" заменить двойные кавычки на одинарные, чтобы проверка на соответствие рекомендациям не ругалась на такие места. Ведь текст рекомендации откровенно подталкивает к созданию метки.

Как мне кажется, здесь просто перестарались "проверяльщики". В смысле, авторы кода "Проверка на соответствие рекомендациям". Когда слишком хорошо - это уже не хорошо


PS: Метки сами по себе "ухудшают читаемость кода". Вне зависимости от содержания. Впрочем, это скорее вопрос привычки...

Кроме того, не вижу каким образом подобные метки могут повлиять на логику работы. Вы что, пишите код, который делает ветвление кода в зависимости от текста?

Как мне кажется, максимум, у Вас будет "кривой" текст чего-либо, но лигика работы от этого не пострадает никак.

[Logger]

Цитата:

Сообщение от Владимир Максимов

Кроме того, не вижу каким образом подобные метки могут повлиять на логику работы. Вы что, пишите код, который делает ветвление кода в зависимости от текста?

Я - не пишу Если вы внимательно прочитали тему

Если метки использовать в качестве разделителя "," или заменителя "%1" при наложении фильтра при построении кверей, то функциональность системы может нарушиться при переводе или при отсутствии меток.

А не должна.

P.S.
см. например :
\Classes\RLedgerSheetEngine_TurnoverBalance\originalDocuments

range.value(strfmt("@FAV2145", SysQuery::value(LedgerPostingType::TransferOpeningClosing)));

Почему бы не написать просто :

range.value(SysQuery::valueNot(LedgerPostingType::TransferOpeningClosing));

Если нет меточного файла или не так перевели или еще чего, то оборотка отваливается.

[Владимир Максимов]

А... Вот Вы про что! Да. Был не внимателен. Я так понял, Вы были возмущены именно содержимым метки, а не способом ее применения.

Между прочим, метод SysQuery::valueNot() написан точно также. Разве что, использованы одинарные кавычки и, как следствие, проверка на соответствие рекомендациям не настивает на том, что текст надо заменить меткой. Т.е. еще раз косвенно подтверждается "вина" слишком хорошей проверки.

Я так понимаю, что изначально была "тупо" написана текстовая константа в двойных кавычках, а когда запустили проверку на соответствие рекомендациям, то получили рекомендацию оформить текстовую константу как метку. Ну, и получили, то, что имеем

PS: Каким образом можно неправильно "перевести" запятую или восклицательный знак - не понимаю Как "потерять" меточный файл, тоже плохо представляю. Нет, теоретически, конечно, все возможно, только, по-моему, проблема несколько надуманная .