IMAGE ALEATOIRE AVEC DESCRIPTION DANS BALISE "ALT" ET AU DESSUS DE L'IMAGE

Commentaire de massacr le 06/08/2005 10:59:31

Salut
"grace a "else" et "if" que tout le monde connait a mon avis." Ha, non, je ne connais pas. C'est quoi ? lol
<code>
# if(file_exists(config.php)){
# include('config.php');
# }
# else
# {
# echo 'ce module n\'a pas encore t install, veuillez l\'installer a la page suvante: <a href="install.php">cliquez ici</a>';
# $copy = '&copy; 2005 -<a href=mailto:morgan@vagao.com>&nbsp;morgan abraham</a>-';
# }
</code>
Je me demande si ca vaut vraiment le coup de mettre, ca, étant donné que le fichier install.php n'existe pas, et qu'on ne voit pas pourquoi quelqu'un aurait supprimé le fichier config.php. De plus, pour trois variable, autant les mettre dans le fichier index.php. Enfin...
<code>
# ?>
# <?php
</code>
???????????????? Pourquoi fermer la syntaxe PHP si c'est pour la rouvrir après ?
<code>
# // On vrifie les arguments ...
# $arg_list = func_get_args();
# if (count($arg_list) < 2) return false;
</code>
Dis donc, t'es méticuleux. Tu vérifie tout. Mais étant donné que c'est toi qui appelle la fonction, pourquoi y aurait-il une erreur dans les arguments ? En régle générale, on vérifie plein de choses quand l'utilisateur peut choisir des choses, mettre un texte, etc... Mais quand c'est le programmeur, et que le programme marche bien, allège ta source !!
<code>
if (preg_match($ext_value, $file_in)) {
</code>
Testes plutot le type MYME. Là, on peut faire passer autant les .bmp (en les renommant), que les .jpg.exe. Penses y quand tu feras l'ajout automatique d'images.
<code>
#  while ($file_in = readdir($dir_in)) {
</code>
La vrai syntaxe est
<code>
while (false !== ($file_in = @readdir($dir_in))) {
</code>
<code>
echo '<br>';
</code>
Autant se mettre au XHTML :
<code>
echo '<br />'';
</code>

Pour ton premier code, c'est de la très bonne programmation. Tu respecte la syntaxe PHP, etc... Mais je trouve que tu compliques beaucoup par rapport à l'utilité de ton code. Quand tu appelle une seule fois une fonction (get_file), autant ne pas en faire une fonction, et la mettre directement dans le code.
Par contre, se servir d'un fichier texte par photo, là, il faut compliquer un peu, pour que la gestion de 5000 photos ne relèvent pas du cauchemar. Donc, sois un seul fichier texte avec la correspondance nom_fichier => description, soit une base (code trop simple pour l'utilisation d'une base), soit la description dans le fichier image. Et oui, c'est possible.
Bon voilà. Je te met 7/10, et bravo pour ton premier code. Mais améliores le !!