Как лучше оформлять несколько условий в select where?

[gl00mie]

Уже столько всего сказано... я предпочитаю форматировать запросы так:

while select createdbBy, createdDateTime, RecId
    from    bankClientPayment_RU
    where   bankClientPayment_RU.RContractAccount
    notexists join RContractTable
    where       (   (   RContractTable.RContractPartnerType == RContractPartnerType::Cust
                    &&  BankClientPayment_RU.PartnerType    == ModuleCustVend::Cust 
                    )
                ||  (   RContractTable.RContractPartnerType == RContractPartnerType::Vend
                    &&  BankClientPayment_RU.PartnerType    == ModuleCustVend::Vend 
                    )
                )
            &&          RContractTable.RContractAccount     == bankClientPayment_RU.RContractAccount
            &&          RContractTable.RContractCode        == bankClientPayment_RU.RContractCode

С notexists join, конечно, засада - все отступы рушит, а так, если после where/join/from использовать табуляцию, получается все выровнено одинаково. А еще в сложных логических выражениях или в одиночных запросах select (если только они совсем уж не какие-нить тривиальные) предпочитаю точку с запятой ставить на новой строке - так, в случае чего, удобно новые условия добавлять.
Собственно, основной принцип, как уже отмечали тут, - это возможность править впоследствии код, затрагивая минимум исходных строк, в идеале - лишь добавляя новые строки, чтобы на сравнении они были бы четко отделены от исходного варианта кода. В этом отношении очень раздражает, что в стандартном функционале какой-нить вызов метода с тучей параметров бывает если и написан даже с каждым параметром на отдельной строке, то последний обязательно оказывается на одной строке с закрывающей скобкой. Ну <censored> просто, ни фига о других не думают...

[vanokh]

А в Best practice по Ax2009 хоть кто-нибудь заглядывал?
http://www.microsoft.com/download/en...ng=en&id=23084
рекомендуют исключительно в начале строки (до условия)...

[mazzy]

Цитата:

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

А в Best practice по Ax2009 хоть кто-нибудь заглядывал?

1. спасибо
2. вопрос то был о предпочтениях.
3. многие еще на трешке, там это не регламентировалось, но сами разработчики часто писали операторы ПОСЛЕ условий.

[vanokh]

Цитата:

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

2. вопрос то был о предпочтениях.

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

Цитата:

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

3. многие еще на трешке, там это не регламентировалось, но сами разработчики часто писали операторы ПОСЛЕ условий.

хм, и вот это странно - та chm'ка, которую я читал при обучении (Best Practice Handbook) и потом почти ни разу не открывал, говорит, что "the boolean operators (&&) are in the end of the line (and in columns)"

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

[mazzy]

Цитата:

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

хм, и вот это странно - та chm'ка, которую я читал при обучении (Best Practice Handbook) и потом почти ни разу не открывал, говорит, что "the boolean operators (&&) are in the end of the line (and in columns)"

Спасибо.
По ключевым словам нашел и для истории оставлю

ax3.0


ax2009

[kaw]

Цитата:

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

Wamr меня опередил. В точности так. С той поправкой, что "текущее ощущение прекрасного" как правило заставляет выровнять знаки равенства:

&& RContractTable.RContractAccount == bankClientPayment_RU.RContractAccount
&& RContractTable.RContractCode    == bankClientPayment_RU.RContractCode

Возможно, кому такое выравнивание кажется плохой привычкой.

А как "текущее ощущение прекрасного" сочетается со знаком "!" ?

&& ! RContractTable.RContractAccount

или

&&  RContractTable.RContractAccount == ''

[Gustav]

mazzy, а можно синхронизировать понятия "вариант 1" и "вариант 2" из стартового сообщения с понятиями "до условия" и "после условия" ? например, явно прописать "вар 1" и "вар 2" в пунктах голосования, а то сейчас мне кажется, что они перекрестились (наверное, торможу к вечеру и к концу недели).

Мой выбор будет:
where (...)
&& (...)
&& (...)
это значит в какой пункт надо ткнуть?

Спасибо.

[mazzy]

дописал.

Цитата:

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

это значит в какой пункт надо ткнуть?

вариант 2.

[Gustav]

Цитата:

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

Мой выбор будет:
where (...)
&& (...)
&& (...)

Одно время даже страдал таким выпендражем, как вставка в строку c where фиктивного условия типа where 1==1 или where true. В этом случае все реальные условия ниже единообразно начинались с && - было удобно комментировать условия в процессе отладки.

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

Цитата:

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

это значит в какой пункт надо ткнуть?

Вот и я тоже не понял, пока mazzy только что не разъяснил. Почему написано "после", когда оно "до"? Ну, в смысле, в одной строке "до". Хотя, если "после" - это после первого условия, то как тогда понимать "до"? Ведь до первого условия поставить ничего нельзя. Совсем запутался.

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

[Link]

Последние несколько лет пишу так, кажется более удобным для чтения.
Заметил, что некоторые участники форума придерживается такой же структуры.

        if (    (   prodLabelType   == KAB_ProdLabelType::FinalLabel
                ||  prodLabelType   == KAB_ProdLabelType::ControlledLabel)
            &&  confirmedQty    > 0)
        {

    while select salesTable
        index StatusCustAccIdx
        where salesTable.SalesStatus  == SalesStatus::Backorder
            join TableId from CustTable
                index AccountIdx
                where       CustTable.AccountNum    == salesTable.CustAccount
                        &&  CustTable.CustGroup     != 'KSS'
                join forupdate salesLine
                    index SalesLineIdx
                    where       salesLine.SalesId               == salesTable.SalesId
                            &&  salesLine.RemainSalesPhysical   > 0

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

Просто ради проверки моего предположения. Все ли голосовавшие поняли, что пункты голосования означали следующее:

Я предпочитаю указывать операторы до условия

where (...) &&
          (...) &&
          (...)

Я предпочитаю указывать операторы после условия

where (...)
          && (...)
          && (...)

[mazzy]

Цитата:

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

Просто ради проверки моего предположения. Все ли голосовавшие поняли, что пункты голосования означали следующее:

Я предпочитаю указывать операторы до условия

where (...) &&
          (...) &&
          (...)

Я предпочитаю указывать операторы после условия

where (...)
          && (...)
          && (...)

О_о!

понятно. закрываем тему, пересоздаем.
Как лучше оформлять несколько условий в select where? Повторная попытка

пожалуйста, проголосуйте заново.