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

[mazzy]

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

Прошу переголосовать с измененными формулировками.

В коде часто встречаются select c несколькими условиями (условными выражениями).
Как лучше оформлять несколько условий в select where?

Вариант 1. (операторы после условия)

    select firstonly paymentTrans
        where paymentTrans.AccountNum            == factureJour.CustVendInvoiceAccount &&
              paymentTrans.Voucher               == factureJour.Voucher                &&
              paymentTrans.CurrencyCode          == factureJour.CurrencyCode           &&
              paymentTrans.PrepaymentFactureId_W == factureJour.FactureId              &&
              paymentTrans.Prepayment            == NoYes::Yes;

Вариант 2. (операторы до условия)

    select firstonly paymentTrans
        where paymentTrans.AccountNum            == factureJour.CustVendInvoiceAccount
           && paymentTrans.Voucher               == factureJour.Voucher
           && paymentTrans.CurrencyCode          == factureJour.CurrencyCode
           && paymentTrans.PrepaymentFactureId_W == factureJour.FactureId
           && paymentTrans.Prepayment            == NoYes::Yes;

Как вы предпочитаете ставить операторы между условиями?
Перед условием или после условия?
Почему?

А в более сложны случаях? Например,

    while select 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

Добавлено: понятно, что при правке существующих select'ов скорее всего будете оставлять так как есть. Вопрос как вы предпочитаете, когда сами создаете select'ы

[mazzy]

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

по наводке vanokh
процитирую Best Practice

ax3.0


ax2009

[gl00mie]

Цитата:

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

Как лучше оформлять несколько условий в select where?
Вариант 1. (операторы после условия)
Вариант 2. (операторы до условия)
Как вы предпочитаете ставить операторы между условиями?

Вспомнилась что-то классика:

Цитата:

- Вы хотите курить, как я вижу? - неожиданно обратился к Бездомному неизвестный, - вы какие предпочитаете?
- А у вас разные, что ли, есть? - мрачно спросил поэт, у которого папиросы кончились.
- Какие предпочитаете? - повторил неизвестный.
- Ну, "Нашу марку", - злобно ответил Бездомный.
Незнакомец немедленно вытащил из кармана портсигар и предложил его Бездомному:
- "Наша марка".

Возвращаясь к нашим баранам, в связи с чем, собственно, возник вопрос?

[mazzy]

Цитата:

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

Вспомнилась что-то классика:Возвращаясь к нашим баранам, в связи с чем, собственно, возник вопрос?

ни с чем. давно хотел спросить. правил очередной запрос - спросил

[Alexanderis.ua]

Операторы до условия.
В самом начале знакомства с аксаптой увидел такую запись. Мне показалась более наглядной.
Теперь уже привык и не задумываюсь.
Со сложными условиями примерно так.

    while select bankClientPayment_RU
           where bankClientPayment_RU.RContractAccount
       notexists join RContractTable
           where (  (  RConractTable.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

Видел, что некоторые еще и скобки на разные строки разносят. Наверно так даже лучше, но не привык и как-то пока не хочу переучиваться

[kvan]

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

select firstonly paymentTrans
where paymentTrans.AccountNum == factureJour.CustVendInvoiceAccount
   && paymentTrans.Voucher    == factureJour.Voucher
   && paymentTrans.Prepayment == NoYes::Yes
   && (paymentTrans.CurrencyCode == "UAH" ||
       paymentTrans.CurrencyCode == "USD");

[IKA]

Ставлю "До", тк если нужно строку потом закомментировать, то проще и аккуратней поставить "//" в начале строки, чем комметировать оператор строкой выше, а потом комментировать строку условия

[Raven Melancholic]

Тоже пишу до условных выражений. Причина та же, что и у других - легкость добавления и комментирования. Кроме того, люблю выравнивать части кода, а если делать операторы в конце строки, то выравнивание не такое красивое, особенно, если строка длинная - бывает, что на небольшом мониторе просто не видно как связаны условия - операторы выходят слишком далеко вправо. Если же они слева, то сразу видно как связаны строки оператора друг с другом.

[Daiver]

Указываю всегда после. Оно может не очень удобно, но БестПрактикс. Если каждый будет лепить в коде так как ему удобно, то сопровождать такой код будет совсем не весело.

Кстати, можно устроить еще один опрос, В какой степени программеры Акс следуют рекомендациям БестПрактикс?

[gl00mie]

Цитата:

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

Указываю всегда после. Оно может не очень удобно, но БестПрактикс.

ага, только устаревший годков эдак на несколько...

Цитата:

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

Если каждый будет лепить в коде так как ему удобно, то сопровождать такой код будет совсем не весело.

Кто бы спорил, однако, это, по-моему, не самое страшное, что может случиться по ходу сопровождения чужого кода. Вот тот же <CENSORED> <BEEP> <CENSORED> copy-paste - он куда <CENSORED> %$ТЬ <CENSORED> больше э... выводит из состояния душевного равновесия, нежели положение амперсандов в условиях where.

Цитата:

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

Кстати, можно устроить еще один опрос, В какой степени программеры Акс следуют рекомендациям БестПрактикс?

О-о-о, да-а-а-а!.. (с) м/ф "Гадкий я". Например, кто перевел весь свой код на использование шаблона "инкапсулированный конструктор"? Если бы локализаторы потрудились это сделать хотя бы с классом ComExcelDocument_RU, не говоря уже про остальные, мне бы, к примеру, не пришлось парить себе мозг тем, что делать со 100+ местами в коде, где экземпляры этого класса создаются через new, в попытках перевести его на рельсы .NET. Вот с тем же семейством SysExcel* в этом плане все просто: взял и подменил в construct() возвращаемый класс в полном соответствии с принципом подстановки Лисков, ну там еще парочку десятков мест подправил, где вся инкапсуляция рушится из-за прямой работой с Excel через COM (см. также Ошибка времени выполнения в ComExcelDocument_RU.findRange()).

[mazzy]

соотношение не слишком изменилось