Поиск:

Ответ в темуСоздание новой темы Создание опроса
> [qt4.4] Удаление объекта из его конструктора 
V
    Опции темы
GrishinUS
Дата 31.10.2008, 08:35 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 125
Регистрация: 5.9.2006
Где: Самара

Репутация: нет
Всего: нет



Добрый день!

Есть объект, в конструкторе которого проверяются некие условия. Если условия не выполняются нужно вывести сообщение и закрыть форму.

Код

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 не работает
    }
}


вообще реально удалить самого себя из конструктора? Или же нужно генерировать сигнал родителю чтоб он удалял объект?
PM MAIL ICQ   Вверх
feanor59
Дата 31.10.2008, 09:28 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 66
Регистрация: 24.1.2008
Где: Пермь

Репутация: нет
Всего: нет



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

Это сообщение отредактировал(а) feanor59 - 31.10.2008, 09:32
PM MAIL ICQ GTalk   Вверх
GrishinUS
Дата 31.10.2008, 11:02 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 125
Регистрация: 5.9.2006
Где: Самара

Репутация: нет
Всего: нет



Цитата(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

Это сообщение отредактировал(а) GrishinUS - 31.10.2008, 11:03
PM MAIL ICQ   Вверх
VK_Techno
Дата 31.10.2008, 13:13 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Новичок



Профиль
Группа: Участник
Сообщений: 10
Регистрация: 29.9.2007

Репутация: нет
Всего: нет



Теперь каждый вызов конструктора DoOutImpl вкладывай в блок try/catch, лови исключение типа int и обрабатывай его, например, показав пользователю сообщение об ошибке.
PM MAIL   Вверх
SABROG
Дата 31.10.2008, 15:05 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Hacker
****


Профиль
Группа: Завсегдатай
Сообщений: 2481
Регистрация: 18.9.2006

Репутация: 81
Всего: 91



Вообще не самая светлая идея прописывать проверки в конструкторе. Лучше его оставить пустым и создать метод Initialization(), который и вызывать после того как объект создан.

Представь, если бы разработчики Qt пользовались таким методом ? Скажем конструктор QString выкинет исключение, а ты даже не прописал блок try{}catch(...){}.

Это сообщение отредактировал(а) SABROG - 31.10.2008, 15:08


--------------------
Национальная группа Russian Federation на QtCentre.
PM MAIL   Вверх
mes
Дата 31.10.2008, 15:16 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


любитель
****


Профиль
Группа: Участник Клуба
Сообщений: 7954
Регистрация: 14.1.2006

Репутация: 7
Всего: 250



Цитата(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 ()) ...
};


Это сообщение отредактировал(а) mes - 31.10.2008, 15:58


--------------------
PM MAIL WWW   Вверх
GrishinUS
Дата 31.10.2008, 15:21 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 125
Регистрация: 5.9.2006
Где: Самара

Репутация: нет
Всего: нет



Цитата(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, понял.
PM MAIL ICQ   Вверх
Любитель
Дата 31.10.2008, 20:46 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Программист-романтик
****


Профиль
Группа: Комодератор
Сообщений: 3645
Регистрация: 21.5.2005
Где: Воронеж

Репутация: 16
Всего: 92



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


--------------------
PM MAIL ICQ Skype   Вверх
mes
Дата 31.10.2008, 21:18 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


любитель
****


Профиль
Группа: Участник Клуба
Сообщений: 7954
Регистрация: 14.1.2006

Репутация: 7
Всего: 250



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

 smile А нельзя ли на зле остановится подробней ?  что именно подпадает под Вашу характеристику в данном примере?

Это сообщение отредактировал(а) mes - 31.10.2008, 21:19


--------------------
PM MAIL WWW   Вверх
GrishinUS
Дата 1.11.2008, 10:45 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 125
Регистрация: 5.9.2006
Где: Самара

Репутация: нет
Всего: нет



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

А можно попросить  smile или ссылку на пример?

PM MAIL ICQ   Вверх
GrishinUS
Дата 1.11.2008, 14:26 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 125
Регистрация: 5.9.2006
Где: Самара

Репутация: нет
Всего: нет



Цитата(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();
                }


код выше никак не влияет на видимость диалога :(

Это сообщение отредактировал(а) GrishinUS - 1.11.2008, 14:29
PM MAIL ICQ   Вверх
mes
Дата 1.11.2008, 15:01 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


любитель
****


Профиль
Группа: Участник Клуба
Сообщений: 7954
Регистрация: 14.1.2006

Репутация: 7
Всего: 250



Цитата(GrishinUS @  1.11.2008,  14:26 Найти цитируемый пост)
а как это будет выглядеть в терминах exec и close?

диалог же у Вас модальный? все проверки/действия должны быть выполнены до/в exec(). 


Цитата(GrishinUS @  1.11.2008,  14:26 Найти цитируемый пост)
                OuterForm.exec();
                if(!OuterForm.CanDoOut())

ну "после драки кулаками не машут"©. Вы перейдете  к следующей строчке за  OuterForm.exec() тогда, когда покинете форму диалога, а скрывать ее на тот момент уже смысла нет  smile 

В обшем или перегружаете  метод exec () так, как Вам нужно или вызываете его  по условию. (выбор способа зависит от самой задачи)

Это сообщение отредактировал(а) mes - 1.11.2008, 15:02


--------------------
PM MAIL WWW   Вверх
GrishinUS
Дата 7.11.2008, 08:20 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Шустрый
*


Профиль
Группа: Участник
Сообщений: 125
Регистрация: 5.9.2006
Где: Самара

Репутация: нет
Всего: нет



сделал вызов exec() по условию. Помечаю вопрос как решенный.
PM MAIL ICQ   Вверх
  
Ответ в темуСоздание новой темы Создание опроса
Правила форума "С/С++: Кроссплатформенное программирование, QT/Gtk+/wxWidgets"
JackYF
Любитель
  • В заголовке темы в квадратных скобках обозначьте используемую вами библиотеку, например: [QT],[GTK],[wx].
  • Если вопрос актуален только для некоторой версии библиотеки, либо, если вы пользуетесь не самой последней версией, укажите это. Например: [QT4], [GTK2].
  • Все начинающие изучать Qt - не забудьте зайти сюда.
  • Проставьте несколько ключевых слов темы, чтобы её можно было легче найти.
  • В вопросе укажите полную версию версию библиотеки, а также все дополнительные используемые программные пакеты.
  • Не забывайте пользоваться кнопкой "Код".
  • Телепатов на форуме нет! Задавайте чёткий, конкретный и полный вопрос. Указывайте полностью ошибки компилятора и компоновщика.
  • Новое сообщение должно иметь прямое отношение к тематике этого раздела. Флуд, флейм, оффтопик запрещены.
  • Категорически запрещается обсуждение вареза, "кряков", взлома программ и т.д.

Если Вам понравилась атмосфера форума, заходите к нам чаще! С уважением, JackYF, Любитель.

 
0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема | С/С++: Кроссплатформенное программирование, Qt/Gtk+/wxWidgets | Следующая тема »


 




[ Время генерации скрипта: 0.0854 ]   [ Использовано запросов: 22 ]   [ GZIP включён ]


Реклама на сайте     Информационное спонсорство

 
По вопросам размещения рекламы пишите на vladimir(sobaka)vingrad.ru
Отказ от ответственности     Powered by Invision Power Board(R) 1.3 © 2003  IPS, Inc.