Forum PHP.pl

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??

login=$login
na
login='$login'
tekst w mysql trzeba brac w ciapki

prosze uzywac bbcode

Dzięki za odpowiedź,zrobiłem tak jak powiedziałeś,jednak teraz wyświetla sie komunikat : "Już taki login istnieje!Rejestracja zakończona pomyślnie." Chyba musiałem popełnić jakiś błąd w SQL bo nie dodaje użytkowników do bazy.

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)

Teraz po kliknięciu Rejestruj wyskakuje błąd : "Column count doesn't match value count at row 1"

no juz moglbys sam poszukac. Mowi to, iz nie pasuje liczba kolumn ktore wkladasz
Mowisze ze wkladasz:
imie,nazwisko,email,login, haslo,rodzaj, data
a w rzeczywostosci wkladasz:
imie,nazwisko,email,haslo,haslo2,rodzaj
wiec chyba cos sie nie zgadza

Bardzo dziękuje,rejestracja prawie idealnie działa,ale uważam że to tylko drobny błąd.Mianowicie w formularzu , mam pole rodzaj zablokowane i wpisana wartość jest zawsze "1",natomiast po rejestracji w bazie danych zapisuje mi rodzaj jako 0.

1) przez jakie pole jest reprezentowane w formularzu to zablokowane (pokaz jego html)
2) w tabeli jaki typ ma to pole
3) skoro zawsze jest 1, to po co je tam walic? wkladaj do bazy od razu 1

Ad.1
<td><input type="text" disabled="disabled" value="1" name="rodzaj" /></td>
Ad.2
W tabeli pole rodzaj ma wartość int.
Ad.3
Pole rodzaj jest mi potrzebne ,ponieważ przy logowaniu skrypt sprawdza rodzaj.Jeżeli jest 0 to otwiera się strona admina,1-strona użytkownika.Jestem początkujący w php,więc nie wiem w jak inny sposób zapisać automatycznie przy każdej rejestracji rodzaj 1

disabled="disabled"
pola tego typu nie sa przesylane formularzem, wiec masz 0 (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)
daj readonly albo najlepiej zrob hidden

ad3) no zamiast wpisywac to z posta przez $_POST['rodzaj'] to wpisz bezczelnie 1. no chyba ze jednak ten rodzaj w jakis posob sie zmienia gdzies tam to zrob jak pisalem wczesniej

Zakładając że chce "bezszczelnie wpisywać 1" to jak ma wyglądać zapytanie,żeby zawsze dodawało 1 i nie potrzebne będzie do tego pole w formularzu??
Obecne zapytanie:

[PHP] pobierz, plaintext 
<?
$zapis="INSERT INTO users(imie,nazwisko,email,login, haslo,rodzaj) VALUES('".$_POST['imie']."', '".$_POST['nazwisko']."', ''".$_POST['email']."', ''".$_POST['login']."', ''".$_POST['haslo']."', ' '".$_POST['rodzaj']."', ')";
?>
[PHP] pobierz, plaintext 

Po prostu zamiast zmiennej wpisujesz ręcznie do zapytania 1 (w zmiennej tez możesz (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[PHP] pobierz, plaintext 
<?php
$zapis="INSERT INTO users(imie,nazwisko,email,login, haslo,rodzaj) VALUES('".$_POST['imie']."', '".$_POST['nazwisko']."', ''".$_POST['email']."', ''".$_POST['login']."', ''".$_POST['haslo']."', '1', ')";
?>
[PHP] pobierz, plaintext 

Zrobiłem to jak napisałeś ,niestety przy rejestracji zwraca 0(pewnie interpretuje to jako false).Wiem że napewno sie da zrobić tak żeby zawsze 1 zapisywało,problem tkwi zapewne w szczegółach.

ale to pole to int? jak tak to zamiast
.., '1',...
ma byc
..., 1,....