、それは使用するのはよくないですなぜ引数が本から

Question

を渡されたかどうかをテストするためにメンテナンス可能JavaScriptはそれが言及：

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

が、私はここに!= nullを使用すると、非常に効果的であると感じ、それを濾過し、両方の引数が渡されなかった場合、またはヌルとして渡された場合

著者は、それが悪いと示唆する何らかの理由はありますか？

Answer 1

!= nullは、undefinedの場合はfalse、基本的にはx !== null && x !== undefinedを意味します。 !=はタイプ強制を行い、その効果の1つはnullとundefinedを等価（null == undefinedが真）として扱うことです。

あなたが望む場合を除き、!=とnull（またはundefined）を使用しないでください。

そして、あなたはコードが2つ（だけでなく実際には3つ）の可能性を防御することを非常におっしゃるとおり

1. 何もすべてで引数のために渡されませんでした。

2. nullを通過した。

3. undefinedを渡した。それはnullまたはundefinedを渡してキャッチしていないので：

しかし、コメントは、それは引数が渡​​されたかどうかをテストするための悪い方法だし、それがあると言います。 （nullに合格し、のいずれかを入力しない場合は、が異なる場合があります）。必ずしも特定の関数の引数をチェックするのは悪い方法です。

Answer 2

関数に何かを渡すことができるため、nullもあります。以下に、あなたが希望した場合：

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

あなたは、引数に合格しているだろうが、渡された参照がヌルあるので、あなたのチェックが失敗します。以下を必ずご使用ください：

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

Answer 3

確かに。 JavaScriptでnullの値とundefinedの値を持つことができます。そして彼らは同じではありません。

if (arg4)

そして、あなたはARG4を確認し、この方法は偽、未定義のヌルではないではない、とない：argが有効であるかどうかを確認するための迅速かつ簡単なアプローチはおそらくだろう。この最後の状態に注意してください。あなたは意識的に誤った値をここに伝えるかもしれません:-)

EDIT：T.J.クラウダーがここにあります。これを使うのは簡単ですが、ほとんどの場合はうまくいくでしょうが、注意してください。

Answer 4

http://www.infoq.com/presentations/Null-References-The-Billion-Dollar-Mistake-Tony-Hoareをご覧になることを強くお勧めします。この件については、自分で作成した人からの素晴らしいプレゼンテーションです。