Conversation
a-badin
left a comment
a-badin
left a comment
There was a problem hiding this comment.
Тесты проверяют только вылеты с ошибками, логика приложения не тестируется
| Q_OBJECT | ||
|
|
||
| public: | ||
| explicit Changer(QString name = "", int value = 30, int max_value = 255, QWidget *parent = 0); |
There was a problem hiding this comment.
Когда параметров по-умолчанию больше 1 это создает проблемы, если вы хотите изменить только max_value, то вам придется продублировать name и value из заголовка. Кладите в конструктор только необходимые параметры. Остальные можно будет поменять через сеттер
| ~Changer() = default; | ||
|
|
||
| private: | ||
| int _value; |
There was a problem hiding this comment.
это не ошибка, но считается что имена которые начинаются c _ или содержать два __ зарезервированы за компилятором/реализацией стандартной библиотеки. Это даже прописано в стандарте
| private: | ||
| int _value; | ||
|
|
||
| QSpinBox *spin_box; |
There was a problem hiding this comment.
имена членов данных лучше называть m_spin_box или spin_box_ чтобы они не конфликтовали с параметрами конструктора или аргументами функций класса
| public slots: | ||
| void SetBrushSizeSlot(int size); | ||
| void SetTransparencySlot(int transparency); | ||
| void SetRedSlot(int transparency); |
There was a problem hiding this comment.
нужно сделать 1 сигнал с QColor или чем то подобным
| // Status bar | ||
| statusBar()->showMessage("Status bar"); | ||
|
|
||
| timer = new QTimer(); |
There was a problem hiding this comment.
Утечка памяти. В Qt специфичная система управления памятью, каждый объект QObject принимает на вход указатель на родителя. При уничтожении объекта в деструкторе он удаляет всех потомков. Поэтому если вы создаете объект в куче без parent, то это утечка
2 participants
No description provided.