Discussion :

[thebarbarius]

Bonjours,

Je rencontre une nouvelle erreur.

Voici le code :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
 
public function login($pseudo, $password) {
echo $pseudo . '<br />';
echo $password;
$dbh = mysql::getInstance();
$sql = "SELECT count(*) AS nombreUser
FROM account
WHERE playername = '".$pseudo."'
AND password = '".$password."'";
$requete = $dbh->prepare($sql);
$requete->execute();
$nombreUser = $requete->fetch(PDO::FETCH_ASSOC);
 
if (empty($pseudo) && empty($password) || !isset($pseudo) && !isset($password)) {
echo "Erreur : Toutes les donnés ne sont pas fournis !";
return FALSE;
}
else {
if ($nombreUser['nombreUser'] < 1) {
echo "Erreur : Compte innexistant !";
return FALSE;
}
if ($nombreUser['nombreUser'] > 0) {
echo "Felicitation : Connecter avec succes !";
$this->retour = TRUE;
 
$_SESSION['user']['pseudo'] = $prenom;
$_SESSION['user']['password'] = $password;
return TRUE;
}
}
}

Sa me met met que le compte inexistant alors qu'il existe dans la BDD

Merci

PS : ne clôturer pas le topic il est important.

Merci

[andry.aime]

Bonjour,

1- Tu utilise PDO prepare or que tu mets directement les valeurs dans la requête. C'est pas bon, regarde un exemple sur le manuel php.

2- C'est après que tu as exécuté la requête que tu as testé si les données existent ou non.

A+.

[thebarbarius]

Et bien vs script fonctionne très bien sur mon autre script.

Essaye ma fonction et si tu trouve la solution ce sera un miracle.

[thebarbarius]

J'ai trouvé une erreur j'ai oublié le s a account (table).

Mais sa marche pas quand meme.

[andry.aime]

1- Tu ne dois pas mettre en dur les valeurs dans la requête mais les mettre en paramètre de la fonction execute.
2- Tu as testés l’existence de variable "(empty($pseudo) && empty($password)" après avoir exécuté la requête, or que tu devrais le faire avant la requête.

A+.

[thebarbarius]

Oui j'ai testé ces valeurs.

Sa retourne ce qu'il faut.