Discussion :

[ETVigan]

Bonjour;
J'ai tour petit souci, je réecris un script et transforme une boucle for en foreach...

Voici le code original...

[CODE]
$tb .= "<tr".$this->getClassTag("tr").">\n";
// for($i=0;$i<count($this->tbdesigner);$i++) {
foreach ($this->data as $key => $value) {
$tb .= "<td".$this->getClassTag("td")."><span".$this->getClassTag("tbdesigner").">".$this->tbdesigner[$key]."</span></td>\n";
}
$tb .= "</tr>\n";
[CODE]

Mon souci est l'utilsation de => $value,$value qui n'est utilisé dans le code
Si je le supprime; je n'ai rien d'affiché....
J'ai pourtant relu la doc mais ne percute pas...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
 The foreach construct provides an easy way to iterate over arrays. foreach works only on arrays and objects, and will issue an error when you try to use it on a variable with a different data type or an uninitialized variable. There are two syntaxes:
 
foreach (array_expression as $value)
    statement
foreach (array_expression as $key => $value)
    statement
 
The first form loops over the array given by array_expression. On each iteration, the value of the current element is assigned to $value and the internal array pointer is advanced by one (so on the next iteration, you'll be looking at the next element).
 
The second form will additionally assign the current element's key to the $key variable on each iteration.

Ce n'est pas une question d'anglais mais... ne voit pas

Merci de m'éclairer...

[moogli]

salut,


plutôt

Code :

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

1
2
3
4
5
6
 
<?php
foreach ($this->tbdesigner as $key => $value) {
$tb .= "<td".$this->getClassTag("td")."><span".$this->getClassTag("tbdesigner").">".$value."</span></td>\n";
?>
}

$value représente ici la valeur de l’élément courant du tableau, et $key la clef

donc dans le for tu utilisais $this->tbdesigner[$key] pur avoir la valeur, dans le foreach $value


@+

[ETVigan]

Merci pour le temps pris et votre coup de main...
J'ai procédé à la modif, peut être mal compris mais cela ne donne pas le résultat souhaité.
Voici le code original et ces résultats...

Code :

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

1
2
3
4
5
6
7
8
			$tb .= "<tr".$this->getClassTag("tr").">\n";
//			for($i=0;$i<count($this->tbdesigner);$i++)	{
			foreach ($this->data as $key => $value) {
//				$tb .= "<td".$this->getClassTag("td")."><span".$this->getClassTag("tbdesigner").">".$this->tbdesigner[$key]."</span></td>\n";
				$tb .= "<td".$this->getClassTag("td")."><span".$this->getClassTag("tbdesigner").">".$this->tbdesigner[$key]."</span></td>\n";
			}
			$tb .= "</tr>\n";

qui donne à l'écran, le bold est la ligne concernée !

Je précise qu'il s'agit bien des résultats attendus

Code :

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

1
2
3
4
5
6
7
8
9
10
11
Array
Array
Array
Array
Click Me to toggle
Logo 	Col 1 	Col 2 	Col 3
NEW 	Row1 Col1 	Row1 Col2 	Row1 Col3
NEW 	Row2 Col1 	Row2 Col2 	Row2 Col3
NEW 	Row3 Col1 	Row3 Col2 	Row3 Col3
NEW 	This Software is 	under GPL 3 License. 	Daniele Monti - monska13 [at] gmail.com

La même ligne avec $this->tbdesigner[$key] modifié par $value donne à l'écran...

Code :

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

1
2
3
4
5
6
7
8
9
10
11
Array
Array
Array
Array
Click Me to toggle
Array 	Array 	Array 	Array
NEW 	Row1 Col1 	Row1 Col2 	Row1 Col3
NEW 	Row2 Col1 	Row2 Col2 	Row2 Col3
NEW 	Row3 Col1 	Row3 Col2 	Row3 Col3
NEW 	This Software is 	under GPL 3 License. 	Daniele Monti - monska13 [at] gmail.com

Conscient qu'il est difficile de plonger ainsi dans un code...
Je donne la source complète en pièce attachée avec autre souci en attente mais cela fera l'affaire d'un autre post..

Je vous remercie pour tout conseil !

--
Dernière minute et une correction supplémentaire qui fait que cela fonctionne...

Voici

Code :

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

1
2
3
4
5
6
7
//			for($i=0;$i<count($this->tbdesigner);$i++)	{
			foreach ($this->tbdesigner as $key => $value) {
//				$tb .= "<td".$this->getClassTag("td")."><span".$this->getClassTag("tbdesigner").">".$this->tbdesigner[$key]."</span></td>\n";
				$tb .= "<td".$this->getClassTag("td")."><span".$this->getClassTag("tbdesigner").">".$value."</span></td>\n";
			}
			$tb .= "</tr>\n";

[rawsrc]

Dernière minute et une correction supplémentaire qui fait que cela fonctionne...

Donc post résolu, non ?

[moogli]

Dernière minute et une correction supplémentaire qui fait que cela fonctionne...

Voici

Code :

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

1
2
3
4
5
6
7
//			for($i=0;$i<count($this->tbdesigner);$i++)	{
			foreach ($this->tbdesigner as $key => $value) {
//				$tb .= "<td".$this->getClassTag("td")."><span".$this->getClassTag("tbdesigner").">".$this->tbdesigner[$key]."</span></td>\n";
				$tb .= "<td".$this->getClassTag("td")."><span".$this->getClassTag("tbdesigner").">".$value."</span></td>\n";
			}
			$tb .= "</tr>\n";

ça me rappel mon code

le premier paramètre du foreach est le tableau que l'on souhaite parcourir, dans les 1er exemple on ne sais d’où sort data

@+

[rawsrc]

ça me rappel mon code

le premier paramètre du foreach est le tableau que l'on souhaite parcourir, dans les 1er exemple on ne sais d’où sort data

@+

Faire un helper pour la génération de table à partir d'un tableau (ou un objet Iterable) c'est une bonne idée, mais là le code est trop brouillon et je dirais même sale.