2011-12-20 13 views
0

ボタンを押して最初から最後まで実行するバスケットボールの武道を設定しています。しかし、エラーコンソールには私のTeamarrayが定義されていないと言われています。私のコードで何が間違っていますか?バスケットボールトーナメントシミュレーション:配列が定義されていません

function teamArray(list) 
//Assumes: all basketball teams 
//Returns: a random team from the list 
{ 
var teamArray, randomIndex; 

teamArray = ['Sales', 'HR', 'Accounting', 'Marketing', 'Engineering', 'Manufacturing', 'IT', 'Legal']; 

randomIndex = RandomInt(0, list.length-1); 

index=0; 
while (index < teamArray.length) { 
teamArray[index] = randomIndex; 
index = index * 1; 
} 

return teamArray; 

document.getElementById('team1').value = teamArray[0]; 
document.getElementById('team2').value = teamArray[1]; 
document.getElementById('team3').value = teamArray[2]; 
document.getElementById('team4').value = teamArray[3]; 
document.getElementById('team5').value = teamArray[4]; 
document.getElementById('team6').value = teamArray[5]; 
document.getElementById('team7').value = teamArray[6]; 
document.getElementById('team8').value = teamArray[7]; 

} 

EDIT:あなたのコードで

<input type="button" value="Start Tournament" onclick="teamArray(list); AddBonus(); TotalScore(); Winner(); "> 
+0

エラーはどこにありますか? –

+0

関数からteamArrayを返して、その後にdomでいくつかの変更を加えます!次に、正しいスペルを使用していることを確信していますか(何度も起こりましたが、それは私に何度か起こりました) –

+0

あなたのボタン。あなたのコンソールに "T"と小文字の "a"のような "Teamarray"と表示されたら、それが最初の問題です。JavaScriptは大文字と小文字を区別し、あなたの機能は "TeamArray"ではなく "teamArray"です。それ以外にも、以下で説明するRobGを含む(ただしこれに限定されない)、機能には多くの問題があります。 – nnnnnn

答えて

0

> randomIndex = RandomInt(0, list.length-1); 

RandomeInt復帰? 0とlist.length-1の間の乱数ですか?それはグローバル変数になり、この時点でのよう

> index=0; 

識別子indexが宣言されていません。 1で番号を掛ける

> while (index < teamArray.length) { 
>  teamArray[index] = randomIndex; 
>  index = index * 1; 

だけで、おそらくあなたは意味、これは無限ループを生成する必要があり、番号を返します。

index += 1; 

または

++index; 

を。

} return teamArray;

返却後のコードはありません。私はあなたが何をしようとしているのか分からず、teamArrayは何を期待していますか?数字の配列ですか?元のteamArrayのメンバーのうち?宣言されたメンバーを数字で置き換えているようです。

+0

また、ループ( '* 1'が固定された後)は配列のすべての要素を同じ乱数で置き換えます。 – nnnnnn

関連する問題