Russian Qt Forum
Август 17, 2018, 13:06 *
Добро пожаловать, Гость. Пожалуйста, войдите или зарегистрируйтесь.
Вам не пришло письмо с кодом активации?

Войти
 
  Начало   Форум  WIKI (Вики)FAQ Помощь Поиск Войти Регистрация  

Страниц: [1]   Вниз
  Печать  
Автор Тема: QMessageBox из конструктора  (Прочитано 464 раз)
DarkHobbit
Чайник
*
Offline Offline

Сообщений: 71


Просмотр профиля
« : Июль 13, 2018, 11:37 »

Доброй пятницы.
Сейчас ковыряю _чужой_ Qt/C++ код, в котором то и дело из конструкторов разных классов вызывается QMessageBox::critical, а то и QMessageBox::information. И меня гложет сомнение: а такое вообще допустимо? Нет, я не про разделение логики и UI, оно-то как раз где-то оправдано, а где-то нет.
Я про то, что вроде бы в конструкторах в принципе не рекомендуют писать код, который может "зависнуть" на неопределённое время. Или я чего-то путаю?
Да, вопрос, возможно, даже не столько про Qt, сколько про хороший тон в C++, но QMessageBox может добавлять свою специфику.
Записан

Мои проекты на Qt: DoubleContact, LInvert
ViTech
Бывалый
*****
Offline Offline

Сообщений: 478



Просмотр профиля
« Ответ #1 : Июль 13, 2018, 11:57 »

И что происходит после вызова QMessageBox::critical? Можно кусок кода показать, прям интересно Улыбающийся. Хотя при написании кода "в формах" может быть всё что угодно.

Лично я бы в конструкторах остерегался какие-либо окна показывать Улыбающийся.
Записан

Пока сам не сделаешь...
Igors
Джедай : наставник для всех
*******
Offline Offline

Сообщений: 10110


Просмотр профиля
« Ответ #2 : Июль 14, 2018, 07:15 »

Нет, я не про разделение логики и UI, оно-то как раз где-то оправдано, а где-то нет.
В один прекрасный день возникает необходимость "нужна версия приложения с командной строкой". Или данный класс потребовался для использования в таком приложении. Достаточно пережить хотя бы один такой случай - и никаких "где-то" уже не будет  Улыбающийся

И меня гложет сомнение: а такое вообще допустимо?
Да, напр хотелось сразу показать ошибку которую необходимо исправлять. Что-то типа Q_ASSERT(0). А прокладку писать было лень - вот и влепил MessageBox.

Записан
Racheengel
Джедай : наставник для всех
*******
Offline Offline

Сообщений: 2458

чтоб работа волком не казалась :)


Просмотр профиля WWW
« Ответ #3 : Июль 20, 2018, 02:02 »

тут еще вопрос в реализации этого разделения.  Например,  в конструкторе можно вызвать что то типа абстрактного MessageManager::showMessage (blabla), которая будет уже  в конкретной прилинковываемой из гуи-либы реализации, например,  сделана в виде враппера над QMessageBox, либо в виде нотификатора, либо еще как.
Записан

What is the 11 in the C++11? It’s the number of feet they glued to C++ trying to obtain a better octopus.

http://sintegrial.com - всякий хороший Qt софт (был Грустный )
DarkHobbit
Чайник
*
Offline Offline

Сообщений: 71


Просмотр профиля
« Ответ #4 : Июль 25, 2018, 14:55 »

И что происходит после вызова QMessageBox::critical? Можно кусок кода показать, прям интересно Улыбающийся
Код там довольно монструозный, но если в двух словах - там устанавливается флаг ошибки и конструктор завершается. Потом вызывающий класс этот флаг смотрит.
В принципе, если бы вместо флага (а то и вместе с ним) заполнялась строка с сообщением об ошибки, а уже где-то вне класса это сообщение выводилось - у меня бы вопросов и не было, тогда и message box в классе был не нужен...
Просто одно дело, если я человека буду убеждать переписывать код на основании того, что это некрасиво и немножко другое - на основании того, что это вызывает серьёзные проблемы.
Записан

Мои проекты на Qt: DoubleContact, LInvert
Страниц: [1]   Вверх
  Печать  
 
Перейти в:  

Страница сгенерирована за 0.052 секунд. Запросов: 22.