Optimiser du code

**Mzr** · 09/07/2013, 11h39

Bonjour à tous,

Voilà, j'ai écris une page de code (qui vient en appuie d'un forum phpbb) et le script mets trop longtemps à s'exécuter et le serveur l'envoie des fois dans sa sandbox...

En gros, le script va lire la BDD en plusieurs requêtes pour créer un tableau en HTML. Quand il y a 500 références, ça va, mais dès qu'on arrive aux alentours de 1000 réfs, les 20 secondes du timeout claquent souvent...

Voici mon code (surement très crade, mais j'ai fait suivant ma vieille logique du langage C, enfin, de ce qui m'en reste

) :

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
255
256
257
258
259
260
261
262
<?php
 
//$UserId = $_GET["UserId"];
//echo ($UserId);
 
define('IN_PHPBB', true);
 
$phpbb_root_path = (defined('PHPBB_ROOT_PATH')) ? PHPBB_ROOT_PATH : './';
 
$phpEx = substr(strrchr(__FILE__, '.'), 1);
 
$_GET = array();
 
include($phpbb_root_path . 'common.' . $phpEx);
 
 
 
// Start session management
$user->session_begin();
$auth->acl($user->data);
$user->setup();
 
$Pseudo = $user->data['username'];
$UserIDlook = $user->data['user_id'];
if ($user->data['user_id'] == ANONYMOUS)
{
    login_box('', $user->lang['LOGIN']);
} 
 
page_header('Les fiches perso de la Bible');
 
 
$template->set_filenames(array(
    'body' => 'JeLai.html',
 
));
 
//début affichage
 
 
//récupération des ID des forums enfants
mysql_query("SET NAMES UTF8"); 
 
$countTotal=0;
 
 
if($UserId<>"")
{
// Récupération des Thanks avec l'ID du user en argument
$requete = "SELECT thanks_from, post_id FROM  `phpbb3_thanks` WHERE  `thanks_from` =$UserId";
 
$Resultat = mysql_query($requete);
 
$count = 0;
 
$table2[] = array('ForumType' => "Forum/Type", 'Modele' => "Modele", 'Lien' => "Lien");
 
while ($row = mysql_fetch_array($Resultat))
{
	$count++;
 
// Sélection des infos de la fiche du thanks
$requeteF = "SELECT Bible_Ref, Bible_Fab, Bible_Ech, Bible_Photo, post_subject
FROM `phpbb3_posts` 
WHERE `post_id` =" . $row[post_id];
$ResultatF = mysql_query($requeteF);
$rowF = mysql_fetch_array($ResultatF);
 
// on remplie le tableau
$table2[$count] = array($count,$rowF[Bible_Ech],$rowF[post_subject],"http://forum.site.fr/viewtopic.php?p=" . $row[post_id] . "#p" . $row[post_id],$rowF[Bible_Fab],$rowF[Bible_Ref],$rowF[Bible_Photo]);
}
 
// suivant le tri en argument, on construit l'entete du tableau HTML
switch ($Sort)
{
	case "Echelle":
		$dataBible=$dataBible."<div class=\"ClassBibleEntete\">
  		<table width=\"100%\" border=\"0\">
   		 <tr>
      <td width=\"40\">#</td><td width=\"63\">
	  <font color=\"#f86900\">Echelle</font><a></td><td width=\"28%\"><a href=\"http://forum.site.fr/JeLai.php?UserId=" . $UserId . "&Sort=Modele\">Mod&egrave;le</a>
	  </a></td><td><a href=\"http://forum.site.fr/JeLai.php?UserId=" . $UserId . "&Sort=Fabricant\">Fabricant</a></td><td width=\"38%\">R&eacute;f&eacute;rence</td><td width=\"50\">Photo</td></tr></table></div>";
 
		break;
	case "Modele":
		$dataBible=$dataBible."<div class=\"ClassBibleEntete\">
  		<table width=\"100%\" border=\"0\">
   		 <tr>
      <td width=\"40\">#</td><td width=\"63\"><a href=\"http://forum.site.fr/JeLai.php?UserId=" . $UserId . "&Sort=Echelle\">Echelle</a><a></td><td width=\"28%\">
	  <font color=\"#f86900\">Mod&egrave;le</font></a></td><td><a href=\"http://forum.site.fr/JeLai.php?UserId=" . $UserId . "&Sort=Fabricant\">Fabricant</a></td><td width=\"38%\">R&eacute;f&eacute;rence</td><td width=\"50\">Photo</td></tr></table></div>";
		break;
 
		case "Fabricant":
		$dataBible=$dataBible."<div class=\"ClassBibleEntete\">
  		<table width=\"100%\" border=\"0\">
   		 <tr>
      <td width=\"40\">#</td><td width=\"63\"><a href=\"http://forum.site.fr/JeLai.php?UserId=" . $UserId . "&Sort=Echelle\">Echelle</a><a></td><td width=\"28%\"><a href=\"http://forum.site.fr/JeLai.php?UserId=" . $UserId . "&Sort=Modele\">Mod&egrave;le</a></a></td><td><font color=\"#f86900\">Fabricant</font></td><td width=\"38%\">R&eacute;f&eacute;rence</td><td width=\"50\">Photo</td></tr></table></div>";
		break;
 
 
		default:
		$dataBible=$dataBible."<div class=\"ClassBibleEntete\">
  		<table width=\"100%\" border=\"0\">
   		 <tr>
      <td width=\"40\">#</td><td width=\"63\"><a href=\"http://forum.site.fr/JeLai.php?UserId=" . $UserId . "&Sort=Echelle\">Echelle</a><a></td><td width=\"28%\"><font color=\"#f86900\">Mod&egrave;le</font></a></td><td><a href=\"http://forum.site.fr/JeLai.php?UserId=" . $UserId . "&Sort=Fabricant\">Fabricant</a></td><td width=\"38%\">R&eacute;f&eacute;rence</td><td width=\"50\">Photo</td></tr></table></div>";
 
}
 
// Obtient une liste de colonnes que l'on va trier
 
switch ($Sort)
{
	case "Echelle":
		foreach ($table2 as $key => $row)
		{
    		$rang[$key]  = $row[1];
 
		}
		break;
	case "Modele":
		foreach ($table2 as $key => $row)
		{
    		$rang[$key]  = $row[2];
 
		}
		break;
	case "Fabricant":
		foreach ($table2 as $key => $row)
		{
    		$rang[$key]  = $row[4];
 
		}
		break;
 
 
		default:
		foreach ($table2 as $key => $row)
		{
    		$rang[$key]  = $row[2];
		}
}
 
 
// Trie les données par rang croissant
// Ajoute $array en tant que dernier paramètre, pour trier par la clé commune
array_multisort($rang, SORT_ASC, $table2);
 
// on poursuit le tableau en alternant les class 1 et 2 
for ($counter=1; $counter<$count+1; $counter++)
{
	//echo $count . "<br>"; echo $TableauFinal[$count];
	if($counter&1)
	{
		$dataBible=$dataBible."<div class=\"ClassBible1\" onmouseover=\"this.style.background='#F8B600'; this.style.color='white'; \" onmouseout=\"this.style.background='#EEEEEE'; this.style.color='black';\" onClick='window.open(\"" . $table2[$counter][3]. "\")'>
  <table width=\"100%\" border=\"0\">
    <tr>
      <td width=\"40\">" . $counter . "</td><td width=\"63\">" . $table2[$counter][1] . "</td><td width=\"28%\">" . $table2[$counter][2] . "</td><td>" . $table2[$counter][4] . "</td><td width=\"38%\">" . $table2[$counter][5] . "</td><td width=\"50\"><img src=\"" . $table2[$counter][6] . "\">
	  </td></tr></table></div>";
	}
	else
	{
		$dataBible=$dataBible."<div class=\"ClassBible2\" onmouseover=\"this.style.background='#F8B600'; this.style.color='white'; \" onmouseout=\"this.style.background='#FFFFFF'; this.style.color='black';\" onClick='window.open(\"" . $table2[$counter][3]. "\")'>
  <table width=\"100%\" border=\"0\">
    <tr>
      <td width=\"40\">" . $counter . "</td><td width=\"63\">" . $table2[$counter][1] . "</td><td width=\"28%\">" . $table2[$counter][2] . "</td><td>" . $table2[$counter][4] . "</td><td width=\"38%\">" . $table2[$counter][5] . "</td><td width=\"50\"><img src=\"" . $table2[$counter][6] . "\">
	  </td></tr></table></div>";
	}
 
}
 
$dataBible=$dataBible."</table>";
 
//On liste les users pour ré utiliser les infos
$requeteName = "SELECT username, username_clean FROM `phpbb3_users` WHERE `user_id` = $UserId";
$ResultatName = mysql_query($requeteName);
if($ResultatName<>0)
{
	$row4 = mysql_fetch_array($ResultatName);
}
 
 
	if($count>0)
	{
		//$dataBible=$dataBible."<br><br><strong>" . $count . "</strong> fiches ont été trouvées";
	}
	else
	{
		$dataBible=$dataBible."<strong><br><br>Aucune fiche de coch&eacute;e :-(</strong>";
	}
}
 
$countUsers = 0;
 
//On construit le menu déroulant des users
$requeteUsers = "SELECT username, user_id, username_clean FROM `phpbb3_users` WHERE user_thanks != 0";
$ResultatUsers = mysql_query($requeteUsers);
 
while ($rowUsers = mysql_fetch_array($ResultatUsers))
{
 
	$tableUsers[$countUsers] = array($rowUsers[user_id], $rowUsers[username], $rowUsers[username_clean]);
	$countUsers++;
}
 
// on tri les users
unset($rang);
foreach ($tableUsers as $keyBis => $rowBis)
		{
    		$rang[$keyBis]  = $rowBis[2];
		}
//array_multisort($rang, SORT_ASC, $tableUsers2);
array_multisort($rang, SORT_ASC, $tableUsers);
//print_r($tableUsers);
/*echo"<br><br>";
print_r($tableUsers2);*/
 
// on construit le menu en HTML
$dataFormUsers = $dataFormUsers . "<form action=\"http://forum.site.fr/JeLai.php\" method=\"get\" name=\"formusers\" target=\"_self\" id=\"formusers\">
  <label>
  <select name=\"UserId\" size=\"1\">";
 
for ($counter=1; $counter<$countUsers+1; $counter++)
{ 
    $dataFormUsers = $dataFormUsers . "<option value=\"" . $tableUsers[$counter][0] . "\"";
 
	if($tableUsers[$counter][0] == $UserId )
	{
		$dataFormUsers = $dataFormUsers . " selected=\"selected\"";
	}
	$dataFormUsers = $dataFormUsers . ">" . $tableUsers[$counter][1] . "</option>";
}
$dataFormUsers = $dataFormUsers . "  </select>
  </label>
  <input type=\"submit\" value=\"Ok\">
</form>";
 
 
$LienProfil = "<a href=\"http://forum.site.fr/memberlist.php?mode=viewprofile&u=" . $UserId . "\"><u>" . $row4[username] . "</u></a>";
 
if($UserId != "")
{
	$titre = "Les <strong><u>" . $count . "</u></strong> fiches coch&eacute;es par ";
}
else
{
	$titre = "Merci de s&eacute;lectionner un utilisateur -->";
}
 
// On envoie les infos au template pour affichage
$template->assign_vars(array(
    'USERID'     => $UserId,
	'PSEUDO'     => $LienProfil,
	'DATA'		 => $dataBible,
	'COUNT'		 => $titre,
	'FORMUSERS'  => $dataFormUsers,
));
 
 
 
//make_jumpbox(append_sid("{$phpbb_root_path}viewforum.$phpEx"));
page_footer();
?>

**Celira** · 09/07/2013, 12h02

Tu fais une requête, une boucle sur cette requête et une requête à un seul résultat par tour de boucle. Plus loin, tu fais un tri manuel sur des éléments que tu as récupéré dans ta requête

Autant faire une seule requête avec une jointure, un tri en base, et une seule boucle dessus

**Mzr** · 09/07/2013, 12h34

Oui, quand j'ai écrit mes requêtes, je savais que je pourrais faire une double requete dans une seule (jointure ?) mais même si j'ai regardé quelques tuto, j'ai des difficultés à extraire les résultats que je veux surune seule requêtes...

**Celira** · 09/07/2013, 13h45

Envoyé par Celira

une seule requête avec une jointure, un tri en base, et une seule boucle dessus

Comme ceci :

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
$requete = "SELECT thanks_from, Bible_Ref, Bible_Fab, Bible_Ech, Bible_Photo, post_subject, post_id
FROM  `phpbb3_thanks` th join `phpbb3_posts` po on th.post_id = po.post_id
WHERE  `thanks_from` = $UserId ";
switch ($Sort)
{
	case "Echelle":
		$Requete .= " ORDER BY Bible_Ech";
		break;
	case "Modele":
		$Requete .= " ORDER BY post_subject";
		break;
	case "Fabricant":
		$Requete .= " ORDER BY Bible_Fab";
		break;
	default:
		$Requete .= " ORDER BY post_subject";
}
 
$Resultat = mysql_query($requete);
 
$count = 0;
$table2[] = array('ForumType' => "Forum/Type", 'Modele' => "Modele", 'Lien' => "Lien");
 
while ($row = mysql_fetch_array($Resultat))
{
	$count++;
	// on remplie le tableau
	$table2[$count] = array($count,$row['Bible_Ech'],$row['post_subject]', "http://forum.site.fr/viewtopic.php?p=" . $row['post_id'] . "#p" . $row['post_id'],$row['Bible_Fab'],$row['Bible_Ref'],$row['Bible_Photo']);
}

**Mzr** · 09/07/2013, 15h25

Visiblement, mysql n'apprécie pas.

Ma requête générée est donc :

Code sql :

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

SELECT thanks_from, Bible_Ref, Bible_Fab, Bible_Ech, Bible_Photo, post_subject, post_id FROM `phpbb3_thanks` th join `phpbb3_posts` po on th.post_id = po.post_id WHERE `thanks_from` = 21 ORDER BY post_subject

Et il me répond :

debug : #1052 - Column 'post_id' in field list is ambiguous

J'ai re regardé, et d'après les tuto (http://sqlpro.developpez.com/cours/sqlaz/jointures/) et ça parait correct...

Code sql :

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

1
2
3
4
SELECT C.CLI_ID, C.CLI_NOM, T.TEL_NUMERO
FROM T_CLIENT C, T_TELEPHONE T
WHERE C.CLI_ID = T.CLI_ID
  AND T.TYP_CODE = 'FAX'

**Celira** · 09/07/2013, 15h36

Ah oui, il faut préciser dans quelle table il va chercher post_id, vu que les 2 tables ont une colonne qui porte ce nom (le fait qu'on fasse une jointure sur la colonne en question n'est visiblement pas pris en compte dans la syntaxe

)

Code sql :

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

1
2
3
SELECT thanks_from, Bible_Ref, Bible_Fab, Bible_Ech, Bible_Photo, post_subject, th.post_id 
FROM `phpbb3_thanks` th join `phpbb3_posts` po on th.post_id = po.post_id 
WHERE `thanks_from` = 21 ORDER BY post_subject

**Mzr** · 09/07/2013, 16h08

Ca fonctionne ! merci ! J'aurai un exemple perso sur les jointures.

Par contre, pas de magie, je me trouve toujours botté en touche...

La raison semble être un CPU à 100% pendant 15 à 20 secondes...

La requête SQL étant maintenant optimisée, n'est-ce pas mon code php générant la page HTMM qui poserait souci ?

**Celira** · 09/07/2013, 16h56

Hum... le problème est peut-être le concept : tu construis un tableau HTML qui contient l'intégralité des données de la requête. Si tu as 1000 lignes retournées, ça fait un tableau HTML à 1000 lignes. ça fait quand même un gros tableau.

Tu as pensé à virer le morceau qui faisait le tri de $table2 (vu que maintenant on le fait dans la requête) ?

**Mzr** · 09/07/2013, 17h38

Oui, j'ai bien fait le message...

A mon avis, le souci vient de la variable $dataBible qui doit conserver le tableau...

Je suis obligé de constituer le tableau pour le passer au template

Reste à faire des pages et limiter chaque page à 500 lignes... Mais ça ne me plait pas...

**Celira** · 09/07/2013, 18h13

Autre possibilité : opter pour un template intelligent qui sait fabriquer les tableaux tout seul.
Auquel cas, tu n'aurais à transférer que les données de la requête, ce qui devrait réduire un peu le problème.