当遍历作者列表查找文章中出现超过一次的作者名时，若无作者满足条件，需返回预设默认 id（如 '1899'）；关键在于将默认返回语句置于循环之后，而非嵌套在 else 中。
当遍历作者列表查找文章中出现超过一次的作者名时，若无作者满足条件，需返回预设默认 id（如 '1899'）；关键在于将默认返回语句置于循环之后，而非嵌套在 else 中。
在 PHP 中，return 语句会立即终止当前函数（或脚本）执行并返回值。因此，若将 return '1899' 写在 foreach 循环体内的 else 分支中，第一次迭代不匹配时就会直接返回默认值，后续作者将永远不会被检查——这正是原代码逻辑错误的根本原因。
正确的做法是：先完整遍历所有作者，仅在确认无人满足 count($match[0]) > 1 条件后，再统一返回默认值。即把默认 return 放在循环之外、末尾，作为“兜底逻辑”。
以下是修正后的完整示例：
<?php
function findRepeatedAuthorId($article, $authors) {
foreach ($authors as $id => $author) {
// 使用 preg_quote 防止作者名含正则特殊字符（如 .、+、* 等）
$escapedAuthor = preg_quote($author, '/');
preg_match_all('/\b' . $escapedAuthor . '\b/i', $article, $matches);
if (count($matches[0]) > 1) {
return $id;
}
}
// 所有作者均未满足条件，返回默认 ID
return '1899';
}
// 测试用例
$article = "This is an article written by the three authors - Smith, Jackson, Peters. Peterson is not author. The main author of the article is Jackson.";
$authors = [1 => 'Smith', 2 => 'Jackson', 3 => 'Peters'];
echo findRepeatedAuthorId($article, $authors); // 输出: 2（Jackson 出现 2 次）
// 修改作者名使无匹配
$authors2 = [1 => 'Smith', 2 => 'Phillips', 3 => 'Peters'];
echo findRepeatedAuthorId($article, $authors2); // 输出: 1899
?>✅ 关键改进说明：
立即学习“PHP免费学习笔记（深入）”；
- 将逻辑封装为函数，提升可复用性与可测试性；
- 使用
preg_quote()对作者名转义，避免因姓名含正则元字符（如O'Reilly、C++）导致匹配异常； - 添加
/i修饰符实现大小写不敏感匹配，更符合实际文本场景； - 默认返回值
1899位于循环之后，确保所有作者均被评估完毕。
⚠️ 注意事项：
- 若该代码运行于非函数上下文中（如全局脚本），请确保其处于可返回值的作用域（例如在函数内，或使用
exit()+echo替代return）； -
return在脚本顶层无效，此时应改用echo '1899'; exit;或重构为函数调用； - 如需返回多个匹配 ID，应改为收集结果数组并在循环结束后判断是否为空。
通过这种结构化处理，既保证了逻辑完整性，又增强了代码健壮性与可维护性。
