Soyez le premier à donner votre avis sur cette source.
Snippet vu 7 901 fois - Téléchargée 15 fois
function ScanDirectory($Directory, $fileTypes = null){ $MyDirectory = opendir($Directory) or die('Erreur'); while(false !== ($Entry = @readdir($MyDirectory))) { if(is_dir($Directory.'/'.$Entry)&& $Entry != '.' && $Entry != '..') { $tmp = ScanDirectory($Directory.'/'.$Entry, $fileTypes); $files = array_merge((array)$files, (array)$tmp); } else { // si le tableau est renseigné if (is_array($fileTypes)) { // on recupère l'extension du fichier $ext = get_file_extension($Entry); // on vérifie qu'elle fait partie de celles autorisée if (in_array ($ext, $fileTypes)) { $files[$Entry] = $Directory.'/'; } // sinon, on mets toutes les occurences } else { $files[$Entry] = $Directory.'/'; } /* note : on stocke les valeurs dans un tableau associatif :
Cordialement.
Mais à ma décharge, l'exaspération (entraînée par la lassitude) l'emporte souvent sur le sang-froid.
L'erreur est corrigée, je laisse le partage dans le cadre où, si un débutant cherche avec ces mots clés, il puisse découvrir ce code (ainsi que ceux en rapport) et, s'il le souhaite, évoluer, se faire sa propre opinion voire même modifier cette source !
while($Entry = @readdir($MyDirectory)) {
n'est pas bonne. Et je ne vais pas m'étendre sur le sujet, parce que je l'ai déjà fait des dizaines de fois et je suis lassé de voir toujours la même chose, mal faite.
Et quand je dis que ta source n'apporte rien de nouveau, je ne me base pas sur ce que tu as écrit, mais sur le fait que, simplement, elle n'apporte rien de nouveau.
On va pas refaire un débat sur "faut-il partager ou non", parce qu'on tournerait en rond. Mais ce que j'ai écrit voulait quand même dire que cette source, n'apportant rien de nouveau, ne mérite pas nécessairement d'être publiée (indépendamment de sa qualité, dont je ne parle pas du tout). C'est pas une critique en soi, juste un constat, toujours le même : phpcs est jonché de sources qui n'apportent rien et qui sont souvent bourrées d'erreurs, de failles, etc. Et c'est dommage, parce qu'au final, ce sont les débutants qui morflent en n'apprenant pas à coder correctement.
Bref, désolé pour la digression.
En ce qui concerne la fonction readdir, tu parles surement du fait que je ne me sers pas de ce qu'elle retourne, mais plutôt que je lui dise "tant que tu lui affecte une valeur, on continue".
En attendant, si tu avais eu la gentillesse d'expliquer en quoi cette source est si mauvaise, je te demanderai la prochaine fois d'apporter un commentaire un poil plus constructif car ce genre d'intervention aussi est très lassante...
Vous n'êtes pas encore membre ?
inscrivez-vous, c'est gratuit et ça prend moins d'une minute !
Les membres obtiennent plus de réponses que les utilisateurs anonymes.
Le fait d'être membre vous permet d'avoir un suivi détaillé de vos demandes et codes sources.
Le fait d'être membre vous permet d'avoir des options supplémentaires.