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

[DreeD]

jesli o to ci chodzi (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)
to dodaj jeszcze ostatnio zalogowal sie:
pokombinuj z aktywacja konta przez mail
co wiecej... to juz zalezy od potrzeb serwisu na ktorym to chcesz wykorzystac... jezeli na kilku roznych to ma byc uzywane to skonstruuj to tak zeby latwo mozna bylo edytowac skrypt (dodawajac lub usuwajac nie potrzebne informacje np. miasto)