Nouveau sujet Répondre Imprimer Syndication RSS 2.0

[Optimisation]test email//login déjà pris

Equipe des MODS
Avatar de Grummfy
  • Age : 23 ans
  • Messages : 5458
  • Inscrit : 16 Septembre 2004
  Lien vers ce message 17 Aout 2006, 21:24
hello,
voici une petit optimisation...
il y a moyen de supprimer une requete lors de l'inscription :
dans ./profil.php
chercher
	// Le pseudo entré existe t'il déjà?
	if ( $mode == 'nouveau' || ( $mode == 'edite' && $login != $membre['membre_login'] && $config['editer_login'] ) )
	{
		$sql = 'SELECT membre_login from ' . MEMBRES_TABLE . ' 
				WHERE membre_login = \'' . $login . '\'';
		if ( ! $result = $bdd->requete($sql) )
		{
			info($langue['er_titre'], sprintf($langue['er_select'], 'membres'), __LINE__, __FILE__, $sql);
		}
		$check_login = $bdd->tableau($result);

		if ($check_login['membre_login'] != '')
		{
			$erreur .= $langue['er_login_pris'] . '<br />';
		}
	}

	// l'adresse email entrée existe t'elle déjà?
	if ( $mode == 'nouveau' || ( $mode == 'edite' && $email != $membre['membre_email'] ) )
	{
		$sql = 'SELECT membre_email from ' . MEMBRES_TABLE . ' 
				WHERE membre_email = \'' . $email . '\'';
		if ( ! $result = $bdd->requete($sql) )
		{
			info($langue['er_titre'], sprintf($langue['er_select'], 'membres'), __LINE__, __FILE__, $sql);
		}
		$stat = $bdd->tableau($result);

		if ( $stat['membre_email'] != '' && ! $erreur_mail )
		{
			$erreur .= $langue['er_email_faux'] . '<br />';
		}
	}

et remplacer par :
	// Le pseudo entré existe t'il déjà?
	// l'adresse email entrée existe t'elle déjà?
	if ( $mode == 'nouveau' || ( $mode == 'edite' && (($login != $membre['membre_login'] && $config['editer_login']) || ($email != $membre['membre_email'])) ))
	{
		$sql = 'SELECT membre_login, membre_email from ' . MEMBRES_TABLE . ' 
				WHERE membre_login = \'' . $login . '\' OR membre_email = \'' . $email . '\'';
		if ( ! $result = $bdd->requete($sql) )
		{
			info($langue['er_titre'], sprintf($langue['er_select'], 'membres'), __LINE__, __FILE__, $sql);
		}
		$check_login = $bdd->tableau($result);

		if ($check_login['membre_login'] != '' && ($mode == 'nouveau' || ( $mode == 'edite' && $login != $membre['membre_login'])))
		{
			$erreur .= $langue['er_login_pris'] . '<br />';
		}
		if ( $check_login['membre_email'] != '' && !$erreur_mail && ( $mode == 'nouveau' || ($mode == 'edite' && $email != $membre['membre_email'])) )
		{
			$erreur .= $langue['er_email_faux'] . '<br />';
		}
	}

remarque faite sur fsb 1.0.6a
voila
a+


"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)
http://www.grummfy.com/download/crea/smileys/fsb2.gif Mods fsb1 - Mods fsb2 - Mon forum - Grummfy's project
 
VIP
Avatar de Dravick
  • Age : 19 ans
  • Messages : 523
  • Inscrit : 04 Mars 2005
  Lien vers ce message 18 Aout 2006, 17:24
Le hic c'est qu'on a pas vraiment besoin d'optimiser le code pour "l'inscription", étant donné que c'est une étape qui arrive une fois, puis plus jamais (pour chaque membre). Ce n'est pas comme une optimisation pour par exemple récupérer les forums ou les groupes, qui se font pratiquement à chaque chargement de page.

Dans mon humble avis, ça n'apporte pas vraiment d'amélioration, et ça peut "screwer" les recherches de certains MODs.

-- Dravick


"Take thy beak from out my heart, and take thy form from off my door!"
[list]Quoth the Raven, "Nevermore."[/list]
 
Equipe des MODS
Avatar de Grummfy
  • Age : 23 ans
  • Messages : 5458
  • Inscrit : 16 Septembre 2004
  Lien vers ce message 19 Aout 2006, 10:35
je sais cependant personellement, je voix que c'est aussi en cas d'édition ...

mais bon c'est pas grave optimise qui veux ....


"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)
http://www.grummfy.com/download/crea/smileys/fsb2.gif Mods fsb1 - Mods fsb2 - Mon forum - Grummfy's project
 
Répondre


.