par
micetf » 11 juin 2009, 13:00
Je pense que lorsque tu testes si $logo est vide,
en fait, ce que tu veux c'est savoir si
$row['logo'] existe,
pour alors l'afficher dans ta cellule.
Si tel est le cas,
pour y voir plus clair,
j'aurais plutôt procédé de cette manière.
A la place de ton if (empty($logo)) {} else {}, j'aurais écrit :
Code : Tout sélectionner
// Je prépare ma cellule
$cellule = '<td align="center"><span class="style1">'; // les balises
$cellule .= '<strong>'.$row['nom'].' </strong>'; // le nom
$cellule .= '<br/>';
if (!empty($row['logo'])) { // le logo s'il existe
$cellule .= '<img src="'.$row['logo'].'" width="100"/>';
$cellule .= '<br/>';
}
$cellule .= $row['presentation']; // la présentation
$cellule .= '<br/>';
$cellule .= '<a href="http://'.$row['photo'].'" target="_blank"><img src="http://'.$row['photo'].'" width="220" border="0"/></a>'; // la photo et son lien
$cellule .= '<br/>';
$cellule .= $row['suite']; // suite ?
$cellule .= '<br/>';
$cellule .= $row['prix']; // le prix
$cellule .= '<br/>';
$cellule .= $row['paypal']; // paypal
$cellule .= '</span></td>';
// et je l'affiche
echo $cellule;
Cela t'évite d'écrire deux fois la même chose,
une fois dans le
if, une autre dans le
else,
la seule différence étant l'affichage du logo.
Fred
Je pense que lorsque tu testes si $logo est vide,
en fait, ce que tu veux c'est savoir si [u]$row['logo'][/u] existe,
pour alors l'afficher dans ta cellule.
Si tel est le cas,
pour y voir plus clair,
j'aurais plutôt procédé de cette manière.
A la place de ton if (empty($logo)) {} else {}, j'aurais écrit :
[code]
// Je prépare ma cellule
$cellule = '<td align="center"><span class="style1">'; // les balises
$cellule .= '<strong>'.$row['nom'].' </strong>'; // le nom
$cellule .= '<br/>';
if (!empty($row['logo'])) { // le logo s'il existe
$cellule .= '<img src="'.$row['logo'].'" width="100"/>';
$cellule .= '<br/>';
}
$cellule .= $row['presentation']; // la présentation
$cellule .= '<br/>';
$cellule .= '<a href="http://'.$row['photo'].'" target="_blank"><img src="http://'.$row['photo'].'" width="220" border="0"/></a>'; // la photo et son lien
$cellule .= '<br/>';
$cellule .= $row['suite']; // suite ?
$cellule .= '<br/>';
$cellule .= $row['prix']; // le prix
$cellule .= '<br/>';
$cellule .= $row['paypal']; // paypal
$cellule .= '</span></td>';
// et je l'affiche
echo $cellule;
[/code]
Cela t'évite d'écrire deux fois la même chose,
une fois dans le [b]if[/b], une autre dans le [b]else[/b],
la seule différence étant l'affichage du logo.
Fred