Codeigniter错误循环插入语句时,为键'PRIMARY'复制条目''

时间:2012-10-22 23:15:57

标签: codeigniter activerecord insert foreach

我在codeigniter中设置此功能以作为cron作业运行,并在X时刻记录所有用户。我在选择一个用户并将该用户注销时成功使用了此代码,但是在添加foreach时,我收到以下错误:

Error Number: 1062

Duplicate entry '3858' for key 'PRIMARY'

INSERT INTO `time` (`id`, `projectid`, `phaseid`, `firstname`, `timest`, `status`, `activityid`, `todate`, `remark`, `time`, `workperiod`, `activitydate`, `entrydate`) VALUES ('3858', '212132 Unilever Nigeria  10000L Toothpaste Storage Tanks x 2', '1', 'BONGANI', '1347610976', '1', '1', '1', '1', 3335907, 'nothing', '2012-09-14 10:22:56', '2012-10-23 01:01:23')

Filename: C:\xampp\htdocs\projectfiles\protime\v2\system\database\DB_driver.php

Line Number: 330

这是该模型的代码:

function punch_out() {
    $this -> load -> database();
    $this -> load -> helper('date');
    //get people
    $this -> db -> select('*');
    $this -> db -> from('person');
    $this -> db -> order_by("department", "asc");
    $this -> db -> order_by("userid", "asc");
    $query = $this -> db -> get();
    foreach ($query->result_array() as $row) {
        $people[] = $row;

    }

    foreach ($people as $row) :

    $this -> db -> from('punch');
    $this -> db -> where('firstname', $row['firstname']);
    $this -> db -> order_by("id", "desc");
    $this -> db -> limit(1);
    $query = $this -> db -> get();
    foreach ($query->result_array() as $row) {
        $data = $row;
    }

    $data['workperiod'] = 'nothing';

    $current_time = getdate();

    if ($current_time['wday'] > '0' && $current_time['wday'] < '5') {
        if ($current_time['hours'] > '7' && $current_time['hours'] < '12') {
            $data['workperiod'] = 'stda';
        } else if ($current_time['0'] > strtotime('4:16am') && $current_time < strtotime('11:59pm')) {
            $data['workperiod'] = 'ota';
        }
    } else if ($current_time['wday'] == '5') {
        if ($current_time['0'] > strtotime('7:00am') && $current_time < strtotime('11:59am')) {
            $data['workperiod'] = 'stdb';
        } else if ($current_time['0'] > strtotime('1:01am') && $current_time < strtotime('11:59pm')) {
            $data['workperiod'] = 'otb';
        }
    } else if ($current_time['wday'] == '6') {
        $data['workperiod'] = 'otc';
    } else if ($current_time['wday'] == '0') {
        $data['workperiod'] = 'dtc';
    }

    if ($data['status'] == 1) {
        //MAKE TIME
        $now = time();
        $data['time'] = elapsed_time($data['timest'], $now);
        $data['entrydate'] = timestamp_to_mysqldatetime($now);
        $data['activitydate'] = timestamp_to_mysqldatetime($data['timest']);
        $this -> db -> insert('time', $data);
    }

    endforeach;
}

2 个答案:

答案 0 :(得分:1)

如果您正在尝试更新要插入的相同ID的数据(time,entrydate,activitydate),我认为如果只将数据放入新数组中会更好:

if ($data['status'] == 1) {
    //MAKE TIME
    $now = time();
    $newdata['time'] = elapsed_time($data['timest'], $now);
    $newdata['entrydate'] = timestamp_to_mysqldatetime($now);
    $newdata['activitydate'] = timestamp_to_mysqldatetime($data['timest']);
    $this -> db -> where('id',$data['id']);
    $this -> db -> update('time', $newdata);
}

或者,如果您尝试将新记录插入数据库,只需更改您要插入的阵列名称:

if ($data['status'] == 1) {
    //MAKE TIME
    $now = time();
    $newdata['time'] = elapsed_time($data['timest'], $now);
    $newdata['entrydate'] = timestamp_to_mysqldatetime($now);
    $newdata['activitydate'] = timestamp_to_mysqldatetime($data['timest']);
    $this -> db -> insert('time', $newdata);
}

通过这种编码方式,这可能会避免将来出现更多错误,并可能节省时间。

干杯!

答案 1 :(得分:0)

如果我正确阅读此内容,$data包含有关每个“打卡”的详细信息,包括$data['id'],该打卡的唯一ID。由于您从未从$data['id']中删除$data字段,因此当您使用time作为源插入$data数据库中的条目时,您还要指定新条目与你所汲取的冲击力具有相同的id。尝试运行两次后,您将插入两行id,并且您的错误似乎表明id表上的time字段设置为PRIMARY - 这意味着它不能有任何重复。

我认为,如果我正确理解您的代码和数据结构,那么在运行插入之前解决方案是unset($data['id'])