Версия для печати темы Нажмите сюда для просмотра этой темы в оригинальном формате

Форум программистов > С/С++: Кроссплатформенное программирование, Qt/Gtk+/wxWidgets > [qt4.4] Удаление объекта из его конструктора

Автор: GrishinUS 31.10.2008, 08:35

Добрый день! Есть объект, в конструкторе которого проверяются некие условия. Если условия не выполняются нужно вывести сообщение и закрыть форму. Код DoOutImpl::DoOutImpl(QString *qstrFormalSomewords, QString *qstrFormalCarID, QString *qstrFormalKartNum, QWidget * parent, Qt::WFlags f)     : QDialog(parent, f) {     setupUi(this);     this->qstrCarID = qstrFormalCarID;     this->qstrKartNum = qstrFormalKartNum;     this->qstrSomewords = qstrFormalSomewords;     labelCarID->setText(*qstrFormalCarID);     labelKartNum->setText(*qstrFormalKartNum);     labelSomewords->setText(*qstrFormalSomewords);          if( CheckNewAndTunning1() == 20 || // проверка по № карты         CheckNewAndTunning2() == 20 || // проверка по "новый" и "тюнинг"         CheckNewAndTunning3() == 20)   // проверка по ключевым словам     {         QMessageBox msgBox;         msgBox.setText(trUtf8("Ошибочка вышла..."));         msgBox.setIcon(QMessageBox::Critical);         msgBox.setDefaultButton(QMessageBox::Save);         msgBox.setInformativeText(trUtf8("Выезд запрещен!"));         msgBox.setStandardButtons(QMessageBox::Ok);         msgBox.setDefaultButton(QMessageBox::Ok);         msgBox.exec();                 // пробовал здесь close(), done, return не работает     } } вообще реально удалить самого себя из конструктора? Или же нужно генерировать сигнал родителю чтоб он удалял объект?

Автор: feanor59 31.10.2008, 09:28

если завершить конструктор по исключению, то объект создан не будет. а удалить себя из конструктора нельзя, т. к. до его (конструктора) корректного завершения объект не создаётся.

Автор: GrishinUS 31.10.2008, 11:02

Цитата(feanor59 @ 31.10.2008,  09:28) если завершить конструктор по исключению, то объект создан не будет. а удалить себя из конструктора нельзя, т. к. до его (конструктора) корректного завершения объект не создаётся. добавил  Код DoOutImpl::DoOutImpl(QString *qstrFormalSomewords, QString *qstrFormalCarID, QString *qstrFormalKartNum, QWidget * parent, Qt::WFlags f)     : QDialog(parent, f) {     setupUi(this);     this->qstrCarID = qstrFormalCarID;     this->qstrKartNum = qstrFormalKartNum;     this->qstrSomewords = qstrFormalSomewords;     labelCarID->setText(*qstrFormalCarID);     labelKartNum->setText(*qstrFormalKartNum);     labelSomewords->setText(*qstrFormalSomewords);          if( CheckNewAndTunning1() == 20 || // проверка по № карты         CheckNewAndTunning2() == 20 || // проверка по "новый" и "тюнинг"         CheckNewAndTunning3() == 20 || // проверка по ключевым словам         CheckNewAndTunning4() == 20)   // проверка сумм в таблице и шапке     {         QMessageBox msgBox;         msgBox.setWindowTitle(trUtf8("Ошибочка вышла..."));         msgBox.setIcon(QMessageBox::Critical);         msgBox.setDefaultButton(QMessageBox::Save);         msgBox.setText(trUtf8("Выезд запрещен!"));         msgBox.setStandardButtons(QMessageBox::Ok);         msgBox.setDefaultButton(QMessageBox::Ok);         msgBox.exec();         throw 7; // <========     }       } теперь вся прога завершается, а мне нужно чтобы только диалог DoOutImpl

Автор: VK_Techno 31.10.2008, 13:13

Теперь каждый вызов конструктора DoOutImpl вкладывай в блок try/catch, лови исключение типа int и обрабатывай его, например, показав пользователю сообщение об ошибке.

Автор: SABROG 31.10.2008, 15:05

Вообще не самая светлая идея прописывать проверки в конструкторе. Лучше его оставить пустым и создать метод Initialization(), который и вызывать после того как объект создан. Представь, если бы разработчики Qt пользовались таким методом ? Скажем конструктор QString выкинет исключение, а ты даже не прописал блок try{}catch(...){}.

Автор: mes 31.10.2008, 15:16

Цитата(SABROG @  31.10.2008,  15:05 ) Вообще не самая светлая идея прописывать проверки в конструкторе.  Согласен. Цитата(GrishinUS @  31.10.2008,  08:35 ) вообще реально удалить самого себя из конструктора? Или же нужно генерировать сигнал родителю чтоб он удалял объект?  вынесите условие из кнструктора, и проверяйте его перед показом диалога , типа такого : Код MyDialog : public QDialog {     MyDialog (..) : .. {}     bool IsOk () { .. } } MyDialog dlg; if (dlg.IsOk()) {      if (dlg.ShowModal ()) ... };

Автор: GrishinUS 31.10.2008, 15:21

Цитата(mes @ 31.10.2008,  15:16) Цитата(SABROG @  31.10.2008,  15:05 ) Вообще не самая светлая идея прописывать проверки в конструкторе.  Согласен. Цитата(GrishinUS @  31.10.2008,  08:35 ) вообще реально удалить самого себя из конструктора? Или же нужно генерировать сигнал родителю чтоб он удалял объект?  вынесите условие из кнструктора, и проверяйте его перед показом диалога , типа такого : Код MyDiaolog : public QDialog {     MyDialog (..) : .. {}     bool IsOk () { .. } } MyDialog dlg; if (dlg.isOk()) {      if (dlg.ShowModal ()) ... }; OK, понял.

Автор: Любитель 31.10.2008, 20:46

Ни в коем случае следуйте такому подходу! Это подход MFC  И это зло! Если это реально надо - используйте фабрику (и прайват конструктор). Нет, ну локально, конечно, где-то можно поизвращаться, но ни в коем случае не выставлять такие классы далеко, что называется

Автор: mes 31.10.2008, 21:18

Цитата(Любитель @  31.10.2008,  20:46 ) Это подход MFC smile И это зло! Если это реально надо - используйте фабрику (и прайват конструктор).    А нельзя ли на зле остановится подробней ?  что именно подпадает под Вашу характеристику в данном примере?

Автор: GrishinUS 1.11.2008, 10:45

Цитата(Любитель @ 31.10.2008,  20:46) Ни в коем случае следуйте такому подходу! Это подход MFC  И это зло! Если это реально надо - используйте фабрику (и прайват конструктор). Нет, ну локально, конечно, где-то можно поизвращаться, но ни в коем случае не выставлять такие классы далеко, что называется  А можно попросить   или ссылку на пример?

Автор: GrishinUS 1.11.2008, 14:26

Цитата(mes @ 31.10.2008,  15:16) Цитата(SABROG @  31.10.2008,  15:05 ) Вообще не самая светлая идея прописывать проверки в конструкторе.  Согласен. Цитата(GrishinUS @  31.10.2008,  08:35 ) вообще реально удалить самого себя из конструктора? Или же нужно генерировать сигнал родителю чтоб он удалял объект?  вынесите условие из кнструктора, и проверяйте его перед показом диалога , типа такого : Код MyDialog : public QDialog {     MyDialog (..) : .. {}     bool IsOk () { .. } } MyDialog dlg; if (dlg.IsOk()) {      if (dlg.ShowModal ()) ... }; а как это будет выглядеть в терминах exec и close? Код         DoOutImpl OuterForm(&qstrSomewords, &qstrCarID , &qstrTMPKartNum, this);                 OuterForm.exec();                 if(!OuterForm.CanDoOut())                 {                     OuterForm.hide();  // пробовал close()                 }                 else                 {                     OuterForm.show();                 } код выше никак не влияет на видимость диалога :(

Автор: mes 1.11.2008, 15:01

Цитата(GrishinUS @  1.11.2008,  14:26 ) а как это будет выглядеть в терминах exec и close? диалог же у Вас модальный? все проверки/действия должны быть выполнены до/в exec().  Цитата(GrishinUS @  1.11.2008,  14:26 )                 OuterForm.exec();                 if(!OuterForm.CanDoOut()) ну "после драки кулаками не машут"©. Вы перейдете  к следующей строчке за  OuterForm.exec() тогда, когда покинете форму диалога, а скрывать ее на тот момент уже смысла нет    В обшем или перегружаете  метод exec () так, как Вам нужно или вызываете его  по условию. (выбор способа зависит от самой задачи)

Автор: GrishinUS 7.11.2008, 08:20

сделал вызов exec() по условию. Помечаю вопрос как решенный.

Powered by Invision Power Board (http://www.invisionboard.com)
© Invision Power Services (http://www.invisionpower.com)