Codeigniter Count/Si

Question

cela me prend trop de temps pour comprendre. J'utilise Codeigniter pour interroger une base de données. Le modèle fait ce

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

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

Le contrôleur

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

La vue

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

fonctionne très bien SAUF SI aucune ligne sont renvoyés

puis

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

Comment puis-je gérer l'if ... else ... scénario

J'ai essayé this Q-A, mais ne fonctionne pas pour moi. PlsHlp.

Answer 1

Il suffit de faire:

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

Vous obtenez une erreur, car foreach attend son premier argument est un tableau. S'il n'y a aucun élément dans la base de données, vos fonctions retournent false.

Answer 2

En effet, lorsque vous exécutez votre code:

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

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

quand il n'y a pas de lignes retournées le code ci-dessus fonctionne très bien, il ne marche pas dans le travail que le point de vue où vous essayez d'exécuter une boucle. Cette boucle for ne doit pas être exécutée si aucune ligne n'est renvoyée. Pourtant, vous essayez d'exécuter un forloop même s'il n'y a pas de lignes à travailler avec.

Ma suggestion est en train de changer le code comme ceci:

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

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

dans la vue que vous aurez quelque chose comme ça avant votre boucle:

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

} 

Hope this helps.

PK

Answer 3

Pourquoi ne pas vous faire ceci:

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

Dans la vue, s'il n'y a pas de résultats, un tableau vide sera retourné et foreach ne jetterai pas un erreur:

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

Beaucoup plus propre IMO.

Si vous souhaitez afficher un message d'erreur dans votre vue, vous pouvez le faire:

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

Cela vaut mieux que de vérifier s'il y a des résultats avec if/else deux fois, comme Halfdan et Pavan suggérez .