Transmettre une valeur $ _POST dans un paramètre de fonction avec une variable PDO

Question

J'ai mis en place une connexion pdo et la passe en tant que variable dans une fonction. Tout cela fonctionne très bien et la fonction retourne correctement. Si j'exécute la fonction dans une instruction conditionnelle avec la variable PDO et un nom qui s'exécute correctement - si le nom est dans la base de données, il résonne correctement sinon il résonne aussi correctement. Ce que je veux faire est de passer la valeur d'un post de formulaire à la fonction afin qu'elle vérifie si elle existe dans la base de données. Voici mon code:

La fonction vérifie si le nombre de colonnes est un.

function user_exists($pdo, $username) { 
$stmt = $pdo->prepare('SELECT COUNT(uid) FROM users WHERE username = :username'); 
$stmt->execute(['username' => $username]); 
$result = $stmt->fetchColumn(); 
return ($result == 1); 
} 

Si l'utilisateur admin existe dans la base de données echo 'exists' - Juste pour le test.

if(user_exists($pdo,'admin') == true) { 
echo "exists"; 
} else { 
echo "doesnt exist"; 
} 

Vérifie des deux champs ont été saisis alors je veux vérifier si le nom d'utilisateur est entré dans la base de données, mais je fais quelque chose de mal.

if(!empty($_POST) === true) { 
$username = $_POST['username']; 
$pwood = $_POST['password']; 
if(empty($username) === true || empty($pwood) === true) { 
echo "You need to enter a username and password"; 
} else if (user_exists($pdo,$_POST['username']) == false){ 
echo 'We can\'t find that username. Please try again or register'; 
} 
} 

Answer 1

mieux ne pas comparer avec des valeurs booléennes, il suffit d'utiliser

//...see below 
require_once('UserRepository.php'); 
$ur = new UserRepostory($pdo); 

if(!empty($_POST)) { 
    if (empty($username) || empty($pwood)) { 
     // something 
    } else if (!$ur->exists($_POST['username'])) { // your user_exists($pdo, $username) works fine, too 
     // something else 
    } 
} 

Surtout le if(!empty($_POST) === true) { initial est difficile à lire et conduire à la cause des erreurs de priorité des opérateurs mal compris.

Mise à jour sur la base des commentaires ci-dessus, voici un exemple avec une classe:

// UserRepository.php 
class UserRepository { 
    private $pdo; 

    // '\PDO' instead of 'PDO', see PHP Namespacing 
    public function __construct (\PDO $pdo) 
    { 
     $this->pdo = $pdo; 
    } 

    public function exists($username) 
    { 
     $sql = 'SELECT COUNT(uid) FROM users WHERE username = :username'); 
     $stmt = $this->pdo->prepare($sql); 
     $stmt->execute(['username' => $username]); 
     $result = $stmt->fetchColumn(); 

     return (bool) $result; 
    } 
}