Tetris_Blinova#14
Conversation
hBuzzy
left a comment
hBuzzy
left a comment
There was a problem hiding this comment.
Исправить замечания для принятия основного задания. Доп задание принято.
Очень много ошибок с пробелами и скобками, не стал писать на каждом месте комментарий. Не всегда соблюдается принятый стиль кода. В остальном - хорошо.
| @@ -0,0 +1,120 @@ | |||
| #include "figure.h" | |||
|
|
|||
| figure::figure() | |||
| int m_x; | ||
| int m_y; | ||
| int m_condition; | ||
| QColor m_color; | ||
| QVector<QVector<QVector<bool>>> m_data; |
There was a problem hiding this comment.
В нашей нотации (по google) приватные поля именуются все же с постфиксом "_". m_x -> x_ и т.д.
| #include "griddrawer.h" | ||
|
|
||
| GridDrawer::GridDrawer(QWidget *parent):QWidget(parent), m_columns(3), m_rows(4), m_sizeCell(30) | ||
| { |
There was a problem hiding this comment.
Раз начали внутри кода не переносить скобки, то они не переносятся и в методах. Везде в одном стиле.
| case 1: | ||
| m_data = {{{1,1,1,1}}, {{1},{1},{1},{1}}}; | ||
| break; | ||
| case 2: | ||
| m_data = {{{1,0},{1,1},{1,0}}, | ||
| {{1,1,1},{0,1,0}}, | ||
| {{0,1},{1,1},{0,1}}, | ||
| {{0,1,0},{1,1,1}} | ||
| }; | ||
| break; | ||
| case 3: | ||
| m_data = {{{1,1},{1,0},{1,0}}, | ||
| {{1,1,1},{0,0,1}}, | ||
| {{0,1},{0,1},{1,1}}, | ||
| {{1,0,0},{1,1,1}} | ||
| }; | ||
| break; | ||
| case 4: | ||
| m_data = {{{1,1},{0,1},{0,1}}, | ||
| {{0,0,1},{1,1,1}}, | ||
| {{1,0},{1,0},{1,1}}, | ||
| {{1,1,1},{1,0,0}} | ||
| }; | ||
| break; | ||
| case 5: | ||
| m_data = {{{1,0},{1,1},{0,1}}, | ||
| {{0,1,1},{1,1,0}} | ||
| }; | ||
| break; | ||
| case 6: | ||
| m_data = {{{0,1},{1,1},{1,0}}, | ||
| {{1,1,0},{0,1,1}} | ||
| }; | ||
| break; | ||
| default: | ||
| break; |
There was a problem hiding this comment.
case должны быть на одну табуляцию правее от switch
| } | ||
| } | ||
|
|
||
| void figure::setSpecificData(QVector<QVector<QVector<bool>>> data_){ |
There was a problem hiding this comment.
Постфикс "_" для параметров методов не нужен. Пробел перед { (касается всего проекта, унифицируйте написание скобок и поправьте отступы)
| for(int j = 0; j < grid->rows()+5; j++){ | ||
| if(i == 0 || i == grid->columns() + 1 || j == grid->rows() + 4){ | ||
| gridInt[i].push_back(1); | ||
| }else{ | ||
| gridInt[i].push_back(0); | ||
| } | ||
| } | ||
| } |
There was a problem hiding this comment.
Пробел после for, if и перед { и else.
5 - магическое число. Отделить оператор сложения с двух сторон пробелами.
| for(int i = 0; i < figure_->data()[cond].size(); i++){ | ||
| for(int j = 0; j < figure_->data()[cond][i].size(); j++){ | ||
| if(figure_->data()[cond][i][j]){ | ||
| gridInt[i + x][j + y] = 2; | ||
| } | ||
| } | ||
| } |
There was a problem hiding this comment.
Пробелы после for, if и перед {. Касается всего проекта. Дальше уже повторять комментарий не будут.
| for(int j = grid->rows() + 3; j >= 0; j--){ | ||
| bool ind = true; | ||
| for(int i = 0; i < grid->columns()+2; i++){ | ||
| if(gridInt[i][j] == 0){ | ||
| ind = false; | ||
| break; | ||
| } | ||
| } | ||
| if(ind){ | ||
| emit fullRow(); | ||
| for(int i = 0; i < grid->columns()+2; i++){ | ||
| gridInt[i].remove(j); | ||
| if(i == 0 || i == grid->columns()+1){ | ||
| gridInt[i].push_front(1); | ||
| }else{ | ||
| gridInt[i].push_front(0); | ||
| } | ||
| } | ||
| j++; | ||
| } | ||
| } | ||
| } |
There was a problem hiding this comment.
3, 2 - магические числа. ind ? Во-первых не соответствует нотации названия bool переменных / полей, во-вторых название не несет вообще никакого смысла.
|
|
||
| bool tetris::moveFigureDown(){ | ||
| int maximumHeightFigure = 4; | ||
| bool ind = true; |
| void onFigureChanged(figure* newFigure) { | ||
| figure_ = newFigure; | ||
| } | ||
| void newGame(); |
2 participants
No description provided.