Qt. Как передавать данные между формами?

Question

[VerNika]

Имеется основная форма:

class MainWindow
{
    ...
private:
    ...
    QList<Institution> institutions;
};

И вторая форма:

class AddItem
{
    ...
};

Вторая форма вызывается из первой:

void MainWindow::on_action_add_item_triggered()
{
    AddItem *form_2 = new AddItem(this);
    form_2->show();
}

Как можно передать QList<Institution> institutions во вторую форму, чтобы коллекцию можно было редактировать в ней?

Answer 1

[AtomKrieg]

По неконстантной ссылке (или указателю):

class AddItem
{
    AddItem(MainWindow *parent, QList<Institution>& ins);
...
private:
QList<Institution>& parent_institutions;
};

Но вообще это плохая практика. Свою приватную переменную должен редактировать класс который ей владеет. Для этого нужны свои методы - MainWindow::AddInstitution, MainWindow::RemoveInstitution, etc. Так как это у вас QT, то тут надо использовать слоты и сигналы.

Answer 2

[Vitaly]

На самом деле вам не нужно ничего и никуда передавать, как мне кажется. Т.к. вторая форма используется только для создания новых элементов. Но обо всём по порядку. В приведённом фрагменте кода я вижу несколько архитектурных проблем (если не интересно, можете игнорироваться все, кроме пункта 3):
1) Вторая форма создаётся каждый раз, когда пользователь кликает на кнопку add_item. Это утечка памяти, т.к. все формы будут удалены только после удаления главного окна, но не ранее. Почему бы не создать форму один раз и просто её не показывать на действие? Создали в классе поле QScopedPointer m_addItemDlg (ну или std::unique_ptr), один раз выделили память в конструкторе и радуетесь жизни.
2) Это не очень хорошо, что Institution хранится в контейнере QList по значению т.к., скорее всего вы не реализовали семантику перемещения и всё это копируется... Хотя бы по этой причине (и по многим другим) стоит подумать о том, чтобы хранить все элементы как shared_ptr.
3) Наконец-то мы добрались и до вашего вопроса! В Qt есть отличный механизм слотов и сигналов. В вашем случае идея проста: у главной формы надо реализовать слот onInstitutionAdded а у второй формы дописать сигнал institutionAdded и соединить их в конструкторе главной формы. Когда новый объект создан выслать сигнал из второй формы и в слоте главного окна реализовать добавление в коллецию.
Если же нужно будет редактировать какой-то итем в коллекции, то его конечно лучше передать просто по ссылке перед тем как показать форму. Ну и в этом случае посмотреть уже в сторону Qt model-view.
4) Название класса второй формы слишком общее и ни о чем не говорит.

Comments

[VerNika]

1) Полностью согласна, так и сделаю;
2) Вы же отвечали на мой предыдущий вопрос про сериализацию класса со вложенными коллекциями, для этого я и сделала коллекцию непосредственно самих объектов. "Перемещение" я сделала по-простому:
Institution *bf_inst = new Institution(...);
institutions.append(*bf_inst);
delete bf_inst;
3) Ну а с этим стоит разбираться...
4) Переименуем)

[Vitaly]

VerNika: Упс) По пункту 2 вышло некоторое недопонимание. Я имел ввиду подход вроде этого (ну только у вас будет QFile с минимальными отличиями в коде а не QByteArray): https://gist.github.com/vt4a2h/45aa8071de2cbfcf8a10
Про семантику перемещения я имел ввиду вот это: en.cppreference.com/w/cpp/language/move_constructor
en.cppreference.com/w/cpp/language/move_assignment

Answer 3

[Mercury13]

Вариантов много, в зависимости от того, форма модальная или нет, и нужно ли каким-то моделям передавать сообщение «обновись».

Вариант 1.

class AddItem : public QWidget
{
public:
  AddItem(QWidget* aParent, QList<Institution>& aInstitutions)
      : QWidget(aParent), institutions(aInstitutions) {}
private:
  QList<Institution>& institutions;
};

Во втором варианте у нас модальная форма, но редактировать можно только копию (например, институции первой формы задействованы в какой-то модели, или нужны каскадные удаления, или что-то ещё).

class AddItem : public QDialog
{
public:
  int exec(QList<Institution>& aInstitutions);
private:
  QList<Institution> institutions;
};

int AddItem::exec(QList<Institution>& aInstitutions)
{
  institutions = aInstitutions;
  int r = QDialog::exec();
  if (r) {
     aInstitutions = std::move(institutions);
  }
  return r;
}

И много-много других вариантов.