Время от времени натыкаюсь в сети на рекомендацию использовать константы с левой стороны операции сравнения.
К примеру вместо

следует писать

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

стоит привыкнуть.

ЗЫ сам даже в Java пишу так, а в AS3 например можно нарваться если пользоваться 1ым вариантом

chaos, вы за вариант: "константу слева" ?

semibug,  я за константу слева, что с уст моих коллег звучит как "писать сравнения в стиле Йоды"

Я раньше тоже писал как в первом варианте, потом пересилил себя и отвык. теперь пишу ТОЛЬКО как во втором, потому что возможные ошибки лучше убирать ( если это возможно, конечно ) на этапе компиляции, а не на этапе отладки или эксплуатации.

Вот ещё пример того, как уйти от ошибок на этапе компиляции:

лучше заменить на

чтобы если нужно заремить do_if_expr() do_anyway() не выполнялась по условию.
Есть ещё рекомендация использовать шаблоны вместо макросов ( там есть проверка типов )

semibug, не возражаешь, если я задам вопрос в твоей теме ?
Кто-нибудь знает ещё такие же trick'и для того, чтобы избегать ошибок на этапе компиляции ?

для этого коментится и if тоже
в моей ИДЕ (msvs+VisualAssistX) это делается одной кнопкой "/"

константа слева

константа справа

я никогда не замечал, чтобы в обсуждении алгоритмов люди говорили "и если 2 равно иксу, то...", а читабельность - немаловажное свойство кода

а для поисков ошибок нужно смотреть на предупреждения компилятора, при желании можно его даже в ошибку переделать

константа слева

а что скажет компилятор?:

boostcoder, MSVC даёт у меня С4706 (assignment within conditional expression), если включить Warning Level 4.

а если так?:

любопытно, на основании чего msvc принимает такое решение?
согласен, в первом варианте это логично предположить. а что компилятор скажет во втором случае?

В моей ИДЕ ( MSVS + VisualAssistX ) есть snippet

и потом, в expr может быть какое-то действие, например




если включить Warning Level 4 у меня на Qt начинает матом ругаться, поэтому ограничиваюсь 3-м.
Так всё-таки, по поводу ещё таких же ухищрений. Кто-нибудь знает такого рода "правила" ?

константа слева

Смотря какая IDE. Например FDT подчеркивает это и помечает на полях. Так что не заметить невозможно.

не только msvc.. 
Предполагается что в _условиях_ правильна лишь инициализация, но не присваивание.. 
т.е. если этот код вызовет предупреждение:

то этот вполне легален:


Это сообщение отредактировал(а) mes - 14.7.2010, 09:25

Не обращаю внимания на эту мелочь, пишу так, как с языка слетает (что с чем сравнивать). ИМХО проблема надуманая, но странно, что идею еще никто не довел до идиотизма

и отказу от == в коде.

Это сообщение отредактировал(а) djamshud - 14.7.2010, 09:25

как с языка слетает, зависит от привычки. У всех ( с опытом ) кто написал _слева_, именно так и слетает с языка 

константа справа.
Ошибался только в начале (когда после паскаля пересел на С), сейчас крайне редко (все случаи, что я помню, исправлял тут же). Думаю, потому что компилятор выдает предупреждение. Оно конечно раздражает, когда надо действительно использовать присваивание внутри условия (if (f = fopen(...))... ), но никто не мешает использовать еще одни скобки.

borisbn, такой ещё трик он же сюжет для холивара заметил - писать бесконечный цикл лучше



чем



Рекомендовано варнингом студии. Первое, что пришло в голову, в while можно по ошибке воткнуть переменную l вместо 1, а как на самом деле?

имхо, из за того что условие безусловное.

semibug, я использую Qt, а там есть для этого forever:

Это не холивар. Дело в том, что VC 6.0 при стандартных настройках оптимизации генерировал разный код.

При использовании while( 1 ) { __asm nop; } компилятор генерировал что то вроде:



А при использовании for( ;; ) { __asm nop; } было что то вроде:



(по памяти)

--
Но сей час проверил на VC 2008 -- там генерируется одинаковый код что для фор, что для ваил:



Так что по сути сей час разницы нету что использовать.
--
Ну это я описал с точки зрения оптимизации. А так сам использую for( ;; ) уже по привычке.

константа справа  

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

Это сообщение отредактировал(а) azesmcar - 19.7.2010, 10:22