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

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

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

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