requete SQL qui bug

Version imprimable

requete SQL qui bug

Bonjours,

Je rencontre une nouvelle erreur.

Voici le code :

Code:

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

05/10/2012, 14h15
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+.
05/10/2012, 14h20
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.
05/10/2012, 14h35
thebarbarius

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

Mais sa marche pas quand meme.
05/10/2012, 14h55
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+.
05/10/2012, 14h59
thebarbarius

Oui j'ai testé ces valeurs.

Sa retourne ce qu'il faut.

Code:

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."'  // Tu ne dois pas mettre ces valeurs directement dans la requête
AND password = '".$password."'";
$requete = $dbh->prepare($sql); 
$requete->execute(); // Tu dois mettre en paramètre de cette fonction les valeurs que tu as mis dans $sql
$nombreUser = $requete->fetch(PDO::FETCH_ASSOC);
 
if (empty($pseudo) && empty($password) || !isset($pseudo) && !isset($password)) {  // C'est avant de lancer la requête que tu dois tester ça
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;
}
}
}

05/10/2012, 15h19
rawsrc

A savoir que empty() englobe isset().
C'est un pléonasme que d'utiliser les deux simultanément.

S'il vous plait aider moi.

Voici le sql :

Code:

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
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
 
-- phpMyAdmin SQL Dump
-- version 3.3.9
-- http://www.phpmyadmin.net
--
-- Serveur: localhost
-- Généré le : Ven 05 Octobre 2012 à 14:03
-- Version du serveur: 5.5.8
-- Version de PHP: 5.3.5
 
SET SQL_MODE="NO_AUTO_VALUE_ON_ZERO";
 
 
/*!40101 SET @OLD_CHARACTER_SET_CLIENT=@@CHARACTER_SET_CLIENT */;
/*!40101 SET @OLD_CHARACTER_SET_RESULTS=@@CHARACTER_SET_RESULTS */;
/*!40101 SET @OLD_COLLATION_CONNECTION=@@COLLATION_CONNECTION */;
/*!40101 SET NAMES utf8 */;
 
--
-- Base de données: `darkminevqs4`
--
 
-- --------------------------------------------------------
 
--
-- Structure de la table `accounts`
--
 
CREATE TABLE IF NOT EXISTS `accounts` (
  `id` int(10) unsigned NOT NULL AUTO_INCREMENT,
  `playername` varchar(255) NOT NULL,
  `password` varchar(255) NOT NULL,
  `pwtype` tinyint(2) unsigned NOT NULL DEFAULT '0',
  `email` varchar(100) NOT NULL,
  `registerdate` date NOT NULL,
  `registerip` char(45) NOT NULL,
  `lastlogindate` datetime NOT NULL,
  `lastloginip` char(45) NOT NULL,
  `active` tinyint(1) unsigned NOT NULL DEFAULT '0',
  PRIMARY KEY (`id`)
) ENGINE=InnoDB  DEFAULT CHARSET=latin1 AUTO_INCREMENT=7 ;
 
--
-- Contenu de la table `accounts`
--
 
INSERT INTO `accounts` (`id`, `playername`, `password`, `pwtype`, `email`, `registerdate`, `registerip`, `lastlogindate`, `lastloginip`, `active`) VALUES
(6, 'anju', 'anju', 0, 'anju', '2012-10-05', '', '0000-00-00 00:00:00', '', 0);

05/10/2012, 16h17
andry.aime

Citation:

Envoyé par thebarbarius

S'il vous plait aider moi.

On ne t'aide pas là ?
N'attend pas qu'on va coder à ta place.

A+.
05/10/2012, 16h49
thebarbarius

J'ai trouvé l'erreur et je vous remercie.