Стоит ли делать в таком случае базовый класс и наследоваться от него?

Question

[Николай Шепелев]

В общем есть несколько классов форм на Qt типа:

//Главное окно
class MainWindow : public QMainWindow
{
//
private:
//
int m, n;
double **data, scaleX, scaleY;
}

//И много других форм, открываемых из главной формы - MainWindow
class OtherWindow : public QMainWindow
{
//
private:
//
int m, n;
double **data, scaleX, scaleY;
}

Как видно, у них есть общие поля - int m, n; double **data, scaleX, scaleY;, стоит ли делать базовый класс (с точки зрения ООПа), от которого будут наследоваться эти поля или нет? Что скажете?

Answer 1

[Adamos]

Нет, так делать не стоит.
Вы таким наследованием нарушаете L в SOLID.

Comments

[Николай Шепелев]

А можете, пожалуйста, разъяснить, каким образом такое наследование нарушает данный принцип?

[Adamos]

Тем, что базовый класс будет "классом, у которого есть такие переменные", а не классом, имеющим некие осмысленные, общие для всех дочерних классов, свойства. Например, в data эти два класса, скорее всего, пишут совершенно разные вещи.

Answer 2

[Vitaly]

Если даные только для внутреннего пользования, то намного лучше создать структуру Data (имя можно менее общее) или две структуры, если данные нельзя скомпоновать по смыслу в одну структуру.
Ну и наследовать все классы от QMainWindow это не очень правильно. Т.к. по смыслу в большинстве случаев главное окно одно. Этот класс отличается от обычного QWidget наличием дополнителных компонентов, добавленых по умолчанию (тулбар, например).

Comments

[Николай Шепелев]

Спасибо, именно так и сделал, ибо это оказалось наиболее логично. Да, эти данные описывают одну и ту же штуку.

[Николай Шепелев]

И про наследование, да главное окно наследуется от QMainWindow, вызываемые окна от QWidget. Здесь просто копи-паста сработала.

Answer 3

[Яков Е]

Если три поля — единственное, что их связывает, то смысла в наследовании мало. Может быть есть смысл создать отдельный класс двумерного массива и использовать его?

Comments

[Николай Шепелев]

На самом деле там пять полей (вопрос дополнил), и эти поля - данные, которые обрабатываются в разных формах своими методами.

[Николай Шепелев]

А вот вопрос об отдельном классе в некотором смысле идея, надо обдумать.

Answer 4

[Rou1997]

Базовый, конечно не стоит, сделайте иначе, создайте свой тип (класс или структуру) с этими полями и используйте везде вместо этих переменных.

Answer 5

[Даниил Игоревич]

Сделайте protected в базовом MainWindow, и эти поля будут наследоваться. И зачем вообще отдельный класс для трёх членов?

Comments

[Николай Шепелев]

Вопрос не про то как наследование реализовать, а в том, что уместно ли оно здесь.

[Николай Шепелев]

В этом то и вопрос, эти поля встречаются во всех формах, и представляют собой некую сущность данных. Вот стоит ли эту сущность выделят?