UPLOAD AND ZIP YOUR FILES

Commentaire de malik7934 le 28/01/2005 11:09:28

Salut,

Ok, je vais corriger sous peu... d'autant que mon code oublie quelque chose d'imprtant: il n'efface pas le fichier non-zippé du serveur! Du coup, y a plus d'intérêt!

Je m'empresse de mettre à jour ;o)

Commentaire de eXon le 28/01/2005 11:39:02

Tu devrais codé avec ton error_reporting à E_ALL parce que tu as fait une petite erreur:
$submit = $_POST['submit'];
if (!($submit)){  
    $max_size = 50; // 50kb. Mettez ce que vous voulez ici!

à remplacer par:

$submit = (isset($_POST['submit']))?$_POST['submit']:NULL;
if($submit === NULL){
    $max_size = 50; // 50kb. Mettez ce que vous voulez ici!

Commentaire de eXon le 28/01/2005 13:49:15

Je te conseil de remettre if($submit === NULL) c'est une comparaison par objet. Donc avec if($submit === NULL il va retourné true si c'est NULL mais pas s'il est vide, quand est-ce qu'avec == il va retourné true s'il est vide aussi.

Commentaire de Anthomicro le 28/01/2005 18:28:42

Salut ;-)

Tu peux remplacer ta fonction getFileName() par celle-ci :

function getFileName($fichier)
{
         $fichier = basename ($fichier);
         return substr($fichier, 0,strlen($fichier)-4);
}

Ensuite les multiples ouvertures/fermetures des tags c'est crade. Pour ce qui est du ===NULL ou ==NULL, tu peux remplacer ça :

$submit = (isset($_POST['submit']))?$_POST['submit']:NULL;
if($submit === NULL){
    
    $max_size = 50; // 50kb. Mettez ce que vous voulez ici!

par ça :

if(!isset($_POST['submit'])){
    
    $max_size = 51200; // 50kb. Mettez ce que vous voulez ici!
echo '<form enctype="multipart/form-data" action="',$_SERVER['SCRIPT_NAME'],'" method="post">
<p><input type="hidden" name="MAX_FILE_SIZE" value="',$max_size,'" />
Envoyez ce fichier : <input name="userfile" type="file" />
<input type="submit" name="submit" value="Envoyer le fichier" /></p>
</form>';

}

//Ensuite dans le else toujours vérifier la taille du fichier, le html seul ne suffit pas et quelqu'un de mal intentionné pourra envoyer 5 Mo si il veut, malgré ta limite.
else
{
        $clef=array_keys($_FILES);
if($_FILES[$clef[0]]['size']>$max_size)
{
            //taille trop grosse
        }
}


a ++

Commentaire de Anthomicro le 29/01/2005 12:05:46

"Ensuite dans le else toujours vérifier la taille du fichier, le html seul ne suffit pas et quelqu'un de mal intentionné pourra envoyer 5 Mo si il veut, malgré ta limite."

F-A-U-X! PHP t'empêche d'uploader un fichier de plus de 2MB, donc pas de soucis de ce côté là! Regarde dans phpinfo(4) la limite de "upload_max_filesize""

Arf il va contester chacun de mes messages...

On va donc répondre point par point.

Tout d'abord effectivement par défaut l'upload max est à
2 Mo. Et alors ? il n'en veut que  50 Ko. A partir du moment où la limite souhaitée est inférieure au max_upload_size de php tu DOIS verifier la taille du fichier. (1er argument cassé)

On passe au suivant :

"[blabla] d'eXon pour $submit, je préfère la sienne, je la trouve plus "complète". Mais chacun fait comme il veut ;o)"

Oui, tu la trouves plus chargée, je suis d'accord avec toi.

Ensuite pour verifier l'existence d'une variable on n'utilise pas if($variable===NULL) mais if(isset($variable))

M'enfin bon je t'ai prouvé l'autre fois avec le XHTML que j'avais raison, j'ai pas envie de recommencer pour ça...

Ce qui m'arrangerait c'est que tu arrêtes d'essayer de me casser juste parce que j'ai critiqué je ne sais plus quoi sur ta manière de coder.

a +

Commentaire de malik7934 le 29/01/2005 13:49:49

Encore un truc:

"Ensuite pour verifier l'existence d'une variable on n'utilise pas if($variable===NULL) mais if(isset($variable))"

... si tu regardes un peu mieux le code que m'a suggéré eXon, c'est

$submit = (isset($_POST['submit']))?$_POST['submit']:NULL;
if($submit === NULL)

... le isset à bien lieu ;o)

Commentaire de eXon le 30/01/2005 05:11:42

Pour ceux qui veulent plus en savoir sur le triple égal (===), c'est très bien expliquer sur le site de PHP: http://www.php.net/manual/fr/language.oop5.object-comparison.php

Commentaire de pehel le 29/04/2005 21:42:55

Une petite amélioration de getFileName
function getFileName($fichier)
{
         $fichier = basename ($fichier);
         return substr($fichier,0,-strlen(strrchr($fichier,'.')));
}

histoire de couper les cheveux en quatre.

Mais où serait le plaisir sinon.