Kurzes Programm, welches überpüft ob der Benutzer eine Variable "resolution" definiert hat. Wenn ja, wird versucht die Benutzereingabe in eine Zahl umzuwandeln. Wenn nein, wird ein default Wert von 1 angenommen.
Variante 1:
const QString resolution = options().text("resolution");
bool userResolution = false;
// check if user defined cellsize
qreal newCellSize = 1;
if(!resolution.isEmpty()) {
userResolution = true;
bool ok;
const qreal userCellSize = resolution.toDouble(&ok);
if(ok) {
newCellSize = userCellSize;
}
else {
qDebug() << "Cant parse user defined resolution:" << resolution;
}
}
Die Variable userResoltion ist nicht als const declariert und könnte überschrieben werden. Machen wir sie read-only!
Variante 2:
const QString resolution = options().text("resolution");
const bool userResolution = !resolution.isEmpty();
// check if user defined cellsize
qreal newCellSize = 1;
if(!resolution.isEmpty()) {
bool ok;
const qreal userCellSize = resolution.toDouble(&ok);
if(ok) {
newCellSize = userCellSize;
}
else {
qDebug() << "Cant parse user defined resolution:" << resolution;
}
}
Der Code sieht noch etwas aufgebläht aus. Trennen wir den normalen Programm Path und den Error Path, der durchlaufen wird, wenn die Benutzereingabe ungültig ist.
Variante 3:
qreal toDouble(const QString& s) throw(Exception) {
bool ok;
qreal x = s.toDouble(&ok);
if(!ok)
throw Exception("Cant convert to double" + s);
return x;
}
...
const QString resolution = options().text("resolution");
const bool userResolution = !resolution.isEmpty();
// check if user defined cellsize
qreal newCellSize;
try {
newCellSize = toDouble(resolution);
}catch(...) {
newCellSize = 1.0;
if(!resolution.isEmpty())
qDebug() << "Cant parse user defined resolution:" << resolution;
}
Nun möchten wir die Variable newCellSize auch noch read-only haben, da sie ein Parameter ist und niemals mehr geändert werden darf. Dafür lagern wir nun die komplette Logik für das Parser der Benutzereingabe in eine Funktion aus.
Variante 4:
qreal toDouble(const QString& s, qreal defaultValue = 0.0) {
if(s.isEmpty())
return defaultValue;
bool ok;
qreal x = s.toDouble(&ok);
if(!ok) {
qDebug() << "Cant convert to double" + s;
return defaultValue;
}
return x;
}
...
const QString resolution = options().text("resolution");
const bool userResolution = !resolution.isEmpty();
// check if user defined cellsize
const qreal newCellSize = toDouble(resolution, 1.0);
Der wesentliche Code ist jetzt nur noch 4 Zeilen lang, statt 15 Zeilen.
Er ist auf das wesentliche reduziert, verständlicher und gegen das versehentliche überschreiben der Variablen geschützt.
Nachteile gibt es allerdings auch. Die Fehlermeldung wenn das Konvertieren fehl schläng ist nicht anpassbar. Mit einem zustäzlichen Funktionsargument könnte man das aber nachbessern.