Основные замечания:
-Все лежит в одном месте без конкретной структуры. Разобраться что к чему не так сложно, но наличие пустых файлов и отсутствие ридми сбивает с толку.
-Разный стиль написания кода, порой бросается в глаза, но в целом не так заметно. if-else, switch-case и тд.
Пример: widget.cpp
if (snake.getTimer()->isActive()) {
snake.stopTimer();
startBtn->setEnabled(true);
pauseBtn->setEnabled(false);
}
else {
snake.startTimer();
pauseBtn->setEnabled(true);
}
В других файлах:
if (isPaused) {
timer.stop();
} else {
timer.start(timeoutTime(), this);
}
-Встречаются не очень понятные названия. Например, PPWindow - что это?
-Отсутствие комментариев как таковых. Код во многом говорит сам за себя, но порой комментарии к происходящему не помешали.
По мелочи:
-Встречается мертвый код, причем никак не отмечено комментариями зачем и почему он таков.
-В startdialog.h функции реализованы сразу, это не замечание, скорее наблюдение. Больше одной строчки реализации в хедере привлекает внимание.
В целом код аккуратный, глазу приятно. Выдержанный стиль, в большинстве своем понятные описания и действия, грамотное написание с учетом const, static и т.д.
Основные замечания:
-Все лежит в одном месте без конкретной структуры. Разобраться что к чему не так сложно, но наличие пустых файлов и отсутствие ридми сбивает с толку.
-Разный стиль написания кода, порой бросается в глаза, но в целом не так заметно. if-else, switch-case и тд.
Пример: widget.cpp
if (snake.getTimer()->isActive()) {
snake.stopTimer();
startBtn->setEnabled(true);
pauseBtn->setEnabled(false);
}
else {
snake.startTimer();
pauseBtn->setEnabled(true);
}
В других файлах:
if (isPaused) {
timer.stop();
} else {
timer.start(timeoutTime(), this);
}
-Встречаются не очень понятные названия. Например, PPWindow - что это?
-Отсутствие комментариев как таковых. Код во многом говорит сам за себя, но порой комментарии к происходящему не помешали.
По мелочи:
-Встречается мертвый код, причем никак не отмечено комментариями зачем и почему он таков.
-В startdialog.h функции реализованы сразу, это не замечание, скорее наблюдение. Больше одной строчки реализации в хедере привлекает внимание.
В целом код аккуратный, глазу приятно. Выдержанный стиль, в большинстве своем понятные описания и действия, грамотное написание с учетом const, static и т.д.