Discussion :

[s0ak68]

Bonjour, j'aimerais optimiser ma fonction, mais je ne vois pas trop comment..
Si vous avez des suggestions, je vous en remercie d'avance.

Return 0 : non lu
Return 1 : lu

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
private function ForumLu($id)
{
	if(isset($_SESSION['logged']) && $_SESSION['logged']===true)
	{	
		$sql = 'SELECT * FROM `s_sujet` 
		LEFT JOIN `s_membres` ON `mbm_id`='.$_SESSION['id'].'
		WHERE `s_f`='.$id.'';
		$req = mysql_query($sql) or die('Erreur SQL : '.$sql.'<br />'.mysql_error());
		while ($res = mysql_fetch_object($req)) {
 
			if($res->mbm_inscrit < $res->s_date)
			{
				$sql2 = 'SELECT * FROM `s_lu` WHERE `lu_auteur`='.$_SESSION['id'].' AND `lu_s`='.$res->s_id.'';
				$req2 = mysql_query($sql2) or die('Erreur SQL : '.$sql2.'<br />'.mysql_error());
				$res2 = mysql_fetch_object($req2);
 
				if($res2->lu_mess != $res->s_mess)
				{
					return 0;
				} 
			}
			else	
			{
				return 1;
			}
		}
		return 1;
	}
	else
	{
		return 1;
	}
}

[Grepsd]

Bonjour,

pour t'aider il me faudra la description des champs, tables et de ce qu'elles doivent faire, ainsi que la fonction principale de ta fonction x)

Ensuite, pour optimiser tu pourrais déjà commencer par ne sélectionner dans tes requêtes SQL que les champs dont tu vas te servir et non utiliser * pour récupérer tout les champs.

De plus tu fais une joindre avec une condition ON assez étrange puisque que tu ne spécifie qu'un nom de champs et non table.champs. Ce genre de chose est souvent source d'erreurs, fais bien attention à cela.

Puis au niveau du code:

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
 
private function ForumLu($id)
{
	if(isset($_SESSION['logged']) && $_SESSION['logged']===true) {	
		$sql = 'SELECT * FROM `s_sujet` 
		LEFT JOIN `s_membres` ON `mbm_id`='.$_SESSION['id'].'
		WHERE `s_f`='.$id.'';
		$req = mysql_query($sql) or die('Erreur SQL : '.$sql.'<br />'.mysql_error());
		while ($res = mysql_fetch_object($req)) {
 
			if ($res->mbm_inscrit < $res->s_date) {
				$sql2 = 'SELECT * FROM `s_lu` WHERE `lu_auteur`='.$_SESSION['id'].' AND `lu_s`='.$res->s_id.'';
				$req2 = mysql_query($sql2) or die('Erreur SQL : '.$sql2.'<br />'.mysql_error());
				$res2 = mysql_fetch_object($req2);
 
				if($res2->lu_mess != $res->s_mess)
					return false;
			}
		}
	}
	return true;
}

Tu peux faire ainsi et donc t'éviter trop de ligne de conditions qui ne servent à rien.