本文详解如何在 codeigniter 中正确实现“签到/签到/签退”双态逻辑，重点解决因未传递日期参数导致的 `checktimein()` 始终返回 false、进而重复插入签到记录的问题，并提供健壮的模型层校验与事务安全建议。
在开发考勤系统（如车辆进出登记）时，一个常见需求是：同一车牌号（V_ID）在当天内仅允许一次签到（TIMEIN），再次扫描应触发签退（TIMEOUT）。但如你所见，当前代码始终新增记录而非更新，根本原因在于模型方法 checktimein() 的调用与定义存在关键参数缺失——它在查询中依赖 $date 变量过滤当日记录，却未在控制器中传入该值。
? 问题定位：$date 变量未传递
查看模型中的 checktimein() 方法：
public function checktimein($v_id){
$row = $this->db->where('V_ID', $v_id)
->where('LOGDATE', $date) // ⚠️ 这里使用了未定义的 $date！
->where('STATUS', 0)
->get('attendance')
->num_rows();
return $row > 0;
}此处 $date 是未声明的局部变量，PHP 会静默转为 NULL 或空字符串，导致 WHERE LOGDATE = NULL 永远不匹配有效数据，num_rows() 恒为 0 → checktimein() 恒返回 FALSE → 系统永远执行 save() 而非 update()。
✅ 正确修复方案
1. 修改控制器：显式传入 $date
在 save() 方法中，调用时补充日期参数：
// ✅ 修正：传入 $date $signin = $this->attendance_model->checktimein($v_id, $date);
2. 更新模型方法签名与实现
// ✅ 修正：增加 $date 参数，并使用严格比较
public function checktimein($v_id, $date) {
$this->db->where('V_ID', $v_id);
$this->db->where('LOGDATE', $date); // 确保匹配当日
$this->db->where('STATUS', 0); // 仅检查未签退的签到记录
return $this->db->count_all_results('attendance') > 0;
}? 提示：推荐使用 count_all_results() 替代 get()->num_rows()，更高效且避免加载完整结果集。
3. 强化 save() 和 update() 的健壮性
当前 save() 方法存在风险：未限制唯一性，可能产生多条同日同 ID 记录。建议改为「先查后插」或添加数据库唯一索引（V_ID + LOGDATE 组合唯一）。同时 update() 应精准定位待更新行：
// ✅ 改进 save()：避免重复插入（可选）
public function save($data) {
// 先检查是否已存在当日签到（防御性设计）
$exists = $this->db->where('V_ID', $data['V_ID'])
->where('LOGDATE', $data['LOGDATE'])
->count_all_results('attendance');
if ($exists === 0) {
return $this->db->insert('attendance', $data);
}
return FALSE; // 已存在，拒绝插入
}
// ✅ 改进 update()：明确更新最新一条未签退记录（防多条并存）
public function update($data) {
$this->db->where('V_ID', $data['V_ID']);
$this->db->where('LOGDATE', $data['LOGDATE']); // 限定当日
$this->db->where('STATUS', 0);
$this->db->order_by('ID', 'DESC'); // 假设 ID 自增，取最新签到
return $this->db->update('attendance', $data);
}4. （进阶）添加事务保障（推荐）
为防止并发场景下重复签到，可在控制器中启用事务：
$this->db->trans_start();
$signin = $this->attendance_model->checktimein($v_id, $date);
if ($signin) {
$this->attendance_model->update(['V_ID'=>$v_id, 'TIMEOUT'=>$time, 'STATUS'=>1]);
} else {
$this->attendance_model->save(['V_ID'=>$v_id, 'TIMEIN'=>$time, 'LOGDATE'=>$date, 'STATUS'=>0]);
}
$this->db->trans_complete();
if ($this->db->trans_status() === FALSE) {
$this->session->set_flashdata('error', '操作失败，请重试');
} else {
$this->session->set_flashdata('success', $signin
? "成功签退 {$v_id}"
: "成功签到 {$v_id}");
}? 总结与最佳实践
- 必做：所有依赖日期的业务逻辑，必须显式传参，禁止依赖未声明变量；
- 推荐：在数据库层面添加 UNIQUE KEY idx_vdate (V_ID, LOGDATE)，从源头杜绝脏数据；
- 注意：date('h:i:s A') 生成 12 小时制时间（含 AM/PM），确保数据库字段类型为 VARCHAR 或 TIME 并兼容；若用 TIME 类型，建议改用 date('H:i:s')（24 小时制）；
- 扩展：可增加超时自动签退（如当日 18:00 未签退则系统补签），通过定时任务 + STATUS=0 AND TIMEOUT IS NULL 查询实现。
遵循以上修正，你的考勤系统将准确识别“当日已签到”，真正实现“一卡一签到、二次即签退”的核心逻辑。
