([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

[Sabistik]

Jestem otwarty na wszelką krytykę oprucz ORTOGRAFJ!! ;p

Wybacz, ale to forum to nie śmietnik. Temat Zamykam dopóki nie wyślesz mi wykazu wszystkich błędów wraz z poprawna formą. Dodatkowo dostajesz moderacje na nowo powstałe tematy.

[Scoout]

Błędy poprwaiłem i nadal proszę o pomoc...

[Trobin]

Przede wszystkim nazwa klasy nie bardzo odzwierciedla to co ona robi - napisane jest rejestracja a sprawdzasz tam czy nick istnieje, ale nie dodajesz nowego - to nie jest rejestracja. To jest raczej walidowanie danych do późniejszej rejestracji (czy nick nie jest za długi, krótki lub nie istnieje już w bazie). Jeszcze takie moje czepialstwo - nie lubie nazw klas zaczynających się n=od dużej litery A tak poważniej to zwyczajowo przyjęło się, żeby używać angielskich nazw klas . Piszesz

[PHP] pobierz, plaintext 
<?php
if(strlen($nick)>$this->nick[dlugosc][max]) $blad=3;
?>
[PHP] pobierz, plaintext 

a potem

[PHP] pobierz, plaintext 
<?php
case 2: echo'Nick jest za krótki'; break;
case 2: echo'Nick jest za długi'; break;
?>
[PHP] pobierz, plaintext 

mogę się mylić, ale sprawdzanie czy nick jest za długi chyba nie zadziała.
Poza tym skoro już używasz OOP w PHP to zastanów się nad używaniem obiektowego sterownika do bazy danych - ot dla wygody i możliwości późniejszej rozbudowy. Samej stricte obiektowości nie komentuję, ponieważ sam nie czuję się w tej dziedzinie pewnie .