История изменений
Исправление hobbit, (текущая версия) :
Ну тогда всё же замечу, что операции сравнения уже возвращают логическое значение. Т.е. вместо
bool tmp = (n > 1) ? true : false;
ты вполне мог написать просто
bool tmp = (n > 1);
Название tmp мне тоже не особо нравится, оно вызывает ассоциацию с чем-то временным или отладочным, что потом будет выброшено из программы. Я бы назвал эту переменную res (от result). Но это уже вопрос стиля.
А в остальном, я смотрю, у тебя в последнем варианте даже стиль радикально улучшился. Как будто два разных человека писали…
Исходная версия hobbit, :
Ну тогда всё же замечу, что операции сравнения уже возвращают логическое значение. Т.е. вместо
bool tmp = (n > 1) ? true : false;
ты вполне мог написать просто
bool tmp = (n > 1);
Название tmp мне тоже не особо нравится, оно вызывает ассоциацию с чем-то временным или отладочным, что потом будет выброшено из программы. Я бы назвал эту переменную res (от result). Но это уже вопрос стиля.