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]

Przeniesione z: Hydepark.

Linie 5-8:
- Lepiej $_POST[]
- Linijka jest zbędna - przecież możesz się bezpośrfednio tablicami superglobalnymi posługiwać.
Linia 10:
niepotrzebue wybierasz wszystko z bazy:
[sql:1:a150fb6072]SELECT COUNT(*) FROM user WHERE user=$_POST['user'][/sql:1:a150fb6072]I sprawdzasz, czy wynik jest równy 0 - jeśli tak, znaczy, że nie ma takiego loginu w bazie.
Linie20 - 26:
Warunki nie mają prawa prawidłowo działać, jako, że dotyczą ostatniego pobranego z bazy rekordu.

Polecam login i hasło potraktowac przez trim() na samym początku. Zapobiegnie to stworzeniu konta " " z hasłem " ".
Pozatym skrypt jest napisany dość brzydko - jeśli chodzi o sam sposób kodowania.
Polecam uwadze: isset(), empty();

(dodawajac lub usuwajac nie potrzebne informacje np. miasto)

Podczas rejestracji niezbędne dane to: Login, hasło, email, data rejestracji, stan konta (aktywne, nieaktywne, ban) i data aktywacji. Reszta danych powinna byc w module Profil a nie Rejestracja.