Pseudo existe déjà ?

**jejesochalion** · 28/08/2006, 21h54

Bonjour a tous, j'espere que j'ai choisi la bonne rubrique pour le sujet.
Voici le code qui me permettrai de ne pas avoir deux fois le meme pseudo sur mon site mais il y a un probleme que e ne trouve pas.Pouvez vous m'aider?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
$sql = mysql_query("SELECT COUNT(*) AS nb_pseudo FROM pseudo WHERE pseudo='".$pseudo."'");
                                if ($sql == 0)
                                {mysql_select_db("info_perso");
 
								// On ajoute une entrée avec mysql_query
								mysql_query("INSERT INTO pseudo VALUES('', '$pseudo', '$password', '$email')");
 
								mysql_close();
								}
                               else
                                   echo 'Erreur : le pseudo existe déjà !';

Merci a tous

**bkill** · 28/08/2006, 22h04

Salut,

hmmm, je crois qu'il y a un petit problème de compréhension par rapport à ce que font les fonctions que tu tentes d'utiliser...

Code :

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

1
2
3
4
 
$sql = mysql_query("SELECT COUNT(*) AS nb_pseudo FROM pseudo WHERE pseudo='".$pseudo."'");
if ($sql == 0)
{mysql_select_db("info_perso");

Dans cette petite partie déjà...
- Pourquoi faire le mysql_select_db après avoir exécuté la requête?
- $sql = mysql_query(...); cela te retourne une ressource mysql, tu ne peux donc pas la comparer avec "0" comme tu le fais ici.

Voila une façon de faire, corrigée:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
$conn = mysql_connect( ... ) or die ("Echec de connexion");
mysql_select_db('info_perso');
$sql = "SELECT * FROM pseudo WHERE pseudo = '" . $pseudo .'";
$res = mysql_query($sql) or die ("Erreur SQL : " . mysql_error());
if (mysql_num_rows($res)) {
  echo "pseudo existant!";
}
else {
  // code pour l'insertion...
}

C'est une manière de procéder.

**jejesochalion** · 29/08/2006, 13h36

J'ai fais ca et ca ne marche pas:

Code :

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

1
2
3
4
5
6
7
8
9
10
mysql_select_db('info_perso');
$sql = "SELECT * FROM pseudo WHERE pseudo = ' . $pseudo .'";
$res = mysql_query($sql) or die ("Erreur SQL : " . mysql_error());
if (mysql_num_rows($res)) {
  echo "pseudo existant!";
}
else {
mysql_query("INSERT INTO pseudo VALUES('', '$pseudo', '$password', '$email')");
mysql_close();
}

Les variables sont bonnes, j'arrive a ecrire dasn la BDD mais je peux mettre deux fois le meme pseudo

**gerald2545** · 29/08/2006, 13h39

$sql = "SELECT * FROM pseudo WHERE pseudo = ' . $pseudo .'";

devient

$sql = "SELECT * FROM pseudo WHERE pseudo = '$pseudo'";

ou

$sql = "SELECT * FROM pseudo WHERE pseudo = '" . $pseudo ."'";

**jejesochalion** · 29/08/2006, 14h55

J'ai fais comme ca:

Code :

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

$sql = "SELECT * FROM pseudo WHERE pseudo = '$pseudo'";

mais maintenant il s'affiche aucune base selectionne alors que je l'ai fais plus haut.

**gerald2545** · 29/08/2006, 15h47

donne ton code source et l'erreur exacte renvoyée

**jejesochalion** · 29/08/2006, 16h12

Voici le 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
<?php
   $pseudo=$_POST['pseudo'];
   $password=$_POST['password'];
   $email=$_POST['email'];
 
mysql_connect("localhost", "root", "");
if (strlen(trim($_POST['pseudo'])) > 3 && strlen(trim($_POST['pseudo'])) < 35)
				   mysql_select_db('info_perso');
					$sql = "SELECT * FROM pseudo WHERE pseudo = '$pseudo'";
					$res = mysql_query($sql) or die ("Erreur SQL : " . mysql_error());
					if (mysql_num_rows($res)) {
					echo "pseudo existant!";
					}
					else
					mysql_query("INSERT INTO pseudo VALUES('', '$pseudo', '$password', '$email')");
                    mysql_close();
?>

L'erreur me dit:

Aucune table SQL selectionnée

**Arnaud F.** · 29/08/2006, 16h20

Il faut selectionner la table dans laquelle tu va travailler grâce à : mysql_select_db je crois

http://php.developpez.com/faq/?page=...mysqlconnecter

@+

Arnaud

**Yogui** · 29/08/2006, 16h28

Mon cher Arnaud, tu me sembles fatigué :/

Le plus simple et le plus fiable dans ce genre de situation est d'utiliser une clef UNIQUE sur ton nom d'utilisateur dans la structure de la table MySQL. Ainsi, une insertion d'un username déjà présent donnera une erreur de doublon (dont je ne connais pas le code numérique par coeur mais que tu peux récupérer facilement à l'aide de mysql_errno() et qui te sera très très utile).

**jejesochalion** · 29/08/2006, 16h29

Je l'ai fait a la 7ème ligne du code

**Arnaud F.** · 29/08/2006, 16h31

Envoyé par Yogui

Mon cher Arnaud, tu me sembles fatigué :/

No comment... Je le suis

Arnaud

**jejesochalion** · 29/08/2006, 17h12

Est ce que ce code pourrai marcher?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
<?php
   $pseudo=$_POST['pseudo'];
   $password=$_POST['password'];
   $email=$_POST['email'];
 
mysql_connect("localhost", "root", "");
mysql-select-db('info_perso');
if (strlen(trim($_POST['pseudo'])) > 3 && strlen(trim($_POST['pseudo'])) < 35)
					$res = "SELECT pseudo FROM pseudo WHERE pseudo='$pseudo'";
					if $res==0{
					echo "Tu as reussi, t'es trop fort!";
					mysql_query("INSERT INTO pseudo VALUES('', '$pseudo', '$password', '$email')");}
					else{echo "Pseudo deja existant"}
mysql_close(); ?>

S'il ne marche pas, dites moi pourquoi?merci

**Yogui** · 29/08/2006, 17h15

Non : il t'a déjà été dit que $res ne doit pas être comparé à zéro. En outre, pourquoi ne pas essayer ce que je t'ai proposé ? C'est de loin la méthode la plus fiable.

**jejesochalion** · 29/08/2006, 17h24

Parce que je ne la comprend pas tout simplement.
Si je l'ameliore, est ce que le code marchera?

**Yogui** · 29/08/2006, 17h35

Commence par utiliser un bon éditeur de code, car pour le moment il y a de jolies erreurs de syntaxe qui sont très simples à résoudre. Nous avons un débat à ce sujet dans le forum des outils ainsi qu'un comparatif dans l'un des liens que j'ai en signature.

Ensuite, met ton code en forme car, pour le moment, 5 tabulations me semblent être une très bonne manière de rendre du code inexploitable.

Ensuite, renseigne-toi sur les bonnes pratiques à mettre en oeuvre suite à l'envoi d'un formulaire : http://g-rossolini.developpez.com/tu...page_2#LII-B-1

Sinon, essaie d'adopter une méthode uniforme et homogène pour tes chaînes : Apostrophes ou guillemets dans le code PHP : lesquels choisir ?

Enfin, la réponse que tu attends se trouve probablement dans les exemples de ce tutoriel : Comment débuter avec MySQL et PHP

**Arnaud F.** · 29/08/2006, 17h56

Au passage =>

Code :

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

mysql-select-db('info_perso');

devient

Code :

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

mysql_select_db('info_perso');

Sinon écoute ce que te dis Yogui, et reviens nous poser des questions par la suite

( t'en aura plus beaucoup

)

Arnaud

**jejesochalion** · 29/08/2006, 20h05

J'ai essayer ca:

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
<?php
   $pseudo=$_POST['pseudo'];
   $password=$_POST['password'];
   $email=$_POST['email'];
 
mysql_connect("localhost", "root", "");
mysql_select_db('info_perso');
if (strlen(trim($_POST['pseudo'])) > 3 && strlen(trim($_POST['pseudo'])) < 35)
					$sql = "SELECT * FROM pseudo WHERE pseudo= '" . $pseudo ."''";
					$result = mysql_query($sql) or die("Erreur SQL :  " . mysql_error());
					if (mysql_num_rows($result)){
					echo "Le pseudo que vous avez taper existe déjà.";}
					else{mysql_query("INSERT INTO pseudo VALUES('', '$pseudo', '$password', '$email')");
					echo "Vos information on ete envoyer avec succès.";}
mysql_close(); ?>

Il s'affiche alors:

Erreur SQL : erreur de syntaxe vers "" a la ligne 1

Dans les "" c'est ce que j'ai tape dans le champ pseudo de la page du formulaire

**Yogui** · 29/08/2006, 20h18

Tu n'as pas mis d'accolades à ton premier if() : cela te donnera des problèmes par la suite.
Sinon, protège tes valeurs de la requête à l'aide de la fonction mysql_real_escape_string() ou, mieux, de la fonction proposée dans la doc de cette fonction. Et affiche mysql_error() dans le or die()...

**jejesochalion** · 29/08/2006, 20h23

J'ai change le if(), pour or die je l'ai fais non?
Sinon pourquoi il y a marquer erreur de syntaxe, je ne comprends pas.

**Yogui** · 29/08/2006, 20h27

Non, or die() est à utiliser pour toute communication avec la base de données. Fais-le surtout pour les requêtes afin de connaître le détail de tes erreurs de syntaxe. Bien sûr, il faut lire le message qui t'est proposé. Affiche aussi la requête, cela t'aidera. Vas-y pas à pas et ne viens pas nous voir avant d'avoir essayé plusieurs solutions, je t'en supplie ! Nous pouvons t'aider mais nous n'allons pas tout faire à ta place !

Pseudo existe déjà ? [MySQL]

PHP & Base de données

Discussions similaires

Partager

Partager