Welcome Guest, Not a member yet? Register   Sign In
question: is there a better way of doing this database select?
#1

[eluser]jerry01[/eluser]
I want to get the email from some user.

the only way i know how to parse the result set is as follows.


Code:
function _get_email($username)
    {
        $this->load->model('dc_profiles_model');
        $query = $this->dc_profiles_model->get_email($username);
      foreach($query->result() as $row)
        {
          foreach($row as $fname => $fvalue)
            {
              if ($fname == 'email')
              {
                  $email=$fvalue;
              }
            }
        }
        return $email;
    }

it looks very ugly, and i'm wondering if someone would be kind enough to suggest a more tidy, regular way...
#2

[eluser]Emkay[/eluser]
If it's just one user (if the username is unique) then just use

Code:
$this->load->model('dc_profiles_model');
$query = $this->dc_profiles_model->get_email($username);
if($query->num_rows() == 1)   {
     $user = $query->row();
     return $user->email;
}
else   {
     return false;
}

You can omit the num rows check if the username has already been verified somewhere else.

Code:
$this->load->model('dc_profiles_model');
$query = $this->dc_profiles_model->get_email($username);
$user = $query->row();
return $user->email;
#3

[eluser]jerry01[/eluser]
top man,

many thanks




Theme © iAndrew 2016 - Forum software by © MyBB