Удобство чтения кода - залог и определенная гарантия его работоспособности

[RVS]

Пять копеек:

- налицо уход от темы, КМК. Макросы - это, конечно, здорово, "принцип разумной достаточности" при их применении - безусловно, должен соблюдаться. И InventDimJoin и код, построенный целиком на макросах, приведенный выше, безусловно, все же разные вещи , но речь изначально была не о том.
- "Программы пишутся для людей" (откуда? ). В этом смысле код, написанный "в стиле системы", безусловно, предпочтителен. Как в армии - "безобразно, но единообразно". Иначе (в пределе) получается вариант бессмертных "лебедя, рака и щуки".
- В принципе, все уже сказано в заголовке темы. Думаю, мало кому захочется это оспаривать, и еще меньше тех, у кого это получится.

[imir]

Цитата:

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

не знаю. мне лично #InventDimJoin читать намного удобнее, чем многоэтажный прямой джойн инвентдима. особенно когда он стопиццот раз повторяется в разных местах.
нужно соблюдать принцип разумной достаточности, а не бросаться в крайности

Его легко переделать на query, динамически собираемый в коде, просто кто-то один раз связался с этой конструкцией и тащится она с 2.5-й версии

[pitersky]

Цитата:

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

Его легко переделать на query, динамически собираемый в коде, просто кто-то один раз связался с этой конструкцией и тащится она с 2.5-й версии

переделать легко, только читаться такой Query будет ещё тяжелее, чем джойн
да и нет смысла писать отдельный Query для ситуации, в которой тридцать раз до тебя уже использовали макрос (это к слову о единообразии).

[Zabr]

Цитата:

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

мне лично #InventDimJoin читать намного удобнее, чем многоэтажный прямой джойн инвентдима.

Если в этом запросе должна была быть другая табличная переменная для InventDim, а InventDim тоже есть но используется в другом месте, а кто-то по привычке зафигачил в запрос #InventDimJoin, то результат представить не трудно. Причем замучаетесь искать, в чем проблема. Так что макросы лучше только для констант использовать.

[ice]

Цитата:

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

Если в этом запросе должна была быть другая табличная переменная для InventDim, а InventDim тоже есть но используется в другом месте, а кто-то по привычке зафигачил в запрос #InventDimJoin, то результат представить не трудно. Причем замучаетесь искать, в чем проблема. Так что макросы лучше только для констант использовать.

вообще-то в макрос передаются имена переменных

[pitersky]

Цитата:

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

Если в этом запросе должна была быть другая табличная переменная для InventDim, а InventDim тоже есть но используется в другом месте, а кто-то по привычке зафигачил в запрос #InventDimJoin, то результат представить не трудно. Причем замучаетесь искать, в чем проблема. Так что макросы лучше только для констант использовать.

На мой взгляд, это не самая большая проблема, причём я не совсем понимаю трудности с локализацией ошибки - по-моему, это как раз отыскивается достаточно быстро. Большая проблема - это когда макросы регулярно создаются для разовых вставок или когда в макрос забабахивают сильно параметризуемую логику (это да, сложно и неудобно поддерживать). #InventDimJoin - макрос достаточно прозрачный и простой в применении, поэтому в его использовании я не вижу ничего фундаментально плохого (особенно тогда, когда это часть стиля приложения).
Словом, везде нужна мера и разумность.

[lev]

Кстати, вспомнил удобный макрос #InventDimDevelop, только почему то именно про него частенько забывают, и не добавляют там где надо...

P.S. интересно, в AX2012 появился макрос #FinancialDimDevelop?

[pitersky]

Цитата:

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

Кстати, вспомнил удобный макрос #InventDimDevelop, только почему то именно про него частенько забывают, и не добавляют там где надо...

P.S. интересно, в AX2012 появился макрос #FinancialDimDevelop?

В 2012 такого пока нету
а #InventDimDevelop там пустой. так что может и хорошо, что его активно не использовали

[RVS]

Цитата:

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

#InventDimDevelop там пустой. так что может и хорошо, что его активно не использовали

InventDimDevelop : /* used to locate code with direct dimension references */

Он всегда был "пустой"..

===

Полезен при добавлении Dimension-а. Только вот не везде его MSBS аккуратно ставил. Началось с добавления ГТД, уже там были "пропуски". В 2012-й, как я понял, "пропусков" будет больше.. жаль..

[Romb]

Придерживаюсь следующих убеждений в большинстве случаев.

Если макрос можно заменить inline функцией с похожим или даже идентичным кодом, значит его использование лишнее.

И наоборот.

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