Вход на сайт
Работа тестировщиком после переезда в Германию
11076 просмотров
Перейти к просмотру всей ветки
в ответ Murr 23.12.20 21:25
Возможно было бы достаточно выбрать другое имя переменной или добавить коммент...
Согласен. Хотя бы переименовать "int_minute" в что-то вроде "full_quarter_minute". Не стал, чтобы не перенапрягать оппонента, а то вообще ничего знакомого в коде не увидит...
В идеале - объявить константу "четверть часа в минутах" (а то я два раза 15 использовал) и функцию "округлитьМинутыДоПолнойЧетверти"и её уже вызывать. Но в функции уже стоит проверять параметр, что он >=0 и < 60 и кидать ошибку... И вот это всё ради одной строчки было лень. Так как используется это в одном единственном месте, то просто добавить строчку с комментарием "округляем минуты до полной четверти часа, например 29 => 15" должно хватить.