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

[lev]

на прошлом месте работы, я писал операторы до условия:

while select inventTable
    where inventTable.ItemId      == '1'
        && inventTable.ItemName == '2'

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

while select inventTable
    where inventTable.ItemId      == '1'
        //&& inventTable.ItemName == '2' // удобно

while select inventTable
    where inventTable.ItemId      == '1'  //&& // неудобно
              //inventTable.ItemName == '2'

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

P.S. если говорить кратко, то сейчас пользуюсь первым вариантом (по привычке), но считаю более удобным второй (почему написал выше).
Вот такой вот, парадокс

[pitersky]

Цитата:

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

в итоге получил ответ, что так требует BP и что так красивее (кстати понятие красоты у всех разное ).

Насчёт ВР я тоже слышал - мол МС в сертификационных тестах разработчиков по 2009 в обязательном порядке требует условие справа. Хз правда или нет...

Насчёт красоты - всё-таки эстетичнее, когда границы текста ровные и справа, и слева. Не зря же текстовые редакторы умеют выравнивать текст

[lev]

Цитата:

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

Насчёт красоты - всё-таки эстетичнее, когда границы текста ровные и справа, и слева. Не зря же текстовые редакторы умеют выравнивать текст

Ага, люблю когда все выровнено и слева и справа и посередине

[Zabr]

Вы удивитесь, но я предпочитаю ставить && слева, а || справа. (Если кто-то хочет пофлудить, то это отличная тема для флуда ). Аргументы:
- && слева - не требует дополнительного выравнивания
- && слева - удобнее вставлять, убирать и комментировать условия
- && слева, а || справа - улучшает восприятие, и уменьшает вероятность ошибки при последующем возможном изменении условия (!). Знаков && как правило много, а знак || как правило один, поэтому проблем с выравниванием с ним не возникает..

Если пишу сам с нуля - ставлю слева. Если модифицирую код, где стоят справа - то не меняю (ну разве что если условие сильно запутанное или я его сильно меняю).

А еще я предпочитаю переносить "точку с запятой" отдельно на последнюю строку. Хотя иногда не делаю от лени. Аргументы для переноса:
- удобнее вставлять дополнительную строку в условие
- удобнее убирать (удалять или комментировать) последнюю строку в условии
- дополнительная строка между этим и следующим оператором, содержашая только одну точку с запятой, улучшает визуальное восприятие кода

[mazzy]

Цитата:

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

Вы удивитесь, но я предпочитаю ставить && слева, а || справа.

а можно попросить отформатировать третий пример из первого в этой ветке сообщения.
любопытно было бы посмотреть

[Zabr]

Цитата:

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

а можно попросить отформатировать третий пример из первого в этой ветке сообщения. любопытно было бы посмотреть

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

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

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

if (X.value()) 
     Y = PurchStatus::Invoiced;
else Y = PurchStatus::Received;

[kaw]

Цитата:

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

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

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

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

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

&& ! RContractTable.RContractAccount

или

&&  RContractTable.RContractAccount == ''