EXPLORATEUR DE FICHIER PHP
codefalse
Messages postés
1123
Date d'inscription
mardi 8 janvier 2002
Statut
Modérateur
Dernière intervention
21 avril 2009
-
2 déc. 2007 à 00:50
Arnaud - 20 avril 2015 à 15:57
Arnaud - 20 avril 2015 à 15:57
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.
https://codes-sources.commentcamarche.net/source/44888-explorateur-de-fichier-php
https://codes-sources.commentcamarche.net/source/44888-explorateur-de-fichier-php
20 avril 2015 à 15:57
dans mon dossier www ( apache ) j'ai dossier 1 ( perso ) et dossier 2 ( public avec l'explorateur )
mon probleme c'est que explorateur va dans le dossier 1 alors que j'aimerai qu'il voie le contenu que du dossier 2
merci de ton aide
16 févr. 2009 à 11:39
Merci pour ton script :)
3 déc. 2007 à 15:36
Moi vu que je suis débutant en prog PHP je fais pas de commentaires sur le code.
Bonxp
3 déc. 2007 à 03:22
2 déc. 2007 à 23:22
++
2 déc. 2007 à 22:30
2 déc. 2007 à 19:00
Je vais revoir tout ca la prochaine MAJ je vais essayez de réorganisé tout ca
je vais faire les gestion de toute les erreur
merci pour le commentaire ++
Kevin
2 déc. 2007 à 15:19
au-delà de ce qui a déjà été dit, ce code est bordélique. déjà, j'aimerais comprendre pourquoi tu ne mets pas les 2 couleurs définies en début de fichier dans tes css ? Ensuite, pourquoi balances tu le html dans des echo comme ça ? Tu redéfinis tranquillement le Content-type qui plus est.
Côté séparation du code, c'est pas top tout ça : tout est mélangé. Ca ne donne pas envie de lire le code, du tout. En tous cas moi, ça ne me donne pas envie.
Le str_replace sur le %20 me laisse rêveur, au passage...
Tu utilises des @ à tout va, sans gérer les erreurs éventuelles. En gros, tu ne veux pas voir les erreurs, mais tu n'en fais rien si elles surgissent. C'est la politique de l'autruche ça.
Ce commentaire est bordélique, mais c'est parce que je lis et commente le code comme je le vois.
Bref, Kiki, ça manque sérieusement de travail et d'organisation tout ça.
2 déc. 2007 à 12:39
2 déc. 2007 à 11:56
il y a quelques erreurs qui apparaissent notamment :
Notice: Undefined variable: i
Notice: Undefined variable: replien
ET lors de la création de répertoire !!!!!
Notice: Undefined variable: mkdir
Sinon le point de vue sécurité est a revoir et les liens peuvent être simplifiés
?dir=[..]/../explorateur/file/../file/../file/../img/../file/../file
?dir=[..]&file=...
2 déc. 2007 à 02:07
Voilà j'ai MAJ la source et pour la sécurité si tu utilise l'interface normalement tu as pas de problème
Pour les classe j'ai jeter un oeil elle et bien ta source codefalse :)
Voilà @++
Kevin
2 déc. 2007 à 01:37
+1 avec Codefalse
Et puis :
if( !is_uploaded_file($tmp_file) )
{
exit;
}
C'est pas un peu bourrin ? Afficher un message d'erreur serait le MINIMUM (mieux : une exception)
Et puis ta fonction pour supprimer un fichier est particulièrement dangereuse pour cause d'absence totale de sécurité...
2 déc. 2007 à 00:50
Mais une classe (ou un ensembles de classes) aurait été préférés !
Tu connais DirectoryIterator ?
SplFileInfo ?
Renseigne toi sur ces classes, tu y trouvera des fonctions utiles pour faire des classes plus performantes !