В очередной раз столкнулся с этой темой на форуме, решил провести опрос.
Тех, кто не пишет на этих языках, прошу выбирать соответствующий пункт и в остальной части опроса не участвовать.
Существует, в целом, три варианта отношения к конструкциям вида if (a=f). Первый — конструкция полностью нормальная, нет причин на неё гнать. Второй — опасения спутать = и == и на этом основании объявление данной конструкции вредной. Третий (почему-то про него вспоминают реже чем про первые два) — заявления о том, что не-булевы выражения (речь тут не конкретно про присваивание) в качестве условия if вообще так или иначе не совсем нормальная ситуация.
Часто сторонники второго варианта начинают потом писать про опечатки «которые у всех бывают», но я считаю нужным данную ситуацию прояснить: следует чётко отличать спутывание = и == по причине забывания как в Си пишется сравнение (в этом случае проблемой будет только =/== и ни что другое, и происходит такое, в первую очередь, у сильно неопытных программистов), и опечатки по причине, условно, нажимания не тех кнопок на клавиатуре — такое действительно случается у всех, но в этом случае конкретно присваивание никакой особенной роли не играет, речь идёт вообще о разных «способах» написать в скобках после if что-то неправильное и не заметить это. При этом, поскольку if по смыслу означает проверку условия, логично ожидать в скобках что-то булевое, а все остальные варианты объявить симптомами опечаток, подлежащими как минимум пристальному рассмотрению.
Те, кто так или иначе считает такую конструкцию проблемной (и включают соответствующий варнинг компилятора), дальше делятся ещё на два варианта: одни призывают вообще её избегать в любом виде, вторые же допускают её применение, но с явной подсказкой компилятору/программисту в виде дополнительных круглых скобок вокруг: if((a=f)).
И ещё один момент: даже те, кто не усматривает в данной конструкции опасность, иногда всё равно её избегают, по эстетическим/стилевым причинам (для удобства чтения итд). Это, как и всё вышенаписанное, учтено в опросе.
-
практически не использую Си и C++ или не понял в чём суть проблемы
105 (28%)
********************************************************************************************************************************************************************************************************************************************************************************************************************************
-
конструкцию осуждаю и включаю предупреждение: снижает читаемость кода, при коллективной разработке провоцирует баги
92 (25%)
****************************************************************************************************************************************************************************************************************************************************************************************
-
нормальная конструкция, сам пользуюсь (без вторых скобок)
34 (9%)
*******************************************************************************************************
-
в целом не осуждаю (у других), но в мой стиль не вписывается (неэстетично, неудобно читать итд)
33 (9%)
****************************************************************************************************
-
другой вариант (напишу в комментариях)
31 (8%)
**********************************************************************************************
-
конструкцию осуждаю и включаю предупреждение: у всех бывают опечатки, и дело не в =/== а вообще в автоконверсии int->bool
23 (6%)
**********************************************************************
-
пользуюсь с двойными скобками, но не из-за опечаток, а ради упрощения чтения кода, зрительного выделения "тут не сравнение а меняется переменная"
21 (6%)
****************************************************************
-
боюсь перепутать присваивание со сравнением, конструкцию осуждаю, включаю на неё предупреждение
17 (5%)
***************************************************
-
пользуюсь с двойными скобками, на одинарные включаю варнинг: у всех бывают опечатки, и дело не в =/== а вообще в автоконверсии int->bool
15 (4%)
*********************************************