回答編集履歴

5と6を追記しました。

2017/08/09 10:12

投稿

raccy

スコア21735

test CHANGED Viewed

@@ -2,6 +2,16 @@
+**【注意】**
+`var`がない`a = a || {}`や`window.a = window.a || {}`という書き方や、`||`の左辺が`a`ではない`var a = window.a || {}`という書き方を否定する訳ではありません。
+この文章はES5以上を想定しており、ES3は一切考慮していません。ES3についてはthink49さんの回答を参考にしてください。
 ###varの巻き上げ
@@ -60,6 +70,10 @@
 4. グローバル変数として定義している。
+5. 他ファイルでグローバル変数として定義している。
+6. 最初からグローバル変数として定義されている。
 それぞれ見ていきます。
@@ -334,8 +348,76 @@
+####5. 他ファイルでグローバル変数として定義している。
+scriptタグで単純に二つのJavaScriptファイルを読み込んだ場合は、グローバル変数が衝突する場合があります。その場合に、既にグローバル変数が定義されていれば、それをそのまま使うというのは有効のように見えます。
+```JavaScript
+var a = 1;
+```
+```JavaScript
+var a = a || {};
+// ...
+```
+しかし、このようにして衝突を回避するという考え方自体が間違っています。そもそも、グローバル変数が衝突しないように即時関数やモジュールベースの書き方を採用すべきです。
+```JavaScript
+(function() {
+  var a = a || {};
+  // ...
+})();
+```
+即時関数の中に書いてしまえば、意味が無いことは既に述べたとおりです。
+全体を即時関数にした場合、ライブラリなのでグローバル変数が定義できないとか思うかもしれませんが、グローバル変数はグローバルオブジェクトのプロパティに過ぎませんので、グローバル変数のプロパティとして定義すれば良いだけです。そのとき、`var`の出番はありませんので、該当のコードが現れる事はありません。
+####6. 最初からグローバル変数として定義されている。
+Polyfillに使う場合、既にグローバル変数があればオブジェクトを再設定する必要はありませんので、ある意味それは正しいように思えます。
+しかし、それは完全なPolyfillとはいえません。理由はthink49さんの回答を見てください。また、グローバル変数名衝突回避のためにコード全体を即時関数で囲っているのであれば、`var`の所では必ず再設定になるため、意味がありません。グローバル変数を見たかったら、`var a = window.a || {}`と書く必要があるでしょう。
 ###まとめ
-よって、私の結論は最初の文に戻ります。`var a = a || {};`というコードは、そもそもコードとして不適切な場所にあるか、無意味な物です。そのため、極めて奇妙なコードに思えます。そのようなコードは良くないサンプルであると判断し、私としては、そのような事が書いてあるサイトや本自体を信頼しません。
+よって、私の結論は最初の文に戻ります。`var a = a || {};`というコードは、そもそもコードとしてどこかが不適切である(非推奨な方法を取っている)か、無意味な物です。そのため、極めて奇妙なコードに思えます。そのようなコードは良くないサンプルであると判断し、私としては、そのような事が書いてあるサイトや本自体を信頼しません。