PHPMailer遇错误邮箱时循环终止问题及修复需求

嘿，你的推测完全精准！问题核心就是发送失败时没更新对应记录的sent状态，导致每次循环都揪着这条无效邮箱不放，直接陷入死循环。下面是修复后的代码，还顺手优化了几个细节：

修复后的完整代码 #

// 批量发送邮件逻辑
$continueLoop = true;

while ($continueLoop) {
    // 每次取一条未发送的邮件
    $result = $db->query("SELECT * FROM emails WHERE sent = '' AND mailto <> '' ORDER BY id ASC LIMIT 1");

    if (mysqli_num_rows($result) === 0) {
        $continueLoop = false;
        echo "所有待发送邮件处理完毕！\n";
        continue;
    }    

    $row = $result->fetch_assoc();
    $emailId = $row["id"];
    $recipient = $row["mailto"];

    // 加载当前邮件的内容
    $mailer->AddAddress($recipient);
    $mailer->Subject = $row["mailsubject"];
    $mailer->Body = $row["mailbody"];

    try {
        // 尝试发送邮件
        $mailer->send();
        
        // 标记为发送成功
        $db->query("UPDATE IGNORE emails SET sent='yes', date=NOW() WHERE id='{$emailId}'");
        echo "✅ 邮件发送成功：{$recipient}\n";
    } catch (Exception $e) {
        $errorInfo = $mailer->ErrorInfo;
        echo "❌ 邮件发送失败：{$recipient}，错误原因：{$errorInfo}\n";
        
        // 重中之重：标记为发送失败，避免下次循环再处理这条记录
        $db->query("UPDATE IGNORE emails SET sent='no', date=NOW(), error_msg='{$errorInfo}' WHERE id='{$emailId}'");
        
        // 重置SMTP连接，防止错误状态影响下一封邮件
        $mailer->getSMTPInstance()->reset();
    }

    // 清理当前邮件的地址和附件，为下一封做准备
    $mailer->clearAddresses();
    $mailer->clearAttachments();

    usleep(100000); // 休眠0.1秒，避免触发邮件服务商的频率限制
}

关键修复点 #

1. 在catch块中强制更新数据库状态：发送失败时必须把sent设为'no'（我额外加了error_msg字段用来存储错误信息，你可以根据自己的表结构选择保留或删除），这样这条无效邮箱记录就不会再被查询到，循环就能正常处理后续邮件了。
2. 优化变量命名：把$id改成$emailId，$row["mailto"]存到$recipient变量里，代码可读性更高，调试也更方便。
3. 直观的状态提示：用✅和❌标记成功/失败状态，能实时看到发送进度，排查问题更高效。

额外的优化建议 #

• 防范SQL注入风险：当前直接把$emailId拼进SQL的做法存在安全隐患，建议换成预处理语句，示例如下：

// 发送成功的更新操作
$stmt = $db->prepare("UPDATE IGNORE emails SET sent='yes', date=NOW() WHERE id=?");
$stmt->bind_param("i", $emailId);
$stmt->execute();
$stmt->close();

// 发送失败的更新操作（如果保留error_msg字段）
$stmt = $db->prepare("UPDATE IGNORE emails SET sent='no', date=NOW(), error_msg=? WHERE id=?");
$stmt->bind_param("si", $errorInfo, $emailId);
$stmt->execute();
$stmt->close();

• 批量查询提升效率：每次只查询一条记录的效率偏低，你可以改成一次查询10条（LIMIT 10），然后循环处理这批记录，减少数据库交互次数。
• 日志记录补充：除了打印到控制台，建议把错误信息写入日志文件，方便后续回溯和排查问题。

这样修改后，遇到无效邮箱时会自动标记为失败并跳过，剩下的邮件能正常发送，再也不会陷入死循环啦！

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