Dlaczego to nie jest dobry, aby używać do testowania, jeśli argumentem jest przekazywana

Question

Z książki utrzymaniu JavaScript go wymienić:

// Bad: Testing to see if an argument was passed 
function doSomething(arg1, arg2, arg3, arg4){ 
if (arg4 != null){ 
    doSomethingElse(); 
    } 
} 

ale czuję używając != null tutaj jest dość skuteczny, to filtruje zarówno argument nie został przekazany lub jest przekazywany jako nieważny

jakikolwiek powód, dla którego autor sugeruje, że jest on zły?

Answer 1

!= null jest fałszywe dla undefined, w zasadzie oznacza x !== null && x !== undefined. != ma typ przymusu, a jednym z efektów jest to, że traktuje on jako równoważne null i undefined (null == undefined jest prawdziwe).

Więc jeśli tego nie chcesz, niektórzy ludzie nie zalecają używania != z null (lub undefined).

I masz rację, że kod chroni przed dwoma (no właściwie trzy) możliwości:

1. Nic nie został przekazany do argumentu wcale.

2. null został przekazany.

3. undefined został przekazany.

Ale komentarz mówi, że to zły sposób na sprawdzenie, czy argument został przekazany, a to: Bo to nie złapać przechodzącej null lub undefined. (Przechodząc w null i nie przechodząc w wszystko, co, to są różne przypadki.) To nie musi oznaczać, że jest to zły sposób sprawdzenia argumentu dla określonej funkcji.

Answer 2

Ponieważ można przekazać cokolwiek do funkcji, także null. Jeśli chcesz się do następujących:

var obj = new MyObject(); 
// ... 
obj = null; 
// ... 
doSomething(arg1, arg2, arg3, obj); 

Można by minęło argument ale twój sprawdzanie nie powiedzie się, ponieważ minęło odniesienia jest zerowy. Zawsze używaj:

if(typeof arg4 != 'undefined') 
    // ... 

Answer 3

Pewnie. W javascript możesz mieć wartości null i wartości undefined. I nie są one takie same. Szybkie i proste podejście do sprawdzenia czy arg jest ważna będzie prawdopodobnie:

if (arg4)

iw ten sposób sprawdzić Arg4 nie jest zerowa, nie zdefiniowana, a nie fałszywy. Bądź ostrożny z tym ostatnim warunkiem, możesz świadomie poinformować o fałszywej wartości :-)

EDYCJA: T.J. Crowder jest tutaj. Jest łatwy w użyciu i będzie działał w większości przypadków, ale bądź ostrożny.

Answer 4

Gorąco polecam oglądanie http://www.infoq.com/presentations/Null-References-The-Billion-Dollar-Mistake-Tony-Hoare - wspaniałej prezentacji na ten temat od człowieka, który sam ją stworzył.