回答編集履歴
1
校正
test
CHANGED
@@ -1,4 +1,4 @@
|
|
1
|
-
Node.jsのコールバック関数は、
|
1
|
+
JavaScriptやNode.jsのコールバック関数は、第一引数が`Object or null`のerror引数という慣習がある。
|
2
2
|
|
3
3
|
更にerror引数はコールバックの第一引数として定義されるので、変数未定義のエラーは原則ありえない。
|
4
4
|
|
@@ -6,7 +6,7 @@
|
|
6
6
|
|
7
7
|
また、[truthy - MDN](https://developer.mozilla.org/ja/docs/Glossary/Truthy)で紹介されているように、紛らわしいエラーが起こる事もまずない。
|
8
8
|
|
9
|
-
(現場に慣習を無視して第一引数に
|
9
|
+
(現場に慣習を無視して第一引数にNumberやStringを返すようなエンジニアが居れば話は別)
|
10
10
|
|
11
11
|
|
12
12
|
|
@@ -20,7 +20,7 @@
|
|
20
20
|
|
21
21
|
結果、記述量を減らす為に以下の書き方をするエンジニアが多い。
|
22
22
|
|
23
|
-
私もコールバック関数の[ガード節](http://qiita.com/kaneko_tomo/items/43e824411076bbd757b8)で記載する場合は
|
23
|
+
私もコールバック関数の[ガード節](http://qiita.com/kaneko_tomo/items/43e824411076bbd757b8)で記載する場合はこれ一択。
|
24
24
|
|
25
25
|
|
26
26
|
|
@@ -40,9 +40,7 @@
|
|
40
40
|
|
41
41
|
|
42
42
|
|
43
|
-
変数未定義エラーを回避することは出来ないけど、
|
44
|
-
|
45
|
-
undefinedとnullを一気に確認するイディオムがある。
|
43
|
+
更にundefinedとnullを一気に確認するイディオムがある。
|
46
44
|
|
47
45
|
多少厳密にチェックしたいエンジニアはこちらの書き方を採用するケースも。
|
48
46
|
|
@@ -152,6 +150,6 @@
|
|
152
150
|
|
153
151
|
|
154
152
|
|
155
|
-
下の書き方がNGな理由は、undefinedは予約語ではないので
|
153
|
+
下の書き方がNGな理由は、undefinedは予約語ではないので別の値に上書きされる可能性があるから。
|
156
154
|
|
157
155
|
以前の質問で別の人が紹介してた理由だね。
|