LINUX.ORG.RU

История изменений

Исправление 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). Но это уже вопрос стиля.