PHP函数select_History_Date执行后未显示参考链接问题求助

先看你提供的select_History_Date($link)函数，我发现几个关键问题，咱们一个个拆解：

1. 第一条查询结果被意外跳过 #

你在执行mysqli_query之后，先单独执行了一次$row1 = mysqli_fetch_array($ret, MYSQLI_ASSOC);，然后才进入while循环再次调用fetch。这会导致第一条查询结果被提前读取并丢弃，循环只会从第二条数据开始输出，这大概率是你看不到完整记录的核心原因。

2. 错误处理顺序完全颠倒 #

你现在的逻辑是先尝试fetch数据，再判断$ret是否执行失败。如果SQL执行出错，$ret会是false，这时候调用mysqli_fetch_array会直接抛出致命错误，根本走不到后面的错误提示逻辑。正确的做法是在mysqli_query执行后立刻检查是否失败。

3. SQL逻辑可能存在歧义（需结合业务确认） #

你的WHERE条件用了OR：Loaner_Laptop_Id IS NOT NULL OR HDD_Id IS NOT NULL OR Phone_Id IS NOT NULL，这会返回只要其中一个字段非空的记录。如果你的业务需求是「至少有一个设备被借出/归还」，这个逻辑没问题；但如果是要求「三个字段中至少一个非空且存在归还日期」，那可能需要补充AND date_returned IS NOT NULL的条件，这点你可以结合实际场景确认。

4. HTML拼接的细节优化 #

你用<tr>这种实体字符拼接HTML，虽然能生效，但直接写<tr>会更易读（只要后续输出时正确设置页面Content-Type为text/html）。同时，一定要对从数据库取出的字段内容做转义，避免XSS攻击，推荐用htmlspecialchars()处理。

修复后的完整代码示例 #

function select_History_Date($link){
    $status = '';
    // 优化SQL格式，提升可读性
    $sql = "SELECT Student_Id, Loaner_Laptop_Id, HDD_Id, Phone_Id, date_returned 
            FROM equipment_history 
            WHERE Loaner_Laptop_Id IS NOT NULL 
               OR HDD_Id IS NOT NULL 
               OR Phone_Id IS NOT NULL";
    
    $ret = mysqli_query($link, $sql);
    
    // 先检查SQL执行是否失败，用mysqli_error返回具体错误信息（比错误码更直观）
    if (!$ret) {
        // 生产环境建议用日志记录错误，不要直接die暴露敏感信息
        die('Could not execute select statement: ' . mysqli_error($link));
    }
    
    // 直接进入循环，无需提前fetch
    while ($row = mysqli_fetch_array($ret, MYSQLI_ASSOC)) {
        $status .= '<tr>';
        // 对每个字段做转义处理，避免XSS风险
        $status .= '<td>' . htmlspecialchars($row['Student_Id']) . '</td>';
        $status .= '<td>' . htmlspecialchars($row['Loaner_Laptop_Id']) . '</td>';
        $status .= '<td>' . htmlspecialchars($row['HDD_Id']) . '</td>';
        $status .= '<td>' . htmlspecialchars($row['Phone_Id']) . '</td>';
        $status .= '<td>' . htmlspecialchars($row['date_returned']) . '</td>';
        $status .= '</tr>';
    }
    
    // 手动释放结果集（可选，但属于良好编程习惯）
    mysqli_free_result($ret);
    
    return $status;
}

额外小建议 #

• 尽量使用预处理语句（prepared statements），即使当前查询没有用户输入，也能养成避免SQL注入的好习惯。
• 不要在生产环境用die()直接终止程序，应该将错误信息写入日志，给用户返回友好的提示页面。
• 如果date_returned是日期类型，输出时可以用date('Y-m-d H:i:s', strtotime($row['date_returned']))格式化，让显示更友好。

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