Eviter les doublons à l'insertion dans une base

**leloup84** · 26/01/2006, 13h48

Bonjour,

J'ai crée un formulaire que voici :

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
<form name="style" method="post" action="validation_rubrique.php" enctype="multipart/form-data">
<table width="849">
  	<tr>
		<td align="center">&nbsp;</td>
		<td colspan="2" align="center"><div align="left" class="Style5">Saisir une nouvelle rubrique </div></td>
	</tr>			  	
	<tr>
		<td width="265" align="left"><strong>Nom de la nouvelle rubrique :</strong></td>
		<td colspan="2" align="left"><input type="text" name="rubrique" size="50"></td>
	</tr>
	<tr>
		<td width="265" align="left"><strong>Nom du fichier :</strong></td>
		<td colspan="2" align="left"><input type="text" name="nomfic" size="50"></td>
	</tr>		
	<tr>
		<td height="39" align="center">&nbsp;</td>
		<td width="77" align="center"><div align="left"><input type="button" name="Submit3" value="Valider" onClick="securite();"></div></td>
		<td width="420" align="center"><div align="left"><input type="reset" name="annuler" value="Effacer"></div></td>
	</tr>
	<tr>
		<td colspan="3"><hr></td>				
	</tr>
</table>
</form>

Et je voudrais qu'à l'insertion il vérifie si les données n'existe pas déjà. Pour l'instant j'ai fais comme ceci mais le controle ne se fait pas, comment faire ?

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
<?
 
	require("config.php");
 
	$req2="select nom_style FROM style;";
	$res2=mysql_query($req2);							
	echo"$res2";
		if($_POST['rubrique']!=$res2)
		{
			$req = "insert into style values('', '".$_POST['rubrique']."', '".$_POST['nomfic']."')";
			$verif = mysql_query($req);
		}
 
	/*include("liens_internet.php");*/
?>

Merci davance

**Yogui** · 26/01/2006, 13h52

Salut

Je pense que tu devrais revoir le schéma de ta base de données. Dans l'autre sujet (ici) c'est un peu le même thème.

Tu dois utiliser la clef primaire ou les clefs uniques dans la structure de ta table. MySQL se chargera tout seul de ne pas introduire des tuples dont l'identifiant existe déjà.

Le souci avec ta vérification actuelle est que tu n'as pas mis de WHERE dans ton SELECT, ce qui fait qu'il va te donner tous les tuples de ta base. Après le mysql_query(), tu dois parcourir tous ces résultats avec mysql_fetch_...().
Le mieux est d'utiliser la clef primaire ou les clefs uniques.

**xave** · 26/01/2006, 13h53

Tu ne peux pas faire un echo de $res2 directement!

Tu dois obligatoirement récupérer tes données avec un mysql_fetch_array ou un mysql_fetch_row.

Par exemple:

Code :

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

$resultat=mysql_fetch_row($res2);

Et ensuite pour ton test, tu fais:

Code :

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

if($_POST['rubrique']!=$resultat[0])

Je t'invite à aller consulter les excellents tutoriels et faq ici: http://php.developpez.com/

Tu y trouveras beaucoup de réponses.

**leloup84** · 26/01/2006, 13h58

J'ai ce que tu as dis mais l'insertion se fait comme même

**leloup84** · 26/01/2006, 14h02

J'ai le champ rubrique et nomfic en unique dans ma base de donnée désormais donc logiquement l'insertion ne se fait pas. Le problème c'est que rien ne le dit à l'utilisateur que l'insertion n'a pas pu avoir lieu.

**xave** · 26/01/2006, 14h06

Envoyé par leloup84

J'ai ce que tu as dis mais l'insertion se fait comme même

Pourtant ça n'apparaît pas dans ton code...

**leloup84** · 26/01/2006, 14h07

oui mais j'ai fais cette midification depuis que tu me l'as dit 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
<?
 
	require("config.php");
 
	$req2="select nom_style FROM style where nom_style<>'".$_POST['rubrique']."';";
	$res2=mysql_query($req2);
	$resultat=mysql_fetch_row($res2);							
	if($_POST['rubrique']!=$resultat[0])
		{
			$req = "insert into style values('', '".$_POST['rubrique']."', '".$_POST['nomfic']."')";
			$verif = mysql_query($req);
		}
 
	/*include("liens_internet.php");*/
?>

**Yogui** · 26/01/2006, 14h09

leloup, maintenant que ta table est correctement structurée, tu n'as plus besoin de SQL pour vérifier si l'insertion peut se faire ou non. À la place, récupère la valeur de retour de mysql_query() et gère les erreurs avec mysql_error().

**xave** · 26/01/2006, 14h09

Fais un echo de $resultat[0] et de $_POST['rubrique'] pour vérifier s'ils sont différents ou non.

**leloup84** · 26/01/2006, 14h17

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
<?
 
	require("config.php");
 
	$req2="select nom_style FROM style where nom_style<>'".$_POST['rubrique']."';";
	$res2=mysql_query($req2);
	$resultat=mysql_fetch_row($res2);							
	if($_POST['rubrique']==$resultat[0])
	{
		msgbox("Rubrique déjà existente");
	}
	else
	{
		$req = "insert into style values('', '".$_POST['rubrique']."', '".$_POST['nomfic']."')";
		$verif = mysql_query($req);
	}
 
	/*include("liens_internet.php");*/
?>

Voilà comment je mis prend donc si je fais une insertion d'une donnée qui existe déjà çà devrait m'afficher le message non ?

**xave** · 26/01/2006, 14h21

Oui, ça me semble correct, c'est pour ça que je te demande de faire les echos de mon post précédent, pour essayer de vérifier les données...

**leloup84** · 26/01/2006, 14h23

pour $resultat il m'affiche Array

**leloup84** · 26/01/2006, 14h25

et pour $_POST['rubrique'] il m'affiche bien le nom de ma rubrique

**xave** · 26/01/2006, 14h26

Tu fais bien un

Code :

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

echo $resultat[0];

:

**leloup84** · 26/01/2006, 14h36

ok c'est bon çà marche maintenant çà coince au niveau du message d'erreur pourquoi ?

Code :

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

Fatal error: Call to undefined function msgbox()

**xave** · 26/01/2006, 14h41

Envoyé par leloup84

ok c'est bon çà marche maintenant çà coince au niveau du message d'erreur pourquoi ?

Code :

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

Fatal error: Call to undefined function msgbox()

C'était quoi l'erreur?

L'erreur sur msgbox, c'est parce que tu appelles une fonction qui n'est pas définie. Cette fonction est-elle codée dans la même page ou dans un autre script?
Si c'est un autre script, as-tu fais l'include correspondant?

**leloup84** · 26/01/2006, 14h43

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
<?
 
	require("config.php");
 
	$req2="select nom_style FROM style where nom_style='".$_POST['rubrique']."';";
	$res2=mysql_query($req2);
	$resultat=mysql_fetch_row($res2);
 
	if($_POST['rubrique']==$resultat[0])
	{
		msgbox("Rubrique déjà existente");
	}
	else
	{
		$req = "insert into style values('', '".$_POST['rubrique']."', '".$_POST['nomfic']."')";
		$verif = mysql_query($req);
	}
 
	/*include("liens_internet.php");*/
?>

Tu vois je fais le msgbox dans le même script et il me met l'erreur au niveau dee cette ligne

**xave** · 26/01/2006, 14h48

Envoyé par leloup84

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
<?
 
	require("config.php");
 
	$req2="select nom_style FROM style where nom_style='".$_POST['rubrique']."';";
	$res2=mysql_query($req2);
	$resultat=mysql_fetch_row($res2);
 
	if($_POST['rubrique']==$resultat[0])
	{
		msgbox("Rubrique déjà existente");
	}
	else
	{
		$req = "insert into style values('', '".$_POST['rubrique']."', '".$_POST['nomfic']."')";
		$verif = mysql_query($req);
	}
 
	/*include("liens_internet.php");*/
?>

Tu vois je fais le msgbox dans le même script et il me met l'erreur au niveau dee cette ligne

Non, là tu fais un appel à la fonction msgbox() mais elle n'est définie nul part!

Plus simple, remplace

Code :

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

msgbox("Rubrique déjà existente");

par

Code :

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

echo "Rubrique déjà existente";

Et alors quelle était l'erreur pour ton précédent problème?

**leloup84** · 26/01/2006, 14h58

Ok comme cela çà marche mais le message s'affiche sur une page toute blanche ce n'est pas super. Moi je voulais que cela s'affiche comme un message d'alerte (les petites fenêtres grise comme quand l'on fait un contrôle sur un formulaire)