循环插入数据时触发SQLSTATE[HY093]错误：参数未定义

嘿，这个错误我太熟了——本质就是PDO绑定的参数名和你SQL语句里定义的参数名对不上，咱们直接看你代码里的问题点：

错误根源 #

你在执行INSERT的代码块里，有这么一行绑定参数的代码：

$save_genStdsData->bindparam(":$user_code", $user_code);

这里的:$user_code会把变量$user_code的实际值（比如某个学生的school_id，比如S001）拼进去，导致你绑定的参数名变成了:S001，但你的INSERT语句里对应的参数明明是:user_code：

INSERT INTO generated_results (evaluation_type, user_code, f_id, semester, semestral_term, school_year) 
VALUES (:evaluation_type, :user_code, :f_id, :semester, :semestral_term, :school_year)

PDO找不到:S001这个参数的定义，自然就抛出Invalid parameter number的错误了。

修复方案 #

把那行错误的绑定代码改成下面这样，保持参数名和SQL语句里的:user_code完全一致：

$save_genStdsData->bindparam(":user_code", $user_code);

额外优化建议 #

为了避免以后再踩类似的坑，给你几个小提示：

• 提前prepare语句：你现在在循环里每次都执行prepare，这会重复解析SQL语句，效率很低。建议把INSERT的prepare操作移到循环外面，只做一次，然后在循环里绑定参数并执行就行。
• 开启PDO严格错误模式：在数据库初始化的时候加上这行代码，能让PDO抛出更详细的错误信息，方便快速定位问题：

  $this->db->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
  

• 避免变量复用混乱：你在代码里多次复用$user_code变量（一会儿是随机选中的学生ID，一会儿是当前循环的学生ID），很容易造成逻辑混淆，建议给不同场景的变量起不同的名字，比如$selected_user_code和$current_std_code。

修复后的关键代码片段参考：

// 把prepare移到循环外面，提升性能
$save_genStdsData = $this->db->prepare("INSERT INTO generated_results (evaluation_type, user_code, f_id, semester, semestral_term, school_year) VALUES (:evaluation_type, :user_code, :f_id, :semester, :semestral_term, :school_year)");

for ($i = 1; $i <= $select_stds_evaluation->rowCount(); $i++) { 
    $select_stds_evaluation_data = $select_stds_evaluation->fetch(PDO::FETCH_ASSOC); 
    $selected_user_code = $select_stds_evaluation_data['user_code']; 
    $showStds = $this->db->prepare("SELECT * FROM students WHERE school_id = :user_code"); 
    $showStds->bindparam(":user_code", $selected_user_code); 
    $showStds->execute(); 

    while ($stdsData = $showStds->fetch(PDO::FETCH_ASSOC)) { 
        try { 
            $evaluation_type = "student"; 
            $current_std_code = $stdsData['school_id']; 
            $f_id = $faculty_data['f_id']; 

            // 绑定正确的参数
            $save_genStdsData->bindparam(":evaluation_type", $evaluation_type); 
            $save_genStdsData->bindparam(":user_code", $current_std_code); 
            $save_genStdsData->bindparam(':f_id', $f_id); 
            $save_genStdsData->bindparam(":semester", $today_sem); 
            $save_genStdsData->bindparam(":semestral_term", $today_sem_term); 
            $save_genStdsData->bindparam(":school_year", $today_sy); 
            $save_genStdsData->execute(); 
        } catch (PDOException $ex) { 
            echo $ex->getMessage(); 
        } 
        // ... 其他代码
    }
    // ... 其他代码
}

内容的提问来源于stack exchange，提问作者H. Ferrer