J'ai une question rapide sur refactoring php code. Voici trois fonctions. Les deux premiers semblent assez similaires et ne diffèrent que par une déclaration if. Le troisième combine les deux premiers grâce à l'utilisation d'un drapeau. Est-ce la meilleure pratique? Ici, il semble correct d'utiliser un drapeau, mais que se passe-t-il si nous devons ajouter plus de drapeaux dans le futur? Quelle est la meilleure pratique?Question de refactorisation codeigniter simple - meilleure pratique
Merci.
function check_contact_email($email)
{
$this->db->select('COUNT(login) AS count');
$this->db->from('users');
$this->db->where('email', $email);
$query = $this->db->get();
$row = $query->row();
return ($row->count > 0);
}
function check_contact_email_id($email)
{
$this->db->select('COUNT(login) AS count');
$this->db->from('users');
$this->db->where('email', $email);
$this->db->where('user_id !=', $_POST['user_id']);
$query = $this->db->get();
$row = $query->row();
return ($row->count > 0);
}
function check_contact_email($email, $id = FALSE)
{
$this->db->select('COUNT(login) AS count');
$this->db->from('users');
$this->db->where('email', $email);
if ($id) $this->db->where('user_id !=', $_POST['user_id']);
$query = $this->db->get();
$row = $query->row();
return ($row->count > 0);
}
Quelle version de gâteau utilisez-vous? ce n'est pas 'n'importe où même proche de la bonne façon de le faire ... – user103219
désolé * c'est codeigniter (j'ai eu mes projets mélangés). Que suggérez-vous? – Dirk
ahh ok, eh bien dans ce cas, je ne suis pas familier avec le codeigniter ... dans cakephp toutes ces fonctions peuvent être combinées en une seule fonction $ this-> Model-> Find() – user103219