ページの読み込みが終わりません。

Question

どこが間違っているのかわかりません。
コンパイルエラーを表示してくれる無料ソフトでもあればいいのに。

s = document.getElementById("sp");
n = new Array();
i = 0;
a = 0;
max = 10;
while(i < max){
	n[i] = Math.floor(Math.random() * max);
	while(a < n.length && i > 0){
		if(n[i] == n[a]){
			i--;
			a = 0;
			break;
		}
		a++;
	}
	i++;
}
while(i > 0){
	s.innerHTML += "<br>" + n[i];
	i --;
}

Answer 1

なにをしようとしているのか分からないので回答しにくいですが、i=1の時に内側のループ2周目でa=1に必ずなるので、if(n[i] == n[a])は必ず真となりますから、i--が実行されループを抜けてi++されてi=1になるので、外側が無限ループになっているのでは。

おそらく、「連続して同じ値を持たない配列」を作ろうとしているのかな、、と思いますので、while(a < n.length-1 && i > 0)とか書いてみてはどうですか。

Comments

[otftrough]

そのとおりでした。
普段、while文では配列の全ての要素を操作していたので、癖で(a < n.length)になってたようです。

Answer 2

firefoxのプラグインであるfirebugを使うとブレークポイントの設置やそのタイミングでの変数の状態等が確認出来るので、手軽にjavascriptのデバッグ環境を整えるにはお勧めです。
参考URL

他のブラウザでも近い機能を持っているアドオンは存在するので、そのあたりは好みで。

Comments

[otftrough]

ページの読み込みが永遠に終わらないのでデバッグできませんでした。

Answer 3

すみません、自分はそんなに書けない人なので大したこと言えませんけど、気がついたことだけ……

while(i < max){ の行と、
while(a < n.length && i > 0){ の行の間に、
alert(i);
と記述して実行すると、ずーっと1 が表示されるんで、無限ループになっちゃってます。その辺りに何かあるのかもしれませんね。
がんばってください＞＜b