Kilka klas i ich implementacja. Opcje

[marcio]

WItam w jednym z poprzednich tematow byl link do faq z wikibooks i jako zadanie dla siebie postanowilem rozbudowac sobie tamten przyklad zrobilem jak narazie 2 klasy tworze 3.

1 klasa sluzy do logowania dla mysql jak i dla plikow *.txt -> http://3paste.com/s/374
2 klasa do rejestracji w 2 wyborach jak wyzej. -> http://3paste.com/s/378

Ogolnie plik mysqlDriver.php zawiera prosta klase z debilnym funkcjami tak tylko zeby pocwiczyc (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

Chodzi mi o to czy dobrze wszystko zaimplementowalem, co byscie poprawili, co jest do dupy a co jest good.

Czy np logowanie lepiej by wygladalo gdybym w klasie Login mial 2 publiczne pola np $nick i $pass i logowanie np tworzyl tak zamiast przekazywac zmienne z posta do funkcji login():

[PHP] pobierz, plaintext 
<?php
session_start();
error_reporting(E_ALL);
include('form.php');
include('classAuth.php');
 
 if(isset($_POST['login']) && isset($_POST['pass'])) {
 
  if(TYPE == 'SQL') $user = new Login(new MySqlAuth(new MySqlDriver('***', '***', '***', '***')));
 
  else $user = new Login(new FileAuth('users.txt'));
   
   $user -> nick = $_POST['login'];
   $user -> password = $_POST['pass'];
   echo($user -> AuthLogin());
 
 }
?>
[PHP] pobierz, plaintext 

I tak samo jesli chodzi o rejestracje jak i 3 klase ktora pisze edytowanie/usuwanie user'ow lepiej przekazywac wszystko do funckji lub ustawiac pola po instancji klasy lub to zalezy co sie robi?

Jak wy robicie co polecacie??

Ogolnie czekam na szczery komentarz ewentualne zmiany w kodzie i przedstawienie mi go bym mogl sie w koncu czegos pozytywnego nauczyc (IMG:http://forum.php.pl/style_emoticons/default/tongue.gif)

P.S kod dalem na inny srv by nie robic burdelu w watku.

P.S2 tak wyglada np podstrona dla zalogowanych user'ow: http://3paste.com/s/376

Jakies sugestie??

EDIT:Link zostal poprawiony poprostu przegladalem ostatnio dodane kody na 3paste i przez przypadek skopiowalem nie ten link.

Ten post edytował marcio 11.05.2009, 11:08:45