CodeIgniter Active Record一次删除多个记录

时间:2014-11-07 21:47:39

标签: php arrays codeigniter

这是我的控制器功能之一。它从名为' users'的数据库表中抓取所有行。并将其放入数组中。然后它加载一个名为' deleteuser'通过数组数据传递。

function deleteuser(){



    $this->load->model('users');
    $employees['staff'] = $this->users->getStaff();
    $this->load->view('deleteuser', $employees);
}

在deleteuser视图中,有一个从数组生成的复选框类型输入。表单操作调用一个名为remove的函数。

<form action="remove" method = "post">

<?php foreach($staff as $row): ?>
<div class="checkbox">
        <label>
        <input type="checkbox" name="delete[]" value="<?php echo $row->id ?>" />
        <?php echo $row->lastName . ", " . $row->firstName; ?>
        <br />
        </label> 

        <?php endforeach; ?>
        <br /> 
        <br />
        <input type="submit" name = "remove" value = "Delete" class = "btn btn-danger btn-lg pull-left" />

    </div>

remove函数从输入中获取delete数组并将其传递给名为deleteUser的模型函数。

function remove(){
    $data = $this->input->post('delete');

    $this->users->deleteUser($data);

}

现在这就是我遇到麻烦的地方。下面是deleteUser的模型函数,但它给出了一个未定义的偏移量:1个错误。任何帮助将不胜感激。

function deleteUser($data)
{
    if ($data) {
        for ($i = 0; $i <= count($data); $i++)
        {
        $this->db->where('id', $data[$i]);
        $this->db->delete('users');
        }
    }
}

2 个答案:

答案 0 :(得分:8)

虽然@DamienPirsy的回答在解决您的问题时是正确的,但我建议采用另一种方法。我会一次删除所有记录,而不是循环。这将最大限度地减少您对数据库的查询次数。

function deleteUser($data)
{
    if (!empty($data)) {
        $this->db->where_in('id', $data);
        $this->db->delete('users');
    }
}

答案 1 :(得分:4)

删除相等的=符号,应为

for ($i = 0; $i<count($data); $i++) 

你提取1个元素超出范围。即,如果count($data)为4,则你循环:0 1 2 3 4(&#34;计数次要或等于4&#34;),这确实是五个元素:)