Der richtige Einsatz von versuchen .. fangen

Question

Sollte ich diese Methode zu werfen Fehler werden:

if (isset($this->dbfields[$var])) { 
    return $this->dbfields[$var]; 
} else { 
    throw new FieldNotFoundException($var); 
} 

oder diesen Stil:

try { 
    return $this->dbfields[$var]; 
} catch (Exception $e) { 
    throw new FieldNotFoundException($var); 
} 

... oder etwas ganz anderes?

schnelle Erläuterung des Codes:$this->dbfields ist ein Array. isset() prüft, ob eine Variable gesetzt ist, in diesem Fall, ob das Array-Element existiert.

Answer 1

Das zweite Beispiel ist schlecht. Sie benötigen eine Menge Aufwand, um eine Ausnahme zu finden, wenn Sie, wie Sie demonstrieren, die Ausnahmebedingung von vornherein genauso einfach verhindern können. Außerdem wird davon ausgegangen, dass Sie wissen, warum diese Ausnahme ausgelöst wurde. Wenn es eine andere Ausnahme gab, z. B. nicht genügend Arbeitsspeicher oder etwas, melden Sie dies als "Feld nicht gefunden", auch wenn dies nicht der Fall war.

Denken Sie daran, dass try/catch in Sprachen wie C++ und Java sehr teuer sind, da sie alle speichern und wiederherstellen müssen. Python andererseits hat sehr billige Ausnahmen und sie ermutigen Sie dazu, eine try/except für einfache Validierung zu verwenden. Aber trotzdem ist es immer noch schlecht, alles einzufangen und so zu tun, als wäre es eine Ausnahme.

Answer 2

Fangen „Exception“ nicht der Fall, als die meiste Zeit, eine gute Praxis, aus der zwei, die angezeigt werden, würde ich Option 1.

verwenden, um alle Ausnahmen fangen kann eine andere Ausnahme verstecken und maskieren sie als eine FileNotFoundException.

Answer 3

//First let's do the checks. 
if(!isset($this->dbfields[$var])) 
    throw new FieldNotFoundException($var); 
//Now we're in the clear! 
return $this->dbfields[$var]; 

Answer 4

Ich ziehe die ersten, aber wenn dbfields [var $] etwas Vernünftiges wirft, wenn Sie ein nicht vorhandenes Element zugreifen, dann würde ich es vorziehen, zurückkehr es einfach, ohne zu überprüfen.

Ich mag es nicht besonders, den Ausnahmetyp zu ändern, es sei denn, ich habe einen guten Grund - auch wenn Sie dies tun, sollten Sie versuchen, die ursprüngliche Ausnahme und Stack-Trace beizubehalten.

Answer 5

Lesen Sie Ihre Erklärung erneut.

Ich denke, Ihre Methode dort in # 1 wird alle Ausnahmen fangen, die geworfen werden können und einfach einen Bool zurückgeben. Ich mag es definitiv nicht, die generische Ausnahme die meiste Zeit zu fangen, also wäre # 2 nicht meine Wahl.

Answer 6

"... oder etwas ganz anderes?"

Weder ist sehr gut, so etwas anderes wäre angemessen.

Fix Version 2, um die richtige Ausnahme zu finden, nicht jede mögliche Ausnahme. Poste das als Option 3. Ich werde etwas upzotten, das eine bestimmte Ausnahme statt Exception fängt.

Answer 7

Dies ist alles andere als sprachunabhängig.

Einige Sprachen werden keine Fehler für den Zugriff auf nicht-existente Felder werfen, und das bevorzugte Muster hängt viel von den Implementierungen der Arrays, Tabellen, Objekte usw.