Discussion :

[AyManoVic]

salut tout le monde
cette requête prend du temps pour afficher les topics dans mon forums
je me demande si je peux l'optimiser et rendre l'execution plus rapide

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
function topics($name, $id){
	$sql = mysql_query("SELECT * FROM ".prefix."TOPICS WHERE TOPIC_ID = '".intval($id)."' ") or die(mysql_error());
	if(mysql_num_rows($sql)>0){
		$rs = mysql_fetch_array($sql);
		$topics = Array(
			$rs[CAT_ID], $rs[FORUM_ID], $rs[T_STATUS], $rs[T_SUBJECT],
			$rs[T_MESSAGE], $rs[T_REPLIES], $rs[T_COUNTS], $rs[T_AUTHOR],
			$rs[T_DATE], $rs[T_LAST_POST_AUTHOR], $rs[T_LAST_POST_DATE], $rs[T_ARCHIVE_FLAG],
			$rs[T_STICKY], $rs[T_HIDDEN], $rs[T_TOP], $rs[T_LINKFORUM],
			$rs[T_LASTEDIT_DATE], $rs[T_LASTEDIT_MAKE], $rs[T_LOCK_DATE], $rs[T_LOCK_MAKE],
			$rs[T_OPEN_DATE], $rs[T_OPEN_MAKE], $rs[T_ENUM], $rs[T_SURVEYID],
			$rs[T_UNMODERATED], $rs[T_MODERATED_BY], $rs[T_MODERATED_DATE], 
			$rs[T_HOLDED], $rs[T_HOLDED_BY], $rs[T_HOLDED_DATE], 
			$rs[T_MOVED], $rs[T_MOVED_BY], $rs[T_MOVED_DATE], 
			$rs[T_HIDDEN_BY], $rs[T_HIDDEN_DATE], 
			$rs[T_DELETED_BY], $rs[T_DELETED_DATE], $rs[T_ARCHIVED], $rs[T_UNHIDDEN_BY], $rs[T_UNHIDDEN_DATE],
			$rs[T_STICKY_BY], $rs[T_STICKY_DATE], $rs[T_UNSTICKY_BY], $rs[T_UNSTICKY_DATE]
		);
	}
	if ($name == "CAT_ID"){$nom = 0;}
	if ($name == "FORUM_ID"){$nom = 1;}
	if ($name == "STATUS"){$nom = 2;}
	if ($name == "SUBJECT"){$nom = 3;}
	if ($name == "MESSAGE"){$nom = 4;}
	if ($name == "REPLIES"){$nom = 5;}
	if ($name == "COUNTS"){$nom = 6;}
	if ($name == "AUTHOR"){$nom = 7;}
	if ($name == "DATE"){$nom = 8;}
	if ($name == "LAST_POST_AUTHOR"){$nom = 9;}
	if ($name == "LAST_POST_DATE"){$nom = 10;}
	if ($name == "ARCHIVE_FLAG"){$nom = 11;}
	if ($name == "STICKY"){$nom = 12;}
	if ($name == "HIDDEN"){$nom = 13;}
	if ($name == "TOP"){$nom = 14;}
	if ($name == "LINKFORUM"){$nom = 15;}
	if ($name == "LASTEDIT_DATE"){$nom = 16;}
	if ($name == "LASTEDIT_MAKE"){$nom = 17;}
	if ($name == "LOCK_DATE"){$nom = 18;}
	if ($name == "LOCK_MAKE"){$nom = 19;}
	if ($name == "OPEN_DATE"){$nom = 20;}
	if ($name == "OPEN_MAKE"){$nom = 21;}
	if ($name == "ENUM"){$nom = 22;}
	if ($name == "SURVEYID"){$nom = 23;}
	if ($name == "UNMODERATED"){$nom = 24;}
	if ($name == "MODERATED_BY"){$nom = 25;}
	if ($name == "MODERATED_DATE"){$nom = 26;}
	if ($name == "HOLDED"){$nom = 27;}
	if ($name == "HOLDED_BY"){$nom = 28;}
	if ($name == "HOLDED_DATE"){$nom = 29;}
	if ($name == "MOVED"){$nom = 30;}
	if ($name == "MOVED_BY"){$nom = 31;}
	if ($name == "MOVED_DATE"){$nom = 32;}
	if ($name == "HIDDEN_BY"){$nom = 33;}
	if ($name == "HIDDEN_DATE"){$nom = 34;}
	if ($name == "DELETED_BY"){$nom = 35;}
	if ($name == "DELETED_DATE"){$nom = 36;}
	if ($name == "ARCHIVED"){$nom = 37;}
	if ($name == "UNHIDDEN_BY"){$nom = 38;}
	if ($name == "UNHIDDEN_DATE"){$nom = 39;}
	if ($name == "STICKY_BY"){$nom = 40;}
	if ($name == "STICKY_DATE"){$nom = 41;}
	if ($name == "UNSTICKY_BY"){$nom = 40;}
	if ($name == "UNSTICKY_DATE"){$nom = 41;}
 
	return($topics[$nom]);
}

je dois selectionner tous les champs de la table TOPICs donc select * est raisonnable
merci

[sabotage]

Meme si ce n'est pas ça qui va doubler la vitesse utilise if/elseif ou switch{} pour tes conditions ; cette suite de if est vraiment une mocheté.

Pourquoi créés-tu un gros tableau pour au final ne retourner qu'un seul élément ?
D'ailleurs a quoi sert cette fonction ? tu dis avoir besoin de tous les champs mais si j'ai les yeux en face des trous, tu n'en utilises qu'un seul.

La requête en elle même ne doit prendre que quelques millisecondes. Est-ce que tu ne la repeterais pas des centaines de fois ?

[AyManoVic]

Creer un gros tableau pour en retourner une seule valeur

Au fait c'est sur ce tableau qu'agissent les conditions (la moche suite de if que je viens de transformer en switch :p)

cette fonction aide a lister les topic d'un certain forum

sinon une suggestion ?

[sabotage]

Pourquoi la fonction ne retourne qu'un seul champ ?
Tu appelles la fonction et donc interroge la base pour lire chacune des x valeurs dont tu as besoin ?

[AyManoVic]

Pourquoi la fonction ne retourne qu'un seul champ ?
?

en effet c'est ça mon probleme
à chaque $nom la requête se refait c'est pour ça je demande une possibilité d'optimisation
la fonction je l'appele dans un autre fichier quand j'ai besoin par exemple d'afficher le titre du topic je fais

Code :

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

$subject = topics("SUBJECT", $t);

vous me saisissez mantenant ?

[sabotage]

Récupère tous les champs d'un coup.

[AyManoVic]

vous voulez dire quoi ?
je vous ai pas compris :s

[sabotage]

Actuellement tu lances une requête chaque fois que tu as besoin d'une donnée.
Je te propose de recuperer toutes les données d'un topic d'un coup :

Code :

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

1
2
3
4
5
6
7
8
9
function topics($id){
	$result = mysql_query("SELECT * FROM ".prefix."TOPICS WHERE TOPIC_ID = ".intval($id));
       $row = mysql_fetch_assoc($result);
       return $row;
}
 
$infotopic = topics(6);
 
echo 'date du message : ' . $infotopic['T_DATE'];

[AyManoVic]

Bonne idée , Merci beaucoup Sabotage ^^