довольно часто замечал запись вида:

причем в совершенно различных источниках
зачем так? по идее ведь anotherVariable=someConditionalExpression и красивее и эффективнее?

первый вариант:


второй вариант:


Это сообщение отредактировал(а) nmn - 8.1.2009, 19:19

Не понял, а что не так? Если a=b, то c=истина, иначе c=ложь.  

nmn, первый вариант просто читается легче...

Гм. Я привык, что первый вариант иначе как индусским кодом не называют.

Void, окей...сравним два варианта:

1)


2)

второй лучше ? 

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

nmn, вы абсолютно правы, вторая запись "красивее и эффективнее" и понятнее.

Partizan, скажите пожалуйста что первично в приведённом вами отрывке: установка свойства или проверка условия? Отражает ли условие в данном месте реальное ветвление алгоритма?

Я ничего против первого варианта не имею, но в его отношении прав Void 

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

второй вариант

visible = о.О...

конечно и то и другое - читается...я говорю о том, что первый вариант - более очевиден =)

Bishop,  против второго также ничего не имею =) Про красоту и эффективность даже не спорю =) Тут в общем то и спорить не о чем)

Поддерживаю Partizan-а. Первая запись читабельнее. А еще она позволяет легко поставить отдельные breakpoint-ы на каждую ветку, например, поймать момент когда this.Visible выставляется в false и пропустить его в дебаге.

Это сообщение отредактировал(а) PashaPash - 8.1.2009, 19:11

чем эффективнее?

Обычно код нужно поддерживать. Если это трудно делать (как во втором случае), то его красота сомнительна.

ну если только ради брейпоинта подстраивать код, по моему это неразумно
ведь существует такое понятие как условный брейкпоинт, вот туда эти условия и записать

Сравни тык по F9 с выделением условия и вписыванием его в нужное поле (про которое тут 50% народа не догадывается). А вообще 1-й вариант читабельнее потому что он именно "читается" как "если срока непустая и blah-blah-blah то сделать видимым, иначе - невидимым". А первый вариант - "установить видимость непустоте строки или истинности blah-blah-blah". В спеках и вообще при описании поведения обычно используют первую формулировку. Первый вариант легче дописывается до случая когда нужно сделать еще что-то только в одной ветке, например:


З.Ы. я проголосовал за 2, тупо промахнулся 

Это сообщение отредактировал(а) PashaPash - 9.1.2009, 01:01

Partizan, ты уж извини, но ты чушь написал. Первый вариант как раз таки скрывает суть происходящего. Одной булевой перемонной (anotherVariable) присваивается значение другой (someConditionalExpression). Что и написано во втором варианте. If statement нужен для конструкций вида: 

где ... - есть некий набор инструкций.
 

Если не секрет, что ты в приведённом пример отлаживать собрался? a = b?   


Void, поддерживаю!  Типичный примерчик индусского кода.

Добавлено через 1 минуту и 35 секунд
PashaPash, вот сейчас ты и подтвердил мои слова. Не о том немного думаете. Взгляните внимательнее на обсуждаемый код.

Это сообщение отредактировал(а) kemiisto - 9.1.2009, 01:01

kemiisto, всё же определённая логика в этом есть, когда речь идёт о свойствах. component.Visible = booleanVar — это не просто присвоение, но и сопутствующие действия. Кстати, если считать значение этого свойства на следующей строчке, оно не обязательно будет равно только что присвоенному.