javascript - forEach 循环返回未定义的值

Question

我想从 graphiql gui 中查看虚拟数据(书籍)。但是当我使用 forEach 循环遍历书籍，寻找特定的 id 时，它会返回未定义的值，但是如果我使用普通的 for 循环它工作正常。

这是我的代码:

let books = [
    { name: 'Name of the Wind', genre: 'Horror', id: '1', authorID: '3' },
    { name: 'The Final Empire', genre: 'Fantasy', id: '2', authorID: '1' },
    { name: 'The Long Earth', genre: 'Sci-Fi', id: '3', authorID: '2' },
];
const RootQuery = new GraphQLObjectType({
    name: 'RootQueryType',
    fields: {
        book: {
            type: BookType,
            args: { id: { type: GraphQLString } },
            //this forEach is not working
            resolve(parent, args){
                books.forEach( function(book) {
                    if(book.id == args.id) {
                        console.log(book);
                        return book;
                    }
                }); 
            }
        }
    }
});

当我打印图书数据时，它在控制台中显示了该特定图书，但在 GUI 响应中没有显示:

request:
{
  book(id: "2") {
    name
    genre
  }
}
response: 
{
  "data": {
    "book": null
  }
}

Answer 1

A return <value>在forEach回调没有意义。返回值无处可去，循环也没有中断。

改为使用 .find :

return books.find(function(book) {
    return book.id == args.id;
}); 

当性能很重要并且你有很多书时，最好先预处理这些书并创建一个集合:

let books = [
    { name: 'Name of the Wind', genre: 'Horror', id: '1', authorID: '3' },
    { name: 'The Final Empire', genre: 'Fantasy', id: '2', authorID: '1' },
    { name: 'The Long Earth', genre: 'Sci-Fi', id: '3', authorID: '2' },
];
let bookIds = new Set(books.map(({id}) => id));

...然后不需要循环就可以知道图书 ID 是否有效:

return bookIds.has(args.id);