[php]rejestracja błąd Opcje

[riquelme]

Oto mój skrypt do rejestracji nowych użytkowników:

<?
$polaczenie = mysql_connect("localhost", "w", "e");
mysql_select_db("w", $polaczenie) or die ("Nie mozna sie połączyć");
$imie=$_POST['imie'];
$nazwisko=$_POST['nazwisko'];
$email=$_POST['email'];
$login=$_POST['login'];
$haslo=$_POST['haslo'];
$haslo2=$_POST['haslo2'];
$rodzaj=$_POST['rodzaj'];

if ($haslo != $haslo2)
{
echo "Hasła nie są takie same!";
}

$wynik=mysql_query("SELECT * FROM users WHERE login=$login");
echo mysql_error();
if ($wynik!==0)
{
echo "Już taki login istnieje!";

}
$zapis="INSERT INTO users(imie,nazwisko,email,login, haslo,rodzaj, data) VALUES('".$_POST['imie']."', '".$_POST['nazwisko']."', ''".$_POST['email']."', ''".$_POST['haslo']."', ''".$_POST['haslo2']."', ' '".$_POST['rodzaj']."', ')";
$wynik=mysql_query($zapis);

if (!$wynik)
{
echo "Rejestracja zakończona pomyślnie";
exit;
}
else
{
echo ('Rejestracja nie powiodła sie! ');
exit;
}

?>

Przy wypełnianiu formularza np we wszystkie pola wpisuje xxx tp wyświetla się komunikat : "Unknown column 'xxx' in 'where clause'Już taki login istnieje!Rejestracja zakończona pomyślnie". Co należy zmienić żeby prawidłowo dodawało użytkowników do bazy??

[nospor]

dziwnie masz to skonstruowane. mimo ze bedziesz mial, iz login istnieje, to i tak przechodzisz dalej do dodawania usera. gdzie sens i logika? (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
pozatym zle sprawdzasz istnienie usera. mysql_query o ile nie ma bledow mysql zwraca dla selecta resource, czyli twoj warunek zawsze bedzie sprawdzony. powiino to byc tak:

[PHP] pobierz, plaintext 
<?php
$wynik=mysql_query("SELECT * FROM users WHERE login='$login'");
echo mysql_error();
if (mysql_num_rows($wynik) > 0)
{
echo "Już taki login istnieje!";
exit;
}
?>
[PHP] pobierz, plaintext 

co do niedodawania to walnij se :

[PHP] pobierz, plaintext 
<?php
$wynik=mysql_query($zapis) or die(mysql_error());
?>
[PHP] pobierz, plaintext 

i zobacz czy ci nie pluje bledami

pomine juz ten szczegol:

[PHP] pobierz, plaintext 
<?php
if (!$wynik)
{
echo "Rejestracja zakończona pomyślnie";
exit;
}
?>
[PHP] pobierz, plaintext 

ze jezeli zapytanie wypluje ci bledy to ty piszesz ze wszystko jest ok (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)