PHP中do while循环下函数重声明错误解决咨询

这个错误太常见啦——cannot redeclare exists_in_db 本质是你的exists_in_db函数被定义了不止一次，PHP不允许同一个作用域里重复定义同名函数。下面给你一步步解决：

核心原因&解决方案 #

1. 检查文件引入方式 #

如果这段代码所在的文件被你用include或require多次引入（比如在多个页面或者循环里），立刻换成include_once或require_once。这两个函数会确保文件只被加载一次，从根源避免函数重复定义。

2. 给函数加存在性校验 #

哪怕你不确定文件会不会被重复引入，也可以在定义函数前先判断它是否已经存在，这样绝对安全：

if (!function_exists('exists_in_db')) {
    function exists_in_db($str) {
        // 记得替换成你的实际数据库连接和查询语句
        global $conn; // 或者直接传入连接实例
        $check_badge_in_paids = mysqli_query($conn, "SELECT * FROM your_table WHERE badge_code = '$str'");
        return mysqli_num_rows($check_badge_in_paids) > 0;
    }
}

3. 别在循环/重复执行的代码块里定义函数 #

如果你的函数定义写在了循环、条件判断或者某个会多次执行的代码段里，赶紧把它移到全局作用域（比如文件最顶部），确保只定义一次。

额外优化：解决潜在的SQL注入问题 #

顺便提一句，你原来的代码直接把$str拼进SQL语句里，有严重的SQL注入风险！建议改成预处理语句，既安全又稳定：

if (!function_exists('exists_in_db')) {
    function exists_in_db($str, $conn) {
        // 预处理语句避免注入
        $stmt = mysqli_prepare($conn, "SELECT 1 FROM your_table WHERE badge_code = ?");
        mysqli_stmt_bind_param($stmt, "s", $str);
        mysqli_stmt_execute($stmt);
        mysqli_stmt_store_result($stmt);
        return mysqli_stmt_num_rows($stmt) > 0;
    }
}

function rand_string($conn) {
    $str = "";
    do {
        // 也可以用更可控的随机字符串生成方式，比如用str_shuffle
        $str = substr(md5(microtime()), 0, 5);
    } while(exists_in_db($str, $conn));
    return $str;
}

这样改完，不仅解决了函数重复声明的错误，还能避免安全问题，随机字符串的唯一性校验也更可靠~

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