未定義配列プッシュ

Question

後、私は、最小の一番高いから山の配列をソートするために取得しようとしているが、印刷しようとしたとき、私は私がすべてのステップが欠落していないと思う

TypeError: mountainArray[i] is undefined 

を取得します解決策が、私は未定義になる。事前に

おかげ以下

コード：

var position; 
 
var height; 
 

 
function Mountain(position, height) { 
 
    this.position = position; 
 
    this.height = height; 
 
} 
 

 
function sortMountain(a, b) { 
 
    return b.height - a.height; 
 
} 
 

 
var mountainArray = []; 
 

 
// game loop 
 
while (true) { 
 
    for (var i = 0; i < 8; i++) { 
 
    var mountainH = parseInt(readline()); // represents the height of one mountain, from 9 to 0. 
 

 
    mountainArray.push(Mountain(i, mountainH)); 
 
    mountainArray.sort(sortMountain); 
 

 

 
    } 
 
    for (var i = 0; i < 8; i++) { 
 
    print(mountainArray[i].position); 
 
    } 
 
}

Answer 1

あなたのクラスが書かれている方法：

function Mountain(position, height) { 
    this.position = position; 
    this.height = height; 
} 

あなたはときnewキーワードを使用する必要がありますあなたはプッシュします：

while (true) { 
    for (var i = 0; i < 8; i++) { 
    var mountainH = parseInt(readline()); // represents the height of one mountain, from 9 to 0. 

    mountainArray.push(new Mountain(i, mountainH)); // HERE 
    mountainArray.sort(sortMountain); 


    } 
    for (var i = 0; i < 8; i++) { 
    print(mountainArray[i].position); 
    } 
} 

コメントで指摘されているように、ループで毎回それを行う必要はありません。