Projekt

Allgemein

Profil

Aktionen
Link kopieren

Fehler #642

geschlossen

Sortierte Ausgabe bei EntityInspector und Logger

Von Sebastian Diel vor mehr als 7 Jahren hinzugefügt. Vor mehr als 6 Jahren aktualisiert.

Status:
Erledigt
Priorität:
Normal
Zugewiesen an:
-
Zielversion:
-
Beginn:
21.05.2017
Abgabedatum:
% erledigt:

100%

Geschätzter Aufwand:

Beschreibung

Bei der Fehlersuche umfangreicher Programme wird die ungeordnete Ausgabe der inspizierten Entities sehr unübersichtlich und lässt sich nicht sinnvoll automatisch diffen. Ein QStringList::sort kostet nicht allzuviel (bei ClassInTouch2: t.elapsed() < 1 ms!) und bringt dafür einen deutlichen Zugewinn in der Nutzbarkeit. Dafür muss nur eine Zeile eingefügt werden:

bool EntityInspector::checkRegisteredEntities() {
    QStringList classes = EntityInstanceFactory::getRegisteredClasses();
    classes.sort(); //  <<<<<<<<<<<<<<<<<<<<<<<<<  NEUE ZEILE  <<<<<<<<<<<<<<<<<<<<<<<<<
    QString msg = QDateTime::currentDateTime().toString(Qt::ISODate) +
                  " - Start checking entities\n";
    this->logger->logMsg(msg, MsgType::INFO);
    bool ok = true;
    for (int i = 0; i < classes.size(); ++i) {
        bool r = this->checkEntity(classes.at(i));
        if (!r) {
            msg = "###############################";
            msg += "\n";
            msg += classes.at(i) + " is erroneous!";
            msg += "\n";
            msg += "###############################\n";
            this->logger->logMsg(msg, MsgType::CRITICAL);
            ok = false;
        } else {
            msg = "Entity class " + classes.at(i) + " seems ok.\n";
            this->logger->logMsg(msg, MsgType::INFO);
        }
    }
    msg = QDateTime::currentDateTime().toString(Qt::ISODate) +
          " - End checking entities\n";
    this->logger->logMsg(msg, MsgType::INFO);
    return ok;
}

Dateien

Alle Dateien heruterladen
mergeNonDiffable.png (109 KB) mergeNonDiffable.png Sebastian Diel, 23.05.2017 00:24
mergeDiffable.png (98,7 KB) mergeDiffable.png Sebastian Diel, 23.05.2017 00:24
Aktionen
Link kopieren

Auch abrufbar als: Atom PDF