forEach anstelle von Schleife

Question

Ich habe mit dieser Funktion zu sehen, ob eine bestimmte Zahl in einem Array von Zahlen ist. Wenn es so ist, sollte es wahr oder falsch zurückgegeben werden. Ich habe dieses Stück Code, das unten perfekt funktioniert. Allerdings könnte man es noch mehr (zB Vermeidung einer for-Schleife und stattdessen forEach verwenden?) Vereinfachen

Dank

function findNumber(number){ 
    for (var i=0;i<array.length;i++){ 
    if (array[i] === number){ 
    return true } 
    } 
    return false 
    } 

Answer 1

// I guess var array is already declared 
    function findNumber(number){ 
    let test = false 
    array.forEach(function(nbr){ 
     if(nbr ==(=) number){ 
      test = true 
     }  
    }) 
    if(test == true) { 
     return true 
    } else { 
     return false 
     } 
    } 

Answer 2

Etwas, das semantisch, um Ihren Code ähnlich ist und „functional sieht“ könnte be:

const findNumber = number => array.includes(number); 

Es gibt wirklich keinen Grund, Array.prototype.forEach zu verwenden. Und um ehrlich zu sein, wird der Array.prototype.forEach in einem "wirklich funktionalen Stil" fast nie benutzt (da sein Zweck ist, Nebenwirkungen zu machen, und das ist, was Sie versuchen, mit FP zu vermeiden).

Referenzen:

• Array.prototype.includes()
• Arrow functions

Answer 3

function findNumber(arr, number){ 

var result = false; 

arr.forEach(function(item, index){ 

    if(item === number) result = true; 
}); 

return result; 

}

Btw. Ich denke, es wäre einfacher und einfacher, einfach Array.prototype.indexOf() oder Array.prototype.includes() Methoden zu verwenden.