この関数はなぜ未定義に戻っていますか？

Question

私は今、練習をしており、なぜこれが未定義に戻っているのか理解できません。

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

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


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

ご覧のとおり、私は真と偽になるはずですが、代わりに私は定義されていません。これはなぜですか？私はisNaNを正しく理解していないからだと思うが、100％確実ではない。

Answer 1

isNaN return引数がNaNの場合、フィルタ関数はNaNを保持しますが、それ以外はすべて削除します。配列内にNaNがない場合、その長さは0になります。そのような配列の場合、関数はfalseを返します。

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

これはあなたが望むものとは逆のようです。あなたはあなたの関数をeveryと呼んでいましたが、それはsomeと呼ばれるべきです。

他のケースではreturn文を使用しないため、それ以外の場合はundefined（trueではなく）が返されます。唯一のreturnステートメントはifステートメント内にあるため、条件がtrueyでない場合はundefinedを返します。

代替は、あなたのフィルタコールバック内 compareの戻り値を否定することである ​​

（Shomzの答え@参照）、その後、return newArray.length === 0;：それを修正する一つの方法は、それが比較newArray.length === array.lengthの結果を返すように、それを変更することです。

Answer 2

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

これは、新しい配列が空または未定義の場合にのみトリガーされます。それ以外の場合は、関数はundefinedを返します。値が数値でない場合はisNaNがtrueであるため、それを修正してコールバック関数の戻りを取り消してください。

はこちらをご覧ください：

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