Warum gibt diese Funktion undefiniert zurück?

Question

Ich mache gerade eine Übung, und ich kann nicht herausfinden, warum dies undefiniert zurückkehrt.

Wie Sie sehen können, soll ich wahr und falsch werden, aber stattdessen werde ich undefiniert und undefiniert. Warum ist das der Fall? Ich denke es liegt daran, dass ich IsNaN nicht richtig verstehe, aber ich bin mir nicht 100% sicher.

Answer 1

isNaN zurück wahr, wenn es Argument ist NaN, das bedeutet, dass Ihre Filterfunktion NaN hält, aber alles andere entfernt. Seine Länge ist nur 0, wenn das Array keine NaNs enthielt. Für solch ein Array gibt Ihre Funktion false zurück:

console.log(every([4], isNaN)); // false 

Das scheint das Gegenteil von dem zu sein, was Sie wollen. Sie haben Ihre Funktion every aufgerufen, aber es verhält sich eher so, als sollte es some heißen.

Für alles andere gibt es undefined (statt true) zurück, weil Sie in keinem anderen Fall eine Rückgabeanweisung haben. Ihre einzige return-Anweisung befindet sich in der if-Anweisung. Wenn die Bedingung nicht truthy ist, wird undefined zurückgegeben. Ein Weg, um es zu beheben ist, es zu ändern, so dass sie das Ergebnis des Vergleichs gibt newArray.length === array.length:

function every (array, compare){ 
    var newArray = array.filter(
     function (property){ 
      return compare(property); 
     } 
    ); 

    return newArray.length === array.length; 

} 


console.log(every([NaN, NaN, NaN], isNaN)); 
// → true 
console.log(every([NaN, NaN, 4], isNaN)); 
// → false 

Die Alternative ist, den Rückgabewert von compare in Ihrem Filter Rückruf zu negieren (siehe @ Shomz Antwort) und dann return newArray.length === 0; .

Answer 2

if (newArray === undefined || newArray.length == 0) 
    return false; 

Dies wird nur ausgelöst, wenn Ihr neues Array leer oder undefiniert ist. Andernfalls gibt die Funktion undefined zurück. Korrigieren Sie das und kehren Sie auch die Rückgabe Ihrer Callback-Funktion um, weil isNaN wahr ist, wenn ein Wert keine Zahl ist.

Siehe hier:

function every(array, compare) { 
 
    var newArray = array.filter(
 
    function(property) { 
 
     return !compare(property); 
 
    } 
 
); 
 
    return newArray.length == 0; 
 
} 
 

 

 
console.log(every([NaN, NaN, NaN], isNaN)); 
 
// → true 
 
console.log(every([NaN, NaN, 4], isNaN)); 
 
// → false