ループ内の関数の戻り値をプッシュすることができません

Question

私は素数の配列を返します。この機能を持っている：私は数字の配列内のすべての数字の素因数を取得しにそれをプッシュする

function getPrimeFactors(n) { 
    var factors = []; 
    for (i = 2; i <= Math.sqrt(n); i++) { 
    if (n % i === 0) { 
     var count = 0; 
     while (n % i === 0) { 
     n = n/i; 
     count++; 
     } 
     for (j = 1; j <= count; j++) { 
     factors.push(i); 
     } 
    } 
    } 
    if (n !== 1) { 
    factors.push(n); 
    } 
    return factors; 
} 

var numbers = [2, 3, 4, 5]; 
var array = []; 

を新しい配列（配列）

for(i = 0; i < numbers.length; i++) {  
    array.push(getPrimeFactors(numbers[i])); 
} 

私は間違っていますか？

Answer 1

削除された答えでDamien Goldが述べたように、forループでvarを使用して変数をローカルにし、2つの関数が互いに干渉しないようにします。

function getPrimeFactors(n) { 
    var factors = []; 
    for (var i = 2; i <= Math.sqrt(n); i++) { 
    if (n % i === 0) { 
     var count = 0; 
     while (n % i === 0) { 
     n = n/i; 
     count++; 
     } 
     for (var j = 1; j <= count; j++) { 
     factors.push(i); 
     } 
    } 
    } 
    if (n !== 1) { 
    factors.push(n); 
    } 
    return factors; 
} 

そして：あなたはあなたのgetPrimeFactorsアレイに含まれる値を追加するのではなく、あなたの配列に配列を推進している 追記として

var numbers = [2, 3, 4, 5]; 
var array = []; 

for(var i = 0; i < numbers.length; i++) {  
    array.push(getPrimeFactors(numbers[i])); 
} 

。代わりに結果の数値を1つの配列にプッシュしたい場合は、次のようにしてください：

for(var i=0; i<numbers.length; i++) {  
    array = array.concat(getPrimeFactors(numbers[i])); 
}