Discussion :

[popy67]

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 :

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
 
//$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

[sabotage]

Enleve tes deux "break"

[popy67]

Enleve tes deux "break"

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).

[Séb.]

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.

[popy67]

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".

[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 :

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

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

[popy67]

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 :

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

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 :

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
$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 :

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
$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

[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 :

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

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.

[popy67]

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 :

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

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