Optimisation de mon formulaire

**Zarvox** · 17/07/2009, 14h34

Bonjour à tous,

Pour faciliter l'exploitation de la base MySQL par les non informaticiens de mon entreprise je développe des formulaires en php.

Je voudrais soumettre mon formulaire ici car je débute et il y à sûrement des moyens plus rapide de faire ce que je fait.

Je suis ouvert à tout les conseils.

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
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
254
<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"
"http://www.w3.org/TR/html4/loose.dtd">
<html>
  <head>
    <meta name="generator" content="Bluefish 1.0.7">
    <meta content="text/html; charset=iso-8859-1" http-equiv="content-type">
    <title>
      Recherche
    </title>
    <link rel="stylesheet" href="lib/base.css" type="text/css">
  </head>
  <body>
    <table border="0" cellpadding="0" cellspacing="0" height="100%" width="100%">
      <tbody>
        <tr>
          <td align="center" valign="middle">
            <div align="center"></div>
            <table border="0" cellpadding="2" cellspacing="0" width="600">
              <tbody>
                <tr>
                  <td align="center">
                    <h2>
                      Recherche de renseignements
                    </h2>
                  </td>
                </tr>
                <tr>
                  <td>
                    <b>La recherche peut porter uniquement sur une partie du nom ou du
                    prénom. Ce dernier étant facultatif, dans ce cas la recherche
                    affichera uniquement le contenu de la première fiche rencontrée.</b>
                  </td>
                </tr>
                <tr>
                  <td align="left">
                    <form>
                      <center>
                        <table border="0" cellpadding="0" cellspacing="0" width="550">
                          <tbody>
                            <tr height="50" valign="bottom">
                              <td align="right" valign="middle" width="100">
                                Nom :&nbsp;
                              </td>
                              <td valign="middle" width="430">
                                <input name="nom" size="18">&nbsp;Prénom
                                :&nbsp;<input name="prenom" size="18">
                              </td>
                            </tr>
                          </tbody>
                        </table><input name="Recherche" value="Lancer la recherche" type=
                        "submit">
                      </center>
                    </form>
                  </td>
                </tr>
                <tr>
                  <td align="left" valign="top">
                    <?php include("lib/connexion.inc.php");
                    // Adaptation du format
                    function dateFR($daten) {
                    list($yyyy,$mm,$dd)=explode("-",$daten);
                    if (is_numeric($yyyy) && is_numeric($mm) && is_numeric($dd)) {
                    return "$dd/$mm/$yyyy";
                    }
                    }
                    function datentFR($datent) {
                    list($yyyy,$mm,$dd)=explode("-",$datent);
                    if (is_numeric($yyyy) && is_numeric($mm) && is_numeric($dd)) {
                    return "$dd/$mm/$yyyy";
                    }
                    }
                    function datsortFR($datsort) {
                    list($yyyy,$mm,$dd)=explode("-",$datsort);
                    if (is_numeric($yyyy) && is_numeric($mm) && is_numeric($dd)) {
                    return "$dd/$mm/$yyyy";
                    }
                    }
                    function telbe($tel) {
                    if(strlen($tel) ==10){
                    $number = 4;
                    } else {
                    $number = 3;
                    }
                    $tel = substr_replace($tel," ",$number,0);
                    list($pref, $num) = split(' ',$tel);
                    $num = chunk_split ($num, 2);
                    $tel = $pref . " " . $num;
                    return $tel;
                    }
                    // Déclaration des variables
                    $nom = addslashes($_GET['nom']);
                    $prenom = $_GET['prenom'];
                    // Création et envoi de la requête
                    $result = mysql_query("SELECT f.`num_registre`, f.`entreprise`, f.`rue`, f.`code_postal`, f.`ville`, f.`datenaiss`, ROUND(DATEDIFF(CURDATE(),`datenaiss`)/365), f.`villenaiss`, f.`niss`, f.`pays`, f.`nationalite`, f.`etat_civil`, f.`sexe`, f.`tel`, f.`tel2`, f.`gsm`, f.`gsm2`, d.`date_entree`, d.`date_sortie`
                    												  FROM fiche AS f
                    												  INNER JOIN datpro AS d ON f.`num_registre` = d.`num_registre`
                    												  WHERE f.`nom` LIKE '%$nom%' AND f.`prenom` LIKE '%$prenom%'");
                    // Récupération et affichage des résultats
                    while ( $row = mysql_fetch_row($result)){
                    $num = $row[0];
                    $ent = $row[1];
                    $rue = $row[2];
                    $code_postal = $row[3];
                    $ville = $row[4];
                    $daten = $row[5];
                    $age = $row[6];
                    $villen = $row[7];
                    $niss = $row[8];
                    $pays = $row[9];
                    $nation = $row[10];
                    $eciv = $row[11];
                    $sexe = $row[12];
                    $tel = $row[13];
                    $tel2 = $row[14];
                    $gsm = $row[15];
                    $gsm2 = $row[16];
                    $datent = $row[17];
                    $datsort = $row[18];
                    $daten = dateFR($daten);
                    $datent = datentFR($datent);
                    $datsort = datsortFR($datsort);
                    $tel = telbe($tel);
                    $gsm = telbe($gsm);
                    }
                    // Déconnexion de la base de données
                    mysql_close();
                    ?>
                    <table border="0" cellpadding="0" cellspacing="0" height="152" width=
                    "550">
                      <tbody>
                        <tr>
                          <td align="right" width="100">
                            N°, entreprise :
                          </td>
                          <td colspan="1" width="270">
                            <input size="5" name="nom" value=
                            "<?php echo $num ?> ">&nbsp;<input size="18" name="prenom"
                            value="<?php echo $ent ?> ">
                          </td>
                          <td align="right" width="70">
                            Entrée :
                          </td>
                          <td width="110">
                            <input size="7" name="datent" value="<?php echo $datent ?> ">
                          </td>
                        </tr>
                        <tr>
                          <td align="right" width="100">
                            Adresse :
                          </td>
                          <td colspan="1" width="270">
                            <input size="30" name="rue" value="<?php echo $rue ?> ">
                          </td>
                          <td align="right" width="70">
                            Sortie :
                          </td>
                          <td width="110">
                            <input size="7" name="datesort" value="<?php echo $datsort ?> ">
                          </td>
                        </tr>
                        <tr>
                          <td align="right" height="40" valign="top" width="100">
                            CP, ville :
                          </td>
                          <td height="40" valign="top" width="270">
                            <input size="6" name="code_postal" value=
                            "<?php echo $code_postal ?> "> &nbsp;<input value=
                            "<?php echo $ville ?> " name="ville" size="18">
                          </td>
                          <td align="right" width="70"></td>
                          <td width="100"></td>
                        </tr>
                        <tr>
                          <th colspan="2"><center>Naissance</center></th>
                        </tr>
                        <tr>
                          <td align="right" width="100">
                            Date :
                          </td>
                          <td width="270">
                            <input size="8" name="daten" value=
                            "<?php echo $daten ?> ">&nbsp;Age :<input size="2" name="age"
                            value="<?php echo $age ?>">
                          </td>
                          <td colspan="1" rowspan="2" align="right" width="70">
                            Téléphone :
                          </td>
                          <td width="110">
                            <input size="8" name="tel" value="<?php echo $tel ?> ">
                          </td>
                        </tr>
                        <tr>
                          <td align="right" height="40" valign="top" width="100">
                            Lieu :
                          </td>
                          <td height="40" valign="top" width="270">
                            <input size="10" name="villen" value=
                            "<?php echo $villen ?> ">
                          </td>
                          <td width="110">
                            <input size="8" name="tel2" value="<?php echo $tel2 ?> ">
                          </td>
                        </tr>
                        <tr>
                          <td align="right" width="100">
                            Pays, nationalité :
                          </td>
                          <td width="270">
                            <input size="15" name="nation" value=
                            "<?php echo $nation ?> ">&nbsp;<input size="12" name="pays"
                            value="<?php echo $pays ?> ">
                          </td>
                          <td colspan="1" rowspan="2" align="right" width="70">
                            GSM :
                          </td>
                          <td width="110">
                            <input size="10" name="gsm" value="<?php echo $gsm ?> ">
                          </td>
                        </tr>
                        <tr>
                          <td align="right" width="100">
                            Etat civil, sexe :
                          </td>
                          <td width="270">
                            <input size="8" name="sexe" value=
                            "<?php echo $sexe ?> ">&nbsp;<input size="12" name="eciv"
                            value="<?php echo $eciv ?> ">
                          </td>
                          <td width="110">
                            <input size="10" name="gsm2" value="<?php echo $gsm2 ?> ">
                          </td>
                        </tr>
                        <tr>
                          <td align="right" width="100">
                            N° national :
                          </td>
                          <td colspan="1" width="270">
                            <input size="20" name="niss" value="<?php echo $niss ?> ">
                          </td>
                          <td align="right" width="70"></td>
                          <td width="110"></td>
                        </tr>
                      </tbody>
                    </table>
                  </td>
                </tr>
              </tbody>
            </table>
          </td>
        </tr>
      </tbody>
    </table>
  </body>
</html>

**Djakisback** · 17/07/2009, 15h03

Salut,
plusieurs points à mon avis ^^
- Il vaut mieux inclure ton fichier de connexion en haut de page
- Idem déclarer tes fonctions soit en haut de page soit dans un fichier à part
- Ce code est un peu lourd :

Code :

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

1
2
3
4
5
6
7
 while ( $row = mysql_fetch_row($result)){
                    $num = $row[0];
                    $ent = $row[1];
                    $rue = $row[2];
                    $code_postal = $row[3];
                    $ville = $row[4];
                    $daten = $row[5];

Tu peux accéder directement aux noms des champs en utilisant mysql_fetch_assoc() qui te crée un tableau :

Code :

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

1
2
while ( $row = mysql_fetch_assoc($result)){
echo $row['code_postal'];

- Il faut que tu utilises mysql_real_escape_string() pour protéger tes données GET, addslashes() ne suffit pas.

- Il faut que tu testes si ta requête a marché, en général tu fais :

Code :

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

$result = mysql_query(".....") or die('Erreur sql');

- Dans tes fonctions tu peux te passer du list (c'est du détail) :

Code :

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

1
2
3
4
5
6
 function dateFR($daten) {
                    $date = explode("-",$daten);
                    if (is_numeric($date[0]) && is_numeric($date[1]) && is_numeric($date[2])) {
                    return $date[2].'/'.$date[1].'/'.$date[0];
                    }
                    }

**Zarvox** · 20/07/2009, 09h15

Les fonctions on été déportées en haut de page (dans le head).

J'ai modifié le formulaire pour accéder directement au colonnes :

Code :

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

1
2
3
4
5
6
7
8
echo "<table>";
                    while ( $row = mysql_fetch_assoc($result)){
                    echo "<tr>
                    <td>N° :&nbsp<input size='5' name='nom' value='" . $row['num_registre']."'></td>
                    <td>Entreprise :&nbsp;<input size='18' name='prenom' value='".$row['entreprise']."'></td>
                    <td>Date d'entrée :&nbsp;<input size='7' name='datent' value='".$row['date_entree']."'></td></tr>";
                    }
                    echo "</table>";

Je ne comprend pas dès le début le formulaire affiche un enregistrement (toujours le même) de la base mysql alors qu'aucune recherche n'à encore été faite.

- Dans tes fonctions tu peux te passer du list (c'est du détail) :

Pour ce qui est des fonctions c'est un autre qui les as développées moi j'y comprend rien

**Djakisback** · 20/07/2009, 15h13

Envoyé par Zarvox

Les fonctions on été déportées en haut de page (dans le head).

Colle-les avant toute sortie HTML ce sera encore plus propre.

C'est normal que tu aies directement des résultats car tu fais LIKE '%$nom%'
étant donné que $nom est vide ça te donne : LIKE '%%' ce qui correspond à chercher n'importe quel caractère. Ce qui est bizarre c'est que normalement il devrait te sortir tous les enregistrements de la base et non pas un seul.

Pour ne pas lancer ta recherche directement il faut que tu testes sir les variables sont définies. Ex :

Code :

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

1
2
3
4
5
6
7
8
9
if(isset($_GET['nom'], $_GET['prenom'])) {
// là tu mets tout ton code de requête et d'affichage avec entre autres :
 
$result = mysql_query("SELECT f.`num_registre`, f.`entreprise`, f.`rue`, f.`code_postal`, f.`ville`, f.`datenaiss`, ROUND(DATEDIFF(CURDATE(),`datenaiss`)/365), f.`villenaiss`, f.`niss`, f.`pays`, f.`nationalite`, f.`etat_civil`, f.`sexe`, f.`tel`, f.`tel2`, f.`gsm`, f.`gsm2`, d.`date_entree`, d.`date_sortie`
                    												  FROM fiche AS f
                    												  INNER JOIN datpro AS d ON f.`num_registre` = d.`num_registre`
                    												  WHERE f.`nom` LIKE '%".mysql_real_escape_string($_GET['nom'])."%' AND f.`prenom` LIKE '%".mysql_real_escape_string($_GET['prenom'])."%'") or die('Erreur sql');
 
}

**Zarvox** · 23/07/2009, 15h16

Envoyé par Djakisback

Colle-les avant toute sortie HTML ce sera encore plus propre.

C'est fait.

Ce qui est bizarre c'est que normalement il devrait te sortir tous les enregistrements de la base et non pas un seul.

Cela venait de ma méthode d'affichage.

Au lieu de procéder à l'affichage dans le code php je le faisais dans le code HTML ce qui faisait que je n'avait de place que pour un seul enregistrement.

Effectivement lorsque j'ai fait afficher le résultat par php j'ai bien eu tout les enregistrements de la base.

Code :

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

1
2
3
4
5
6
7
8
9
if(isset($_GET['nom'], $_GET['prenom'])) {
// là tu mets tout ton code de requête et d'affichage avec entre autres :
 
$result = mysql_query("SELECT f.`num_registre`, f.`entreprise`, f.`rue`, f.`code_postal`, f.`ville`, f.`datenaiss`, ROUND(DATEDIFF(CURDATE(),`datenaiss`)/365), f.`villenaiss`, f.`niss`, f.`pays`, f.`nationalite`, f.`etat_civil`, f.`sexe`, f.`tel`, f.`tel2`, f.`gsm`, f.`gsm2`, d.`date_entree`, d.`date_sortie`
                    												  FROM fiche AS f
                    												  INNER JOIN datpro AS d ON f.`num_registre` = d.`num_registre`
                    												  WHERE f.`nom` LIKE '%".mysql_real_escape_string($_GET['nom'])."%' AND f.`prenom` LIKE '%".mysql_real_escape_string($_GET['prenom'])."%'") or die('Erreur sql');
 
}

La détection fonctionne

Il n'y a pas d'affichage si mes variables sont vides.

Par contre j'ai un problème pour l'appel de mes fonctions qui me permettent d'afficher la date à la française.

J'ai aussi un probléme avec les noms qui comprennent une apostrophe, quand je lance la recherche je n'ai au mieux que les caractères précédant l'apostrophe.

Optimisation de mon formulaire [MySQL]

PHP & Base de données

Vue hybride

Discussions similaires

Partager

Partager