Микропост.
Я регулярно вижу условные конструкции с условиями всяких разных видов. И простыми и сложными и с экономией строчек, чтобы вместо переменной с названием надо было гадать, что там в ифе происходит.
Мой недавний пример. Здоровенный иф с двумя ветками. if (!group) { мало кода } else { много кода }
Понятно, почему человек инвертировал условие. Он хотел иметь более короткую ветку под ифом, а длинную под элсом. Но блин! Это сложно читать. Лучше делать прямые условия и с малым количеством операторов. Если условий два или больше, то лучше их вытащить в отдельную переменную, чтобы в самом if/while уже было понятное название условия.
