Récupération des données d'un formulaire avec checkbox

**Solhunken** · 28/04/2015, 09h57

Bonjour,
j'ai pas mal cherché sur le forum sans trouvé de solution à mon problème alors je viens demander de l'aide,
désolé par avance si le titre n'est pas explicite mais je ne savais pas trop comment le définir...

J'ai une page php qui affiche une liste de produits tirée d'une base SQL.

Je peux modifier des options sur chaque ligne puis valider les modifications faites sur la page avec un bouton $_POST.

Je traite le tout avec une page de traitement (code ci-dessous) mais je rencontre un souci avec des checkbox,
il s'agit de $_POST['Creserved'], je souhaite modifier la valeur avant écriture dans la BDD mais si j'ai une liste de 3 produits
<td>PROD1</td><td>checkbox</td>
<td>PROD2</td><td>checkbox</td>
<td>PROD3</td><td>checkbox</td> (je coche cette checkbox)
et que je coche seulement le dernier de la liste, après traitement celui qui est coché est le premier de la liste.
résultat :
<td>PROD1</td><td>checkbox</td> (checkbox cochée)
<td>PROD2</td><td>checkbox</td>
<td>PROD3</td><td>checkbox</td> (checkbox non cochée)

J'ai ce problème seulement avec la partie du code qui traite les checkbox :

Code :

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

1
2
3
if (isset(Creserved[$key]))
$Creserved_format[$key] = '1';
else $Creserved_format[$key] = '0';

Je ne parviens pas à résoudre ce problème... Mes connaissances en php ne sont pas hyper développées,
j'apprends tout seul. Merci d'avance de votre aide.

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
<?php
require_once('_connexion.php'); 
 
/*Récupérer les données */
$CKey = $_POST['CKey'];
$Cid = $_POST['Cid'];
$Clanguage = $_POST['Clanguage'];
$Cetat = $_POST['Cetat'];
$Ccote = $_POST['Ccote'];
$Ccommentaire = $_POST['Ccommentaire'];
$Creserved = $_POST['Creserved'];
 
foreach ($CKey as $key => $value)
{
 
if (isset(Creserved[$key]))
$Creserved_format[$key] = '1';
else $Creserved_format[$key] = '0';
 
/* requete de MAJ */ 
$query_collec_modif = ("UPDATE collection SET 
 	Clanguage = '$Clanguage[$key]', Cetat = '$Cetat[$key]', Ccote = $Ccote[$key], Ccommentaire = '$Ccommentaire[$key]', Ctab = '1', Creserved = '$Creserved_format[$key]'
	WHERE CKey=$CKey[$key];"); 
 
$result = mysql_query($query_collec_modif) or die('Erreur SQL !<br>'.$query_collec_modif.'<br>'.mysql_error());
echo $query_collec_modif.'<br/>';
}
header('Location: collection_modif.php');  
?>

**Celira** · 28/04/2015, 10h48

Regarde bien la coloration syntaxique :if (isset(Creserved[$key])). Il ne manquerait pas un $ quelque part ?

Au passage, pendant la phase de développement, il vaut mieux activer toutes les erreurs PHP, y compris les NOTICES, car une notice peut signaler l'arbre qui cache la forêt.

**Solhunken** · 28/04/2015, 11h16

Oui effectivement, je l'ai oublié mais même en rectifiant cette erreur le comportement est identique.

Pour ce qui est d'afficher les erreurs PHP je n'ai pas la main dessus le fichier php.ini

**Celira** · 28/04/2015, 11h47

Pour afficher les erreurs, tu peux ajouter error_reporting(E_ALL); au début de ton script.

Pour tes checkboxes, est-ce que la valeur est bien mise à jour en base dans le bon champ ?

**Solhunken** · 28/04/2015, 12h00

Dans la base, ce n'est pas la bonne ligne qui est mise à jour.
Pour les autres champs c'est bon mais seule la valeur issue de la checkbox s'enregistre sur la 1ère ligne produit visible.

voilà le résultat issu de

Code :

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

echo $query_collec_modif.'<br/>';

UPDATE collection SET Clanguage = 'KR', Cetat = 'M', Ccote = 15, Ccommentaire = 'comment1', Ctab = '1', Creserved = 1 WHERE CKey=26; <<< Creserved est à 1 au lieu de 0
UPDATE collection SET Clanguage = 'IT', Cetat = 'NM', Ccote = 0, Ccommentaire = 'comment2', Ctab = '1', Creserved = 0 WHERE CKey=28;
UPDATE collection SET Clanguage = 'FR', Cetat = 'NM', Ccote = 40, Ccommentaire = 'comment3', Ctab = '1', Creserved = 0 WHERE CKey=23;
UPDATE collection SET Clanguage = 'FR', Cetat = 'NM', Ccote = 40, Ccommentaire = 'comment4', Ctab = '1', Creserved = 0 WHERE CKey=24;
UPDATE collection SET Clanguage = 'CN', Cetat = 'PL', Ccote = 40, Ccommentaire = 'À supprimer', Ctab = '1', Creserved = 0 WHERE CKey=17;
UPDATE collection SET Clanguage = 'JP', Cetat = 'NM', Ccote = 80, Ccommentaire = '\'insert \'', Ctab = '1', Creserved = 0 WHERE CKey=27; <<< Creserved est à 0 au lieu de 1

**Celira** · 28/04/2015, 13h43

Il y a peut-être un problème au niveau du formulaire, on peut voir le code du formulaire ?
Sinon, question bête : ce que tu veux en base, c'est bien cochée = 1, non cochée = 0 ?

**Solhunken** · 28/04/2015, 14h00

ce que tu veux en base, c'est bien cochée = 1, non cochée = 0 ?

Oui, tout à fait.

Voilà le code 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
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
134
135
136
137
138
139
140
141
142
143
144
145
146
<?
session_start();  // Démarrage de la session. A placer obligatoirement avant tout code html
$username = $_GET['username'];//On récupère les données envoyées par la méthode POST du formulaire d'identification
$id = $_GET['id'];//On récupère les données envoyées par la méthode POST du formulaire d'identification
$Nnamepage = $_GET['Nnamepage']; // depuis index.php
$Nnamepage_FR = $_GET['Nnamepage_FR']; // depuis index.php
$Nset = $_GET['Nset']; // depuis index.php
$Nid = $_GET['Nid']; // depuis index.php
$Language = $_GET['Language']; // Depuis carte/carteVO.php
 
?>
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01//EN" "http://www.w3.org/TR/html4/strict.dtd">
<head>
		<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
		<meta name="Details" content="">
		<meta name="author" content="K">
		<meta name="robots" content="index,follow">
		<meta name="keywords" content=""/>
		<link rel="stylesheet" href="Css.css" />
		<title>Collection</title>
	</head>
<body>
<?php
///si l'utilisateur est connecté, on affiche le control panel
if(isset($_SESSION['username']))
{
echo '<div id="log_in"><table border="0" width="600">';
include ("log_ok.php");
echo '</table></div>';
}
else
{
//Sinon, on affiche le panel de login
echo '<div id="log_in"><table border="0" width="600">';
include("log_in.php");
echo '</table></div>';
header("Location: index.php");
 
}
try
{	
// initialisation des variables 
require_once('_connexion.php'); 
// connection à la DB
$bdd = new PDO('mysql:host='.$host.';dbname='.$bdd.'', ''.$user.'', ''.$pass.'') or die ("Impossible de se connecter");
}
catch(Exception $e)
{// En cas d'erreur, on affiche un message et on arrête tout
        die('Erreur : '.$e->getMessage());
}// Si tout est ok, on continu
 
$collection= mysql_query ('SELECT 
CKey, Cid, Cusername, Ctab, NKey_card, Cqty, Clanguage, Cetat, Ccote, Ccommentaire, Creserved, Cdate,
Ncards.Nname,
Ncards.Nname_FR,
Ncards.Nset,
Ncards.Nrarity,
Ncards.Nnumber,
Nsets.Nname_FR as "NSetName_FR",
Nsets.Nname as "NSetName"

FROM collection
left outer join Ncards on collection.NKey_card = Ncards.NKey
left outer join Nsets on Ncards.Nset = Nsets.Ncode

WHERE Cid = \''.$_SESSION['userid'].'\'
order by Ncards.Nset, Ncards.Nname_FR ');
 
//-----------------------------------------------------------------------
echo '<style> <!--body{	background-color: #d9d4d0;}	--></style>'; 
 
echo '<div id="Header"><table border=0 width=300>';
include("Header.php");
echo '</table></div>';
echo '<div id="search"><table border=0 width=900>';
include("search.php");
echo '</table></div>';
 
echo '<div id="classeur"><table border=0 width=900 class=txt><tr>
<td width=320>Nom</td>
<td witdh=30 align="center">Édition</td>
<td witdh=10 align="center">Langue</td>
<td witdh=10 align="center">État</td>
<td witdh=30 align="center">Qté</td>
<td witdh=40 align="center">Côte</td>
<td>Commentaire</td>
<td>Réservée</td>
</tr>';
 
$linenumber = 0;
while($datacollection = mysql_fetch_array($collection))
{
echo '<style>
a:hover { 
	color:red;
	text-decoration:none;
}
</style>';
if ($data["Nname_FR"] ==''){$Nname = $data["Nname"];} else {$Nname = $data["Nname_FR"];}
 
$linenumber = $linenumber+1;
if ($linenumber%2==1){ 
echo '<tr bgcolor="#EEEEEE">';} elseif ($linenumber%2<>1){ echo '<tr>';}
if ($data["NSetName_FR"] =='' ) {$NSetName = $data["NSetName"]; } else {$NSetName = $data["NSetName_FR"]; }
 
echo '<form action="collection_modif_process.php" method="post">';
echo '<td>'.$datacollection['Nname'].'</td>
<td align="center"><img src="/pics/pics_page/'.$datacollection["Nset"].'/'.$datacollection["Nrarity"].'.png"></a></td>
<td><SELECT name="Clanguage[]" size="1">
		<OPTION VALUE="FR"'; if($datacollection['Clanguage']=='FR'){echo 'selected="selected"';} echo '>Français</OPTION>
		<OPTION VALUE="VO"'; if($datacollection['Clanguage']=='VO'){echo 'selected="selected"';} echo '>Anglais</OPTION>
		<OPTION VALUE="DE"'; if($datacollection['Clanguage']=='DE'){echo 'selected="selected"';} echo '>Allemand</OPTION>
		<OPTION VALUE="IT"'; if($datacollection['Clanguage']=='IT'){echo 'selected="selected"';} echo '>Italien</OPTION>
		<OPTION VALUE="ES"'; if($datacollection['Clanguage']=='ES'){echo 'selected="selected"';} echo '>Espagnol</OPTION>
		<OPTION VALUE="PT"'; if($datacollection['Clanguage']=='PT'){echo 'selected="selected"';} echo '>Portugais</OPTION>
		<OPTION VALUE="CN"'; if($datacollection['Clanguage']=='CN'){echo 'selected="selected"';} echo '>Chinois</OPTION>
		<OPTION VALUE="JP"'; if($datacollection['Clanguage']=='JP'){echo 'selected="selected"';} echo '>Japonais</OPTION>
		<OPTION VALUE="KR"'; if($datacollection['Clanguage']=='KR'){echo 'selected="selected"';} echo '>Koréen</OPTION>
		<OPTION VALUE="RU"'; if($datacollection['Clanguage']=='RU'){echo 'selected="selected"';} echo '>Russe</OPTION>
</SELECT></td>
<td><SELECT name="Cetat[]" size="1">
		<OPTION VALUE="M"'; if($datacollection['Cetat']=='M'){echo 'selected="selected"';} echo '>Mint</OPTION>
		<OPTION VALUE="NM"'; if($datacollection['Cetat']=='NM'){echo 'selected="selected"';} echo '>Near Mint</OPTION>
		<OPTION VALUE="EX"'; if($datacollection['Cetat']=='EX'){echo 'selected="selected"';} echo '>Excellent</OPTION>
		<OPTION VALUE="GO"'; if($datacollection['Cetat']=='GO'){echo 'selected="selected"';} echo '>Good</OPTION>
		<OPTION VALUE="PL"'; if($datacollection['Cetat']=='PL'){echo 'selected="selected"';} echo '>Played</OPTION>
		<OPTION VALUE="PO"'; if($datacollection['Cetat']=='PO'){echo 'selected="selected"';} echo '>Poor</OPTION>
</SELECT></td>';
 
echo '<td align="right">'.$datacollection['Cqty'].'</td>

<td><input size="5" maxlength="4" format="NNNN" style="text-align: right" type="text" value ="'.$datacollection['Ccote'].'" name="Ccote[]"></td>
<td><input size="12" type="text" value ="'.$datacollection['Ccommentaire'].'" name="Ccommentaire[]"></td>';
if ($datacollection['Creserved'] == '1') {$Creserved="checked=checked";} elseif ($datacollection['Creserved'] == '0') {$Creserved="";} 
echo '<td width=10 align=center><input type="checkbox" '.$Creserved.' name="Creserved[]"></td>';
echo '
	<input type="hidden" name="Cid[]" id="Cid" value="'.$datacollection["Cid"].'"/>
	<input type="hidden" name="CKey[]" id="CKey" value="'.$datacollection["CKey"].'"/>
	<input type="hidden" name="Cusername[]" id="Cusername" value="'.$_SESSION['username'].'"/>
</tr>';
}
echo '<tr><td><table border=1>
<tr><td><input type="submit" name="Submit" value="Valider les modifications" /></td></tr>
</table></td></tr></form></form>
</table></div>';
?>

Merci de ton aide.

edit : J'ai peut être une idée sur le problème, dans le traitement je vérifie

Code :

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

1
2
3
if (isset($Creserved[$key]))
$Creserved_format[$key] = '1';
else $Creserved_format[$key] = '0';

Une checkbox non cochée ne renvoi rien comme valeur (je crois bien), la valeur [$key] semble rester identique à la 1ère ligne, et non a celle cochée.
Je me relis mais je sens que je ne suis pas très clair...

**Celira** · 28/04/2015, 14h20

Ah je sais ! lorsque tu envoies des checkboxes, seules les checkboxes cochées sont effectivement envoyées. Donc si tu as un tableau de checkboxes et que tu en coches une seule, tu envoies un tableau d'un élément. Si les index ne sont pas explicites dans ton tableau, ta checkboxe cochée sera automatiquement à l'index 0 de ton tableau.

Donc ce qu'il faut faire, c'est forcer l'index du tableau de checkboxes, par exemple avec une variable que tu incrémentes à chaque tour.

**Solhunken** · 28/04/2015, 14h24

Donc je modifie le for each en incrémentant une valeur à chaque ligne plutôt que d'utiliser la valeur $key ?
Je vais faire des tests, merci pour la piste.

**Celira** · 28/04/2015, 14h32

Le plus simple serait peut-être d'indexer TOUS les tableaux du formulaire avec la même variable, comme ça tu pourras conserver ton script d'enregistrement.

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
$i = 0;
while($datacollection = mysql_fetch_array($collection))
{
	echo '<td><SELECT name="Clanguage['.$i.']" size="1">';
	// ...
 
	echo '<td><SELECT name="Cetat['.$i.']" size="1">';
	// ...
	echo '<td width=10 align=center><input type="checkbox" '.$Creserved.' name="Creserved['.$i.']"></td>';
	echo '
		<input type="hidden" name="Cid['.$i.']" id="Cid" value="'.$datacollection["Cid"].'"/>
		<input type="hidden" name="CKey['.$i.']" id="CKey" value="'.$datacollection["CKey"].'"/>
		<input type="hidden" name="Cusername['.$i.']" id="Cusername" value="'.$_SESSION['username'].'"/>
	</tr>';
	$i++;
}

**Solhunken** · 28/04/2015, 14h50

Super, ton code fonctionne parfaitement comme je le souhaite. Mille merci.

**Celira** · 28/04/2015, 14h51

Tant mieux !

Récupération des données d'un formulaire avec checkbox [MySQL]

PHP & Base de données

Discussions similaires

Partager

Partager