Codeigniter cannot run a loop

Question

My model returns array with "user ids" and I want to run loop for each of those user ids but I get errors "Undefined variable: users" and "Invalid argument supplied for foreach()". Could you please check what is wrong with my controller code.

My Model:

 public function get_user_id($post_id){

    $this->db->select('user_id');
    $this->db->from('comments');   
    $this->db->where('post_id', $post_id);

    $query = $this->db->get();

if ($query && $query->num_rows() >= 1){ 

return $query->result();
    }    
     else {
    return false;
    }

   }

My controller:

$this->model_a->get_user_id($post_id);
$data["users"] = $this->model_a->get_user_id($post_id);         

foreach($users as $user){

$user_id = $user['user_id'];
//loop code 
}

Roni · Accepted Answer · 2014-10-30 09:49:31Z

2

Try

$data["users"] = $this->model_a->get_user_id($post_id);

foreach($data["users"] as $user){

$user_id = $user['user_id']; //loop code

}

There is no variable called $users

answered Oct 30, 2014 at 9:49

Roni

19410 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

EducateYourself Over a year ago

Fatal error: Cannot use object of type stdClass as array

Shijin TR Over a year ago

@EducateYourself use result_array() insted of result()

EducateYourself Over a year ago

Thanks Roni and Shijin. I have combined your answers and it works now.

Akhil Clement · Accepted Answer · 2014-10-30 09:54:13Z

1

   $this->model_a->get_user_id($post_id);
$data["users"] = $users = $this->model_a->get_user_id($post_id);         

foreach($users as $user){

$user_id = $user['user_id'];
//loop code 
}

$users must be defined.

answered Oct 30, 2014 at 9:54

Akhil Clement

6951 gold badge10 silver badges18 bronze badges

Comments

Sougata Bose · Accepted Answer · 2014-10-30 09:55:56Z

1

try -

in the model -

if ($query && $query->num_rows() >= 1){ 
   return $query->result_array();
} else {
   return false;
}

in controller -

$data["users"] = $this->model_a->get_user_id($post_id);

foreach($users as $user){
     $user_id = $user['user_id'];
     //loop code 
}

answered Oct 30, 2014 at 9:55

Sougata Bose

31.8k8 gold badges52 silver badges89 bronze badges

Collectives™ on Stack Overflow

Codeigniter cannot run a loop

3 Answers 3

3 Comments

Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

3 Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related