Почему запускается первым конструктор mainwindow.cpp?

Question

[Oleg Seledets]

Здравствуйте.
есть файл autoriz который должен запускаться первым, а при нажатии на кнопку у его формы, открывается mainwindow.
Но при запуске первым подгружается конструктор mainwindow. как это и править или можно ли события, которые в конструкторе mainwindow (подключение к бд, и загрузка настроек из QSettings) повешать в какое-либо событие, по открытию формы??

Конструктор mainwindow

MainWindow::MainWindow(QWidget *parent) :
    QMainWindow(parent),
    ui(new Ui::MainWindow)
{
    ui->setupUi(this);
    /* код на подключение БД и настроек*/
}

Конструктор autoriz

autoriz::autoriz(QDialog *parent) :
    QDialog(parent),
    ui(new Ui::autoriz)
{
    ui->setupUi(this);
    QTextCodec::setCodecForLocale(QTextCodec::codecForName("Windows-1251"));

    mainW = new MainWindow(); // Инициализируем  окно для диспетчера
    // подключаем к слоту запуска окна авторизация по кнопке в окне диспетчера
    connect(mainW, &MainWindow::firstWindow, this, &autoriz::show);
}

переход с авторизации на mainwindow

mainW->show(); //показать окно регистрации
        this->close(); // Закрываем основное окно

main.cpp

#include "autoriz.h"    //авторизация
#include "ui_autoriz.h"
#include <QApplication>
int main(int argc, char *argv[])
{
    QApplication a(argc, argv);
    autoriz au;
    au.show();
    return a.exec();
}

Answer 1

[Mercury13]

Я бы переписал архитектуру программы.
1. Перед вами модальное окошко. Вот его и вызывайте, как модальное — au.exec().
2. Чтобы проверить, что авторизация верна, надо соединиться. Так что сделайте невизуальный объект «соединение» и протащите в exec. Ну или создайте в exec.
3. А уж там собирайте основную программу.

QApplication a(argc, argv);
FmAutoriz au;
std::unique_ptr<SomeConnection> connection = au.exec();   // SomeConnection и exec придётся написать
if (connection) {
  FmMain m(connection);
  m.show();
  return a.exec();
} else {
  return 0;
}

std::unique_ptr<SomeConnection>FmAutoriz::exec()
{
  bool b = QDialog::exec();
  if (b) {
    return std::move(this->connection);
      // connection — поле FmAutoriz.
      // Такой же unique_ptr, в обработчиках кнопок пытаемся создать соединение.
  } else {
     return nullptr;
  }
}

void FmAutorize_on_btOk_click()
{
   connection = new SomeConnection();
   if (connection.connect(someLogin, somePassword)) {
      accept();
   } else {
      // сообщи об ошибке
   }
}

Вижу побочку этого (но, может, и не надо избавляться) — на панели задач анимацией сменяется кнопка с диалогом авторизации на кнопку с главным окном.

Comments

[Roman]

тут судя по прошлому вопросу (не могу найти его) дело в том, что читаются настройки из файла - при создании MainWindow, которые в свою очередь пишутся autoriz.
Получается, что настройки считываются до того как запишутся.

[Mercury13]

Роман, Если тут дело ещё и в настройках — что мешает сделать объект ProgSettings и прочитать его до создания формы?

ProgSettings progSets;   // глобальный объект в каком-нибудь модуле

QApplication a(argc, argv);
progSets.read(settingsFileName);
FmAutoriz au;
// что дальше — уж сами придумайте.

[Roman]

Mercury13, тут уже переписывать надо)
а если лепить из того что есть в лоб. я уже написал.

P.S.
Я за правильный вариант, который Ваш.

[Mercury13]

Роман, Если ошибка в считывании настроек — то мой вариант с настройками вполне встраивается и в вашу систему. Например, я сделал настройки глобальным объектом, потому что крайне мала вероятность, что потребуются два объекта настроек.

[Oleg Seledets]

Mercury13, Роман, Спасибо за ответы, скорее всего да, я полез "разбираться" с QT не зная QT.
Хорошо, что только на относительно начальном уровне задал вопрос, буду разбираться)
потому что даже это:

Так что сделайте невизуальный объект «соединение» и протащите в exec. Ну или создайте в exec.

понимается с трудом. если вообще понимается)

Но соединение у меня же происходит просто при совпадении логина из Combobox и пароля из qsetting? Или это не то, что надо?))
------------
Насчет настроек была тема, удалил.
В авторизации, имя пользователя, которое выбираем через Сombobox, должно при нажатии на клавишу записаться в настройки. и после открытия главной формы, оно должно считаться из настроек и подставить значение в lineEdit. о как

[Oleg Seledets]

Mercury13, вами приложенный код

spoiler

std::unique_ptr<SomeConnection>FmAutoriz::exec()
{
  bool b = QDialog::exec();
  if (b) {
    return std::move(this->connection);
      // connection — поле FmAutoriz.
      // Такой же unique_ptr, в обработчиках кнопок пытаемся создать соединение.
  } else {
     return nullptr;
  }
}

необходимо писать в классе autoriz? и заменять FmAutorize на autorize?

[Mercury13]

Oleg Seledets,
Да, в этом классе. И я предлагаю для форм придумать какой-то префикс, я использую Fm. Кто-то использует F.

[Mercury13]

Oleg Seledets, Я считаю венгерскую запись не формальностью, а сокращением, и «FmAuthorize» будет читаться как «форма авторизации».
https://mercury13-kiev.livejournal.com/15789.html

[Oleg Seledets]

Mercury13, QScopedPointer заменяет же std::unique_ptr в QT?
и еще очень , наверное тупой вопрос
SomeConnection и есть не визуальный объект «соединение»?
его тоже необходимо писать в autorize.cpp?

[Mercury13]

Oleg Seledets,
С QScopedPointer — да, но с привкусом Си++03. Рекомендую всё же std::unique_ptr.
Лучше в отдельном модуле: соединением будет пользоваться и главная форма, и форма авторизации.

[Oleg Seledets]

Mercury13, Но, что в нём должно быть если это не отдельная форма?? это всегда нужны будут такие сложности, для того, чтобы открыть форму при нажатии на кнопку?

[Mercury13]

Oleg Seledets, Это обычное дело — оттаскивать функциональность подальше от интерфейса. Впрочем, полностью это сделать невозможно: функциональность и интерфейс всегда переплетены. Хотя бы на уровне того, что каждому полю в функциональности надо сделать соответствующее поле в формочке.

Не так давно я решал такую штуку: есть несколько разных событий, изменяющих ключ сортировки. Некоторые вызываются парами, некоторые вызываются, когда ключ по факту и не изменялся — так что нужно запомнить имеющийся ключ и пересортировать, когда реально сменился. По-моему, это интерфейс и в базовую функциональность его не надо.

[Oleg Seledets]

Mercury13, Спасибо.
Пойду лучше учить плюсы, и читать Шлее. Прежде чем пытаться с ходу что-то разобрать
окончательно запутался в (объектах, модулях) думал не придется пока что это использовать. а будет достаточно как то получить сигнал при открытии формы и менять уже в нем то, что должно постоянно обновляться при открытии) а тут я даже не понял, что должно быть в этом самом SomeConnection)))