Klasa obsługi bazy danych, moj pierwszy singleton Opcje

[Beynar]

Prosze o troche słów krtyki dla mojej klasy obsługi bazy danych.

Tutaj dokumentacja: http://ww1.pelp.net/forum/viewtopic.php?f=14&t=5
Tutaj źródło w przejzystej formie: http://ww1.pelp.net/new/beynar/libs/db

[PHP] pobierz, plaintext 
<?php
//
// mysql service class
// version 1.12 stable
// refere /docs/db for documentation
// beynar
//
 
class db{
    
    // ini config path
    const DB_CONFIG_PATH = &#092;"./config/config.ini\";
    
    // singleton
    private static $instance;
    private static $link;
    private static $queries=0;
 
    // access properties
    private $host;
    private $login;
    private $database;
    
    // data
    private $resource=false;
    private $data=false;
    private $status=NULL;
    private $rows=0;
 
 
    public static function getInstance(){
        // singleton object creator
    if(!isset(self::$instance))
        self::$instance = new db();
    return self::$instance;
    }
 
 
    private function __construct(){
    // read config
    $cfg=parse_ini_file(self::DB_CONFIG_PATH, true);
 
    // establish connection to rdbms
    if($link=mysql_connect($cfg[&#092;"db\"][\"host\"], $cfg[\"db\"][\"login\"], $cfg[\"db\"][\"password\"]))
        self::$link = $link;
    else $this -> error();
        $this -> host = $cfg[&#092;"db\"][\"host\"];
    $this -> login = $cfg[&#092;"db\"][\"login\"];
 
    // select db        
    if($db=mysql_select_db($cfg[&#092;"db\"][\"database\"]))
        $this -> database = $cfg[&#092;"db\"][\"database\"];
    else $this -> error();
    }
 
        public function query($query){
    if(!$resource=mysql_query($query, self::$link)){
        $this -> error($query);
        return false;
        }
    self::$queries++;
        // gets data from resource as array
    if(is_resource($resource)){
        $this -> resource = $resource;
        $this -> data = null;
        while($res=mysql_fetch_array($this -> resource, MYSQL_ASSOC))
        $this -> data[]=$res;
        mysql_free_result($resource);
        }
    else if(is_bool($resource))
        $this -> status = $resource;
    return true;
    }
 
 
    public function numRows(){
    if(is_array($this -> data)){
        $rows=count($this -> data);
        $this -> rows = $rows;
        return $rows;
        }
    }
 
        public function numQueries(){
    return self::$queries;
    }
 
            public function getData(){
    return $this -> data;
    }
            public function getStatus(){
    return $this -> status;
    }
            public function getResource(){
    return $this -> resource;
    }
            public function getHostname(){
    return $this -> host;
    }
            public function getDbName(){
    return $this -> database;
    }
    
    public function getUsername(){
    return $this -> login;
    }
 
 
    // DEBUGING METHOD
    // ONLY FOR DEVELOPING TIME
    private function error($query=false){
        if($query)
        echo &#092;"\".$query.\"\";
    echo &#092;"\".mysql_errno().\" = \".mysql_error().\"\";
    }
            public function __destruct(){
    mysql_close(self::$link);
    }
}
?>
[PHP] pobierz, plaintext 

metoda error finalnie bedzie spisywala błędy do logow lub po prostu throw exception...

Ten post edytował Beynar 27.12.2007, 20:45:22

Powód edycji: poprawiłem bbcode (cysiaczek)

[Cysiaczek]

Przenoszę na Oceny

Szczerze mówiąc, to nie bardzo jest co oceniać :|
Kod da się napisać w 15 minut, a podobny efekt da użycie PDO
Obsługuje tylko MySQL, więc powinna się nazywać mysql_db

Nie mówię, że jest zupełnie źle, ale, jeszcze troszkę musisz popracować ;]

Pozdrawiam i powodzenia