Bonjour à tous,

Je suis débutant en PHP et je souhaiterais optimiser mon code. Celui-ci permet déjà d'envoyer plusieurs données selon qu'elles soient de type texte ou de type tableau, à l'aide d'un formulaire, pour intégrer ces données dans une base de données.

Dans ma base de données, il y a trois tables qui sont les suivantes :
- formulaire (id, nom, prenom, adresse, .... devis, contact, client);
- siteformulaire (id_formulaire, id_site);
- site (id, n_site).

La table intermédiaire nommée "siteformulaire" permet d'attribuer plusieurs données d'un même champ à un formulaire (voir le champ checkbox tout en bas du fichier newfile.php faisant suite).

Voici mon code mis dans un fichier nommé newfile.php :

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
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
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
 
<?php 
include ("configuration.php");// connexion à la base de données
 
	//déclaration des variables à poster dans le formulaire
	$id=$_POST['id'];
	$nom=$_POST['nom'];
	$prenom=$_POST['prenom'];
	$adresse=$_POST['adresse'];
	$code_postal=$_POST['code_postal'];
	$ville=$_POST['ville'];
	$telephone=$_POST['telephone'];
	$fax=$_POST['fax'];
	$email=$_POST['email'];
	$devis=$_POST['devis'];
	$contact=$_POST['contact'];
	$client=$_POST['client'];
 
	//************* AFFICHAGE DES DONNEES SUITE A LA SAISIE DU FORMULAIRE ******************
 
    echo "Bonjour !"."</br>";
 
    echo "Votre nom est " .$nom. " !"."</br>";
	echo "Votre prénom est " .$prenom. " !"."</br>";
	echo "Votre adresse est " .$adresse. " !"."</br>";
	echo "Votre code postal est ".$code_postal. " !"."</br>";
	echo "Votre ville est ".$ville. " !"."</br>";
	echo "Votre téléphone est ".$telephone. " !"."</br>";
	echo "Votre fax est ".$fax. " !"."</br>";
	echo "Votre email est ".$email. " !"."</br>";
 
	if ($devis!="Oui")
	echo "Vous ne souhaitez pas effectuer de demande de devis"."</br>";
	else echo "Vous avez effectué une demande de devis"."</br>";
 
	if ($contact!="Oui")
	echo "Vous ne souhaitez pas effectuer de demande de contact"."</br>";
	else echo "Vous avez effectué une demande de contact"."</br>";
 
	if ($client!="Oui")
	echo "Vous n'êtes pas un client de l'entreprise"."</br>";
	else echo "Vous êtes un client de l'entreprise"."</br>";
 
    echo '<b>Vous  connaissez le site  :</b><ul>';
    foreach($_POST['site'] as $n_site){
        echo '<li>'.$n_site.'</li>';
    }
    echo '</ul><br />';
 
		//**************************** ENVOI DES DONNEES DANS LA BASE **************************
 
    $sql = "INSERT INTO formulaire(id, nom, prenom, adresse, code_postal, ville, telephone, fax, email, devis, contact)
    VALUES('','$nom','$prenom','$adresse','$code_postal','$ville','$telephone','$fax','$email', '$devis', '$contact')";
    mysql_query($sql) or die('Erreur SQL !'.$sql.'<br>'.mysql_error());
 
  	$id_formulaire = mysql_insert_id();
    // on récupère l'id_formulaire qui vient d'être généré
 
    $sql = "SELECT s.id
    FROM site s
    LEFT JOIN siteformulaire sf 
    ON s.id=sf.id_site
	LEFT JOIN formulaire f
	ON f.id=sf.id_formulaire
    WHERE n_site='$n_site'";
 
$req = mysql_query($sql) or die('Erreur SQL !'.$sql.'<br>'.mysql_error());
		$res= mysql_num_rows($req);
 
foreach($_POST['site'] as $n_site){
    $sql = "INSERT INTO site(id, n_site) VALUES('','$n_site')";
    mysql_query($sql) or die('Erreur SQL !'.$sql.'<br>'.mysql_error());
    	// on récupère l'id_site qui vient d'être généré
     $id_site = mysql_insert_id();
       $sql = "INSERT INTO siteformulaire(id_formulaire, id_site) VALUES ('$id_formulaire','$id_site')";
        mysql_query($sql) or die('Erreur SQL !'.$sql.'<br>'.mysql_error());
}
 
	echo "</br></br>"."Vos données ont été envoyées au webmaster !"; 
	mysql_close($oConnect); // on ferme la connexion à mysql
 
?>
 
 
<form method="post" action="newfile.php">
 
 
	<fieldset style="margin:50px">
	<legend><b>Vos coordonnées</b></legend>
	<table>
		<tr>
			<td width="220"><label for="nom">Nom :</label></td>
			<td><input type="text" name="nom" value="<?php if(isset($nom)){$sValue = htmlentities($nom, ENT_QUOTES);}?>"/></td>
		</tr>
 
		<tr>
			<td width="220"><label for="prenom">Prénom :</label></td>
			<td><input type="text" name="prenom" value ="<?php if(isset($prenom)){$sValue =  htmlentities($prenom, ENT_QUOTES);}?>"/></td>
		</tr>
 
		<tr>
			<td width="220"><label for="adresse">Adresse :</label></td>
			<td><input type="text" name="adresse" value ="<?php if(isset($adresse)){$sValue = htmlentities($adresse, ENT_QUOTES);}?>"/></td>
		</tr>
 
		<tr>
			<td width="220"><label for="code_postal">Code postal :</label></td>
			<td><input type="text" name="code_postal" value="<?php if(isset($code_postal)){$sValue = htmlentities($code_postal, ENT_QUOTES);}?>"/></td>
		</tr>
 
		<tr>
			<td width="220"><label for="ville">Ville :</label></td>
			<td><input type="text" name="ville" value="<?php if(isset($ville)){$sValue = htmlentities($ville, ENT_QUOTES);}?>"/></td>
		</tr>
 
		<tr>
			<td width="220"><label for="telephone">Téléphone :</label></td>
			<td><input type="text" name="telephone" value="<?php if(isset($telephone)){$sValue = htmlentities($telephone, ENT_QUOTES);}?>"/></td>
		</tr>
 
		<tr>
			<td width="220"><label for="fax">Fax :</label></td>
			<td><input type="text" name="fax" value="<?php if(isset($fax)){$sValue = htmlentities($fax, ENT_QUOTES);}?>"/></td>
		</tr>
 
		<tr>
			<td width="220"><label for="email">Email :</label></td>
			<td><input type="text" name="email" value="<?php if(isset($email)){$sValue = htmlentities($email, ENT_QUOTES);}?>"/></td>
		</tr>
 
	</table>
	</fieldset>
 
	<fieldset style="margin:50px">
	<legend><b>Vos options</b></legend>
	<table>
 
		<tr>
			<td width="220"><label for="devis">Demande de devis :</label></td>
			<td>
	<select name="devis" id="9">
	 	<option value=" " selected="selected"> </option>
	    <option value="Oui">Oui</option>
	    <option value="Non">Non</option>
	</select>
			</td>
		</tr>
 
		<tr>
			<td width="220"><label for="contact">Demande de contact :</label></td>
			<td>
	<select name="contact" id="10">
	 	<option value=" " selected="selected"> </option>
	    <option value="Oui">Oui</option>
	    <option value="Non">Non</option>
	</select>
			</td>
		</tr>
 
	</table>
	</fieldset>
 
	<fieldset style="margin:50px">
	<legend><b>Vos informations</b></legend>
	<table>
 
		<tr>
			<td width="220"><label for="client[]">Etes vous déjà client ?</label></td>
			<td>
			<input type="radio" name="client[]" value="Oui" id="11" /> <label for="oui">Oui</label>
			<input type="radio" name="client[]" value="Non" id="12" /> <label for="non">Non</label>
			</td>
		</tr>
 
		<tr>
			<td width="220"><label for="site[]">Comment avez vous connu le site ?</label></td>
			<td>
			<input type="checkbox" name="site[]" value="par internet" id="14"/> <label >par Internet</label>	
			<input type="checkbox" name="site[]" value="par la presse" id="15"/> <label >par la Presse</label>
			<input type="checkbox" name="site[]" value="par un autre moyen de communication" id="16"/> <label >Autres</label>
			</td>
		</tr>
 
	</table>
	</fieldset>
 
	<fieldset style="margin:50px">
	<legend><b>Opérations</b></legend>
	<table>
		<tr>
			<td width="250"><label for="valider"></label></td>
			<td><input type="submit" value="Valider" name="valider" id="17"/></td>
			<td><input type="reset" value="Effacer" id="18"></td>
		<!--	<td><input type="button" value="Réinitialiser" onclick="reinitialiser()" id="19"/></td>
		-->
		</tr>
	</table>
	</fieldset>
 
</form>
Je voudrais simplifier ce code en utilisant le formulaire sous la forme d'un tableau dans lequel se trouve des données de type texte (simple champ à remplir) et de type tableau (en référence notamment au type checkbox) parcourues avec la fonction foreach. On peut donc schématiser de la manière suivante : le formulaire contient plusieurs champs soit de type texte soit de type tableau. Je sais qu'il y a possibilité d'optimiser mon code mais je ne sais pas comment faire. Peut-être selon le modèle MVC ???

J'ai commencé à faire le code suivant sans être convaincu de ce à quoi il peut mener.

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
 
 
// cas si c'est un tableau
if(isset($sKey)){
 
	$aFormulaire = array();
	if(is_array($_POST)){
 
		foreach($_POST as $sKey=>$sValue){
 
			echo "$sKey = $sValue";
 
 
		}
	}
}
 
		// faire le cas aussi si ce n'est pas un tableau
		// filtrer la variable $aFormulaire['nom'] = 'H&eacute;l&egrave;ne&quote;'; (nom pris comme exemple en référence au premier champ du formulaire)
Voilà si quelqu'un peut me guider ça serait cool Merci