Ajout à la base de donnée

**hanitax** · 23/05/2013, 10h50

bonjour
j'ai un formulaire que je remplie , avec des images à upoader , lorsque je clique sur ajouter , l'enregistrement s'ajout correctement à ma base de donnée mais lorsque j'actualise ma page avec F5 ou quand je lance la page de l'ajout il me rajoute une ligne vide à ma base ,je sais pas d'ou vient le probléme voila le code pour vous faciliter les choses.
NB : j'ai pas encore fais un test pour les champs si vides ou non

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
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
<?php
if(isset($_POST['valider']))
 
  @ $titre     = $_POST["titre"] ;
  @ $details = $_POST["details"] ;
  @ $num         = $_POST["num"] ;
  @ $prix_dh     = $_POST["dh"] ;
  @ $prix_euro = $_POST["euro"] ;
  @ $resume1         = $_POST["res1"]; 
  @ $resume2 = $_POST["res2"] ;
  @ $status         = $_POST["status"] ;
  @ $date_creation = $_POST["date"] ;
   @ $photo = $_POST["ph"] ;
  @ $ph1 = $_POST["ph1"] ;
  @ $ph2 = $_POST["ph2"] ;
  @ $ph3 = $_POST["ph3"] ;
  //pour ajout des photo
 	@ $dossier = 'photo/';
	@ $fichier = basename($_FILES['ph']['name']);
	@ $dossiergalerie = 'photo/gal/';
	@ $ph1 = basename($_FILES['photo1']['name']);
	@ $taille_maxi = 10240000;
	@ $taille = filesize($_FILES['ph']['tmp_name']);
	@ $tailleph1 = filesize($_FILES['photo1']['tmp_name']);
	@ $extensions = array('.png','.gif','.jpg','.jpeg','.bmp');
	@ $extension = strrchr($_FILES['ph']['name'], '.');
	@ $extensionph1 = strrchr($_FILES['photo1']['name'], '.');
	@ $extph=$extension;
	@ $extph1=$extensionph1;
	if(!in_array($extension, $extensions)) 
	{
		$erreur = 'Vous devez uploader un fichier de type png, gif, jpg, jpeg, txt ou doc...';
	}
	else
	{
		if($taille>$taille_maxi)
		{
			$erreur = 'Le fichier est trop gros...';
		}
		else
		$testimg=true;
	}
		{
$cheminphoto=$dossier.$num.$extph;
$cheminphoto1=$dossiergalerie.$num.$extph1;
  $cnx = mysql_connect("127.0.0.1","root","");
  $db  = mysql_select_db('immo'); 
 @ $sql = "INSERT INTO bien(reference,titre,description,prix_dh,prix_euro,resume1,resume2,status,date_creation,photo,photo1) VALUES('$num','$titre','$details','$prix_dh' ,'$prix_euro','$resume1' ,'$resume2','$status','$date_creation' ,'$cheminphoto','$cheminphoto1') ";
 
 @ $requete = mysql_query($sql, $cnx) or die(  ) ;
 
  if($requete)
  { 
  move_uploaded_file($_FILES['ph']['tmp_name'],$cheminphoto);
  move_uploaded_file($_FILES['photo1']['tmp_name'],$cheminphoto1);
    echo("L'ajout  à été correctement effectuée") ;
  }
 
  else
  {
    echo("L'ajout à échouée");
 
}
}
  ?>

le code du formulaire :

Code html :

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
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
<form name="ajout" method="post" action="ajout.php" enctype="multipart/form-data">
  <p>Entrez référence du bien:&nbsp;&nbsp; 
    <input type="text" name="num"/> 
  </p>
  <p><br/>
    Titre  : &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp; &nbsp;   &nbsp; &nbsp;
    <input type="text" name="titre"/>
  </p>
  <p><br/>
    Prix Dh : 
    <input type="text" name="dh"/>
    </p>
  <p><br/>
    Prix euro : 
    <input type="text" name="euro"/>
    </p>
  <p><br/>
 
  la description du bien :</p>
  <p>
    <textarea name='details' ></textarea>
  </p>
  <p><br/>
  détails  :</p>
  <p>
    <textarea name='res1' ></textarea>
    </p>
  <p><br/>
  Plus de détails  : </p>
  <p>
    <textarea name='res2' ></textarea>
    </p>
  <p><br/>
    Status
      <select name='status' >
        <option >Afficher</option>
        ";
 
        <option >En attente</option>
      </select  >
  </p>
  <p>Date Création :
    <input name='date'   type='date' >
  </p>
  <p>    <br/>
  Photo du diaporama :</p>
  <p>Photo principale : </br>
 
   <input name='ph'  type='file' maxlength=120 />
  </p>
  <p>Photo 1 :
 
   <input type="file" name="photo1" size=50 />
    </br>
  </p>
<p>
    <input type="submit" name="valider" value="Ajouter"/>
    <input name='annuler' type='reset' value='Annuler' />
  </p>
</form>

**baderahmed** · 23/05/2013, 11h23

ton code n'est pas entre les accolades du if de validation du formulaire

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
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
 
		<?php
		if(isset($_POST['valider'])) {
 
		  @ $titre     = $_POST["titre"] ;
		  @ $details = $_POST["details"] ;
		  @ $num         = $_POST["num"] ;
		  @ $prix_dh     = $_POST["dh"] ;
		  @ $prix_euro = $_POST["euro"] ;
		  @ $resume1         = $_POST["res1"]; 
		  @ $resume2 = $_POST["res2"] ;
		  @ $status         = $_POST["status"] ;
		  @ $date_creation = $_POST["date"] ;
		   @ $photo = $_POST["ph"] ;
		  @ $ph1 = $_POST["ph1"] ;
		  @ $ph2 = $_POST["ph2"] ;
		  @ $ph3 = $_POST["ph3"] ;
		  //pour ajout des photo
		 	@ $dossier = 'photo/';
			@ $fichier = basename($_FILES['ph']['name']);
			@ $dossiergalerie = 'photo/gal/';
			@ $ph1 = basename($_FILES['photo1']['name']);
			@ $taille_maxi = 10240000;
			@ $taille = filesize($_FILES['ph']['tmp_name']);
			@ $tailleph1 = filesize($_FILES['photo1']['tmp_name']);
			@ $extensions = array('.png','.gif','.jpg','.jpeg','.bmp');
			@ $extension = strrchr($_FILES['ph']['name'], '.');
			@ $extensionph1 = strrchr($_FILES['photo1']['name'], '.');
			@ $extph=$extension;
			@ $extph1=$extensionph1;
			if(!in_array($extension, $extensions)) 
			{
				$erreur = 'Vous devez uploader un fichier de type png, gif, jpg, jpeg, txt ou doc...';
			}
			else
			{
				if($taille>$taille_maxi)
				{
					$erreur = 'Le fichier est trop gros...';
				}
				else
				$testimg=true;
			}
 
		$cheminphoto=$dossier.$num.$extph;
		$cheminphoto1=$dossiergalerie.$num.$extph1;
		  $cnx = mysql_connect("127.0.0.1","root","");
		  $db  = mysql_select_db('immo'); 
		 @ $sql = "INSERT INTO bien(reference,titre,description,prix_dh,prix_euro,resume1,resume2,status,date_creation,photo,photo1) VALUES('$num','$titre','$details','$prix_dh' ,'$prix_euro','$resume1' ,'$resume2','$status','$date_creation' ,'$cheminphoto','$cheminphoto1') ";
 
		 @ $requete = mysql_query($sql, $cnx) or die(  ) ;
 
		  if($requete)
		  { 
		  move_uploaded_file($_FILES['ph']['tmp_name'],$cheminphoto);
		  move_uploaded_file($_FILES['photo1']['tmp_name'],$cheminphoto1);
		    echo("L'ajout  à été correctement effectuée") ;
		  }
 
		  else
		  {
		    echo("L'ajout à échouée");
 
		}
		}
		  ?>

j'ai aussi quelques remarques sur ton code :
1/ il faut tester toutes les variables envoyées par le formulaire, un teste avec la fonction isset et un autre avec empty pour chaque variable et même faire un teste sur le type avec is_numeeric par exemple.

2/ n'utilise pas le @ pour ne pas afficher des erreurs mais fais en sorte que ton code ne génère pas d'erreurs.

3/ tu peux utiliser la fonction extract($_POST) pour créer tes variables à partir du post.

4/ ne jamais inséré les variables envoyées par formulaire directement dans la base de données mais utilise avant la fonction mysql_real_escape_string() ou même le pdo s'il le faut.

**rawsrc** · 23/05/2013, 11h27

Envoyé par baderahmed

3/ tu peux utiliser la fonction extract($_POST) pour créer tes variables à partir du post.

JAMAIS DE LA VIE

Lire l'avertissement en rouge ici

**baderahmed** · 23/05/2013, 11h48

- ben normalement il n' y a plus de register_globals à ON même sans utilisé extruct il ne faut pas le mettre à ON.

- puis j'ai demandé de tester les variables avant l'extruct.

**rawsrc** · 23/05/2013, 12h00

Tu peux très bien tester 3 clés du tableau $_POST en rapport avec les données attendues pour ton traitement alors que ton tableau pourra en contenir beaucoup d'autres (sans même que tu le saches). extract() ne fait pas dans la dentelle, il sort tout le contenu du tableau.
N'oublie pas qu'un utilisateur pourra toujours bidonner la soumission d'un formulaire rien que pour voir comment ton code réagit :

Never trust user input

Généralement, il est très fortement conseillé de ne jamais faire un extract() sur un tableau dont tu n'es pas absolument sûr de son contenu.

**baderahmed** · 23/05/2013, 12h03

Merci rawsrc

**hanitax** · 23/05/2013, 12h04

oui mais aprés avoir ajouter un nouveau enregistrement , comment eviter l'affichage du message (chaque action précédement effectuée sera répétée , souhaitez vous continuer ?

**rawsrc** · 23/05/2013, 13h22

Pour stopper l'exécution du programme, tu peux utiliser une fonction d'extinction comme exit().
Pour continuer à exécuter le programme en repassant le flux d'exécution au script appelant, tu dois utiliser return ou simplement laisser le script arriver à sa fin.

Ajout à la base de donnée

Langage PHP

Discussions similaires

Partager

Partager