Qt5: Come usare qDebug() per accedere a un file, applicazione multi-thread

Question

Ho iniziato a utilizzare Qt5 pochi giorni fa. Avevo bisogno di un logger per la mia app e ho deciso di utilizzare qDebug, ma sembra che debba essere "reindirizzato" per avere i registri in un file.

Ho usato qInstallMessageHandler per farlo e ho scritto il mio gestore come illustrato di seguito (ispirato da altre persone qui).

Sembra che funzioni, ma poiché non sono un guru, devo chiedere: È corretto utilizzarlo in un'applicazione multithread o no?

Inoltre, se è ok/sicuro per l'utilizzo in un'applicazione multi-thread, può essere migliorato in qualche modo?
Grazie!

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

Il tuo approccio è semplice e pulito. Lo terrei così.

C'è una cosa che puoi migliorare: apri il file solo una volta all'avvio dell'applicazione e chiudilo quando chiudi l'applicazione. L'apertura di un file è un'operazione costosa.

È possibile scrivere nello stesso file aperto da più thread, poiché il mutex garantisce che solo un thread scrive nello stesso momento.

Answer 2

Non si può trovare in qualsiasi parte della documentazione Qt che qDebug è thread-safe. Quindi non è sicuro chiamarlo contemporaneamente da più thread e in effetti si incontrerebbero output misti se non si utilizza un meccanismo di blocco.

Il tuo approccio di blocco sarebbe meglio se si utilizza QMutexLocker in quanto è fortemente raccomandato dalla documentazione Qt:

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

Un secondo approccio è quello di fornire una classe lavoratrice che è dotato di slot per la scrittura del registro. È quindi possibile avere un'istanza di esso in una nuova discussione e chiamare gli slot nel gestore dei messaggi utilizzando QMetaObject::invokeMethod con un tipo di connessione Qt::QueuedConnection. In questo modo, ogni chiamata da ciascun thread verrebbe accodata ed elaborata nel thread di lavoro e probabilmente avrebbe prestazioni migliori poiché tutto il lavoro viene eseguito in un thread separato.