Codeigniter 计数/如果

Question

我花了很长时间才弄清楚。我正在使用 Codeigniter 查询数据库。 模型会这样做

 function currentfunction($id)
{ 
    $query = $this->db->get_where('mytable', array("id =" => $id));

    if($query->num_rows() > 0){
        return $query->result_array();
    }else {
        return false;
    }
}

控制器

$this->load->model('Display');
$results = $this->Display->currentfunction($id);
$this->load->view('current_items', array('currentitems' => $results));

景色

foreach($currentitems as $row){
              echo $row['name']
              ///....do more
              }

工作正常，除非没有返回行

然后

Message: Invalid argument supplied for foreach()...

我该如何处理 if...else...场景

我试过this Q-A，但对我不起作用。请告知。

Answer 1

只要做：

if(is_array($currentitems)) {
    foreach($currentitems as $row){
              echo $row['name']
              ///....do more
    }
}
else
{
    echo "No items in database!";
}

你得到一个错误，因为 foreach 期望它的第一个参数是一个数组。如果数据库中没有项目，但是您的函数返回 false。

Answer 2

这是因为当您运行代码时：

$query = $this->db->get_where('mytable', array("id =" => $id));

    if($query->num_rows() > 0){
        return $query->result_array();
    }else {
        return false;
    }

当没有返回行时，上面的代码可以正常工作，只是在您尝试运行 for 循环的视图中不起作用。 如果没有返回行，则不应运行此 for 循环。然而，即使没有可使用的行，您仍试图运行 forloop。

我的建议是像这样更改代码：

$query = $this->db->get_where('mytable', array("id =" => $id));

if($query->num_rows() > 0){
    return $query->result_array();
}else {
    $noResults = true;
}

在视图中，您将在 for 循环之前看到类似的内容：

if($noResults != true){
     foreach($currentitems as $row){
        echo $row['name']
        ///....do more
     } 
}
else{
     //do something 
     echo "No items in database!";

}

希望这会有所帮助。

PK

Answer 3

你为什么不这样做：

function currentfunction($id)
{ 
    return $this->db->get_where('mytable', array("id =" => $id));
}

在视图中，如果没有结果，会返回一个空数组，foreach不会抛出错误：

foreach($currentitems->result_array() as $row)
{
    echo $row['name']
    ///....do more
}

更清洁的 IMO。

如果您想在视图中显示错误消息，您可以这样做：

if($currentitems->num_rows() > 0)
{
    foreach($currentitems->result_array() as $row)
    {
        echo $row['name']
        ///....do more
    }
}
else
{
    // Error message
}

这比使用 if/else 两次检查是否有结果要好，就像 halfdan 和 Pavan 建议的那样。