Qt5: Как использовать qDebug() для входа в файл, многопоточного приложения

Question

Я начал использовать Qt5 несколько дней назад. Мне нужно было logger для моего приложения, и я решил использовать qDebug, но, похоже, он должен быть «перенаправлен», чтобы иметь журналы в файле.

Я использовал qInstallMessageHandler, чтобы сделать это, и я написал свой собственный обработчик, как показано ниже (вдохновленный другими людьми здесь).

Кажется, что это работает, но поскольку я не гуру, я должен спросить: Можно ли использовать это в многопоточном приложении или нет?

Кроме того, если это нормально/безопасно для использования в приложении с несколькими потоками, можете ли это как-то улучшить?
Спасибо!

void MessageHandler(QtMsgType type, const QMessageLogContext & context, const QString & msg) 
{ 
    mutex.lock(); 

    QDateTime dateTime(QDateTime::currentDateTime()); 

    QString timeStr(dateTime.toString("dd-MM-yyyy HH:mm:ss:zzz")); 
    QString contextString(QString("(%1, %2)").arg(context.file).arg(context.line)); 

    QFile outFile("file.log"); 
    outFile.open(QIODevice::WriteOnly | QIODevice::Append); 

    QTextStream stream(&outFile); 
    stream << timeStr << " " << contextString << ": " << msg << endl; 

    mutex.unlock(); 
} 

Answer 1

Ваш подход выглядит простым и чистым. Я бы сохранил это так.

Есть одна вещь, которую вы можете улучшить: открыть файл только один раз при запуске приложения и закрыть его при закрытии приложения. Открытие файла - дорогостоящая операция.

Вы можете писать в один и тот же открытый файл из нескольких потоков, так как ваш мьютекс гарантирует, что одновременно будет записываться только один поток.

Answer 2

Вы не можете найти где-нибудь в документации Qt, что qDebug потокобезопасно. Поэтому небезопасно вызывать его одновременно из нескольких потоков, и действительно, вы столкнетесь с смешанным выходом, если не используете механизм блокировки.

Ваш замок подход был бы лучше, если вы используете QMutexLocker, как настоятельно рекомендуется в документации Qt:

void MessageHandler(QtMsgType type, const QMessageLogContext & context, const QString & msg) 
{ 
    QMutexLocker locker(&mutex); 
    ... 
} 

Второй подход заключается в создании класса работник, который имеет слоты для записи журнала. Затем вы можете иметь экземпляр этого в новом потоке и вызвать его слоты в обработчике сообщений, используя QMetaObject::invokeMethod с типом соединения Qt::QueuedConnection. Таким образом, каждый вызов из каждого потока будет поставлен в очередь и обработан в рабочем потоке, и, вероятно, он будет иметь лучшую производительность, поскольку вся работа выполняется в отдельном потоке.