Czy to jest poprawne - w duchu obiektowości?, proszę o komentarze Opcje

[Aztech]

Zacząłem niedawno pisać na poważnie (w końcu praca duplomowa) w php5 i chciałbym się spytać czy to co napisałem, jest w pełni w duchu obiektowości. Może widzicie tam jakieś błędy, może macie jakieś wskazówki. Za wszelkie rady, spostrzeżenia, w szczególności te bardzo krytyczne, będę niezmiernie wdzięczny.

Gwoli wyjaśnienia, klasa ta ma odpowiadać za obsługę użytkownika:
(*) dodawać
(*) usuwać
(*) uaktualniać
(*) parsować poprawność wprowadzonych danych
itp.

A więc jeszcze raz proszę o jak najwięcej uwag i krytyki (ew. pochwały też przyjmuję (IMG:http://forum.php.pl/style_emoticons/default/biggrin.gif) )

[PHP] pobierz, plaintext 
<?php 
 
 
/*
    UWAGA: wszystkie zapytanie sql sformatować odpowiednio dla adodb
*/
 
require_once("../core/adodb/adodb.inc.php");
 
class TUser {
    private $user_id=-1;
    private $nick ="";
    private $name= "";
    private $group_id=0;
    private $password="";
    private $password_crypt="";
    private $email="";
    private $gg="";
    private $photo="";
    private $description="";
    private $phone="";
    private $last_error="";
    private $is_error=0;   //0 - no error, 1 - error
 
################################## need to update all validation functions #####
##############################
 
/********************************************************************************
*****    
                        VALIDATE FUNCTIONS                    
********************************************************************************
*****/
 
    public function validateName($name) {
        // validate if name is correct, name have 2 words which begin with uppercase and s
pace beetween of them
        if (ereg("^[A-z](a-zA-Z/-)*( A-Za-z/-)*",$name)) {
               $this->unsetError();
               return true;           
        }
        else {
            $this->last_error = "NO_VALID_NAME";
            $this->setError();
            return false;
        }       
    }
    
    public function validateEmail() {
        //validate if email is correct
        if (eregi("^([a-z0-9_-.]{1,25})@([a-z0-9_-.]){1,25}$",$this->$email)){
               $this->unsetError();
               return true;           
        }
        else {
            $this->last_error = "NO_VALID_EMAIL";
            $this->setError();
            return false;
        }
    }
    
    public function validatePassword($pass) {
        //6-25 signs, one big letter and one number        
        if (ereg("[A-Za-z0-9]")) {
               $this->setError();
               return true;           
        }
        else {
            $this->last_error = "NO_VALID_PASSWORD";
            $this->setError();
            return false;
        }
    }
    
/********************************************************************************
*****    
                        COMPARE FUNCTIONS                   
********************************************************************************
*****/
 
    public function comparePassword($value="") {
        if ($this->password == $value) {
            $this->unsetError();            
            return true;            
        }
        else {
            $this->last_error = "PASSWORD_NOT_SAME";
            $this->unsetError();
            return false;
        }           
    }    
 
/********************************************************************************
*****    
                        GET FUNCTIONS                    
********************************************************************************
*****/    
    
    public function getUserInfo(){
        return $this;
    }
 
/********************************************************************************
*****    
                        SET FUNCTIONS                    
********************************************************************************
*****/    
 
    public function setError(){
        $this->is_error = 1;
    }
    
    public function unsetError(){
        $this->is_error = 0;
    }
 
    public function setUserID($value="") {
        $this->user_id = $value;
    }
    
    public function setNick($value="") {
        $this->nick = $value;
    }
    
    public function setName($value="") {
           $this->name = $value;
    }
    
    public function setGroupID($value="") {
           $this->group_id = $value;
    }
    
    public function setPassword($value="") {
           $this->password = $value;
    }
    
    public function setEmail($value="") {
           $this->email = $value;
    }
    
    public function setGG($value="") {
           $this->gg = $value;
    }
    
    public function setPhoto($value="") {
        $this->phone = $value;
    }
    
    public function setDescription($value="") {
           $this->description = $value;
    }
    
    public function setPhone($value="") {
           $this->phone = $value;
    }
    
    
    public function setUserFromSQL($table){
        //set user data using result form SQL query
        $this->user_id          =   $table["id_user"];
        $this->nick             =   $table["nick"];
        $this->name             =   $table["name"];
        $this->group_id         =   $table["name"];
        $this->password         =   $table["password"];
        $this->password_crypt   =   md5($table["password"]);
        $this->email            =   $table["email"];
        $this->gg               =   $table["gg"];        
        $this->photo            =   $table["photo"];
        $this->description      =   $table["description"];
        $this->phone            =   $table["phone"];
        $this->last_error       =   "";
        $this->is_error         =   "0";    //no error
    }
       
    public function __construct() {
        $this->user_id          =   "-1";
        $this->nick             =   "";
        $this->name             =   "";
        $this->group_id         =   "0";
        $this->password         =   "";
        $this->password_crypt   =   "";
        $this->email            =   "";
        $this->gg               =   "";        
        $this->photo            =   "";
        $this->description      =   "";
        $this->phone            =   "";
        $this->last_error       =   "";        
        $this->is_error         =   "0";    //no error
    }
    
    public function clearrData() {
        $this->__construct();
    }
 
/********************************************************************************
*****    
                        SET FUNCTIONS                    
********************************************************************************
*****/ 
 
    public function findAllGroupsSQL(&$groups) {
        //fing all groups where belong user
        //if user is set
        if ($this->user_id != "-1") {
            /*
            $sql =   " SELECT user_id, group_id, id_groups, name, moderator_id"
                    ." FROM rbx_group_users LEFT JOIN rbx_groups"
                    ." ON rbx_group_users.group_id = rbx_groups.id_groups"
                    ." WHERE user_id = 3";            
                    */
        }
        else {
            $this->lastError="NO_USER";
            $this->setError();
            return false;
        }
    }
    
    public function userExistsSQL() {
        //connectToADO($db);
        $sql = "select email from rbx_username where email='".$this->email."'";
        //$result = $db->Execute($sql);  
        if (!$result->FieldCount())
            return true;
            else return false;        
    }
    
    public function createUserSQL() {
    //gdy uzytkownik nie istnieje
        if (!$this->userExists()) {
            //connectToADO($db);
            $sql = "insert into 'rbx_user' ( 'id_user' , 'name' , 'pass' , 'email' , 'gg' , 'photo' , 'description' , 'phone' )".
            " values ('', '".$this->name."', '".$this->password_crypt."', '".$this->email."', '".$this->gg."', '".$this->photo."', '".$this->description."', '".$this->phone."')";
            //$result = $db->Execute($sql);          
            $this->is_error = false;      
        }
        else {
            $this->is_error = true;
            $this->last_error = "USER_ALREADY_EXIST";
        }
    }           
 
    public function updateUserSQL() {    
        //connectToADO($db);        
            $sql = "update 'rbx_user'"
            ." set 'name'=".$this->name.","
            ." 'pass'=".$this->password_crypt.","
            ." 'email'=".$this->email.","
            ." 'gg'=".$this->gg.","
            ." 'photo'=".$this->photo.","
            ." 'description'=".$this->description.","
            ." 'phone'=".$this->phone.","
            ." 'pass'=".$this->password_crypt
            ." WHERE 'id_user'=".$this->user_id;            
        //$result = $db->Execute($sql);          
        $this->is_error = false;      
    }
    
    public function deleteCurrentUserSQL() {
        //connectToADO($db);
            $sql = "delete from 'rbx_user'"
            ."WHERE user_id=".$this->user_id;
        //result = $db->Execute($sql);
        $this->is_error = false;
    }
    
    public function deleteUserSQL($db) {
        //connectToADO($db);
            $sql = "delete from 'rbx_user'"
            ."WHERE user_id=".$db->user_id;
        //result = $db->Execute($sql);
        $this->is_error = false;
    }        
    
        
}
 
?>
[PHP] pobierz, plaintext 

Ten post edytował NuLL 28.11.2005, 23:06:30

[NuLL]

Jak ma sie sprawa jesli chodzi o pozniejszą zmianę struktury (chociazby nazw) w bazie danych ? bieganie po obiektach i zmiany ?

Cos za cos niestety. Samego kodu za duzo zmieniac tak naprawde nie trzeba. Mozna sobie napisac cos ala genericObject czy typowy obiekt i po tym dziedziczyć poprostu. Cala klase z pierwszego posta mozna by powaznie skrocic korzystajac z tego co jest ponizej ( to tylko wzor pisany z palca, nie testowany czy cos )

[PHP] pobierz, plaintext 
<?php
 
abstract class genericObject extends Object 
{
	private 
		$myFields, $databaseFields, $loaded, $modified, $tupleId, $databaseTable, $keyName ;
 
	public function __construct($tupleId=0)
	{	
		$this->loaded=0;
		$this->tupleId=intval($tupleId);
	}
 
	public function setDatabaseParams($table,$key)
	{
		$this->databaseTable=$table;
		$this->keyName=$key;
	}
 
	public function forceLoaded()
	{
		$this->loaded=1;
	}
 
	public function getField($name)
	{	
		if($this->loaded==0)
		{
			$this->load();
		}
 
		return $this->databaseFields[$name];
	}
 
	public function getFields()
	{
		if($this->loaded==0)
		{
			$this->load();
		}
 
		$fields=func_get_args();
 
		$ret=array();
 
		foreach ($fields as $field)
		{
			if(isset($this->databaseFields[$field])) $ret[$field]=$this->databaseFields[$field];
		}
 
		return $ret;
	}
 
	public function getAllFields()
	{
		if($this->loaded==0)
		{
			$this->load();
		}
 
		return $this->databaseFields;		
	}
 
	public function setField($name,$value)
	{
		$this->myFields[$name]=$value;
		$this->modified=1;
	}
 
	public function setFields($data)
	{
		if(is_array($data))
		{
			foreach ($data as $key=> $value)
			{
				$this->setField($key,$value);
			}
		}
	}
 
	public function destroy()
	{
		if($this->tupleId)
		{
			$db=db::instance();
 
			$db->execute("DELETE FROM ".$this->databaseTable." WHERE ".$this->keyName."=".$this->tupleId);
 
			if($db->affectedRows()>0)
			{
				$this->loaded=0;
				return true;
			}
			return false;
		}
	}
 
	public function getId()
	{
		return $this->tupleId;
	}
 
	private function load()
	{
		$db=db::instance();
 
		$this->databaseFields=$db->getRow("SELECT * FROM ".$this->databaseTable." WHERE ".$this->keyName."=".$this->tupleId);
 
		$this->loaded=1;
 
		$this->modifiedFields=array();
 
		$this->modified=0;
	}
 
	public function save()
	{
		$db=db::instance();
 
		if($this->tupleId>0)
		{
			$db->update($this->databaseTable,$this->myFields,$this->keyName."=".$this->tupleId);
 
			$ret=$db->affectedRows();
		}else
		{
			$insertId=$db->insert($this->databaseTable,$this->myFields);
			$ret=$insertId;
			$this->tupleId=$insertId;
		}
 
		$this->databaseFields=$this->myFields;
		$this->databaseFields[$this->keyName]=$this->tupleId;
 
		$this->myFields=array();
 
		return $ret;
	}
}
 
?>
[PHP] pobierz, plaintext