[Tableaux] pb avec une touche

**xtiand4** · 23/06/2006, 10h37

Bonjour voila j ai un pb pour un lien que j ai creer qui est sensé me modifié les donnés de mon formulaire.
alors quand je click sur ce lien il me renvoie les données stockées dans ma bdd prérempli dans mon formulaire, mais j ai une erreur lorsque j envoie les modif dans ma bdd.

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
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
<?php 	//récupérations des données! 
 
 
if(isset($_POST['modele']))      $modele=$_POST['modele'];
else      $modele="";
 
 
if(isset($_POST['type']))      $type=$_POST['type'];
else      $type="";
 
if(isset($_POST['nom']))      $nom=$_POST['nom'];
else      $nom="";
 
if(isset($_POST['processeur']))      $processeur=$_POST['processeur'];
else      $processeur="";
 
 
if(isset($_POST['memoire_video']))      $memoire_video=$_POST['memoire_video'];
else      $memoire_video="";
 
 
if(isset($_POST['memoire_vive']))      $memoire_vive=$_POST['memoire_vive'];
else      $memoire_vive="";
 
 
if(isset($_POST['disque_dur']))      $disque_dur=$_POST['disque_dur'];
else      $disque_dur="";
 
 
if(isset($_POST['taille_ecran']))      $taille_ecran=$_POST['taille_ecran'];
else      $taille_ecran="";
 
 
if(isset($_POST['lecteur_optique']))      $lecteur_optique=$_POST['lecteur_optique'];
else      $lecteur_optique="";
 
 
if(isset($_POST['os']))      $os=$_POST['os'];
else      $os="";
 
 
if(isset($_POST['connectique']))      $connectique=$_POST['connectique'];
else      $connectique="";
 
 
if(isset($_POST['accessoires']))      $accessoires=$_POST['accessoires'];
else      $accessoires="";
 
 
if(isset($_POST['graveur']))      $graveur=$_POST['graveur'];
else      $graveur="";
 
if(isset($_POST['prix_cote_occasion']))      $prix_cote_occasion=$_POST['prix_cote_occasion'];
else      $prix_cote_occasion="";
 
 
if(isset($_POST['prix_achat']))      $prix_achat=$_POST['prix_achat'];
else      $prix_achat="";
 
 
if(isset($_POST['prix_vente']))      $prix_vente=$_POST['prix_vente'];
else      $prix_vente="";
 
 
if(isset($_POST['date_entree_stock']))      $date_entree_stock=$_POST['date_entree_stock'];
else      $date_entree_stock="";
 
 
if(isset($_POST['sous_garantie']))      $sous_garantie=$_POST['sous_garantie'];
else      $sous_garantie="";
 
 
if(isset($_POST['textganrantie']))      $textganrantie=$_POST['textganrantie'];
else      $textgarantie="";
 
 
if(isset($_POST['etat']))      $etat=$_POST['etat'];
else      $etat="";
 
 
$id = $_POST["id_produit"];
 
//connexion à la base
 
require('./db.php');
 
// Requête de modification
 
    $sql = "update stock set
							modele= '$modele',
							type= '$type',
							nom= '$nom',
							processeur= '$processeur',
							memoire_vidéo= '$memoire_video',
							memoire_vive= '$memoire_vive',
							disque_dur= '$disque_dur',
							taille_ecran= '$taille_ecran',
							lecteur_optique= '$lecteur_optique',
							os= '$os',
							connectique= '$connectique',
							accessoires= '$accessoires',
							graveur= '$graveur',
							prix_cote_occasion= '$prix_cote_occasion',
							prix_achat= '$prix_achat',
							prix_vente= '$prix_vente',
							date_entree_stock=  NOW(),
							sous_garantie= '$sous_garantie',
							textgarantie= '$textgarantie',
							etat= '$etat',
							image= '$image'
				where id_produit=$id";
 
 // on insére les informations du formulaire dans la table
    mysql_query($sql) or die('Erreur SQL !'.$sql.'<br>'.mysql_error());
 
    // on affiche le r?sultat pour le visiteur
    $text = 'Vos infos on été modifiées.';
 
    mysql_close();  // on ferme la connexion
 
 
?>

Erreur :

Erreur SQL !update stock set modele= '2', type= 'fixe', nom= ',lhkjln', processeur= 'kkjk,', memoire_vidéo= 'jk,kml,', memoire_vive= 'mkl,lkm,;', disque_dur= 'mk,', taille_ecran= 'km,', lecteur_optique= 'mk,', os= 'mk, kml', connectique= 'm,ml;', accessoires= 'mlk,;', graveur= '', prix_cote_occasion= 'lmkmlkm', prix_achat= 'mlml', prix_vente= '', date_entree_stock= NOW(), sous_garantie= 'Non', textgarantie= '', etat= '1', image= '' where id_produit=
Erreur de syntaxe près de '' à la ligne 23

Merci

**sohnic** · 23/06/2006, 10h47

Bonjour,
Pas de '' encadrant $id ?
where id_produit='$id'";

Sohnic

**xtiand4** · 23/06/2006, 10h50

non le pb reste le même
et sur une autre table j ai mi le même code sans les ' ' et ca marche!

**Amara** · 23/06/2006, 10h59

Il existe bien $id ? Parce que tu fais pas le test avec isset pour id_produit

**xtiand4** · 23/06/2006, 11h03

oui je l ai declarer ici je te montre le code la c est quand j affiche les données dans le 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
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
<?php
	require('./db.php');
 
	$id =$_GET['id_produit'];
	$res = mysql_query("select * from stock where id_produit=$id") or die (mysql_error());
	$row = mysql_fetch_array($res);
	if (!$row)
		die("Demande non trouvée");
?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
<title>Document sans titre</title>
</head>
 
<body>
<form method="POST" action="modifstock1.php">
<div align="center"></div>
<p align="center">
 
 
<table width="566" border="0" align="center">
<p align="center">
</p><tr>
	<td><font face="Verdana" size="2">Modéle</font></td>
			<td>
				<font face="Verdana" size="2">MAC:</font> <input type=radio name="modele" value="1">
 
    			<font face="Verdana" size="2">PC:</font> <input name="modele" type=radio value="2" checked="checked" /></td>
	</tr><tr>
	<td><font face="Verdana" size="2">Type</font></td>
			<td>
				<select name="type">
				<option value="fixe">Fixe</option>
				<option value="portable">Portable</option>
				<option value="peripherique">Périphérique</option>
				<option value="accessoires">Accessoires</option>
				</select>	</td>
	</tr><tr>			
      <td><font face="Verdana" size="2">Nom</font></td>
      <td><input name="nom" type="text"style="background-color: #CCCCCC;font: Verdana; color: #000000; border: 0px; " value="<?= $row["nom"] ?>"></td>
    </tr><tr>
	  <td><font face="Verdana" size="2">Processeur</font></td>
      <td><input name="processeur" type="text"style="background-color: #CCCCCC;font: Verdana; color: #000000; border: 0px;" value="<?= $row["processeur"] ?>"></td>
    </tr><tr>
	  <td width><font face="Verdana" size="2">Mémoire Video</font></td>
      <td width><input name="memoire_video" type="text" style="background-color: #CCCCCC;font: Verdana; color: #000000; border: 0px;" value="<?= $row["memoire_video"] ?>" ></td>
	</tr><tr>
	  <td width><font face="Verdana" size="2">Mémoire Vive</font></td>
      <td width><input name="memoire_vive" type="text" style="background-color: #CCCCCC;font: Verdana; color: #000000; border: 0px;" value="<?= $row["memoire_vive"] ?>"> </td>
	</tr><tr>
	  <td width><font face="Verdana" size="2">Disque Dur</font></td>
      <td width><input name="disque_dur" type="text" style="background-color: #CCCCCC;font: Verdana; color: #000000; border: 0px;" value="<?= $row["disque_dur"] ?>"> </td>
	</tr><tr>
	  <td width><font face="Verdana" size="2">Taille Ecran</font></td>
      <td width><input name="taille_ecran" type="text" style="background-color: #CCCCCC;font: Verdana; color: #000000; border: 0px;" value="<?= $row["taille_ecran"] ?>"></td>
	</tr><tr>
	  <td width><font face="Verdana" size="2">Lecteur Optique</font></td>
      <td width><input name="lecteur_optique" type="text" style="background-color: #CCCCCC;font: Verdana; color: #000000; border: 0px;" value="<?= $row["lecteur_optique"] ?>" ></td>
	</tr><tr>
	  <td width><font face="Verdana" size="2">Systéme d'Opération</font></td>
      <td width><input name="os" type="text" style="background-color: #CCCCCC;font: Verdana; color: #000000; border: 0px;" value="<?= $row["os"] ?>"></td>
	</tr><tr>
	  <td width><font face="Verdana" size="2">Connectique</font></td>
      <td width><input name="connectique" type="text" style="background-color: #CCCCCC;font: Verdana; color: #000000; border: 0px;" value="<?= $row["connectique"] ?>"></td>
	</tr><tr>
	  <td width><font face="Verdana" size="2">Accessoires</font></td>
      <td width><input name="accessoires" type="text" style="background-color: #CCCCCC;font: Verdana; color: #000000; border: 0px;" value="<?= $row["accessoires"] ?>"></td>
	</tr><tr>
	  <td width><font face="Verdana" size="2">Prix de la côte d'occasion</font></td>
      <td width><input name="prix_cote_occasion" type="text" style="background-color: #CCCCCC;font: Verdana; color: #000000; border: 0px;"value="<?= $row["prix_cote_occasion"] ?>"> </td>
	</tr><tr>
	  <td width><font face="Verdana" size="2">Prix d'achat</font></td>
      <td width><input name="prix_achat" type="text" style="background-color: #CCCCCC;font: Verdana; color: #000000; border: 0px;" value="<?= $row["prix_achat"] ?>"></td>
	</tr><tr>
	<td width><font face="Verdana" size="2">Prix de'vente</font></td>
      <td width><input name="prix_vente" type="text" style="background-color: #CCCCCC;font: Verdana; color: #000000; border: 0px;" value="<?= $row["prix_vente"] ?>"></td>
	</tr><tr>
	<td><font face="Verdana" size="2">Sous Garantie</font></td>
			<td>
				<font face="Verdana" size="2">Oui:</font> <input type=radio name="sous_garantie" value="Oui">
 
    			<font face="Verdana" size="2">Non:</font> <input name="sous_garantie" type=radio value="Non" checked="checked" /></td>
	</tr><tr>
		<td width><font face="Verdana" size="2"></font></td>
	  <td><textarea name="textgarantie" cols="50" rows="10" style="background-color: #CCCCCC;font: verdana; color: #000000; border: 0px;"></textarea></td>
	</tr><tr>
	<td><font face="Verdana" size="2">Etat</font></td>
			<td>
				<select name="etat" size="1">
				<option value="1" selected="selected">Non vendu</option>
				<option value="2">Vendu</option>
			        </select>	</td>
	</tr><tr>
      <td valign="top">
      </td><td><div align="center">
<input type="reset" name="Reset" value="Effacer">          
<input type="submit" name="envoi" value="Envoyer">
        </div></td>
    </tr>
  </table>
  <div align="center"></div>
</form>
</body>
</html></html>

**Amara** · 23/06/2006, 11h06

Je vois justement pas d'endroit où tu transmets la valeur de id_produit à la page de traitement...

**xtiand4** · 23/06/2006, 11h25

Envoyé par JWhite

Je vois justement pas d'endroit où tu transmets la valeur de id_produit à la page de traitement...

au début quand je fais $id =$_GET['id_produit']; au debut du script!

**Sylvain71** · 23/06/2006, 11h31

J'ai pas lu tout le script mais :

au début quand je fais $id =$_GET['id_produit']; au debut du script!

ça serait pas plutot $_POST ?

**Amara** · 23/06/2006, 11h32

Oui mais c'est dans ta page de formulaire, après cette valeur n'est pas retransmise à ta page de traitement, je me trompe ?

Si c'est le cas il faut mettre ça dans un champ d type HIDDEN par exemple.

**xtiand4** · 23/06/2006, 11h45

Envoyé par JWhite

Oui mais c'est dans ta page de formulaire, après cette valeur n'est pas retransmise à ta page de traitement, je me trompe ?

Si c'est le cas il faut mettre ça dans un champ d type HIDDEN par exemple.

la je comprend pas ?
et je t assure que j ai fait le même script pour une autre page et ca marche.

alors pourquoi ici c est différent

**Amara** · 23/06/2006, 11h50

et je t assure que j ai fait le même script pour une autre page et ca marche

Mauvaise réponse. Il faut toujours chercher à comprendre même en recopiant du code d'une autre page même si on l'a développée soi-même, le copier-coller est l'ennemi du programmeur car il demande une vigilance accrue.

Avant de continuer une question pour être sûr : le traitement de ton formulaire est bien fait dans une autre page PHP que celle qui contient ledit formulaire ???

**xtiand4** · 23/06/2006, 12h11

Envoyé par JWhite

Mauvaise réponse. Il faut toujours chercher à comprendre même en recopiant du code d'une autre page même si on l'a développée soi-même, le copier-coller est l'ennemi du programmeur car il demande une vigilance accrue.

Avant de continuer une question pour être sûr : le traitement de ton formulaire est bien fait dans une autre page PHP que celle qui contient ledit formulaire ???

Merci du conseil!

**Amara** · 23/06/2006, 12h14

Et pour la question ?!?

**xtiand4** · 23/06/2006, 12h25

Envoyé par JWhite

Mauvaise réponse. Il faut toujours chercher à comprendre même en recopiant du code d'une autre page même si on l'a développée soi-même, le copier-coller est l'ennemi du programmeur car il demande une vigilance accrue.

Avant de continuer une question pour être sûr : le traitement de ton formulaire est bien fait dans une autre page PHP que celle qui contient ledit formulaire ???

oui tout à fait!

**Amara** · 23/06/2006, 12h27

Donc dans ton formulaire tu ajoutes un champ caché comme ça pour transmettre la valeur de l'id du produit :

Code :

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

1
2
3
4
 
<?
echo "<INPUT TYPE='hidden' NAME='id_produit' VALUE='" . $id . "'/>";
?>

Et ça devrait être bon.

**xtiand4** · 23/06/2006, 13h01

Envoyé par JWhite

Donc dans ton formulaire tu ajoutes un champ caché comme ça pour transmettre la valeur de l'id du produit :

Code :

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

1
2
3
4
 
<?
echo "<INPUT TYPE='hidden' NAME='id_produit' VALUE='" . $id . "'/>";
?>

Et ça devrait être bon.

Il ne me mets plus de message d erreur mais il n update pas ma table?

**xtiand4** · 23/06/2006, 13h04

Oh fait voici ou j ai ton echo

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
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" />
<title>Document sans titre</title>
<script language="javascript">
	function valider(){
		if (document.pform.nom.value.length == 0){
			alert("Veuillez saisir votre nom");
			document.pform.nom.focus();
			return;
		}
		
		if (document.pform.processeur.value.length == 0){
			alert("Veuillez saisir le processeur");
			document.pform.processeur.focus();
			return;
		}		
		
		document.pform.submit();
	}
</script>
</head>

<body bgcolor="#CCCCCC">
<? echo ' <tr>
            <td bgcolor="#FFFFFF"><a href="cms.php" <font color="#25AD3D">Retour à la gestion du stock Actimac</a></font></td>
          </tr>';
?>
<form method="POST" action="formstock1bis.php" enctype="multipart/form-data" name="pform">
<div align="center"></div>
<p align="center">

<?
echo "<INPUT TYPE='hidden' NAME='id_produit' VALUE='" . $id . "'/>";
?>

<table width="566" border="0" align="center">
<p align="center">
</p><tr>
	<td><font face="Verdana" size="2">Modéle</font></td>
			<td>
				<font face="Verdana" size="2">MAC:</font> <input type=radio name="modele" value="1">
    
    			<font face="Verdana" size="2">PC:</font> <input name="modele" type=radio value="2" checked="checked" /><br/></td>
	</tr><tr>
	<td><font face="Verdana" size="2">Type</font></td>
			<td>
				<select name="type">
				<option value="fixe">Fixe</option>
				<option value="portable">Portable</option>
				<option value="peripherique">Périphérique</option>
				<option value="accessoires">Accessoires</option>
				</select>	</td>
	</tr><tr>			
      <td><font face="Verdana" size="2">Nom</font></td>
      <td><input name="nom" type="text"style="background-color: #FFFFFF;font: Verdana; color: #000000; border: 0px;"></td>
    </tr><tr>
	  <td><font face="Verdana" size="2">Processeur</font></td>
      <td><input name="processeur" type="text"style="background-color: #FFFFFF;font: Verdana; color: #000000; border: 0px;"></td>
    </tr><tr>
	  <td width><font face="Verdana" size="2">Mémoire Video</font></td>
      <td width><input name="memoire_video" type="text" style="background-color: #FFFFFF;font: Verdana; color: #000000; border: 0px;" ></td>
	</tr><tr>
	  <td width><font face="Verdana" size="2">Mémoire Vive</font></td>
      <td width><input name="memoire_vive" type="text" style="background-color: #FFFFFF;font: Verdana; color: #000000; border: 0px;" ></td>
	</tr><tr>
	  <td width><font face="Verdana" size="2">Disque Dur</font></td>
      <td width><input name="disque_dur" type="text" style="background-color: #FFFFFF;font: Verdana; color: #000000; border: 0px;" ></td>
	</tr><tr>
	  <td width><font face="Verdana" size="2">Taille Ecran</font></td>
      <td width><input name="taille_ecran" type="text" style="background-color: #FFFFFF;font: Verdana; color: #000000; border: 0px;" ></td>
	</tr><tr>
	  <td width><font face="Verdana" size="2">Lecteur Optique</font></td>
      <td width><input name="lecteur_optique" type="text" style="background-color: #FFFFFF;font: Verdana; color: #000000; border: 0px;" ></td>
		</tr><tr> 
	  <td width><font face="Verdana" size="2">Systéme d'Opération</font></td>
      <td width><input name="os" type="text" style="background-color: #FFFFFF;font: Verdana; color: #000000; border: 0px;" ></td>
	</tr><tr>
	  <td width><font face="Verdana" size="2">Connectique</font></td>
      <td width><input name="connectique" type="text" style="background-color: #FFFFFF;font: Verdana; color: #000000; border: 0px;" ></td>
	</tr><tr>
	  <td width><font face="Verdana" size="2">Accessoires</font></td>
      <td width><input name="accessoires" type="text" style="background-color: #FFFFFF;font: Verdana; color: #000000; border: 0px;" ></td>
	</tr><tr>
	  <td width><font face="Verdana" size="2">Graveur</font></td>
      <td width><input name="graveur" type="text" style="background-color: #FFFFFF;font: Verdana; color: #000000; border: 0px;" ></td>
	</tr><tr>
	  <td width><font face="Verdana" size="2">Prix de la côte d'occasion</font></td>
      <td width><input name="prix_cote_occasion" type="text" style="background-color: #FFFFFF;font: Verdana; color: #000000; border: 0px;" ></td>
	</tr><tr>
	  <td width><font face="Verdana" size="2">Prix d'achat</font></td>
      <td width><input name="prix_achat" type="text" style="background-color: #FFFFFF;font: Verdana; color: #000000; border: 0px;" ></td>
	</tr><tr>
	  <td width><font face="Verdana" size="2">Prix de Vente</font></td>
      <td width><input name="prix_vente" type="text" style="background-color: #FFFFFF;font: Verdana; color: #000000; border: 0px;" ></td>
	</tr><tr>
	<td><font face="Verdana" size="2">Sous Garantie</font></td>
			<td>
				<font face="Verdana" size="2">Oui:</font> <input type=radio name="sous_garantie" value="Oui">
    
    			<font face="Verdana" size="2">Non:</font> <input name="sous_garantie" type=radio value="Non" checked="checked" /></td>
	</tr><tr>
	
		<td width><font face="Verdana" size="2"></font></td>	
	  <td><textarea name="textgarantie" cols="50" rows="10" style="background-color: #FFFFFF;font: verdana; color: #000000; border: 0px;"></textarea></td>
	
	</tr><tr>
	
	<td><font face="Verdana" size="2">Etat</font></td>
			<td>
				<select name="etat" size="1">
				<option value="1" selected="selected">Non vendu</option>
				<option value="2">Vendu</option>
			        </select>	</td>
	</tr>
	<tr>
		<td>Photo</td>
		<td><input type="file" size="50" name="photo" /></td>
	</tr>
	<tr>
      <td valign="top">
      </td><td><div align="center">
<input type="reset" name="Reset" value="Effacer" >          
<input type="button" name="envoi" value="Envoyer" onClick="valider()">
        </div></td>
    </tr>
  </table>
  <div align="center"></div>
</form>
</body>
</html>

est ce correcte ?
ps: le echo est en rouge dans le code

**Amara** · 23/06/2006, 13h39

Oui c'est bon le principal c'est que ça soit dans le FORM, après la place importe peu. Je suppose juste qu'avant le code que tu viens de mettre tu as toujours le bout de code PHP qui initialise $id...

Le problème vient maintenant de la requête UPDATE, je te conseille de faire un echo $sql; juste avant ton mysql_query pour voir si ça correspond à ce que tu souhaites faire.

**xtiand4** · 23/06/2006, 14h00

Envoyé par JWhite

Oui c'est bon le principal c'est que ça soit dans le FORM, après la place importe peu. Je suppose juste qu'avant le code que tu viens de mettre tu as toujours le bout de code PHP qui initialise $id...

Le problème vient maintenant de la requête UPDATE, je te conseille de faire un echo $sql; juste avant ton mysql_query pour voir si ça correspond à ce que tu souhaites faire.

excuse moi j ai pas encore les reflex du code.
c est quoi la synthaxe du echo $sql