problème récupération variable contenu dans un tableau

**mac-addict** · 22/08/2011, 09h08

Bonjour à tous ,
Je vous explique mon souci , je débute en php et j'ai un tableau contenant plusieurs variables .
Les 5 premières colonnes contiennent des variables directement récupérées dans ma base de données, jusque la pas de souci

.

La ou ça se complique c concernant les 2 dernières colonnes , l'avant dernière contient un input de type bouton radio j'arrive a récupère la valeur de mon bouton si coché et à la faire s'enregistrer dans la base de données mais impossible le de faire de même pour la dernière colonne.

La dernière colonne contient un input de type text , et je voudrais réussir à enregistrer la valeur saisie dans le champ dans la BDD au changement de page.

J'utilise la method post dans mon formulaire et sur la page de destination récupère mes variables post et les déclarent en variable de session mais ça coince ^^

Je vous copie colle mon formulaire en method post et les lignes de la page suivante pour enregistrer et afficher les valeurs.

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
<table id="hovertable">
<form name="list" METHOD="POST" ACTION="admin6.php">
<?php
 
echo '<tr><th>table:</th><th>personnel:</th><th>service:</th><th>pour la periode du:</th><th>au:</th><th>motif:</th><th>type:</th><th>date:</th><th>effectue:</th><th>à modifier:</th><th>annotation:</th></tr>';
while($ligne = mysql_fetch_object($res))
 
	{	
	 echo '<tr onmouseover="this.style.backgroundColor="#ffff66";" onmouseout="this.style.backgroundColor="#d4e3e5";"><td>'.$ligne->nom_table.'</td><td>'.$ligne->nom.'</td><td>'.$ligne->service.'</td><td>'.$ligne->date_debut.'</td><td>'.$ligne->date_fin.'</td><td>'.$ligne->motif.'</td><td>'.$ligne->type.'</td><td>'.$ligne->date.'</td><td>'.$ligne->effectue.'</td><td><INPUT TYPE="radio" name="modif" value="'.$ligne->nmr.'">'.'</td><td><INPUT TYPE="text" name="annotation">'.'</td></tr>';
	}
 
?>
</table>
<div><input type=submit name=envoyer value=envoyer></div>
<br>
</form>

et sur ma page admin6 j'ai cela:

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
$_SESSION['modif']=$_POST["modif"];
$_SESSION['annotation']=$_POST["annotation"];
$oui="oui";
$sql1 = 'update fusion set effectue = "'.$oui.'" where nmr = "'.$_SESSION['modif'].'"';
$res1= mysql_query($sql1);
 
$sql2 = 'INSERT INTO fusion VALUES ("'.'","'.'","'.'","'.'","'.'","'.'","'.'","'.'","'.'","'.'","'.$_SESSION['annotation'].'")';
$res2 = mysql_query($sql2);
 
$sql="select * from fusion WHERE effectue like 'oui' order by nmr desc";
$res= mysql_query($sql);
 
echo '<tr><th>table:</th><th>personnel:</th><th>service:</th><th>pour la periode du:</th><th>au:</th><th>motif:</th><th>type:</th><th>date:</th><th>effectue:</th><th>annotation:</th></tr>';
while($ligne = mysql_fetch_object($res))
    {	
	 echo '<tr><td>'.$ligne->nom_table.'</td><td>'.$ligne->nom.'</td><td>'.$ligne->service.'</td><td>'.$ligne->date_debut.'</td><td>'.$ligne->date_fin.'</td><td>'.$ligne->motif.'</td><td>'.$ligne->type.'</td><td>'.$ligne->date.'</td><td>'.$ligne->effectue.'</td><td>'.$ligne->annotation.'</tr>';
	}
 
?>
</table>

D'avance merci .

Invité · 22/08/2011, 09h53

Bonjour,
ps :
- toutes les balises html s'écrivent en minuscules.
- les input, br (et img) sont fermantes (<.../>):

Code :

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

1
2
<input type="text" name="annotation" value="" />
<br />

- ta façon d'écrire le code html dans le php, + le fait que tout soit sur la même ligne : c'est vite illisible, et ne facilite pas le débogage.
- une bonne indentation du code est recommandée (pour : lisibilité, maintenance, débogage du code)
-

il faut absolument proteger la BdD des injections sql ! -> mysql_real_escape_string()

Autant prendre de "bonnes habitudes" tout de suite :

-> PHP : Le tutoriel pour grands débutants pressés
-> Cours de PHP 5
-> Les tableaux (XHTML & CSS)

D'après toi, lequel de ton code et du mien est le plus clair et le plus "propre" ?

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
<form name="list" method="post" action="admin6.php">
<table id="hovertable">
<thead>
	<tr>
		<th>table:</th>
		<th>personnel:</th>
		<th>service:</th>
		<th>pour la periode du:</th>
		<th>au:</th><th>motif:</th>
		<th>type:</th><th>date:</th>
		<th>effectue:</th><th>à modifier:</th>
		<th>annotation:</th>
	</tr>
</thead>
<tbody>
<?php
while($ligne = mysql_fetch_object($res))
{
?>
	<tr onmouseover="this.style.backgroundColor='#ffff66';" onmouseout="this.style.backgroundColor='#d4e3e5';">
		<td><?php echo $ligne->nom_table; ?></td>
		<td><?php echo $ligne->nom; ?></td>
		<td><?php echo $ligne->service; ?></td>
		<td><?php echo $ligne->date_debut; ?></td>
		<td><?php echo $ligne->date_fin; ?></td>
		<td><?php echo $ligne->motif; ?></td>
		<td><?php echo $ligne->type; ?></td>
		<td><?php echo $ligne->date; ?></td>
		<td><?php echo $ligne->effectue; ?></td>
		<td><input type="radio" name="modif" value="<?php echo $ligne->nmr; ?>" /></td>
		<td><input type="text" name="annotation" /></td>
	</tr>
<?php
}
?>
</tbody>
</table>
<div><input type="submit" name="envoyer" value="envoyer"/></div>
<br />
</form>

**zeclad01** · 22/08/2011, 10h05

Code illisible, puis merci d'utiliser les balises [ CODE][/CODE ] si tu veux un debug.

Invité · 22/08/2011, 10h06

Code :

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

1
2
3
4
5
6
<table id="hovertable">
<form name="list" METHOD="POST" ACTION="admin6.php">
....
</table>
....
</form>

Vois-tu le problème ?
Ca ira mieux comme ca :

Code :

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

1
2
3
4
5
6
<form name="list" method="post" action="admin6.php">
<table id="hovertable">
....
</table>
....
</form>

Au fait, pour ton problème :

Code :

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

$sql2 = "INSERT INTO fusion (annotation) VALUES ('".mysql_real_escape_string($_SESSION['annotation'])."');";

-> LE SQL de A à Z : 6e partie - les mise à jour en SQL - INSERT, DELETE et UPDATE

**mac-addict** · 22/08/2011, 10h42

Tout d'abord merci pour vos réponses et désolé pour la présentation, je vais faire un effort.

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
 
<table id="hovertable">
<?php
$_SESSION['modif']=$_POST["modif"];
 
$_SESSION['annotation']=$_POST["annotation"];
$oui="oui";
 
$sql1 = 'update fusion set effectue = "'.$oui.'" and annotation = "'.$_SESSION['annotation'].'"  where nmr = "'.$_SESSION['modif'].'"';
$res1= mysql_query($sql1);
 
$sql="select * from fusion WHERE effectue like 'oui' order by nmr desc";
$res= mysql_query($sql);
 
echo '<tr>
              <th>table:</th>
              <th>personnel:</th>
              <th>service:</th>
              <th>pour la periode du:</th>
              <th>au:</th>
              <th>motif:</th>
              <th>type:</th>
              <th>date:</th>
              <th>effectue:</th>
              <th>annotation:</th>
       </tr>';
while($ligne = mysql_fetch_object($res))
    {	
	 echo '<tr>
                       <td>'.$ligne->nom_table.'</td>
                       <td>'.$ligne->nom.'</td>
                       <td>'.$ligne->service.'</td>
                       <td>'.$ligne->date_debut.'</td>
                       <td>'.$ligne->date_fin.'</td>
                       <td>'.$ligne->motif.'</td>
                       <td>'.$ligne->type.'</td>
                       <td>'.$ligne->date.'</td>
                       <td>'.$ligne->effectue.'</td>
                       <td>'.$ligne->annotation.'</td>
                </tr>';
	}
 
?>
</table>

Donc voila une partie du code de admin6 celle sensée afficher l'annotation saisie dans le champ texte a la page précédente et récupérée via le post, j'espère que c'est plus lisible.

Dans mon tableau sur la page précédente je me retrouve donc avec une colonne entière d'input type text mais je ne veux récupérer que la valeur du champ qui est rempli ( celui à côté du bouton radio modifié qui sera coché , le but étant ici de mettre une tâche effectué en traitée : oui et de mettre une annotation sur la manière dont le problème a été résolu).

Je précise que je suis sur un réseau intranet .

**zeclad01** · 22/08/2011, 11h02

Est-ce que tes variables ne sont pas vides lorsque tu les récupères/affectes ?

Code :

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

1
2
echo "SESSION : ".$_SESSION['annotation'];
echo "POST : ".$_POST['annotation'];

**mac-addict** · 22/08/2011, 11h11

effectivement , elles restent vide même si je rempli le champ annotation dans mon tableau.

Je ne comprends pas

Invité · 22/08/2011, 11h26

J'ai vu le problème !

Il faut identifier tes input pour chaque ligne du tableau (sinon, il ne prend que le dernier) : name="modif[]" et name="annotation[]"

Pour chaque enregistrement (ligne), tu dois sûrement avoir un id (int, auto-increment) :
=> on va mettre cet id comme indice des array modif[] et annotation[] :

Code :

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

1
2
		<td><input type="radio" name="modif[<?php echo $ligne->id; ?>]" value="<?php echo $ligne->nmr; ?>" /></td>
		<td><input type="text" name="annotation[<?php echo $ligne->id; ?>]" /></td>

=> tu récupères ici des array :

Code :

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

1
2
$array_modif = $_POST['modif']; // array
$array_annotation = $_POST['annotation']; // array

qu'il faut donc traiter comme suit :

Code :

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

1
2
3
4
5
6
foreach($array_modif as $key => $val) {
   // [...] $key est en fait l'id !
}
foreach($array_annotation as $key => $val) {
   // [...] $key est en fait l'id !
}

Invité · 22/08/2011, 11h31

Et si j'ai bien compris : ce n'est pas une requete INSERT que tu dois faire.

Mais une requete UPDATE pour chaque ligne !

Code :

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

1
2
3
4
5
6
7
8
<?php
foreach($_POST['annotation'] as $key => $val) {
	// $key est l'indice de l'array (ici = l'id de l'enregistrement !)
	// $val est la valeur
	$sql2 = "UPDATE fusion SET annotation = '".mysql_real_escape_string($val)."' WHERE id = '".intval($key)."';";
	// ...
}
?>

**mac-addict** · 22/08/2011, 12h03

Tout d'abord un grand merci , je comprends le principe de parcourir chaque élément un par un et effectivement cela parait très utile.

Cependant dans ma base l'id c nmr (pour numéro) , je dois donc remplacer id par nmr dans mon cas c'et bien cela?

De plus en mettant nmr a la place de id pour numéroter mes array
J'obtiens comme rentrée dans mes bases : Array pour chaque annotation.

Voila ce que j'ai mis sur la première page

Code :

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

1
2
<td><input type="radio" name="modif[<?php echo $ligne->nmr; ?>]" value="<?php echo $ligne->nmr; ?>" /></td>
<td><input type="text" name="annotation[<?php echo $ligne->nmr; ?>]" /></td>

et sur ma 2ième page j'ai fait 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
<?php
$array_modif = $_POST['modif']; // array
$array_annotation = $_POST['annotation']; // array
$oui="oui";
 
foreach($array_modif as $key => $val) 
{
$sql1 = 'update fusion set effectue = "'.$oui.'" where nmr = "'.$array_modif.'"';
$res1= mysql_query($sql1);
}
foreach($array_annotation as $key => $val) 
{
$sql2 = 'update fusion set annotation = "'.$array_annotation.'"';
$res2 = mysql_query($sql2);
}

Je sais que je suis nul lol, mais je pense que la soutien ne doit pas être loin , c'est juste que je m'y prends mal , car je ne maitrise pas ce que j'utilise.
D'avance merci.

Invité · 22/08/2011, 13h04

Moi aussi, j'ai eu du mal avec foreach($blabla as $key => $val) ....
Remplace :

Code :

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

1
2
3
4
5
foreach($array_annotation as $key => $val) 
{
$sql2 = 'update fusion set annotation = "'.$array_annotation.'"';
$res2 = mysql_query($sql2);
}

par (ce que j'avais pourtant écrit ici !) :

Code :

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

1
2
3
4
5
6
7
 
foreach($array_annotation as $key => $val) // se traduit par : "pour chaque couple indice=>valeur de l'array ..."
{
	// ici, $key (indice de l'array) a été défini comme nmr (l'id dans la bdd)
	$sql2 = "UPDATE fusion SET annotation = '".mysql_real_escape_string($val)."' WHERE nmr = '".intval($key)."';";
	$res2 = mysql_query($sql2);
}

Tu peux jeter un oeil ici aussi : Apostrophes ou guillemets : lesquels choisir ?

ps : Si tu veux voir à quoi ressemble l'array (couples [indice] => valeur), affiche :

Code :

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

1
2
3
4
5
<?php
echo '<pre>';
print_r($array_annotation);
echo '</pre>';
?>

**mac-addict** · 22/08/2011, 14h14

si je fais cela :

Code :

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

1
2
3
4
5
<?php
echo '<pre>';
print_r($array_annotation);
echo '</pre>';
?>

j'obtiens ceci :

Array
(
[nmr; ?>] => test4
)

et si je fais ceci :

Code :

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

1
2
3
4
5
<?php
echo '<pre>';
print_r($array_modif);
echo '</pre>';
?>

j'obtiens ceci :

Array
(
[nmr; ?>] => oui
)

oui correspondant bien a la valeur que je veux mettre si le bouton radio est coché et test4 étant la valeur mise dans le champ texte.

Donc désolé de poster à nouveau mais ou est le souci ?

Invité · 22/08/2011, 14h30

Code :

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

1
2
3
4
Array
(
[nmr; ?>] => oui
)

Tu ne vois pas comme un défaut ?
nmr; ?> -> on devrait avoir l'indice (un nombre entier)
Manifestement, une erreur dans le code ... au niveau du formulaire

Peux-tu remontrer le bout de code (tes 2 input) ?

Ca devrait ressembler à ca :

Code :

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

1
2
		<td><input type="radio" name="modif[<?php echo $ligne->nmr; ?>]" value="<?php echo $ligne->nmr; ?>" /></td>
		<td><input type="text" name="annotation[<?php echo $ligne->nmr; ?>]" /></td>

**mac-addict** · 22/08/2011, 14h41

j'ai changé et j'ai mis ça à la place dans mon input :

Code :

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

<td><input type="radio" name="modif[<?php echo '.$ligne->nmr.'; ?>]" value="oui" /></td>

résultat j'obtiens ceci :

Array
(
[] => oui
)

Invité · 22/08/2011, 14h54

C'est sûr qu'en passant le code au shaker ... ca marche moins bien.

Il te suffit de copier-coller ceci (sans rien changer !) :

Code :

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

1
2
		<td><input type="radio" name="modif[<?php echo $ligne->nmr; ?>]" value="oui" /></td>
		<td><input type="text" name="annotation[<?php echo $ligne->nmr; ?>]" /></td>

Astuce : clique sur "sélectionner tout", ci-dessus (en haut à droite du code)

------------------------------^

**mac-addict** · 22/08/2011, 15h35

j'ai rigoureusement copié collé vos 2 lignes et quand je teste le résultat est toujours le même à savoir :

Array
(
[nmr; ?>] => oui
)

Invité · 22/08/2011, 15h39

Si tu ne nous mets pas ton code, on ne pourra pas te dire d'où vient l'erreur ...

**mac-addict** · 22/08/2011, 15h44

donc sur la première page j'ai 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
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
<form name="list" METHOD="POST" ACTION="admin6.php">
<table id="hovertable">
<?php
 
echo '<tr>
<th>table:</th>
<th>personnel:</th>
<th>service:</th>
<th>pour la periode du:</th>
<th>au:</th>
<th>motif:</th>
<th>type:</th>
<th>date:</th>
<th>effectue:</th>
<th>à modifier:</th>
<th>annotation:</th>
</tr>';
while($ligne = mysql_fetch_object($res))
 
	{	
	 echo '<tr onmouseover="this.style.backgroundColor="#ffff66";" onmouseout="this.style.backgroundColor="#d4e3e5";">
	 <td>'.$ligne->nom_table.'</td>
	 <td>'.$ligne->nom.'</td>
	 <td>'.$ligne->service.'</td>
	 <td>'.$ligne->date_debut.'</td>
	 <td>'.$ligne->date_fin.'</td>
	 <td>'.$ligne->motif.'</td>
	 <td>'.$ligne->type.'</td>
	 <td>'.$ligne->date.'</td>
	 <td>'.$ligne->effectue.'</td>
	 <td><input type="radio" name="modif[<?php echo $ligne->nmr; ?>]" value="oui" /></td>
	 <td><input type="text" name="annotation[<?php echo $ligne->nmr; ?>]" /></td>
	 </tr>';
	}
 
?>
</table>
<div><input type=submit name=envoyer value=envoyer></div>
<br>
</form>

et sur la deuxième j'ai 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
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
<table id="hovertable">
<?php
$array_modif = $_POST['modif']; 
$array_annotation = $_POST['annotation']; 
$oui="oui";
 
foreach($array_modif as $key => $val) 
{
$sql1 = "update fusion set effectue = = '".mysql_real_escape_string($val)."' WHERE nmr = '".intval($key)."';";
$res1= mysql_query($sql1);
}
foreach($array_annotation as $key => $val) 
{
 
	$sql2 = "UPDATE fusion SET annotation = '".mysql_real_escape_string($val)."' WHERE nmr = '".intval($key)."';";
	$res2 = mysql_query($sql2);
}
 
$sql="select * from fusion WHERE effectue like 'oui' order by nmr desc";
$res= mysql_query($sql);
 
echo '<tr>
<th>table:</th>
<th>personnel:</th>
<th>service:</th>
<th>pour la periode du:</th>
<th>au:</th>
<th>motif:</th>
<th>type:</th>
<th>date:</th>
<th>effectue:</th>
<th>annotation:</th>
</tr>';
 
while($ligne = mysql_fetch_object($res))
    {	
	 echo '<tr>
<td>'.$ligne->nom_table.'</td>
<td>'.$ligne->nom.'</td>
<td>'.$ligne->service.'</td>
<td>'.$ligne->date_debut.'</td>
<td>'.$ligne->date_fin.'</td>
<td>'.$ligne->motif.'</td>
<td>'.$ligne->type.'</td>
<td>'.$ligne->date.'</td>
<td>'.$ligne->effectue.'</td>
<td>'.$ligne->annotation.'</td>
</tr>';
	}
 
?>
</table>

Invité · 22/08/2011, 15h51

Regarde la couleur syntaxique des lignes 31 et 32 du 1er code ci-dessus :
ca devrait te mettre la puce à l'oreille !
On constate une différence dans les couleurs !

Le fait que tu as bien copié-collé les lignes (qui contiennent des echo)
... sauf que ...
tu les as copiées à l'intérieur d'un autre echo !

Code php :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
	 echo '<tr onmouseover="this.style.backgroundColor="#ffff66";" onmouseout="this.style.backgroundColor="#d4e3e5";">
	 <td>'.$ligne->nom_table.'</td>
	 <td>'.$ligne->nom.'</td>
	 <td>'.$ligne->service.'</td>
	 <td>'.$ligne->date_debut.'</td>
	 <td>'.$ligne->date_fin.'</td>
	 <td>'.$ligne->motif.'</td>
	 <td>'.$ligne->type.'</td>
	 <td>'.$ligne->date.'</td>
	 <td>'.$ligne->effectue.'</td>
	 <td><input type="radio" name="modif['.$ligne->nmr.']" value="oui" /></td>
	 <td><input type="text" name="annotation['.$ligne->nmr.']" /></td>
	 </tr>';

A moins que ton écran soit en noir & blanc, on voit nettement la différence, non ?

ps : je mettrais même (pour afficher les annotations déjà enregistrées) :

Code php :

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

1
2
3
4
5
	 echo '<tr 
....
	 <td><input type="text" name="annotation['.$ligne->nmr.']" value="'.$ligne->annotation.'" /></td>
....
	 </tr>';

**mac-addict** · 22/08/2011, 16h12

Un grand merci à toi jreaux62 , effectivement cela venait bien de là.

Une autre erreur bête de copier coller me bloquait également , ligne 9 dans LA 2ième partie j'avais :

Code :

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

$sql1 = "update fusion set effectue = = '".mysql_real_escape_string($val)."' WHERE nmr = '".intval($key)."';";

Au lieu de :

Code :

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

$sql1 = "update fusion set effectue = '".mysql_real_escape_string($val)."' WHERE nmr = '".intval($key)."';";

Un simple égal en trop , donc encore merci pour tout

problème récupération variable contenu dans un tableau

PHP & Base de données

Discussions similaires

Partager

Partager