Difficulté de concaténation ou mauvaise définition d'array

Version imprimable

Voir 40 message(s) de cette discussion en une page

Bonjour à toutes et tous, et, par avance, merci de vos lumières.
Voici la difficulté rencontrée:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 if ($_POST['Button'] == 'Enregistrer'){ if( !isset( $_POST[''.$compte.'n\''] ) ) $_POST[''.$compte.'n\''] = ''; if( !isset( $_POST[''.$compte.'n1\''] ) ) $_POST[''.$compte.'n1\''] = ''; if( !isset( $_POST[''.$compte.'n2\''] ) ) $_POST[''.$compte.'n2\''] = ''; $montant = array($_POST[''.$compte.'n\''] , $_POST[''.$compte.'n1\''] , $_POST[''.$compte.'n2\'']); $codeHtml = '<tr><td class="libelle">'.ucfirst($compte).'</td> <td class="montant">'. $montant[0].'</td> <td class="montant">'. $montant[1].'</td> <td class="montant">'. $montant[2].'</td> </tr>'; // '<pre>'.print_r($montant).'</pre>'; '<pre>'.print_r($_POST).'</pre>'; }
Le résultat de print_r($montant) est vide, mais celui de print_r($_POST) est :
Array ( [compten] => 1000 [compten1] => 2000 [compten2] => 3000 [Button] => Enregistrer [compten'] => [compten1'] => [compten2'] => )
Évidemment ce que je souhaite obtenir, c'est $montant[0] = 1000 etc. donc comment faire ? et deuzio, j'ai un tableau vide après "enregistrer" d’où cela peut il provenir ?
Encore merci à la bonne âme...

Quel est ce bazar d'apostrophes et d'antislash ?
Il faut faire siiiiimple

Code:

1
2
3
4
5
6
7
8
9
10
11
12
if ($_POST['Button'] == 'Enregistrer'){
	$montant[0] =  isset($_POST['compten']) ? $_POST['compten'] : '';
        $montant[1] =  isset($_POST['compten1']) ? $_POST['compten1'] : '';
        $montant[2] =  isset($_POST['compten2']) ? $_POST['compten2'] : '';
 
	$codeHtml = '<tr>
                <td class="libelle">'.ucfirst($compte).'</td>
		<td class="montant">'. $montant[0].'</td>
		<td class="montant">'. $montant[1].'</td>
		<td class="montant">'. $montant[2].'</td>
		</tr>';
}

26/05/2015, 15h37
jmtrivia
Oupps... Dans la précipitation j'ai oublié ça au début du code d’où les apostrophes et antislash.
Code:

1 2 function record($compte) {
En tout cas merci, tu m'a donné des idées...

Je ne sais pas si ce code est correct ? mais il fonctionne !
Merci encore à toi sabotage.

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
	function record($compte) 
	{	
	if( !isset( $codeHtml ) ) $codeHtml = '';
	if( !isset( $_POST['Button'] ) ) $_POST['Button'] = '';
 
		if ($_POST['Button'] == 'Enregistrer') 
		{
			$montant[0] =  isset($_POST[''.$compte.'n']) ? $_POST[''.$compte.'n'] : '';
        	$montant[1] =  isset($_POST[''.$compte.'n1']) ? $_POST[''.$compte.'n1'] : '';
        	$montant[2] =  isset($_POST[''.$compte.'n2']) ? $_POST[''.$compte.'n2'] : '';	
			$codeHtml = '<tr><td class="libelle">'.ucfirst($compte).'</td>
							<td class="montant">'. $montant[0].'</td>
							<td class="montant">'. $montant[1].'</td>
							<td class="montant">'. $montant[2].'</td>
							</tr>';
 
			 '<pre>montant'.print_r($_POST).'</pre>';		
		}
}

Même avec une partie variable, il n'y a pas besoin de tout ça :

Code:

1
2
3
4
5
6
function record($compte) {
  if ($_POST['Button'] == 'Enregistrer'){
        $compte .= 'n';
	$montant[] =  isset($_POST[$compte]) ? $_POST[$compte] : '';
        $montant[] =  isset($_POST[$compte.'1']) ? $_POST[$compte.'1'] : '';
        $montant[] =  isset($_POST[$compte.'2']) ? $_POST[$compte.'2'] : '';

Citation:
Envoyé par sabotage
Même avec une partie variable, il n'y a pas besoin de tout ça :
Code:

1 2 3 4 5 6 function record($compte) { if ($_POST['Button'] == 'Enregistrer'){ $compte .= 'n'; $montant[] = isset($_POST[$compte]) ? $_POST[$compte] : ''; $montant[] = isset($_POST[$compte.'1']) ? $_POST[$compte.'1'] : ''; $montant[] = isset($_POST[$compte.'2']) ? $_POST[$compte.'2'] : '';
Effectivement, on a gagné en lisibilité:D
Par contre lorsque j'appelle la fonction plusieurs fois, j'ai un tableau vide dès le 2e appel à la fonction (mais des chiffres exacts pour le 1er )

26/05/2015, 18h45
sabotage

Comment est ton formulaire exactement ?

Citation:

Envoyé par sabotage

Comment est ton formulaire exactement ?

Comme ça :)

Code:

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
 
<?php
/* ===== FRAIS GENERAUX ==================== 

1/ Connexion à la base
2/ On récupère l'ID du bien

*/
	function record($compte) 
	{	
	if( !isset( $codeHtml ) ) $codeHtml = '';
	if( !isset( $_POST['Button'] ) ) $_POST['Button'] = '';
 
		if ($_POST['Button'] == 'Enregistrer') 
		{
			$montant[] =  isset($_POST[$compte.'n']) ? $_POST[$compte.'n'] : '';
        	$montant[] =  isset($_POST[$compte.'n1']) ? $_POST[$compte.'n1'] : '';
        	$montant[] =  isset($_POST[$compte.'n2']) ? $_POST[$compte.'n2'] : '';
			$codeHtml = '<tr><td class="libelle">'.ucfirst($compte).'</td>
							<td class="montant">'. $montant[0].'</td>
							<td class="montant">'. $montant[1].'</td>
							<td class="montant">'. $montant[2].'</td>
							</tr>';
			'<pre>'.print_r($montant).'</pre>';				
		}
 
		elseif ($_POST['Button'] == 'Modifier')
		{	
 
		}	
 
		else // (empty($_POST['Button']))
		{			
			$codeHtml= '<tr>
							<td class="libelle"><label for="'.str_replace(' ','', $compte).'">'.ucfirst($compte).'</label></td>
							<td class="montant"><input type="text" class="" id="" size="6" name="'.str_replace(' ','',lcfirst($compte)).'n" /></td>
							<td class="montant"><input type="text" class="" id="" size="6" name="'.str_replace(' ','',lcfirst($compte)).'n1" /></td>
							<td class="montant"><input type="text" class="" id="" size="6" name="'.str_replace(' ','',lcfirst($compte)).'n2" /></td>
							</tr>';					
		}
		return $codeHtml;
	}	
?>	
		<div class="col-sm-8 blog-main">
<?php
if( !isset( $_POST['Button'] ) ) {
?>				
			<form id="" name="frmFraisgeneraux" method="post" action="index.php?page=4">
<?php 
}
?>				
			<table class="table-width" cellspacing='0'>
<?php
			echo '<tr><td colspan="4">ACHATS</td></tr>';
			echo record('fournitures');
			echo record('autres approvisionnements');
			echo record('achat de materiel');
			echo record('achat non stockes');
			/*
			echo '<tr><td colspan="4">AUTRES CHARGES EXTERNES</td></tr>';
			echo record('sous traitance');
			echo record('credit bail');
			echo record('location immobilieres');
			echo record('charges locatives');
			echo record('entretien');
			echo record('assurances');
			echo record('divers');
			echo '<tr><td colspan="4">AUTRES SERVICES EXTERIEURS</td></tr>';
			echo record('sous traitance');
			*/
 
?>				
				<tr>
					<td colspan="4" style="text-align:center;">
					<input type="submit" class="btn btn-primary" name="Button" value="Enregistrer">
					<input type="submit" class="btn btn-primary" name="Button" value="Modifier">
					</td>
				</tr>
<?php //} ?>				
				<thead>
					<th>Compte</th>
					<th class="entete">Année N</th>
					<th class="entete">Année N+1(P)</th>
					<th class="entete">Année N+2(P)</th>
				</thead>			
			</table>			
<?php
if( !isset( $_POST['Button'] ) ) { echo '</form>'; }
?>
		</div><!-- /.blog-main --> 
	</div><!-- /.row --> 
</div><!-- /.container -->

Il faut que tu fasses un effort sur la présentation car ton code n'est pas aligné et donc difficile à lire.

Je te propose les modifications suivantes :
- fournir un nom pour le libellé et un nom pour l'input, cela permet d'être libre sur le libellé (notamment de pouvoir mettre les accents ou n'importe quel symbole sans contrainte)
- j'ai retiré le "label for" en effet cette balise sert à désigner l'id d'un champs de formulaire unique, ce qui n'est pas ton cas
- utiliser des boucles avec une notation en tableau pour les champs et une boucle pour la recuperation

Code:

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
<?php	
	function record($compte) 	{
 
	$codeHtml = '';
	$montant = array();
	$ninput = 3;
 
	$label = htmlspecialchars($compte['label']);
	$input = $compte['input'] . '[]';
 
	if (!isset($_POST['Button'])) {
            $codeHtml = "<tr>\r\n";
            $codeHtml .= '<td class="libelle">'.$label.'</label></td>';
            for ($i = 0; $i <= $ninput, $i++) {
                $codeHtml .= '<td class="montant"><input type="text" size="6" name="'.$input . '" /></td>';
            }
            $codeHtml .= "</tr>\r\n";					
	}
      elseif ($_POST['Button'] == 'Enregistrer') 	{
        $codeHtml = "<tr>\r\n";
        $codeHtml .= '<td class="libelle">'. $label .'</td>';
        for ($i = 0; $i <= $ninput, $i++) {
            $montant = isset($_POST[$input][$i]) ? $_POST[$input][$i] : '';
            $codeHtml .= '<td class="montant">'. htmlspecialchars($montant).'</td>';
        }
        $codeHtml .= "</tr>\r\n";
  }
	return $codeHtml;
}
 
echo record(array('label'=>'fournitures', 'input'=>'fournitures'));
echo record(array('label'=>'autres approvisionnements', 'input'=>'autres_appro'));
echo record(array('label'=>'achat de matériel', 'input'=>'achat_materiel'));
echo record(array('label'=>'achats non stockés', 'input'=>'achat_nonstockes'));

27/05/2015, 14h10
jmtrivia

Bonjour Sabotage,

Bon, alors là, il n'y a plus aucun affichage.
J'ai juste modifié for ($i = 0; $i <= $ninput ; $i++) en for ($i = 0; $i < $ninput ; $i++) puisque je n'affiche que 3 années.
Quand je regarde ce que je poste, je remplis bien le(s) array(s).
Par contre, sur la page ou je suis sensé afficher les résultats de ma saisie, la zone $_POST[$input][$i] est vide. Pourtant en faisant un print_r($_POST)j'ai
Array ( [fournitures] => Array ( [0] => 1000 [1] => 2000 [2] => 3000 ) par exemple.
Je dois te dire que j'ai du mal à comprendre...
27/05/2015, 14h38
sabotage

Quand tu dis "plus aucun affichage" c'est une page blanche ?
27/05/2015, 14h46
jmtrivia

Non un tableau vide.

Ah oui pardon j'ai bougé les [] :

Code:

1
2
3
4
5
6
7
8
9
$input = $compte['input'];
 
	if (!isset($_POST['Button'])) {
            $codeHtml = "<tr>\r\n";
            $codeHtml .= '<td class="libelle">'.$label.'</label></td>';
            for ($i = 0; $i <= $ninput, $i++) {
                $codeHtml .= '<td class="montant"><input type="text" size="6" name="'.$input . '[]" /></td>';
            }
            $codeHtml .= "</tr>\r\n";

27/05/2015, 18h18
jmtrivia

ah non ! maintenant on a un array dans l'array :)
Array ( [fournitures] => Array ( [0] => 1000 [1] => 1000 [2] => 1000 )
27/05/2015, 18h22
sabotage

ba oui, c'est ça qu'on veut.
27/05/2015, 18h29
jmtrivia

Oui, mais la valeur ne s'affiche toujours pas ...
Bon, j'ai corrigé ça : for ($i = 0; $i < $ninput ; $i++) {
27/05/2015, 18h34
sabotage

Tu peux remontrer le code ?

Voili

Code:

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
	function record($compte) 	{
 
	$codeHtml = '';
	$montant = array();
	$ninput = 3;
 	$label = ucfirst(htmlspecialchars($compte['label']));
	$input = $compte['input'].'[]';
 
	if (!isset($_POST['Button'])) 
	{
		$codeHtml = "<tr>\r\n";
      $codeHtml .= '<td class="libelle">'.$label.'</label></td>';
      for ($i = 0; $i < $ninput ; $i++) {
          $codeHtml .= '<td class="montant"><input type="text" size="6" name="'.$input.'" /></td>';
      }
      $codeHtml .= "</tr>\r\n";					
	}
   elseif ($_POST['Button'] == 'Enregistrer')
	{
     $codeHtml = "<tr>\r\n";
     $codeHtml .= '<td class="libelle">'. $label .'</td>';
     for ($i = 0; $i < $ninput ; $i++) {
         $montant = isset($_POST[$input][$i]) ? $_POST[$input][$i] : '';
         $codeHtml .= '<td class="montant">'. htmlspecialchars ($montant).'</td>';
		}
  	}
	else // ($_POST['Button'] == 'Modifier')
	{
	}
	return $codeHtml;
}

27/05/2015, 18h51
jmtrivia
Si je comprends bien dans:
Code:

1 2 3 $montant = isset($_POST[$input][$i]) ? $_POST[$input][$i] : ''; $codeHtml .= '<td class="montant">'. htmlspecialchars ($montant).'</td>';
puisqu'on reçoit :
Array ( [fournitures] => Array ( [0] => 1000 [1] => 1000 [2] => 1000 )
$montant vaut [fournitures], non ?
27/05/2015, 21h58
sabotage

Tu n'as pas appliqué la correction que je viens de te donner.

Voir 40 message(s) de cette discussion en une page