JavaScriptの配列名にnameを使用すると期待の動作をしない

JSで例えば以下の様に配列名をnameとして、for文で回す処理をするも出力結果が期待値と異なる。

JavaScript
1<!DOCTYPE html>
2<html>
3<head>
4<meta charset="UTF-8" />
5<title>サンプルページ</title>
6</head>
7<body>
8<script type="text/javascript">
9	var name = ['山田', '佐藤', '鈴木'];
10	for(var i in name) {
11		document.write(name[i] + 'です。' + '<br>');
12	}
13</script>
14</body>
15</html>

出力結果
山です。
田です。
,です。
佐です。
藤です。
,です。
鈴です。
木です。

ちなみに配列名をname以外にすると（例えばname1）の様にすると以下の様に期待通りの結果が得られるのですが、こうなる理由が分かりません。もしご存じの方がいらっしゃったら教えて頂けると幸いです。

期待する出力結果
山田です。
佐藤です。
鈴木です。

検証ブラウザ
Chrome

行動規範の内容に同意します

回答2件

ベストアンサー

とりあえず簡単に。「name」を使った場合特有の点を言うと、
・nameがグローバルオブジェクトのプロパティとして、すでに存在している
・グローバルオブジェクトのnameを参照してしまう
・String型に強制される
からです。

「name1」でも文字列型に強制すれば同じことが起きます。

var name1 = ['山田', '佐藤', '鈴木'];
name1 += "";
for(var i in name1) {
    document.write(name1[i] + 'です。' + '<br>');
}

それから、配列にfor-inは絶対に使ってはいけません。for-inはオブジェクトに使うものです。配列に使ってもうまく動くことが多いですが、予期せぬ問題を引き起こす場合もあります。

そこが分かっていらっしゃらないということは、JavaScript初心者の方ではないかと思います。しっかりした本で基礎をきちんと勉強したほうがいいと思います。

投稿2015/09/22 20:39

miu_ras

総合スコア902

y_natsui

2015/09/23 12:04

ご回答頂き有難うございました。シンプルで分かりやすい回答を頂き有難うございます。書籍の中のサンプルが期待の動作をしなかったので、やっとその理由が分かって助かりました。

行動規範の内容に同意します

miu_ras さんの仰る通りなのですが、いくつか補足しておきます。

グローバル変数はグローバルオブジェクト(window)のプロパティでもある為、グローバル変数 name は window.name と衝突します。
window.name は String 型のプロパティとして定義されている為、String 型に強制的に変換されます。
String 型は ES5 から配列のように参照可能になっており、string[0] は string.charAt(0) と同じ値を返します。
従って、グローバル変数 name は String 型に変換され、for-in で1文字ずつ値を返すようになってしまいます。
解決法としては関数内で変数を定義して下さい。
変数名を変更することでも回避可能ですが、window のプロパティは新しい仕様が生まれる度に増えていく運命にあります。
仕様が増える度に変数名が衝突する懸念があるのでローカル変数として定義した方が安全度が高いと思われます。
OKWave で同様の回答をした事がある為、そちらも参考にどうぞ。

JavaScriptをgoogle chromeで - JavaScript | 【OKWave】

for-in はプロトタイプ上のプロパティも拾う上、列挙順がランダムであるという性質があります。
配列の繰り返し文は一般に 0 から順番にインクリメントして走査することを期待する為、for-in での走査は推奨されません。
配列の繰り返しには for 文もしくは Array.protoype.forEach を使用して下さい。

解決の為のコード例を載せておきます。
始めは難しく考えず、for 文を使うことをお勧めします。

JavaScript
1(function () {  // window.name との衝突を避ける為、関数式で括る (いわゆる即時関数)
2  var name = ['山田', '佐藤', '鈴木']; // ローカル変数を定義
3
4  for (var i = 0, l = name.length; i < l; ++i) {
5    document.write(name[i] + 'です。' + '<br>');
6  }
7
8  name.forEach(function (value) {   // Array.protoype.forEach
9    document.write(value + 'です。' + '<br>');
10  });
11
12  document.write(name.map(function (value) {  // Array.protoype.map, Array.protoype.join
13    return value + 'です。';
14  }).join('<br>'));
15}());