QMessageBox из конструктора

[DarkHobbit]

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

[ViTech]

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

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

[Igors]

Нет, я не про разделение логики и UI, оно-то как раз где-то оправдано, а где-то нет.

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

И меня гложет сомнение: а такое вообще допустимо?

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

[Racheengel]

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

[DarkHobbit]

И что происходит после вызова QMessageBox::critical? Можно кусок кода показать, прям интересно

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