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

return <value>回调中的forEach是没有意义的。返回的值无处可寻，循环也不会中断。

代替使用.find：

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

当性能很重要并且您有很多书籍时，最好先对书籍进行预处理并创建Set：

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);

forEach循环返回未定义的值

1 个答案: