Discussion :

[Kenshin86]

Salut

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
<?php 
 
// Dossier de destination du fichier
$folder = "C:/Documents and Settings/Easy/Bureau/SiteWeb/www/dep/";
 
// Tableau array des différents types
 
$allowed_types = array("application/pdf");
 
$fsize = $HTTP_POST_FILES['fichier']['size'];
$fname = $HTTP_POST_FILES['fichier']['name'];
$ftype = $HTTP_POST_FILES['fichier']['type'];
$fsize = $HTTP_POST_FILES['fichier']['size'];
$ftmp = $HTTP_POST_FILES['fichier']['tmp_name'];
 
echo "Taille : $fsize";  echo "<br>";
echo "Nom Fichier : $fname";  echo "<br>";
echo "Type : $ftype";  echo "<br>";
echo "Chemin : $ftmp";  echo "<br>";
 
if(copy($ftmp,''.$folder.''.$fname.'')) {$error = 0;}
 
if(!in_array($ftype, $allowed_types)){$error = 1;}
 
switch($error){
 
case'1':
echo("Format de fichier incorrecte.");
break;
case'0':
echo("Copie réussie");
break;
}
?>

Lorsque je sélectionnes un fichier PDF , je dois avoir le message "copie réussie" , c'est le cas donc ça va.

Par contre quand c'est un autre type de fichier. J'ai affiché "Format incorrecte" mais la copie se réalise quand meme...

Cordialement

[supergogol]

bonsoir,

c'est tout à fait normal, ton code n'est pas logique. En effet, tu teste le type de fichier APRES avoir copier ce dernier ! Il faut le faire avant !

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
 
<?php   
// Dossier de destination du fichier 
$folder = "C:/Documents and Settings/Easy/Bureau/SiteWeb/www/dep/";   
// Tableau array des différents types   
$allowed_types = array("application/pdf");   
$fsize = $HTTP_POST_FILES['fichier']['size']; 
$fname = $HTTP_POST_FILES['fichier']['name']; $ftype = $HTTP_POST_FILES['fichier']['type']; 
$fsize = $HTTP_POST_FILES['fichier']['size']; 
$ftmp = $HTTP_POST_FILES['fichier']['tmp_name'];   
 
echo "Taille : $fsize"; 
echo "<br>"; 
echo "Nom Fichier : $fname"; 
echo "<br>"; 
echo "Type : $ftype"; 
echo "<br>"; 
echo "Chemin : $ftmp"; 
echo "<br>";
 
if(!in_array($ftype, $allowed_types))
{
     echo("Format de fichier incorrecte."); 
}  
else
{
      if(copy($ftmp,''.$folder.''.$fname.'')) 
            echo("Copie réussie"); 
}
?>

en plus, comme tu peux le voir, le switch est superflue

[Kenshin86]

Ok merci.

C'est qu'en fait , je comptais intégrer plusieurs cas.

Dont la vérification (si le fichier existe , ainsi que sa taille)

Je croyais ça fesait mieux que d'intégrer plusieurs si. Mais bon si cette solution marche , je vais faire abstraction de l'esthétique.

[mathieugamin]

Salut
Autre chose, dans ton code, tu spécifies des codes erreurs par des entiers :

Code :

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

1
2
3
if(copy($ftmp,''.$folder.''.$fname.'')) {$error = 0;}
 
if(!in_array($ftype, $allowed_types)){$error = 1;}

Mais dans ton switch tu les traites comme des strings. C'est un peu puriste, mas ça évite des surprises:

Code :

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

1
2
3
4
5
6
7
8
switch($error){
case'1':
echo("Format de fichier incorrecte.");
break;
case'0':
echo("Copie réussie");
break;
}

Donc enlèves les quotes dans les case de ton switch, c'est plus propre

[Kenshin86]

Oui mathieugamin , j'ai retiré les quote , mais cela n'as rien changé.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
if($fsize > $MAX_FILE_SIZE)
	{echo("Fichier trop volumineux");}	
elseif
 
	(!in_array($ftype, $allowed_types))
	echo("Format incorect");
 
elseif 
	(!file_exists($folder."m_".$fname))
	echo("existant");
 
elseif
	(copy($ftmp,''.$folder.''.$fname.'')) 
	echo("OK");

J'ai fais ça et il y a un problème lors de la copie (cela m'affiche existant)

[Broswendes]

Hum, si le fichier n'existe pas , alors on affiche qu'il existe ?

Code :

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

1
2
3
elseif 
	(!file_exists($folder."m_".$fname))
	echo("existant");

Tu ne devrais pas plutôt mettre:

Code :

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

1
2
3
elseif 
	(file_exists($folder."m_".$fname))
	echo("existant");