|
Название: QMessageBox из конструктора Отправлено: DarkHobbit от Июль 13, 2018, 11:37 Доброй пятницы.
Сейчас ковыряю _чужой_ Qt/C++ код, в котором то и дело из конструкторов разных классов вызывается QMessageBox::critical, а то и QMessageBox::information. И меня гложет сомнение: а такое вообще допустимо? Нет, я не про разделение логики и UI, оно-то как раз где-то оправдано, а где-то нет. Я про то, что вроде бы в конструкторах в принципе не рекомендуют писать код, который может "зависнуть" на неопределённое время. Или я чего-то путаю? Да, вопрос, возможно, даже не столько про Qt, сколько про хороший тон в C++, но QMessageBox может добавлять свою специфику. Название: Re: QMessageBox из конструктора Отправлено: ViTech от Июль 13, 2018, 11:57 И что происходит после вызова QMessageBox::critical? Можно кусок кода показать, прям интересно :). Хотя при написании кода "в формах" может быть всё что угодно.
Лично я бы в конструкторах остерегался какие-либо окна показывать :). Название: Re: QMessageBox из конструктора Отправлено: Igors от Июль 14, 2018, 07:15 Нет, я не про разделение логики и UI, оно-то как раз где-то оправдано, а где-то нет. В один прекрасный день возникает необходимость "нужна версия приложения с командной строкой". Или данный класс потребовался для использования в таком приложении. Достаточно пережить хотя бы один такой случай - и никаких "где-то" уже не будет :)И меня гложет сомнение: а такое вообще допустимо? Да, напр хотелось сразу показать ошибку которую необходимо исправлять. Что-то типа Q_ASSERT(0). А прокладку писать было лень - вот и влепил MessageBox. Название: Re: QMessageBox из конструктора Отправлено: Racheengel от Июль 20, 2018, 02:02 тут еще вопрос в реализации этого разделения. Например, в конструкторе можно вызвать что то типа абстрактного MessageManager::showMessage (blabla), которая будет уже в конкретной прилинковываемой из гуи-либы реализации, например, сделана в виде враппера над QMessageBox, либо в виде нотификатора, либо еще как.
Название: Re: QMessageBox из конструктора Отправлено: DarkHobbit от Июль 25, 2018, 14:55 И что происходит после вызова QMessageBox::critical? Можно кусок кода показать, прям интересно :) Код там довольно монструозный, но если в двух словах - там устанавливается флаг ошибки и конструктор завершается. Потом вызывающий класс этот флаг смотрит.В принципе, если бы вместо флага (а то и вместе с ним) заполнялась строка с сообщением об ошибки, а уже где-то вне класса это сообщение выводилось - у меня бы вопросов и не было, тогда и message box в классе был не нужен... Просто одно дело, если я человека буду убеждать переписывать код на основании того, что это некрасиво и немножко другое - на основании того, что это вызывает серьёзные проблемы. |