PHP

[PHP] Upload de fichier presque fonctionnel (presk...)

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : [PHP] Upload de fichier presque fonctionnel (presk...)

Donzeky

Le bleu c'est la vie.

Bonsoir,

voilà je tente de créer un formulaire qui permette l'envoi d'images .jpg ou .jpeg uniquement, ne dépassant pas les 50Ko, avec affichage des erreurs en cas de non-respect de ces règles ou envoi de formulaire vide.
Il marchait correctement jusqu'à ce que je rajoute un elseif pour la taille maxi.

Maintenant, il repère correctement une taille trop élevée et un format d'image interdit, mais m'interdit également aussi l'upload de .jpg et .jpeg

J'ai beau chercher, je ne vois pas d'où vient l'erreur.
Alors... merci d'avance à qui trouvera le pb (je précise que je débute en php )

Voici mon code :

Code :

<?php
$nom_fichier = $_FILES['fichier']['name'];
$nom_fichier_temp = $_FILES['fichier']['tmp_name'];
$nom_fichier_size = $_FILES['fichier']['size'];
$nom_fichier_type = $_FILES['fichier']['type'];
$taille_max = 50176;
if (isset($_POST['upload']))
{
if ($nom_fichier_size==0)
{
$message_erreur = '<br />L\'upload a échoué. Veuillez choisir une image.<br />';
/* switch($i)
{
case 0:
echo
}*/
}
elseif($nom_fichier_size > $taille_max)/
{
$message_erreur = '<br />Seuls les fichiers de moins de 50Ko sont acceptés';
}
elseif (!stristr($nom_fichier_type, 'jpg') || !stristr($nom_fichier_type, 'jpeg'))//stristr est semblable à strstr, mais n'est pas sensible à la casse
{
$message_erreur = '<br />Seules les images de type .jpg et .jpeg sont admises.<br />';
}
else
{
$nom_destination = 'fichiers/';
$nom_destination .= $nom_fichier;//sinon $destination vaut fichiers/$_FILES['fichier']['name']
$size = $nom_fichier_size;
move_uploaded_file($nom_fichier_temp, $nom_destination);
echo 'Le fichier '.$nom_fichier.' a été correctement uploadé <p>';
echo 'Il pèse '.$nom_fichier_size.' Octets';
echo '<p>Aperçu :<p>';
readfile($nom_destination);
}
}
?>
<form method="post" action="file_upload_en_un_seul_fichier.php" enctype="multipart/form-data" name="formulaire">
<fieldset>
<legend>Envoi de fichier</legend>
<label for="photo">Chemin :</label>
<input type="file" name="fichier" size="30"><p>
<input type="submit" name="upload" value="Uploader">
<?php
echo $_GET = stripslashes($message_erreur);//afficher sans les antislashes le message d'erreur transmis par GET
?>
</fieldset>
</form>

Message édité par Donzeky le 06-08-2006 à 15:46:25

Publicité

AlphaZone

Tu as des tests qui me font un peu peur comme celui de la ligne 8, et tu as oublier un champ à la ligne 45

Voici un tutorial : http://www.phpfrance.com/tutoriaux [...] e-fichiers

Ca devrait t'aider.

Message cité 1 fois

Donzeky

Le bleu c'est la vie.

AlphaZone a écrit :

Merci pour ta réponse.

J'ai rajouté </fieldset>, le champs label (c'était ça qui manquait :??: ).
J'ai aussi remplacé le
if ($_POST['upload'])
par
if (isset($_POST['upload'])).
Mais ça ne change rien...

Je crois que je vais carrément reprendre depuis le début, en faisant autrement.

EDIT : code modifié dans le premier message

Message cité 1 fois
Message édité par Donzeky le 06-08-2006 à 15:46:49

AlphaZone

Donzeky a écrit :

J'ai rajouté </fieldset>, le champs label (c'était ça qui manquait :??: ).

Non c'est un champ de type hidden qui te permet de fixé la taille max des fichiers envoyés.

Donzeky

Le bleu c'est la vie.

Mais le fait de spécifier la taille max par :

Code :

elseif($nom_fichier_size > $taille_max)
où
$nom_fichier_size
vaut
$_FILES['fichier']['size']
et où
$taille_max
vaut
50176

ne suffit pas :??:
Ou bien c'est moins "conseillé" ?

AlphaZone

Et non, malheureusement cela ne suffit pas. Je pense que tu as compris que je parlais du champ hidden nommé MAX_FILE_SIZE.
Voici un article très intéressant, je pense que cela devrait t'aidé dans la compréhension : http://www.nexen.net/index.php?opt [...] method.php

Extrait de la partie concerné :

Citation :

Le champs caché MAX_FILE_SIZE (mesuré en octets) doit précédé le champ input de type file et sa valeur représente la taille maximale acceptée du fichier. Ceci est considéré comme un conseil pour le navigateur, bien que PHP l'utilise également. Il est très facile de contourner cette restriction. Ne comptez pas sur le respect de cette configuration par le navigateur! La configuration de PHP sur la taille maximale à respecter ( upload_max_filesize ) ne peut être contournée, elle. Vous devez ajouter la variable MAX_FILE_SIZE à votre formulaire dans tous les cas car il prévient le chargement de gros fichiers qui demanderait un long délai d'attente au client et ainsi fera échouer le script.

Autre précision sur ton code :

Code :

$nom_fichier = $_FILES['fichier']['name'];
$nom_fichier_temp = $_FILES['fichier']['tmp_name'];
$nom_fichier_size = $_FILES['fichier']['size'];
$nom_fichier_type = $_FILES['fichier']['type'];
$taille_max = 50176;

Tu devrais faire cela SI et SEULEMENT SI tu es sûr qu'un fichier a été uploadé, sinon cela risque d'afficher des messages d'erreur sur ta page. C'est un peu gênant pour le visiteur et peut-être un plus pour un pirate de passage

Message cité 1 fois
Message édité par AlphaZone le 06-08-2006 à 18:59:32

Donzeky

Le bleu c'est la vie.

AlphaZone a écrit :

Autre précision sur ton code :

Code :

$nom_fichier = $_FILES['fichier']['name'];
$nom_fichier_temp = $_FILES['fichier']['tmp_name'];
$nom_fichier_size = $_FILES['fichier']['size'];
$nom_fichier_type = $_FILES['fichier']['type'];
$taille_max = 50176;

Pour cela j'avais mis :

Code :

if (isset($_POST['upload']))
--> Sinon réafficher le formulaire avec le message d'erreur :
"Veuillez choisir une image".

Mais après, est-ce suffisant pour la sécurité... de toute façon je lis les liens que tu m'as donnés et meme plus, et teste diférentes méthodes

AlphaZone

Justement niveau sécurité ce n'est pas au top, tu vérifi simplement que le formulaire a été envoyé or tu ne sais pas si ton champ fichier a été saisi.

Il faut que tu vérifi si le champ fichier a été défini et qu'il n'est pas vide : if(isset() && !empty(...))

flo850

moi je

il suffit de verifier la taille du tableau $_FILES
au passage, il vaut mieux eviter de tester sur le bouton submit , il me semble que si on se sert de entrer pour valider le formulaire la variable n'est pas passé ( sous IE 6 )

le pluis propre est d'utiliser un champ hidden pour checker si on est passer par le formulaire ou pas

FORUM HardWare.fr

Programmation

PHP

[PHP] Upload de fichier presque fonctionnel (presk...)

Sujets relatifs
Batch - Afficher les fichier du répertoire System Volume Information	Galeire Photo "miniature" en PHP
Version xdebug pour PHP 4.3.8 sous Windows	Effacer la mise en forme d'un fichier
probleme chargement de fichier csv	[PHP] Prog. orientée objet pas gérée par mon serveur O_o
requete multiple PHP SQL : besoin coup de main urgent	Apache, MySQL PHP 5 et Windows ... petite galère à l'installation
Découpage de fichier word	PHP MVC et urls
Plus de sujets relatifs à : [PHP] Upload de fichier presque fonctionnel (presk...)

Page générée en 0.083 secondes