Wie viele Überprüfungen gegen null sind angemessen?

Question

Ich habe dieses Problem, wenn eine Variable ein Feld fehlt und der Benutzer die Warnung sehen kann, dass diese oder jene Variable diese oder jene Eigenschaft nicht hat. Im einfachen Fall ist es sehr einfach.

if(field) 
    doSomething(field.subField); 

In empirischen Situationen habe ich mich jedoch zu dieser absurden Überprüfung gefunden.

if(!data 
    || !data.records 
    || !data.records[0] 
    || !data.records[0].field 
    || !data.records[0].field.id) 
    return null; 
doSomething(data); 

Ich meine, komm schon - das Rohr-ish-Ding sieht aus wie wenn ich einen Klempner, nicht Entwickler. Ich habe also ein sehr starkes Gefühl, dass meine Schecks, obwohl ausreichend, ein bisschen viel zu übertrieben sein könnten. Gibt es in JS eine Vereinbarung darüber, wann eine Prüfung durchgeführt werden soll?

Answer 1

Ich werde nur mit einer kontroversen Meinung kommen.

In JavaScript, überprüfen Sie nicht auf null Werte an Orten, wo dies realistisch nicht auftreten sollte. Mit anderen Worten, Ihre Idee, jede verschachtelte Eigenschaft auf Nullen zu prüfen, ist etwas übertrieben und dient nur dazu, Ihr Skript zu komplizieren.

Nach meiner Erfahrung habe ich gelernt, lassen Sie einfach den Skriptfehler auftreten. Dies ist etwas kontraintuitiv für jemanden, der C-Code oder Datenbankcode schreibt, wo ein unbehandelter null den Server zum Absturz bringen oder Daten korrumpieren könnte, jedoch in der Skriptwelt, um besser über Ihren Fehler früher als später zu erfahren. Wenn Ihre Seite weiterhin geladen wird, ohne dass angezeigt wird, dass etwas Unerwartetes passiert ist, wird sie sich später in Form von seltsamen Fehlern manifestieren, wenn der Benutzer auf eine Schaltfläche klickt oder ein Formular absendet.

My Advice:

prüfen nullnur wenn Sie bereit sind, etwas dagegen zu tun. Wenn Sie einen Web-Service haben, der möglicherweise eine null zurückgibt, wenn etwas schief geht, dann prüfen Sie das und zeigen Sie eine Fehlermeldung an. Wenn Sie einen Wert ungleich null erhalten, nehmen Sie an, dass es sich um einen gültigen Wert handelt, und fahren Sie fort. Es gibt keinen Grund, Ihr gesamtes Skript mit einer Null-Überprüfung zu versehen, die Ihrem Programm keinen wirklichen Nutzen bringt.

Answer 2

Normalerweise würden Sie sicherstellen, dass wenn ein Objekt existiert, es immer eine Grundmenge von Eigenschaften hat, die es nutzbar machen.

Zum Beispiel, wenn die Variable data überhaupt einen Wert hat, wäre es ein Objekt mit der Eigenschaft records, die immer ein Array ist, auch wenn es leer ist. Wenn das Array irgendetwas enthält, sollte es immer Objekte mit einer field -Eigenschaft geben, die ein Objekt ist, das immer eine id Eigenschaft hat. Das würde die Schecks reduzieren auf:

if (!data || data.records.length == 0) { 
    return null; 
}