Vérification de la non présence d'une valeur

Version imprimable

Bonjour,

J'essaye de créer une requete interrogeant la base de données en vue de verifier que la valeur saisie dans le formulaire n'est pas déjà présente dans la base de données je fais ceci :
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 //$nom_hotel = Valeur saisie dans le formulaire include("../include/connex.inc.php"); //requete $sql_selection = "SELECT nom_hotel FROM hotel"; $stmt = $dbh->query($sql_selection); while($row = $stmt->fetch(PDO::FETCH_NUM)) { //Si valeur déjà présente if($row[0] == $nom_hotel) { print($nom_hotel ." -- "); print($row['nom_hotel']); print("EXISTANT !"); break; } //sinon else { print($nom_hotel ." -- "); print($row['nom_hotel']); print("A ENREGISTRER !"); break; }
Mon problème est que la boucle s'arrête à la première valeur contenue dans le tableau et ne fait pas le tour pour comparer chaque valeur une par une.
J'ai donc toujours "A enregistrer" qui s'affiche même si je saisie une valeur déjà présente dans le tableau $row renvoyé par la base de données.

Quel est le problème ?
Merci d'avance pour vos lumières ;)

05/04/2009, 11h13
sabotage

Enleve tes deux "break" :mouarf:
05/04/2009, 11h17
popy67

Citation:

Envoyé par sabotage

Enleve tes deux "break" :mouarf:

Salut,

Toujours pareil sauf que cette fois ci ca m'affiche "A enregistrer" autant que fois que le nombre d'enregistrement déjà présents dans le tableau (sauf une fois ou ca affiche existant car la veleur est identique).
05/04/2009, 11h31
Séb.

Citation:

Envoyé par popy67

Toujours pareil sauf que cette fois ci ca m'affiche "A enregistrer" autant que fois que le nombre d'enregistrement déjà présents dans le tableau (sauf une fois ou ca affiche existant car la veleur est identique).

Me semble que c'est bien ce que tu demandes de faire à ton script non ? Si tu veux juste afficher EXISTANT et pas A ENREGISTRER alors supprime le else et tout ce qu'il contient.
05/04/2009, 11h44
popy67

Citation:

Envoyé par Séb.

Me semble que c'est bien ce que tu demandes de faire à ton script non ? Si tu veux juste afficher EXISTANT et pas A ENREGISTRER alors supprime le else et tout ce qu'il contient.

Je veux faire la chose suivante :

Si par exemple dans mon formulaire je marque "dodo" et que mon tableau $row contient deja la valeur "dodo" j'affiche une seule fois existant; si mon tableau ne contient pas "dodo" j'affiche une seule fois "a enregistrer".
05/04/2009, 11h47
bacchus41
Qu'est ce que tu souhaites faire exactement ?

- Simplement vérifier l'absence du nom d'un hôtel en table ?
=> pourquoi n'utilises tu pas de clause WHERE ?
Code:

1 2 $sql_selection = "SELECT count(nom_hotel) FROM hotel WHERE nom_hotel LIKE '%$nom_hotel%' ";

Citation:

Envoyé par bacchus41

Qu'est ce que tu souhaites faire exactement ?

- Simplement vérifier l'absence du nom d'un hôtel en table ?
=> pourquoi n'utilises tu pas de clause WHERE ?

Code:

1
2
 
$sql_selection = "SELECT count(nom_hotel) FROM hotel WHERE nom_hotel LIKE '%$nom_hotel%' ";

Ah ouais, en effet ^^ je n'avais pas pensé à WHERE :)
Comme tu me l'as dit et avec le code suivant ca marche très bien :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
$sql_selection = "SELECT count(nom_hotel) FROM hotel WHERE nom_hotel LIKE '%$nom_hotel%'";
		$stmt = $dbh->query($sql_selection);
 
		//////////////////////////////////////////////////////////////////////////////
		while($row = $stmt->fetch(PDO::FETCH_NUM))
			{
			if($row[0] == 1)
				{
				print($nom_hotel ." -- ");
				print($row['nom_hotel']);
				print("EXISTANT !");
 
				}
			else
				{
				print($nom_hotel ." -- ");
				print($row['nom_hotel']);
				print("A ENREGISTRER !");
 
				}
		}

Quelques minutes avant de revenir sur le forum j'avais aussi trouvé la solution du isset qui elle aussi fonctionne bien et avec laquelle je peux encore ajouter d'autres valeurs à vérifier; comme ceci :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
$sql_selection = "SELECT nom_hotel FROM hotel";
		$stmt = $dbh->query($sql_selection);
 
		//////////////////////////////////////////////////////////////////////////////
		while($row = $stmt->fetch(PDO::FETCH_NUM))
			{
			if(isset($row[0]))
				{
				print($nom_hotel ." -- ");
				print($row['nom_hotel']);
				print("EXISTANT !");
 
				}
			else
				{
				print($nom_hotel ." -- ");
				print($row['nom_hotel']);
				print("A ENREGISTRER !");
 
				}
		}

Merci beaucoup pour votre aide à tous :king:

05/04/2009, 12h17
bacchus41
OK,

La version avec le WHERE est plus performante, après il y a des petites choses à vérifier.
En règle générale, fais le max de traitement en SQL : le moteur MySQL est là pour ça et est optimisé pour.

1. Dans la clause j'ai utilisé LIKE '%mot%', donc ca te retournera tous les hotels dans le nom contient le mot 'mot'.

2.
Selon la définition de la table, tu peux avoir plusieurs hotel répondant à ce critère il faudrait donc modifier ton code :
Code:

1 2 3 4 5 6 7 8 while($row = $stmt->fetch(PDO::FETCH_NUM)) { if($row[0] >= 1) { print($nom_hotel ." -- "); print($row['nom_hotel']); print("EXISTANT !");
3.
Si tu veux que le nom soit unique, ajoute un INDEX UNIQUE sur la colonne nom_hotel (inutile s'il d'agit de la clé primaire).

4.
Il faut éviter d'intégrer directement une donnée venant d'un formulaire dans une requête, l'utilisateur pourrait y insérer du code SQL.
05/04/2009, 14h12
popy67
Citation:
Envoyé par bacchus41

OK,

La version avec le WHERE est plus performante, après il y a des petites choses à vérifier.
En règle générale, fais le max de traitement en SQL : le moteur MySQL est là pour ça et est optimisé pour.

1. Dans la clause j'ai utilisé LIKE '%mot%', donc ca te retournera tous les hotels dans le nom contient le mot 'mot'.

2.
Selon la définition de la table, tu peux avoir plusieurs hotel répondant à ce critère il faudrait donc modifier ton code :

Code:

1 2 3 4 5 6 7 8 while($row = $stmt->fetch(PDO::FETCH_NUM)) { if($row[0] >= 1) { print($nom_hotel ." -- "); print($row['nom_hotel']); print("EXISTANT !");

3.
Si tu veux que le nom soit unique, ajoute un INDEX UNIQUE sur la colonne nom_hotel (inutile s'il d'agit de la clé primaire).

4.
Il faut éviter d'intégrer directement une donnée venant d'un formulaire dans une requête, l'utilisateur pourrait y insérer du code SQL.
Ok ok merci pour tous ces precieux conseils :) :yaisse2: