使用Express过滤已读书籍数组失败，请求问题排查与代码修正

一、for循环方法的问题 #

你的printBookList函数输出所有元素，核心问题是循环变量i没有用let/var/const声明，导致i成为全局变量。如果调用函数前全局作用域里的i被修改过（比如其他循环改动了i的值），会导致循环的起始或终止条件异常，引发不可预期的输出结果。

虽然判断逻辑bookList[i].alreadyRead == completedReadingBook本身没问题，但全局变量i会破坏循环的独立性。

二、filter方法的问题 #

你的filter写法存在多处错误：

• 完全不需要外层for循环：filter方法本身会遍历整个数组并返回符合条件的元素集合，外层循环属于冗余操作。
• 循环内部直接return：第一次循环就会终止代码执行，后续逻辑根本不会运行。
• 错误使用filtered数组：filtered是筛选后的元素数组，你用bookList[filtered]取值是错误的——数组索引只能是数字，用数组作为索引会得到undefined，自然输出空结果。
• 未定义completedReadingBook变量：代码里没有传入或声明这个变量，导致筛选条件等价于和undefined比较，所以返回空数组。

正确解决方案 #

方案1：修复后的for循环写法 #

给循环变量i加上let声明，避免全局变量污染，确保每次调用函数时循环都从0开始：

function printBookList(bookList, completedReadingBook){
    for(let i = 0; i < bookList.length; i++){ // 用let声明i，限制作用域
        if (bookList[i].alreadyRead === completedReadingBook) {
            console.log(`${bookList[i].title} by ${bookList[i].author}`);
        }
    }  
}

// 调用测试
printBookList(bookList, true);
// 输出：
// Harry Potter by J.K. Rowling
// Zero to One by Peter Thiel

这里用===（严格相等）代替==，是更规范的写法，能避免类型转换带来的潜在问题。

方案2：正确使用filter方法 #

先通过filter筛选目标书籍，再遍历筛选后的数组输出：

function printReadBooks(bookList) {
    // 筛选出已读书籍
    const readBooks = bookList.filter(book => book.alreadyRead === true);
    // 遍历输出格式内容
    readBooks.forEach(book => {
        console.log(`${book.title} by ${book.author}`);
    });
}

// 调用测试
printReadBooks(bookList);
// 输出：
// Harry Potter by J.K. Rowling
// Zero to One by Peter Thiel

如果需要支持灵活筛选已读/未读，可以改成：

function printFilteredBooks(bookList, isRead) {
    const filteredBooks = bookList.filter(book => book.alreadyRead === isRead);
    filteredBooks.forEach(book => {
        console.log(`${book.title} by ${book.author}`);
    });
}

// 筛选已读
printFilteredBooks(bookList, true);
// 筛选未读
printFilteredBooks(bookList, false);

内容的提问来源于stack exchange，提问作者yeeee