optimiser mon code (faire une boucle)

[cFranssen]

j'aimerais optimiser ce code:


j'ai essayer de faire une boucle mais je n'arrive pas a mettre une variable dans "$_FILES['variable']".

Code :

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
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
if(isset($_FILES['photo1']))
{ 
     $dossier = './img/big/'.$marque.'/';
     $fichier = basename($_FILES['photo1']['name']);
     if(move_uploaded_file($_FILES['photo1']['tmp_name'], $dossier . $fichier)) //Si la fonction renvoie TRUE, c'est que ça fonctionne
     {}
 
}
$img1='<img alt="" src="img/';
$img2='"';
$photo1=$img1.$fichier.$img2;
 
$file='./img/mini/'.$marque.'/'.$fichier;// L'emplacement de l'image à redimensionner.
 
copy($dossier.$fichier, $file);// copie du fichier d'origine avant de redim.
 
$x = 100;
$y = 100; // Taille en pixel de l'image redimensionnée
$size = getimagesize($file);
 
if ($size) {
 
 
$img_big = imagecreatefromjpeg($file); // On ouvre l'image d'origine
$img_new = imagecreate($x, $y);
 
 
// création de la miniature
$img_mini = imagecreatetruecolor($x, $y)
or   $img_mini = imagecreate($x, $y);
 
// copie de l'image, avec le redimensionnement.
imagecopyresized($img_mini,$img_big,0,0,0,0,$x,$y,$size[0],$size[1]);
 
imagejpeg($img_mini,$file );
 
}
 
//--------------------image2--------------------------------------------
 
 
if(isset($_FILES['photo2']))
{ 
     $dossier = './img/big/'.$marque.'/';
     $fichier = basename($_FILES['photo2']['name']);
     if(move_uploaded_file($_FILES['photo2']['tmp_name'], $dossier . $fichier)) //Si la fonction renvoie TRUE, c'est que ça a fonctionné...
     {}
}
$img1='<img alt="" src="img/';
$img2='"';
$photo2=$img1.$fichier.$img2;
 
 
$file='./img/mini/'.$marque.'/'.$fichier;// L'emplacement de l'image à redimensionner.
 
copy($dossier.$fichier, $file);// copie du fichier d'origine avant de redim.
 
$x = 100;
$y = 100; // Taille en pixel de l'image redimensionnée
$size = getimagesize($file);
 
if ($size) {
 
 
$img_big = imagecreatefromjpeg($file); // On ouvre l'image d'origine
$img_new = imagecreate($x, $y);
 
 
// création de la miniature
$img_mini = imagecreatetruecolor($x, $y)
or   $img_mini = imagecreate($x, $y);
 
// copie de l'image, avec le redimensionnement.
imagecopyresized($img_mini,$img_big,0,0,0,0,$x,$y,$size[0],$size[1]);
 
imagejpeg($img_mini,$file );
 
}
 
 
//-----------------image3-----------------------------------------------
 
if(isset($_FILES['photo3']))
{ 
     $dossier = './img/big/'.$marque.'/';
     $fichier = basename($_FILES['photo3']['name']);
     if(move_uploaded_file($_FILES['photo3']['tmp_name'], $dossier . $fichier)) //Si la fonction renvoie TRUE, c'est que ça a fonctionné...
     {}
}
$img1='<img alt="" src="img/';
$img2='"';
$photo3=$img1.$fichier.$img2;
 
 
$img1='src="img/';
$img2='" width="80"';
$mini=$img1.$fichier.$img2;
 
 
 
$file='./img/mini/'.$marque.'/'.$fichier;// L'emplacement de l'image à redimensionner.
 
copy($dossier.$fichier, $file);// copie du fichier d'origine avant de redim.
 
$x = 100;
$y = 100; // Taille en pixel de l'image redimensionnée
$size = getimagesize($file);
 
if ($size) {
 
 
$img_big = imagecreatefromjpeg($file); // On ouvre l'image d'origine
$img_new = imagecreate($x, $y);
 
 
// création de la miniature
$img_mini = imagecreatetruecolor($x, $y)
or   $img_mini = imagecreate($x, $y);
 
// copie de l'image, avec le redimensionnement.
imagecopyresized($img_mini,$img_big,0,0,0,0,$x,$y,$size[0],$size[1]);
 
imagejpeg($img_mini,$file );
 
}

[sabotage]

nomme tes champs photo[] au lieu de photo1, photo2 etc. et fais une boucle foreach()

[cFranssen]

je dois donc faire une liste ?

quel est la syntaxe ?

[sabotage]

Je ne vois pas ce que tu veux dire par une liste.
Dans ton formulaire tu remplaces "photo1" par "photo[]" etc.

[cFranssen]

je ne comprend pas en quoi ça va changer le php

[sabotage]

Tu recevras un tableau $_FILES['photo'] que tu pourras traiter dans une boucle foreach()