Discussion :

[niavlys26]

Pour mettre en application ce que je suis en train d'apprendre, j'ai créé ma première classe (historique ).
A ce sujet, déjà deux questions :
Dans cette classe j'ai créé une fonction verifier_utilisateur avec pour argument $var_pseudo qui doit me retourner true si $var_pseudo existe dans la base, ou false dans le cas contraire.

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
 
        function verifier_pseudo( $var_pseudo )
	{
                // La connection à la base est déjà établie
                if( !empty( $var_pseudo ) )
                {
		        $sql = 'SELECT pseudo FROM utilisateurs WHERE pseudo = \'' . $var_pseudo .'\'';
		        $req = mysql_query( $sql );
                        // Autre question
		        $data = mysql_fetch_assoc( $req );
		        if( mysql_num_rows( $req ) > 0 )
		        {
			        return true;
		        }
                }
	}

Alors ma première question est : est-ce que par défaut, si la ligne return true; n'est pas atteinte, ma fonction retournera FALSE ?
La deuxième : J'ai remarqué que si aucun enregistrement ne correspond au pseudo, j'ai des Warning: XXX: supplied argument is not a valid MySQL result resource (XXX étant la fonction fetch_assoc et num_rows.
Dois-je ajouter un if( $req !== false ) {} ?
Merci et bonne journée

[niavlys26]

oups pour la deuxième question je crois que j'ai dit une bêtise... dans ce cas, mysql_query ne me retournait-elle pas NULL et non pas false ?

[Fench]

Salut,

Première question: il faut un return dans tous les cas, c plus propre qd même.
Moi, j'utilise tjs une variable initialisée à true ou false puis modifier en fonction.

Deuxième question: Le pb est que mysql_num_rows peut retourner false, il faut donc tester en premier ce cas.

Voici une ébauche de ta fonction modifié, à terminer.

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
function verifier_pseudo( $var_pseudo ) {
       testPseudo = false;
       if( !empty( $var_pseudo ) ){
            $sql = 'SELECT pseudo FROM utilisateurs WHERE pseudo = \'' . $var_pseudo .'\'';
            $req = mysql_query( $sql );
            $data = mysql_fetch_assoc( $req );
            $nbR = mysql_num_rows( $req );
            if( !$nbR and $nbR>0 ) testPseudo=true;
       }
       return testPseudo; 
}

[niavlys26]

Ok pour tout, merci en effet c'est plus propre

[Herode]

Pour être tout à fait précis :

Code :

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

1
2
3
4
5
6
7
8
function foo() {
}
function foo2() {
    return;
}
 
echo "null ? ", foo() === null ? "oui" : "non", "<br/>";
echo "null ? ", foo2() === null ? "oui" : "non", "<br/>";

Dans les deux cas, la fonction renvoie null 'par défaut'.
Mais en effet, faire des return explicites est conseillé pour la clarté du code

[niavlys26]

Code :

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

1
2
 
if( !$nbR and $nbR>0 ) testPseudo=true;

Oups, Y'a une ch'tite incohérence, je dois plutot mettre

Code :

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

1
2
 
if( !( $nbR == false ) and $nbR>0 ) testPseudo=true;

non ?