([PHP])Rejestracja Opcje

[Scoout]

Pisze własną stronne dopiero co poznałem się na obiektach i chcę się spytać czy ten skrypt jest poprawny czy można w nim coś zmienić czy ulepszyć??

[PHP] pobierz, plaintext 
<?php
 
$mysql=mysql_connect('localhost', 'root', 'aidaid');
mysql_select_db ('gra');
 
class Rejestracja
{
 
  public function __construct() //Dane podstawowe
  {
	$this->nick[dlugosc][min]=3;
	$this->nick[dlugosc][max]=16;
  }
 
  public function nick($nick) //Sprawdzanie poprawnosci Nicku
  {
	$zapytanie='SELECT id FROM usr_akt WHERE nick=''.$nick.''';
	$result=mysql_query($zapytanie);
	$row=mysql_fetch_array($result);
 
	if($row!==false) $blad=1;
	if(strlen($nick)<$this->nick[dlugosc][min]) $blad=2;
	if(strlen($nick)>$this->nick[dlugosc][max]) $blad=3;
 
	if(isset($blad)) $status=$blad; else $status='ok';
 
	return $status;
  }
 
  public function blad($blad) //Kontrola błędów
  {
	if($blad!=='ok') 
	{
	  switch($blad)
	  {
		case 1: echo'Dany nick istnieje już w bazie danych'; break;
		case 2: echo'Nick jest za krutki'; break;
		case 2: echo'Nick jest za długi'; break;
	  }
 
	} else echo('Nick jest prawidłowy');
 
	return;
  }
}
 
 
$user=new Rejestracja;
$nick=$user->nick('qwqw');
$user->blad($nick);
 
?>
[PHP] pobierz, plaintext 

Jestem otwarty na wszelką krytykę oprócz ORTOGRAFJ!! ;p

Ps. Jest to tylko część sprawdzająca Nick ale chce wiedzieć czy ogólne założenia są dobre np bezpieczeństwo...

Wiem ze do tego powinna być dodane sprawdzanie czy w Nicku nie ma Html ale to tyle mojej wiedzy...

Ten post edytował Scoout 26.03.2008, 13:01:35