Discussion :

[link.80]

Bonjour,

J'ai une table MYSQL structurée comme ceci :
ID (type : tinyint(4)) TEXTE (type : text)

ID | TEXTE
1 | blablabla
2 | ahahah
3 | ohohoho
...
24| hihihi


Su ma page je génère 24 input différents à l'aide d'une boucle for correspondant au 24 lignes de ma table.

J'aimerais lors sur submit, faire une update sur toutes les lignes, or je n'arrives pas, j'ai une erreur SQL, ça fait un moment que je retourne le problème dans tous les sens et ne ne trouves pas, ça serait vraiment sympa de votre part de m'aider, voici le code :

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
echo '<form method="post" action="" enctype="multipart/form-data">';
 
		for($i=1; $i<25; $i++)
		{
			$sql="SELECT * FROM calendrier WHERE ID=".$i." ";
			$resU=mysql_query($sql);
			$row=mysql_fetch_array($resU);
 
		echo '<h1>Champ N° '.$i.'</h1>
			<input class="input" name="desc'.$i.'" value="'.$row["TEXTE"].'" type="text" />';
 
		}
		echo '<input type="submit" value="Valider" />';
		echo '</form>';
 
 
	  if ($_POST)
	  {
		  for($i=1; $i<25; $i++)
		  {
			  $qn="UPDATE calendrier SET TEXTE='".$_POST["desc".$i.""]."' ";
			  $qn.="WHERE ID=".$i." ";
 
		  }
		  $res=mysql_query($qn);
		  if (isset($res))
		  {
		  echo "OK";
		  }
		  else 
		  {
                 echo "Erreur";
		  }
	  }

[sabotage]

Et quelle est l'erreur ?

Sinon, ecrire en dur des id est une grosse impasse :

- une seule requête SELECT et pas de SELECT *
- lecture des valeurs recues
- on protège les valeurs recues

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
echo '<form method="post" action="" enctype="multipart/form-data">';
 
$sql="SELECT id, texte FROM calendrier LIMIT 25;";
$resU=mysql_query($sql);
while($row=mysql_fetch_array($resU)) {
 		echo '<h1>Champ N° '.$row["id"].'</h1>
			<input class="input" name="desc['.$row["id"].']" value="'.$row["texte"].'" type="text" />';
		}
		echo '<input type="submit" value="Valider" />';
		echo '</form>';
 
 
	  if (isset($_POST['desc']))
	  {
		foreach($_POST['desc'] as $key=>$value) {
			  $qn="UPDATE calendrier SET TEXTE='".mysql_real_escape_string($value)."' WHERE ID=".intval($key);
              		  $res=mysql_query($qn);
                 }
	  }

[link.80]

Merci pour ta réponse Sabotage,

j'ai essayé ton code mais il ne marche pas, je l'ai alors adapté, il y avait un souci avec la valeur $_POST['desc'], j'ai donc eu l'idée d'ajouter un champ caché pour récupérer l'ID.

Ensuite je ne comprends pas pourquoi il faut mettre un foreach. Je ne fais a aucun moment de tableaux dans mon code, c'est pas plutôt un for qu'il faudrait mettre ?
Et aussi $_POST['desc'] est faux il me semble.

Ensuite tu dis

une seule requête SELECT et pas de SELECT *

aurais-tu la gentillesse de me dire pourquoi et quelle est la différence entre écrire SELECT * ou SELECT ID, TEXTE. Je demande par simple curiosité et parce que ça m'intéresse

Merci d'avance pour ton aide, c'est sympa !

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
 
echo '<form method="post" action="" enctype="multipart/form-data">';
 
		$sql="SELECT ID, TEXTE FROM calendrier";
		$resU=mysql_query($sql);
		while($row=mysql_fetch_array($resU)) 
		{
		echo '<h1>Champ N° '.$row["ID"].'</h1>
			  <input class="input" name="desc['.$row["ID"].']" value="'.$row["TEXTE"].'" type="text" />
			  <input type="hidden" name="id" value="'.$row["ID"].'" />
			  ';
		}
		echo '<input type="submit" value="Valider" />';
		echo '</form>';
 
 
	  if ($_POST)
	  {
		foreach($_POST['id'] as $key=>$value) 
		{
			$qn="UPDATE calendrier SET TEXTE='".mysql_real_escape_string($value)."' WHERE ID=".intval($key);
        	$res=mysql_query($qn);
 
		echo $res;
		exit;
	  	}

[sabotage]

Je ne fais a aucun moment de tableaux dans mon code

Quand j'écris name="desc['.$row["ID"].']", le formulaire renvoit un tableau $_POST['desc'].
Donc c'est bien un foreach sur $_POST['desc'] et isset($_POST['desc']) qu'il nous faut.
Tu n'as pas besoin du champ Hidden, la clef de $_POST['desc'] est l'id.
Pour mieux comprendre le retour tu peux faire un

Code :

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

print_r($_POST);

quelle est la différence entre écrire SELECT * ou SELECT ID, TEXTE

Si dans ta table tu as un champ image, en faisant SELECT *, tu le recupererais alors que tu n'en as pas besoin.

[yaniss321]

Salut.
en gros, Select * prend tout select element1, element2, etc... ne prend que les éléments que tu lui demande

[link.80]

ok, super un grand merci
j'y vois un peu plus clair maintenant !

Dernière question, si je veux maintenant rajouter par exemple un champ d'upload en dessous de chaque input, je me la joue comment ?

c'est juste comme ça ?

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
 
		echo '<form method="post" action="" enctype="multipart/form-data">';
 
		$sql="SELECT ID, TEXTE FROM calendrier";
		$resU=mysql_query($sql);
		while($row=mysql_fetch_array($resU)) 
		{
		echo '<h1>Champ N° '.$row["ID"].'</h1>
			  <input class="input" name="desc['.$row["ID"].']" value="'.$row["TEXTE"].'" type="text" />
			  <input type="file" name="upFile['.$row["ID"].']" />';
		}
		echo '<br />
			  <br />
			  <input type="submit" value="Valider" />
			  </form>';
 
	  if (isset($_POST['desc']))
	  {
		foreach($_POST['desc'] as $key=>$value)
		{
			$qn="UPDATE calendrier SET TEXTE='".mysql_real_escape_string($value)."' WHERE ID=".intval($key);
        	$res=mysql_query($qn);
	  	}	
 
		foreach($_FILES["upFile"] as $key2=>$value2)
		{
 
		  if(move_uploaded_file($value2, "calendrier/".$key2.".jpg"))
		  {
		  echo "OK";
		  }
		  else 
		  {			 
		   echo "Erreur";
		  }
 
	  	}