Discussion :

[moimp]

Bonjour,
Dans le traitement d'un formulaire, je ne veux exécuter un bloc de code que si un fichier a bien été sélectionné. Ce bloc contient bien sûr le téléchargement mais aussi des actions annexes.
J'ai écrit if (!empty($_FILES)) mais la condition est toujours vraie.
Je teste if($_FILES['file']['error'] !== 4 and !empty($_FILES['file']['name'])) mais je ne suis pas sûr que ce soit vrai dans tous les cas ni que ce soit la bonne méthode.

[Pytet]

Bonjour,

La condition est correcte pour vérifier qu'un fichier a bien été sélectionné mais elle ne vérifie pas les autres éventuelles erreurs d'upload, par exemple si le fichier sélectionné est trop grand ou si l'upload a été fait partiellement.

Pour contrôler que l'upload s'est fait sans erreur, on peut vérifier que la variable $_FILES['file']['error'] soit égale à 0. Au passage, on peut utiliser les constantes défini par PHP pour les codes d'erreurs (UPLOAD_ERR_OK, UPLOAD_ERR_NO_FILE, etc..) au lieu d'utiliser directement les valeurs des codes d'erreurs (0, 4, etc...).

https://www.php.net/manual/fr/featur...ost-method.php
https://www.php.net/manual/fr/featur...oad.errors.php

Un exemple :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
 
if ($_FILES['file']['error'] === UPLOAD_ERR_OK) {
    echo 'Le fichier est bien uploadé';
} else {
    echo 'Erreur lors de l\'upload du fichier. Code erreur : ' : $_FILES['file']['error'];
 
    switch ($_FILES['file']['error']) {
        case UPLOAD_ERR_INI_SIZE:
            echo "The uploaded file exceeds the upload_max_filesize directive in php.ini";
            break;
        case UPLOAD_ERR_FORM_SIZE:
            echo "The uploaded file exceeds the MAX_FILE_SIZE directive that was specified in the HTML form";
            break;
        case UPLOAD_ERR_PARTIAL:
            echo "The uploaded file was only partially uploaded";
            break;
        case UPLOAD_ERR_NO_FILE:
            echo "No file was uploaded";
            break;
        case UPLOAD_ERR_NO_TMP_DIR:
            echo "Missing a temporary folder";
            break;
        case UPLOAD_ERR_CANT_WRITE:
            echo "Failed to write file to disk";
            break;
        case UPLOAD_ERR_EXTENSION:
            echo "File upload stopped by extension";
            break;
 
        default:
            echo "Unknown upload error";
            break;
    }
}

[moimp]

La condition est correcte pour vérifier qu'un fichier a bien été sélectionné

C'est en fait tout ce que je voulais.

mais elle ne vérifie pas les autres éventuelles erreurs d'upload

Le bloc sous la condition fait justement appel à une classe qui effectue tous les contrôles, notamment ceux inhérents à $_FILES et la correspondance entre l'extension et le type MIME.
Je serais d'ailleurs intéressé par un œil critique sur cette classe.
Dans ce même bloc, je fais le transfert du fichier sur le site.

https://www.php.net/manual/fr/featur...ost-method.php
https://www.php.net/manual/fr/featur...oad.errors.php

Je connais ces liens. Merci.