if (a == true) - DAX: Программирование

**fed** · 23.05.2012, 10:23

Всегда когда вижу в коде (на любом языке), попытки сравнения логических переменных, у меня возникают сильные сомнения в минимальном профессионализме человека, который писал код. (Неужели автор кода смог научится программировать, но ни разу не прочитал пятистраничную статейку про логические операции ?)
Сравнивать логическую переменную с константой - бессмысленно.
Сравнивать две логических переменных можно, но стилистически не красиво. То есть - если возникло желание сравнить две логических переменных, скорее всего есть некая ошибка в подходе к программированию, поскольку при правильном построении алгоритма, эти две переменные надо было бы как-то по другому обрабатывать (типа if (a&&b) или if(a&&!b).

Тем не менее, каждая новая версия Аксапты содержит все больше и больше попыток сравнения переменной с логической константой. Например в 2009ой поиск в AOD по строке "== true" дает 461 совпадение. А в 2012ой - уже 700.
Причем, что самое любопытное, такие сравнения случаются не в каких-то второстепенных отчетах или формах, которые могут на откуп аутсорсерам и контрактерам отдавать. Такие сравнения можно увидеть, например, в классах ReqCalc, inventAdjustPost, LedgerJournalCheckPost. Я уверен что эти ключевые классы пишутся полноценными штатными разработчиками, причем не в Штатах, а в MDCC.

Я вот много раз слышал, что мол при наборе разработчиков в Микрософт, проверяют не знание аксапты, а типа общую программистскую культуру и знание технологий разработки. Я не верю, что человек смог прочитать книжки про OOD и Agile Development, но так и не научился оператор if нормально писать. Возникает подозрение, что авторы этого кода не только Аксапту не знают, но и вообще программировать не умеют...

Вот я и пытаюсь понять - это типа теперь такой best practice ?

macklakov · 23.05.2012, 10:42

Да в 2012 есть масса перлов. Есть новые, а есть и бережно переносимые из версии в версию.

Цитата:

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

Неужели автор кода смог научится программировать, но ни разу не прочитал пятистраничную статейку про логические операции ?

Можете кинуть ссылку на статейку? Хочется почитать, для общего развития.

**fed** · 23.05.2012, 10:52

Цитата:

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

Можете кинуть ссылку на статейку? Хочется почитать, для общего развития.

Ну я кажется такую статейку читал в Науке и Жизни году в 86ом. Но я думаю можно что-то популярное на хабре или в той же википедии найти.

macklakov · 23.05.2012, 11:03

Цитата:

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

Ну я кажется такую статейку читал в Науке и Жизни году в 86ом. Но я думаю можно что-то популярное на хабре или в той же википедии найти.

А, ну букварь я уже читал.

driller · 23.05.2012, 11:20

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

PHP код:


			
 IF 1 = 2.

    MESSAGE e044(fmup).

 ENDIF.

И ничего, повсеместно используемая фича ABAP

SRF · 23.05.2012, 12:01

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

Но к примеру на JavaScript, можно просто завести переменную, не инициализировать ее и фактически будет три значения true, false, null поэтому для упрощения можно использовать сравнение с какой либо константой, отметая два других значения, например, != true.

X++:

var flagTest;
//..
flagTest=$("CheckBoxTest").selected;
//...

if(flagTest !=true) {

}

И еще в C# существует хитрый тип bool? - логический с null(bool + null), так вот такая рода конструкция, в C# просто не откомпилируется, потребует явного приведения типа

X++:

bool? test = null;

if (test)
{ 
            
}

тогда как

X++:

bool? test = null;

if (test == true)
{ 
            
}

не вызовет никаких нареканий, примечательно, что такое неявное преобразование работает насколько я понимаю только для типа bool?, например если во втором случае написать int, то ничего не прокатит

S.Kuskov · 23.05.2012, 10:48

Ещё очень умиляют конструкции типа

X++:

a = (< >) ? true : false;

db · 23.05.2012, 11:48

Цитата:

Сообщение от S.Kuskov

Ещё очень умиляют конструкции типа

X++:

a = (< >) ? true : false;

тут сразу вспомнилось вот это обсуждение в похожей теме Проверка на RecId

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

**oip** · 23.05.2012, 10:57

Если я вижу у кого-то код вида

X++:

boolean bool;
...
if (bool == true)
...

то в таких случаях говорю, что тип выражения "bool == true" - тоже boolean, а значит если следовать такой логике, то надо писать "if ((bool == true) == true)"... и так далее.

PS А в конце, для надежности, можно добавить "!= false".

Владимир Максимов · 23.05.2012, 12:00

Честно говоря, не вижу особых проблем. Ну, сравнивают с логической константой и что? Для меня, например, так нагляднее, чем использование восклицательного знака, да еще без пробела. Попробуй еще разгляди его! А так, никакой неоднозначности и не надо "с лупой" искать этот восклицательный знак.

В конструкции вида

a = (<логическое условие>) ? true : false;

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

Link · 23.05.2012, 17:25

Цитата:

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

Честно говоря, не вижу особых проблем. Ну, сравнивают с логической константой и что? Для меня, например, так нагляднее, чем использование восклицательного знака, да еще без пробела. Попробуй еще разгляди его! А так, никакой неоднозначности и не надо "с лупой" искать этот восклицательный знак.

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

**fed** · 23.05.2012, 15:37

Ну я бы сказал, что if(a==true) пишут от непонимания и не умения программировать, а switch(true) пишут как раз таки от хорошего понимания того как оператор switch работает и приличного опыта разработки

Вариант switch(true) изрядно повышает читаемость кода из за отсутствия большого количества вложенных if else...

Владимир Максимов · 23.05.2012, 16:08

Цитата:

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

Ну я бы сказал, что if(a==true) пишут от непонимания и не умения программировать, а switch(true) пишут как раз таки от хорошего понимания того как оператор switch работает и приличного опыта разработки

Вариант switch(true) изрядно повышает читаемость кода из за отсутствия большого количества вложенных if else...

Довольно спорное утверждение. Можно ведь и так написать

X++:

ok = true;

if (ok && ...)
{
   ok = false;
   ...
}

if (ok && ...)
{
   ok = false;
   ...
}

Никаких вложенных if(), конструкция довольно "читабельна" и нет опоры на "фичу" со switch(true). Фича, она фича и есть. Лично я считаю что использование подобных фич - это, скорее, показатель НЕ профессионализма. Банальная лень искать штатные инструменты без опоры на фичи.

Raven Melancholic · 23.05.2012, 16:13

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

X++:

if (_ledgerJournalTrans_Project.CostPrice < 0)
            _ledgerJournalTrans_Project.CostPrice = - _ledgerJournalTrans_Project.CostPrice;

Вместо abs(...)
В одних случаях с нулем сравнивается прямо (что, на мой взгляд более понятно), в других полагаются на приведение числа к логическому типу:

X++:

_ledgerJournalTrans_Project.Qty == 0

X++:

!_ledgerJournalTrans.AmountCurDebit

Raven Melancholic · 23.05.2012, 16:27

Если уж быть совсем занудой, то можно вспомнить теорему Джекопини (точное название не помню), смысл которой в том, что любой алгоритм можно представить всего тремя структурами.

**fed** · 23.05.2012, 16:33

Цитата:

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

Довольно спорное утверждение. Можно ведь и так написать

X++:

ok = true;

if (ok && ...)
{
   ok = false;
   ...
}

if (ok && ...)
{
   ok = false;
   ...
}

Ну лично я, протрассировав такой код решил бы что случилась какая-то засада, поскольку Ok = false. Можно конечно переменную поаккуратнее назвать - типа TryNextBranch. Но тогда как-то уж очень громоздко выглядит. Не лучше чем switch(true)
В идеале - нужно было бы добавить в X++ оператор elif или elseif, который бы позволял такие конструкции без вложености писать. Но раз такого оператора нету, остается использовать разные по степени кривости заменители...

S.Kuskov · 23.05.2012, 16:43

Цитата:

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

это, скорее, показатель НЕ профессионализма.

Не согласен. "Фича со switch(true)" не является какой-то лазейкой или брешью в компиляторе. Эта "фича" является прямым следствием возможности использовать в операторе Сase не только константы уровня компиляции, но и динамические структуры. Это преимущество, о котором нужно знать и использовать, а не стыдиться его

Link · 23.05.2012, 17:20

Цитата:

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

Сравнивать логическую переменную с константой - бессмысленно.

Я так пишу. Не вижу криминала. Считаю это более очевидным, сразу видно, что это булевая переменная.

X++:

    boolean                         b;
    Str                             s;
    NoYesId                         n;
;
    if (    b == true
        &&  n == NoYes::Yes
        &&  s != '')
    {

    }
    
    if (    b
        &&  n
        &&  s)
    {
    
    }

**Maxim Gorbunov** · 24.05.2012, 14:35

Цитата:

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

Сравнивать две логических переменных можно, но стилистически не красиво.

А что здесь некрасивого? По сути, сравнение двух логических переменных - это XOR (исключающее "или").

**Maxim Gorbunov** · 24.05.2012, 14:49

А в целом согласен с fed. Качество кода, производимого Майкрософт, стремительно падает от версии к версии. К сожалению.

* это исключительно моё личное мнение

Похожие темы
Тема	Автор	Раздел	Ответов	Посл. сообщение
Бага в Query update(true)	Alexius	DAX: Программирование	5	14.09.2011 14:09
Не срабатывает skipDatabaseLog(true)	jaran	DAX: Программирование	14	09.04.2011 13:22
visible(true) и курсор	mvf	DAX: Программирование	6	20.07.2005 10:09
recordLevelSecurity(true)	sassas	DAX: Программирование	12	23.12.2004 16:44