par
Cyrano » 06 févr. 2007, 15:37
Un seul avis : aère ton code, il est un peu illisible, comme ceci par exemple :
<?php
$xml = '<?xml version="1.0" encoding="iso-8859-1"?>'//entête
.'<playlist>';//première balise avant boucle
$rep = "zik";//répertoire des morceaux à lister
/* si fichier est un répertoire */
if (is_dir($rep))
{
/* ouverture du fichier */
if ($dossier = opendir($rep))
{
/* début de boucle */
while (($fichier = readdir($dossier)) !== false)
{
/* on bannit le . et le .. */
if ($fichier !="." && $fichier !="..")
{
$xml .= "<song>";//balise song
$xml .= $fichier;//chanson
$xml .= "</song>";//fermeture balise song
}//if qui bannit
}//while fermé
closedir($dossier);//fermeture du répertoire listé
}//second if
}//premier if
/* fermeture balise */
$xml .= "</playlist>";
$fp = fopen("playlist.xml", 'w+'); //fichier déjà existant
fputs($fp, $xml); //on inscrit la totale
fclose($fp); //on ferme le fichier playlist.xml
echo 'Export XML effectué !<br><a href="playlist.xml">Voir le fichier</a>';//aperçu
?>
N'oublie jamais un détail : en ce moment, le code est clair dans ta tête, tu retrouves rapidement le bon endroit si un problème survient. Mais dans six mois ou un an ou deux ans, tu voudras apporter une amélioration ou une modification, tu vas perdre un temps considérable à relire ton propre code d'abord pour le comprendre. Les commentaires, c'est très bien, mais du code bien indenté et aéré, c'est encore mieux

Un seul avis : aère ton code, il est un peu illisible, comme ceci par exemple :
[php]<?php
$xml = '<?xml version="1.0" encoding="iso-8859-1"?>'//entête
.'<playlist>';//première balise avant boucle
$rep = "zik";//répertoire des morceaux à lister
/* si fichier est un répertoire */
if (is_dir($rep))
{
/* ouverture du fichier */
if ($dossier = opendir($rep))
{
/* début de boucle */
while (($fichier = readdir($dossier)) !== false)
{
/* on bannit le . et le .. */
if ($fichier !="." && $fichier !="..")
{
$xml .= "<song>";//balise song
$xml .= $fichier;//chanson
$xml .= "</song>";//fermeture balise song
}//if qui bannit
}//while fermé
closedir($dossier);//fermeture du répertoire listé
}//second if
}//premier if
/* fermeture balise */
$xml .= "</playlist>";
$fp = fopen("playlist.xml", 'w+'); //fichier déjà existant
fputs($fp, $xml); //on inscrit la totale
fclose($fp); //on ferme le fichier playlist.xml
echo 'Export XML effectué !<br><a href="playlist.xml">Voir le fichier</a>';//aperçu
?> [/php]
N'oublie jamais un détail : en ce moment, le code est clair dans ta tête, tu retrouves rapidement le bon endroit si un problème survient. Mais dans six mois ou un an ou deux ans, tu voudras apporter une amélioration ou une modification, tu vas perdre un temps considérable à relire ton propre code d'abord pour le comprendre. Les commentaires, c'est très bien, mais du code bien indenté et aéré, c'est encore mieux ;)