Метод фильтра JavaScript, возвращающий фильтрованный массив и пустой массив

Question

Я пытаюсь зарегистрировать текущий статус чтения, но вместо получения только журнала я также получаю пустой массив при вызове displayInformation(). Как я могу получить результат, который я хочу, без дополнительного пустого массива? и почему он возвращает пустой массив?

function displayInformation() { 
    function statusRead(obj){ 
    if (obj.readingStatus === false) { 
     console.log('You still need to read ' + obj.title + ' by ' + obj.author + '.'); 
    } else { 
     console.log('Already read ' + obj.title + ' by' + obj.author + '.'); 
    } 
    } 

    var statusFilter = library.filter(statusRead); 
    console.log(statusFilter); 
} 

var library = [ 
    { 
     title: 'Bill Gates', 
     author: 'The Road Ahead', 
     readingStatus: true 
    }, 
    { 
     title: 'Steve Jobs', 
     author: 'Walter Isaacson', 
     readingStatus: true 
    }, 
    { 
     title: 'Mockingjay: The Final Book of The Hunger Games', 
     author: 'Suzanne Collins', 
     readingStatus: false 
    } 
]; 

displayInformation(); 

При вызове displayInformation() это, Что записывается в консоль

"Already read Bill Gates byThe Road Ahead." 
"Already read Steve Jobs byWalter Isaacson." 
"You still need to read Mockingjay: The Final Book of The Hunger Games by Suzanne Collins." 
[] 

Answer 1

Как я могу получить результат, я хочу без дополнительного пустого массива?

Вы должны использовать .forEach() или обычный for loop, чтобы делать то, что хотите. .filter() использование случай совсем отличается от вашего.

Почему он возвращает пустой массив?

Поскольку метод .filter() возвращает отфильтрованный массив, он возвращает пустой массив в вашем случае, как ваша callBack функция возвращает undefined все время.

---

Ваш код должен быть таким,

function displayInformation(library) { 
library.forEach(function(obj){ 
    if (obj.readingStatus === false) { 
     console.log('You still need to read ' + obj.title + ' by ' + obj.author + '.'); 
    } else { 
     console.log('Already read ' + obj.title + ' by' + obj.author + '.'); 
    } 
}); 
} 

displayInformation(library); 

---

Чистой для цикла версии,

function displayInformation(library) { 
    var i = 0, len = library.length, obj; 
    for (; i < len; i++) { 
    obj = library[i]; 
    if (obj.readingStatus === false) { 
     console.log('You still need to read ' + obj.title + ' by ' + obj.author + '.'); 
    } else { 
     console.log('Already read ' + obj.title + ' by' + obj.author + '.'); 
    } 
    } 
} 

displayInformation(library); 

Answer 2

Это потому, что вы печатаете из statusFilter, который результат фильтрации с использованием statusRead. Поскольку statusRead никогда не возвращается true, результат будет пустым. Способ, которым работает filter, будет создавать новый массив из старого массива, где он включает в себя все значения, которые возвращают правду. Например, вот как вы можете получить все четные числа из списка чисел.

var evens = numbers.filter(function(x) { 
    return x % 2 === 0; 
}); 

Итак, опять же, так как вы никогда не возвращаются true из вашего filter предиката, вы получаете пустой список, который вы переходите к console.log наружу.

Чтобы просто итерацию по списку, вы должны либо использовать for loop:

for (var i = 0; i < library.length; i++) { 
    var obj = library[i]; 
    ... 
} 

или forEach method:

library.forEach(function(obj) { 
    ... 
});