Script d'inscription qui ne fonctionne pas

**brotic** · 30/11/2008, 16h53

Bonjour, je cré actuellement un site et j'ai un problème avec la page d'inscription de celui -ci, le script que j'ai mis en place ne fonctionne pas, le voici :

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
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
<?php
if(isset($_POST['pseudo']) && isset($_POST['mdp']) && isset($_POST['mail']) )
{
	mysql_connect("localhost", "root", "");
	mysql_select_db("brotic");
	$nbr_inscritP = mysql_query( "SELECT * FROM inscrit" );
	$nbr_inscrit = mysql_num_rows( $nbr_inscritP );
	$mdp = $_POST['mdp'];
	$mail = $_POST['mail'];
	$id = 1;
	while( $id <= $nbr_inscrit ) {
		$reponse = mysql_query("SELECT * FROM inscrit WHERE id='$id'" );
		$donnees = mysql_fetch_array($reponse);
		if( $_POST['pseudo'] != $donnees['pseudo'] )
		{
			 mysql_connect("localhost", "root", "");
			  mysql_select_db("brotic");
			  mysql_query("INSERT INTO inscrit VALUES('', '" . $Ppseudo . "', '" . $mail . "', '" . $mail . "', '" . $REMOTE_ADDR . "')" );
			  $chaine = "Inscription en cour d'enregistrement...";
		}
		  else
		  {
			 $chaine = "Le pseudo choisit est dejà utilisé...";
		  }
	$id++;
}
}
else
{
	$chaine = "Vous n'avez pas données toutes les informations demandées...";
}
mysql_close();
?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
	<head>
		<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
		<meta http-equiv="refresh" content="2; URL=index.php" />
		<link rel="stylesheet" media="screen" type="text/css" title="design" href="design.css" />
		<title>Accueil</title>
	</head>
	<body>
		<div class="tableau">
			<p>
				<?php echo $chaine; ?>
			</p>
		</div>
	</body>
</html>

Voila je ne c'est pas du tout où est mon erreur donc je viens ici pour demander au de l'aide au spécialiste, merci d'avance.

p.s : Il est interdit de se moquer j'ai 15 ans et je ne programme que depuis 1ans

.

**sabotage** · 30/11/2008, 19h42

Bonjour et bienvenu sur le forum :

Comme tu es débutant, je vais te corriger

Code :

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

1
2
3
4
5
$nbr_inscrit = mysql_num_rows( $nbr_inscritP );
	$id = 1;
	while( $id <= $nbr_inscrit ) {
		$reponse = mysql_query("SELECT * FROM inscrit WHERE id='$id'" );
		$donnees = mysql_fetch_array($reponse);

Tu ne peux pas parcourir une table de cette facon : si des ids ont été supprimés dans la table (ie : 1,2,3,8,9,10) le compte de ton $id ne correspondra plus aux ids de la table.
Beaucoup plus simplement :

Code :

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

1
2
3
4
5
 
$reponse = mysql_query("SELECT * FROM inscrit" );
while ($donnees= mysql_fetch_array($reponse) {
...
}

Ensuite si tu veux vérifier qu'un element est dans une table, tu n'as pas besoin de lire toute la table en le cherchant.
C'est le boulot de MySql de chercher dans sa table :

Code :

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

SELECT * FROM inscrit WHERE pseudo='$pseudo'

Pour finir, un point sécurité : on n'insere pas dans une requête SQL des données saisies par l'utilisateur car il pourrait détourner ta requête donc :

Code :

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

1
2
$pseudo = mysql_real_escape_string($_POST['pseudo']);
$sql = "SELECT * FROM inscrit WHERE pseudo='$pseudo'";

Sinon tu ne nous as pas dit ce qui ne marchait pas mais :

Code :

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

			  mysql_query("INSERT INTO inscrit VALUES('', '" . $Ppseudo . "', '" . $mail . "', '" . $mail . "', '" . $REMOTE_ADDR . "')" );

tu as mis deux fois $mail et $Ppseudo n'est pas défini dans ton script.
Il ne faut pas utiliser $REMOTE_ADDR mais $_SERVER[REMOTE_ADDR]

**brotic** · 30/11/2008, 20h47

Bah merci maintenant sa fonctionne bien sa devait surement venir du fait que la table n'était pas bien parcouru et par rapport a la sécurité je ne le savais pas donc merci et pour le double $mail c'est une erreur d'écriture en fesant un copier coller^^ mais merci quand même

Script d'inscription qui ne fonctionne pas [MySQL]

PHP & Base de données

Discussions similaires

Partager

Partager