Подсчет чисел в случайном массиве

Question

Итак, я пытаюсь найти, сколько из каждого числа от нуля до десяти генерируется в случайном массиве. Я создал случайный список массива

i=0; 
    var ranList=[]; 
    while (i<20){ 
     i++; 
     ranList.push(Math.floor(10*Math.random())); 
    } 
    //count each number 
    document.write(ranList.sort().join("<br>")); 

/*Then I made a function to count /elements from this array 
*/ 


    function ctnumber(array,elem){ 
     var ct=0; 
     var j =0; 
     while(j<array.length) 
     { 
     j++; 
      if(array[j]==elem){ 
      ct+=1;} 
     } 
     } 
     return ct; 
     } 
     alert(ctnumber(ranList,5)); 

Вторая функция не выполняет, ни малейшего представления, почему?

Спасибо!

Answer 1

Во-первых, вы должны избегать использования массива имен для вас переменной: http://www.w3schools.com/js/js_reserved.asp

Ваши скобки тоже неправильно. Измените функцию на это, и она должна работать:

function ctnumber(arr,elem){ 
     var ct=0; 
     var j =0; 
     while(j<arr.length) 
     { 
     j++; 
      if(arr[j]==elem){ 
      ct+=1;} 
     } 
     return ct; 
     } 

Answer 2

Проблема с кодом, как заявил Пардип в своем комментарии, что у вас есть дополнительные } после ct+=1; в вашем втором while цикла.

Правильный код будет выглядеть так: Fiddle

i = 0; 
var ranList = []; 
while (i < 20) { 
    i++; 
    ranList.push(Math.floor(10 * Math.random())); 
} 
//count each number 
document.write(ranList.sort().join("<br>")); 

function ctnumber(array, elem) { 
    var ct = 0; 
    var j = 0; 
    while (j < array.length) { 
    j++; 
    if (array[j] == elem) { 
     ct += 1; // NOTE NO } HERE NOW 
    } 
    } 
    return ct; 
} 
alert(ctnumber(ranList, 5)); 

Я также предлагаю немного кода очистки:

var i = 0; 
var ranList = []; 
while (i < 20) { 
    i++; 
    ranList.push(Math.floor(10 * Math.random()); 
} 

function countNumbers(list, elem) { 
    var count = 0; 
    // For loops are generally more readable for looping through existing lists 
    for (var i = 0; i < list.length; i++) { 
     if (list[i] == elem) { 
      count++; 
     } 
    } 
    return count; 
} 
alert(countNumber(ranList, 5)); 

Обратите внимание, что console.log() является гораздо более эффективным инструментом отладки, она может быть доступна по F12 в Firefox и Chrome/IE.