par
albat » 20 mars 2012, 14:19
Déjà, je te recommande de clarifier/simplifier ton code.
- Tu peux avantageusement utiliser la fonction extract() pour récupérer tes valeurs avec les noms des champs.
- Lorsque tu affiches qq chose via la fonction echo(), veille à extraire les variables PHP.
- Les éléments statiques à afficher seront préférablement entre apostrophes (') plutôt qu'entre guillemets (").
- C'est plus un détail, mais remplacer les points de concaténation par des virgules de juxtaposition peut être une bonne pratique.
// Boucle de recherche des inscrits
while ($data = mysql_fetch_assoc($result))
{ extract($data);
// Si Moment 1 = Soir
if ($moment == "1")
$moment = "Soir";
echo '<p>',$prenom,' ',$nom,' - ',$prenom2,' ',$nom2,' -';
// Si Moment du Soir = affiché
if ($data['moment'] == "1")
echo ' ',$moment,' -';
// Si Notes = affiché
if (!empty($notes))
echo ' Notes : ',$notes,' -';
// Si contact = affiché
if (!empty($contact))
echo ' Contact : ',$contact;
echo '</p>';
}
// On ferme la connexion MySQL
Déjà, je te recommande de clarifier/simplifier ton code.
- Tu peux avantageusement utiliser la fonction extract() pour récupérer tes valeurs avec les noms des champs.
- Lorsque tu affiches qq chose via la fonction echo(), veille à extraire les variables PHP.
- Les éléments statiques à afficher seront préférablement entre apostrophes (') plutôt qu'entre guillemets (").
- C'est plus un détail, mais remplacer les points de concaténation par des virgules de juxtaposition peut être une bonne pratique.
[php]
// Boucle de recherche des inscrits
while ($data = mysql_fetch_assoc($result))
{ extract($data);
// Si Moment 1 = Soir
if ($moment == "1")
$moment = "Soir";
echo '<p>',$prenom,' ',$nom,' - ',$prenom2,' ',$nom2,' -';
// Si Moment du Soir = affiché
if ($data['moment'] == "1")
echo ' ',$moment,' -';
// Si Notes = affiché
if (!empty($notes))
echo ' Notes : ',$notes,' -';
// Si contact = affiché
if (!empty($contact))
echo ' Contact : ',$contact;
echo '</p>';
}
// On ferme la connexion MySQL[/php]