Forum PHP.pl

[PHP] pobierz, plaintext 
<?php
mysql_connect('localhost','root','');
mysql_select_db("users");
 
$user = $_POST[user];
$pass = $_POST[password];
$mail = $_POST[mail];
 
 
$czyjest = mysql_query("SELECT nick,mail FROM users WHERE user = '$user'");
$czyjest1 = mysql_num_rows($czyjest);
 
if($czyjest1 == 1){ 
$mozna = 0;
}else {
$user1 = md5($user);
mysql_query("INSERT INTO users(user,password,mail) VALUES ($user,$pass,$mail)"); // rejestruje
}
 
 
 
mysql_close();
?>
[PHP] pobierz, plaintext 

Zwraca to komunikat

Nie chce to działać poprawnie.
Z góry dziękuje za pomoc!!
Pozdrawiam,
SzymonTJ

Edit :
Uwaga!
Nie chodzi mi o zabezpieczenia, a o normalne funkcjonowanie (IMG:style_emoticons/default/exclamation.gif)

Ten post edytował SzymonTJ 20.04.2010, 19:04:50

Nie lepiej użyć

[SQL] pobierz, plaintext 
SELECT COUNT(*) FROM users WHERE user='$user'
[SQL] pobierz, plaintext 

MYSQL INJECTION!!!!

Linia 5: nie $_POST[user], a $_POST['user']
Linia 6: to samo
Linia 7: to samo
Poza tym poczytaj o SQL Injection, bo Twój skrypt umożliwia taki właśnie rodzaj ataku.

Poza tym w tabeli users masz pole nick, czy user, bo używasz ich zamiennie?
Na przyszłość zapoznaj się z tematem Jak poprawnie zadać pytanie. Zostały tam opisane podstawowe sposoby wychwytywania błędów w skryptach PHP.