Czy ja dobrze kumam? Opcje

[eS...]

witam
zrobiłem sobie wkońcu rejestracje użytkowników. Sam bez pomocy internetu itd.
i oto co mi wyszło[php:1:8e22e61f8b]<?php
mysql_connect('localhost','root','krasnal');
mysql_select_db('test');

$user=$HTTP_POST_VARS['user'];
$haslo=$HTTP_POST_VARS['haslo'];
$mail=$HTTP_POST_VARS['mail'];
$miasto=$HTTP_POST_VARS['miasto'];

$sql="SELECT * FROM user;";

$wynik=mysql_query($sql)or die('blad');

while($row = mysql_fetch_row($wynik)){
$user1=$row[1];
$haslo1=$row[2];
$mail1=$row[4];
}

if ($user==$user1 ){
echo "podany nick , istnieje w naszej bazie danych!";
}elseif($mail==$mail1){
echo "Podany mail istnieje w bazie danych";

}elseif(!($user||$haslo||$mail)){
echo "Wypełnij wszystkie pola";

}else{
$sql1= "INSERT INTO user ( `id` , `user` , `haslo` , `level`,`mail`,`miasto`,`data_przylaczenia` ) VALUES ( '', '$user', MD5( '".$haslo."' ),'0','$mail','$miasto',now() );";

$wynik1=mysql_query($sql1)or die(mysql_error());


echo ":-)";

}


?>[/php:1:8e22e61f8b]
Wszystko działe, ale powiedzcie co ewentualnie jest zle lub co dodać??
dzięki z góry

[scanner]

co do lini 20-26
to rozumiem ze ma być
[php:1:83cb173d2a]<?php
if ($user=0)
if($mail=0)

?>[/php:1:83cb173d2a]
(IMG:http://forum.php.pl/style_emoticons/default/questionmark.gif)
czy moze moge to potraktować tak:
[php:1:83cb173d2a]<?php
if($user>0)//taki uzytkownik juz istnieje(mysql_num_row??)
?>[/php:1:83cb173d2a]

[php:1:83cb173d2a]<?php
$strSQL = "SELECT COUNT(*) FROM users WHERE user='".$_POST['user']."'";
$resSQLResult = mysql_query($strSQL);
$intFound = intval(mysql_result($resSQLResult, 0));
if($intFound > 0)
die(' Podany login juz istnieje');
?>[/php:1:83cb173d2a]