 |
|
Модераторы: bsa |
 
|
|
Константа в левой части операции сравнения 
| semibug |
|
||||
|
Опытный  Профиль Группа: Участник Сообщений: 323 Регистрация: 27.3.2009 Репутация: нет Всего: нет |
Время от времени натыкаюсь в сети на рекомендацию использовать константы с левой стороны операции сравнения.
К примеру вместо
следует писать
Смысл в том, что в случае написания одного знака равенства в первом случае ошибка может остаться незамеченной, а во втором компилятор выдаст ошибку. Привык писать первым вариантом, как Вы считаете, знатоки, стоит ли отвыкать? |
||||
|
|||||
Загрузка ...
| chaos |
|
|||
 Серийный программист Профиль Группа: Завсегдатай Сообщений: 2979 Регистрация: 7.7.2004 Где: Екатеринбург Репутация: нет Всего: 44 |
стоит привыкнуть. ЗЫ сам даже в Java пишу так, а в AS3 например можно нарваться если пользоваться 1ым вариантом |
|||
|
||||
| semibug |
|
|||
|
Опытный Профиль Группа: Участник Сообщений: 323 Регистрация: 27.3.2009 Репутация: нет Всего: нет |
chaos, вы за вариант: "константу слева" ?
|
|||
|
||||
| SenkraD |
|
|||
 Опытный Профиль Группа: Участник Сообщений: 933 Регистрация: 3.2.2006 Где: Украина::Киев Репутация: нет Всего: 23 |
semibug, я за константу слева, что с уст моих коллег звучит как "писать сравнения в стиле Йоды"
-------------------- Имеющий язык - да не убоится спросить!  |
|||
|
||||
| borisbn |
|
||||
 Эксперт Профиль Группа: Завсегдатай Сообщений: 4875 Регистрация: 6.2.2010 Где: Ростов-на-Дону Репутация: 21 Всего: 135 |
Я раньше тоже писал как в первом варианте, потом пересилил себя и отвык. теперь пишу ТОЛЬКО как во втором, потому что возможные ошибки лучше убирать ( если это возможно, конечно ) на этапе компиляции, а не на этапе отладки или эксплуатации.
Вот ещё пример того, как уйти от ошибок на этапе компиляции:
лучше заменить на
чтобы если нужно заремить do_if_expr() do_anyway() не выполнялась по условию. Есть ещё рекомендация использовать шаблоны вместо макросов ( там есть проверка типов ) semibug, не возражаешь, если я задам вопрос в твоей теме ? Кто-нибудь знает ещё такие же trick'и для того, чтобы избегать ошибок на этапе компиляции ? -------------------- Женщины отличаются от программистов тем, что у них чары состоят из стрингов |
||||
|
|||||
| Abyx |
|
|||
|
Опытный Профиль Группа: Участник Сообщений: 601 Регистрация: 3.11.2009 Репутация: 5 Всего: 10 |
для этого коментится и if тоже в моей ИДЕ (msvs+VisualAssistX) это делается одной кнопкой "/" |
|||
|
||||
| boostcoder |
|
|||
 pattern`щик Профиль Группа: Завсегдатай Сообщений: 5458 Регистрация: 1.4.2010 Репутация: 20 Всего: 110 |
константа слева
|
|||
|
||||
| maxim1000 |
|
|||
 Эксперт Профиль Группа: Участник Сообщений: 3334 Регистрация: 11.1.2003 Где: Киев Репутация: 1 Всего: 110 |
константа справа
я никогда не замечал, чтобы в обсуждении алгоритмов люди говорили "и если 2 равно иксу, то...", а читабельность - немаловажное свойство кода а для поисков ошибок нужно смотреть на предупреждения компилятора, при желании можно его даже в ошибку переделать -------------------- qqq |
|||
|
||||
| Cheloveck |
|
|||
 Эксперт Профиль Группа: Завсегдатай Сообщений: 1578 Регистрация: 26.7.2008 Где: Тула Репутация: 5 Всего: 32 |
константа слева
--------------------  |
|||
|
||||
| boostcoder |
|
||||
|
pattern`щик Профиль Группа: Завсегдатай Сообщений: 5458 Регистрация: 1.4.2010 Репутация: 20 Всего: 110 |
а что скажет компилятор?:
|
||||
|
|||||
| semibug |
|
|||
|
Опытный Профиль Группа: Участник Сообщений: 323 Регистрация: 27.3.2009 Репутация: нет Всего: нет |
boostcoder, MSVC даёт у меня С4706 (assignment within conditional expression), если включить Warning Level 4.
|
|||
|
||||
| boostcoder |
|
|||
|
pattern`щик Профиль Группа: Завсегдатай Сообщений: 5458 Регистрация: 1.4.2010 Репутация: 20 Всего: 110 |
а если так?:
любопытно, на основании чего msvc принимает такое решение? согласен, в первом варианте это логично предположить. а что компилятор скажет во втором случае? |
|||
|
||||
| borisbn |
|
||||||||
|
Эксперт Профиль Группа: Завсегдатай Сообщений: 4875 Регистрация: 6.2.2010 Где: Ростов-на-Дону Репутация: 21 Всего: 135 |
В моей ИДЕ ( MSVS + VisualAssistX ) есть snippet
и потом, в expr может быть какое-то действие, например
если включить Warning Level 4 у меня на Qt начинает матом ругаться, поэтому ограничиваюсь 3-м. Так всё-таки, по поводу ещё таких же ухищрений. Кто-нибудь знает такого рода "правила" ? -------------------- Женщины отличаются от программистов тем, что у них чары состоят из стрингов |
||||||||
|
|||||||||
| mrbrooks |
|
|||
 трололомен Профиль Группа: Завсегдатай Сообщений: 4259 Регистрация: 4.10.2006 Где: Дол Гулдур Репутация: 19 Всего: 306 |
константа слева
|
|||
|
||||
| ksili |
|
|||
 Эксперт Профиль Группа: Завсегдатай Сообщений: 2069 Регистрация: 3.11.2005 Где: Красноярск Репутация: нет Всего: 17 |
Смотря какая IDE. Например FDT подчеркивает это и помечает на полях. Так что не заметить невозможно. -------------------- Ничто так не развивает аналитическое мышление, как отладка сложной программы без возможности пошагового выполнения (с) |
|||
|
||||
|
|
| Правила форума "C/C++: Для новичков" | |
|
|
Запрещается! 1. Публиковать ссылки на вскрытые компоненты 2. Обсуждать взлом компонентов и делиться вскрытыми компонентами
Если Вам понравилась атмосфера форума, заходите к нам чаще! С уважением, JackYF, bsa. |
| 0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей) | |
| 0 Пользователей: | |
| « Предыдущая тема | C/C++: Для новичков | Следующая тема » |
[ Время генерации скрипта: 0.0605 ] [ Использовано запросов: 22 ] [ GZIP включён ]
Реклама на сайте Информационное спонсорство
|
|
По вопросам размещения рекламы пишите на vladimir(sobaka)vingrad.ru
Отказ от ответственности Powered by Invision Power Board(R) 1.3 © 2003 IPS, Inc. |