EFFACER LES FICHIERS IMAGES ( .PNG ), D'UN RÉPERTOIRE TEMPS VIEUX DE PLUS D'UN CERTAIN TEMPS (À DÉFINIR) DE FAÇON RÉCURENTE !

Commentaire de neigedhiver le 28/12/2009 13:03:02

Salut,

Ca devient lassant les codes de ce genre qui en plus de ne pas innover ne sont même pas écrits correctement...
Je cite la doc (http://fr.php.net/manual/fr/class.dir.php) :
"Notez la façon dont la valeur de retour de dir::read()  est vérifiée dans l'exemple suivant. Nous testons si la valeur est identique (égale et de même type que -- voyez opérateurs de comparaison  pour plus de détails) FALSE sinon, toute entrée dans le nom serait évalué à FALSE causera l'arrêt de la boucle (exemple, un répertoire nommé 0)."

Désolé, c'est pas après toi personnellement, mais cette erreur est tellement récurente que vraiment, vraiment, c'est pénible, fatiguant, épuisant, lassant.

Et comme j'en ai aussi marre qu'on m'engueule parce que je dis du mal des sources que j'aime pas, je te remercie d'avoir partagé ce morceau de code avec la communauté de phpcs, c'est une excellente initiative.
Je t'invite cependant à corriger cette petite erreur qui peut éventuellement causer des erreurs inattendues (bien qu'on puisse s'y attendre...).
Je t'invite également à jeter un oeil à la SPL, plus particulièrement RecursiveDirectoryIterator, mais également FilterIterator ou encore ma source XDir (en toute modestie, encore qu'il doit bien trainer une ou deux personnes qui ne seraient pas de cet avis) et qui permet de faire la même chose en 4 lignes.

Je m'excuse encore, j'ai l'air désabusé, c'est probablement parce que je suis un peu blasé.

Commentaire de neigedhiver le 28/12/2009 19:58:52

Ouaip... J'ai été rude. On règle ça en privé par mp, voilà voilà...

Sinon la pêche, pas pour moi. Mais les bières, pas de soucis... Combien de litres je prévois ?

Commentaire de Benjamin37 le 28/12/2009 21:05:10

Bonsoir !

Ils sont où les fichiers ?

Si c'est pour mettre un petit bout de code à quoi cela sert de poster cela pour régler ses comptes :(

Bonne soirée

Commentaire de neigedhiver le 30/12/2009 00:59:17

[HORS SUJET]
La Goudale, pas de problème... Ici, on a la Félis, brassée à Felletin avec de l'eau de source (non pasteurisée, donc fermentation encore en bouteille) et la 1000 (parce que plateau de millevaches).
Quelques litres, l'autre, hé... Pourquoi si peu ?
[/HORS SUJET]

Ah oui, j'avais été tellement obnubilé par mon problème avec readdir() que j'avais pas fait gaffe à ça.
Le mieux pour vérifier qu'un fichier se termine bien pas png est encore de vérifier avec substr :
if (substr($file, -4) == '.png' && .....)

Commentaire de neigedhiver le 30/12/2009 03:54:45

Une explication sur quoi ? Moi je veux bien expliquer, mais je veux bien savoir quoi ! :)
J'ai tendance à dire que les deux méthodes se valent. Je ne sais pas si l'une est plus rapide que l'autre, il faudrait bencher, mais pour la différence que ça représente, je suis certain, sans avoir besoin de vérifier, que c'est complètement négligeable (bon pour traiter 3 millions de fichiers, ça peut commencer à prendre de l'importance, mais bon, de l'ordre du dixième de seconde, alors...)

Bref. J'ai tendance à utiliser pathinfo pour des fichiers avec chemin complet (puisque cette fonction est capable de séparer le répertoire, le nom du fichier et l'extension). Mais bon, c'est une habitude, pas une vérité absolue.

Commentaire de kohntark le 02/01/2010 15:50:54

T'es vraiment archi nul Christophe0511, ta fonction est vraiment pourrie, tu aurais mieux fait de la garder plutôt que venir la vomir ici.
Je viens de filer le sujet à Patch, Nacre et Kalyx (les toutous à papa) et ils font mieux que toi :

function efface_fichier_dans_rep($path,$ext,$temps) {
    if (is_dir($path)) $O = dir($path); else return false;
        
    while (FALSE !== ($file = $O->read())) {
        (substr(strrchr($path.'/'.$file, '.'), 1) === $ext && (filemtime($path.'/'.$file)<= (time() - $temps))) ? unlink($path.'/'.$file) : '';
    }
    $O->close();
    return true;
}


;) Tu l'auras compris, c'est juste une note "d'humour" pour rebondir sur ces points :

- ça me fait toujours marrer que Neige s'en prenne plein la tronche avec ses commentaires;
(a noter que je n'ai aucune action chez lui et que je ne lui dois absolument rien ^^)
Je cite :
"se faire insulter"
"moi une merde !"
"aussi violente"
"traiter d'imbécile"
Je n'ai retrouvé nul part des propos qui pourraient justifier les tiens.
Comme cela a été très justement précisé par Inwebo son message n'est qu'"un peu rude", et ses contributions à CS sont parmi les plus importantes.
Tu aurais dû en tenir compte dans tes réponses, mais j'ai bien noté ton "...à mon tour présente mes excuses à ceux que j'ai pu heurter !"
Je ne doute pas que Neige t'en ai persuadé en 1 ou 2 MP.

Malheureusement, sur le fond, je ne peux être que d'accord avec, il y a toujours quelques nases pour massacrer les sources, en des termes au combien différents de ceux de Neige.
Moi même je ne poste jamais, mais je changerai peut être d'avis, en me faisant un plaisir de remettre à leurs places ces guignols.

En résumé :
Il faut savoir se mettre à la place des débutants, respecter leur travail et les encourager dans leur progression
Il faut savoir se mettre à la place des membres expérimentés qui voient passer des dizaine de sources de qualité contestable alors qu'ils ont répété des centaines de fois certaines "bonnes pratiques"
Il faut savoir casser, même méchamment, les sources ou les messages du forum (surtout) qui relèvent du foutage de gueule (et il y en a crois moi)

- pour en revenir à la source écrite très rapidement par mes potes canins (:o)) elle ne relève qu'à moitié de l'humour :
On cherche ici des fichiers ayant une certaine extension afin de les supprimer selon leur date de modification

Dans ce contexte pourquoi s'embêter à faire trop de tests ? (., .., pathinfo, ...)
Pourquoi tester is_object ?
Les tests de la validité du répertoire et du bon déroulement de la suppression me paraissent quant à eux nécessaires

(en cas de réclamation quant au code vite écrit => s'adresser aux crocs acérés de mes toutous) ^^


Cordialement,


Kohntark