[corrigé][Optimisation]fsb2 v. svn80 - caclul inutile des get_url

Grummfy

Equipe des MODS

Age : 25 ans
Messages : 5989
Inscrit : 16 Septembre 2004

Message édité 1 fois, dernière édition par Grummfy, 28 Juillet 2009, 1:55

12 Janvier 2009, 23:10

Hello,
petit optimisation sur la version svn révision 80 de fsb2,

dans le fichier index.php,
trouver (ligne 241 -> 268)

PHP (Sélectionner le code)

		// On recupere les donnees GET de la page
		$get_url = '&amp;redirect=' . $this->frame_page;
		foreach ($_GET AS $key => $value)
		{
			if (!in_array($key, array('p', 'sid', 'redirect')))
			{
				if (is_array($value))
				{
					foreach ($value AS $subvalue)
					{
						if (preg_match('#^[a-z0-9_\-]*?$#i', $subvalue))
						{
							$get_url .= '&amp;' . $key . '[]=' . $subvalue;
						}
					}
				}
				else if (preg_match('#^[a-z0-9_\-]*?$#i', $value))
				{
					$get_url .= '&amp;' . $key . '=' . $value;
				}
			}
		}

		// On peut desactiver la recuperation automatique des donnees GET, avec la propriete $frame_get_url = false
		if (!$this->frame_get_url)
		{
			$get_url = '';
		}

et remplacer par (ou par autre chose c'est de l'optimisation en veux-tu en voila ...)

PHP (Sélectionner le code)

// On peut desactiver la recuperation automatique des donnees GET, avec la propriete $frame_get_url = false
		if (!$this->frame_get_url)
		{
			$get_url = '';
		}
		else
		{
			// On recupere les donnees GET de la page
			$get_url = '&amp;redirect=' . $this->frame_page;
			foreach ($_GET AS $key => $value)
			{
				if (!in_array($key, array('p', 'sid', 'redirect')))
				{
					if (is_array($value))
					{
						foreach ($value AS $subvalue)
						{
							if (preg_match('#^[a-z0-9_\-]*?$#i', $subvalue))
							{
								$get_url .= '&amp;' . $key . '[]=' . $subvalue;
							}
						}
					}
					else if (preg_match('#^[a-z0-9_\-]*?$#i', $value))
					{
						$get_url .= '&amp;' . $key . '=' . $value;
					}
				}
			}
		}

"La gravité est le bonheur des imbéciles" Charles de Montesquieu > "T'as raison, L'apesanteur c'est plus rigolo" Hébus de Phalompe (Troll de Troy)
Mods fsb1 - Mods fsb2 - Grummfy's project - Zf Planet

Talus

Volcan

Age : 22 ans
Messages : 1743
Inscrit : 28 Septembre 2006

12 Janvier 2009, 23:37

Euh... Ce n'est pas tellement de l'optimisation... Si $get est vide, il ne sera pas parcouru... Si ce n'est que pour avoir un $get_url vide si y'a rien, un ternaire suffit...

Talus' Works - Talus' LaBlog

Grummfy

Equipe des MODS

Age : 25 ans
Messages : 5989
Inscrit : 16 Septembre 2004

12 Janvier 2009, 23:39

sauf que si tu désactives le truc (avec un false), il va charger (et donc calculer des trucs) le get puis le vider ... donc des manipulation pour rien

"La gravité est le bonheur des imbéciles" Charles de Montesquieu > "T'as raison, L'apesanteur c'est plus rigolo" Hébus de Phalompe (Troll de Troy)
Mods fsb1 - Mods fsb2 - Grummfy's project - Zf Planet

Talus

Volcan

Age : 22 ans
Messages : 1743
Inscrit : 28 Septembre 2006

13 Janvier 2009, 0:34

Calculer rien, oui... Puisque $_GET est, de toutes manières, défini. Donc, pas défaut, $_GET = array().. hors, parcourir un array vide, ca revient strictement à la même chose que rien parcourir. Limite, ton if, il prend plus de perf qu'avec le comportement par défaut...

Talus' Works - Talus' LaBlog

Grummfy

Equipe des MODS

Age : 25 ans
Messages : 5989
Inscrit : 16 Septembre 2004

13 Janvier 2009, 0:58

dans tout les cas on a un if (avant et après)

comme dit ici ... [code]// On peut desactiver la recuperation automatique des donnees GET, avec la propriete $frame_get_url = false[/code
donc si je désactive cela via un $this->frame_get_url = false, il va m'analyser le $_GET (on suppose que le get contient quelque chose, sinon de toute manière la question ne se pose pas ...), avec l'ancienne version, puis me mettre tout à zéro, bref tout le travail pour rien ... avec la nouvelle version il ne travail que si nécessaire ...

"La gravité est le bonheur des imbéciles" Charles de Montesquieu > "T'as raison, L'apesanteur c'est plus rigolo" Hébus de Phalompe (Troll de Troy)
Mods fsb1 - Mods fsb2 - Grummfy's project - Zf Planet

mob

Equipe de support

Age : 19 ans
Messages : 2016
Inscrit : 10 Septembre 2005

13 Janvier 2009, 2:44

Citation (Talus, 12 Janvier 2009, 23:37)

Euh... Ce n'est pas tellement de l'optimisation... Si $get est vide, il ne sera pas parcouru...

Mais si $_GET n'est pas vide, et frame_get_url vaut FALSE, $_GET sera analysé, au lieu de mettre directement = ''. Mais pas avec la version de Grummfy.

En parlant de la version sur le SVN, dans le fichier Captcha_gd.php, il y a une petite erreur de frappe qui empêche l'affichage du captcha à l'inscription.
imagetruecolor() est écrit image true color().

Anciennement B.Moncef.

Grummfy

Equipe des MODS

Age : 25 ans
Messages : 5989
Inscrit : 16 Septembre 2004

13 Janvier 2009, 2:48

je pense que c'est un problème due au dépot svn et non à fsb2 mais à confirmer ...

"La gravité est le bonheur des imbéciles" Charles de Montesquieu > "T'as raison, L'apesanteur c'est plus rigolo" Hébus de Phalompe (Troll de Troy)
Mods fsb1 - Mods fsb2 - Grummfy's project - Zf Planet

BlackJowy

Equipe de support

Age : 3008 ans
Messages : 4038
Inscrit : 01 Mars 2007

03 Mars 2010, 0:05

Sujet archivé pour la raison suivante : Bug corrigé dans la version en développement.

[corrigé][Optimisation]fsb2 v. svn80 - caclul inutile des get_url