[logowanie]-sprawdzanie użytkownika Opcje

[dzesi]

Witam mam problem ze spraszaniem użytkowników czy dany login występuje już w bazie

[PHP] pobierz, plaintext 
<?php
 
include_once('config_database.php');
//odbieranie danych 
$login=$_POST['login'];
$haslo=$_POST['haslo'];
$haslotwo=$_POST['haslotwo'];
 
//////////////////////////////////////////////////////////////////////////////
 
 
$polaczenie=mysql_connect(connect,user,password)or die('connect_error'.mysql_error());
mysql_select_db(database)or die('database_error'.mysql_error());
//tu powinno sprawdzać czy jest taki login czy go nie ma 
$zapytanielog= "SELECT * FROM user login='$login'";
$sprawdzlog= mysql_query($zapytanielog)or die(mysql_error());
$numrows=mysql_num_rows($sprawdzlog);
if($numrows > 0)
{
	echo "Podany login już istnieje";
}
if($haslo!=$haslotwo)
{
	echo "Hasło jest błedne";
}
 
$zapisrej="INSERT INTO user(`login`,`haslo`)VALUES('$login','$haslo')";
$touser=mysql_query($zapisrej);
if($touser)
{
	echo"błąd nie można dodać użytkownika";
}
 
?>
[PHP] pobierz, plaintext 

nigdy nie maiłem potrzeby masłowej rejestracji , zawsze było kilka osób dodawane przez phpmyadmin i zapytania sql, mam nadzieje ze pokażecie co robie zle

[Jarod]

Załóż klucz unikalny na pole login i podczas dodawania sprawdzasz:

[PHP] pobierz, plaintext 
<?php
if (mysql_errno() == 1062 )
{
	   echo '<p>Konto o takim loginie już istnieje !</p>';
 
(...)
}
?>
[PHP] pobierz, plaintext 

[dzesi]

Nadal nic , próbowałem tak

[PHP] pobierz, plaintext 
<?php
$numrows= mysql_num_rows($sprawdzlog);
if($numrows > 0)
{
	echo "Podany login już istnieje";
}
i tak 
$zapytanielog= "SELECT count(*) FROM `user` WHERE `login` = '$login '";
$sprawdzlog= mysql_query( $zapytanielog ) or die ( mysql_error() );
if( $sprawdzlog['count(*)'] > 1)
{
	echo " nie ma takiego loginu";
}
?>
[PHP] pobierz, plaintext 

Ale nic z tego nadal pokazuje ciągle ze login jest w bazie przy każdej rejestracji , ale tak na prawdę go nie ma
wiec co jest nie tak ?, jeśli możecie to podzielcie sie wiedzą jak wy tworzycie takie rzeczy ?, jakim sposobem i wykonanie itd.Bo może[na pewno] są lepsze sposoby niż ten co sam wymyśliłem

[Jarod]

1. Jeżeli chcesz sprawdzać (przed dodaniem usra do bazy) czy login jest zajęty to nie rób tego w ten sposób:

[SQL] pobierz, plaintext 
SELECT count(*) FROM `user` WHERE....
[SQL] pobierz, plaintext 

tylko

[SQL] pobierz, plaintext 
SELECT login FROM `user` WHERE....
[SQL] pobierz, plaintext 

Wydaje mi się, że będzie szybciej - ale nigdy tego nie sprawdzałem.


2.

[PHP] pobierz, plaintext 
<?php
(...)
$result = mysql_query('INSERT INTO user VALUES (...odpowiednie, przefiltrowane dane...)');
 
if (mysql_errno() == 1062)
{
	   // nie udało się dodać user do bazy - podany login już istnieje, odpowiednia obsługa tej sytuacji
}
else
{
		// pomyślnie dodano usera do bazy, przekierowanie lub jakiś komunikat...
}
?>
[PHP] pobierz, plaintext 

Ale na pole login (bo ono jest unikalne) dodajesz klucz unikalności (odsyłam do manuala). Kod błędy 1062 oznacza duplikat danych, czyli próbujesz wstawić do bazy coś co już jest a powinno być unikalne. Kod pisany z głowy.