PHP读取数据库词表循环批量替换单词时代码异常如何解决

现有代码错误点 #

• 语法错误：while($row = mysqli_fetch_array($Fruits))) 行多了1个多余的右括号，不符合PHP语法规范，会直接触发语法报错导致代码无法运行
• 作用域错误：数据库连接对象$mysqli没有传入函数内部，函数局部作用域无法读取全局定义的$mysqli变量，会报未定义变量错误
• 缺少异常判断：未判断数据库查询是否执行成功，当表不存在、连接断开等场景下查询返回false时，后续遍历操作会抛出二次错误

另外你原有实现也存在逻辑问题：第二次替换调用时使用的是原始$content变量，会直接覆盖第一次替换的结果，最终只有orange的替换生效。

修正后可运行代码 #

// 将数据库连接作为参数传入，解决作用域问题
function replaceString ($content, $mysqli){
    // 增加查询失败判断逻辑
    $Fruits = $mysqli->query("SELECT * FROM Fruits ORDER BY id DESC");
    if($Fruits === false) {
        // 可根据业务需要自定义错误处理逻辑，此处直接返回原内容
        return $content;
    }
    // 移除多余的右括号
    while($row = mysqli_fetch_array($Fruits)){
        $content = str_replace($row['word'], $row['replace'], $content);
    }
    return $content;
}

性能优化建议 #

如果待替换的词对数量较大，不需要循环调用str_replace，可以一次性取出所有词对组装成数组，单次调用str_replace即可完成全部替换，执行效率更高：

function replaceString ($content, $mysqli){
    $Fruits = $mysqli->query("SELECT word, `replace` FROM Fruits ORDER BY id DESC");
    if($Fruits === false) return $content;
    $pairs = $Fruits->fetch_all(MYSQLI_ASSOC);
    $search = array_column($pairs, 'word');
    $replace = array_column($pairs, 'replace');
    return str_replace($search, $replace, $content);
}

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