mejora mi coneccion a la db

[thehack]

hola alguno quien pueda ayudarme como mejorar mi conexion a la db implementando el patron singleton aceptare algunas criticas constructivas y ayudas para mejorar este codigo saludos espero puedan ayudarme aca siemrpe lo han echo y de verdad me a servido mucha su ayuda siempre cuando tngo duda vengo hasta aca para esperar alguna respuesta de alguien bueno aca el codigo php saludos

Código PHP:

 class Database{ 
 
     
    public    $debug = true; 
 
     
    private static $instance; 
 
    private    $server   = "localhost"; //database server 
    private    $user     = "root"; //database login name 
    private    $pass     = ""; //database login password 
    private    $database = "forodescarga"; //database name 
 
    private    $error = ""; 
 
     
    public    $affected_rows = 0; 
 
    private    $link_id = 0; 
    private    $query_id = 0; 
 
 
 
private function __construct($server=null, $user=null, $pass=null, $database=null){ 
    // error catching if not passed in 
    if($server==null || $user==null || $database==null){ 
        $this->oops("error"); 
    } 
 
    $this->server=$server; 
    $this->user=$user; 
    $this->pass=$pass; 
    $this->database=$database; 
} 
 
 
 
public static function obtain($server=null, $user=null, $pass=null, $database=null){ 
    if (!self::$instance){  
        self::$instance = new Database($server, $user, $pass, $database);  
    }  
 
    return self::$instance;  
} 
 
 
public function connect($new_link=false){ 
    $this->link_id=@mysql_connect($this->server,$this->user,$this->pass,$new_link); 
 
    if (!$this->link_id){//open failed 
        $this->oops("errro: <b>$this->server</b>."); 
        } 
 
    if(!@mysql_select_db($this->database, $this->link_id)){ 
        $this->oops("error: <b>$this->database</b>."); 
        } 
 
    // unset the data so it can't be dumped 
    $this->server=''; 
    $this->user=''; 
    $this->pass=''; 
    $this->database=''; 
} 
 
 
 
public function close(){ 
    if(!@mysql_close($this->link_id)){ 
        $this->oops("Connection close failed."); 
    } 
} 
 
 
 
public function escape($string){ 
    if(get_magic_quotes_runtime()) $string = stripslashes($string); 
    return @mysql_real_escape_string($string,$this->link_id); 
} 
 
 
 
public function query($sql){ 
    // do query 
    $this->query_id = @mysql_query($sql, $this->link_id); 
 
    if (!$this->query_id){ 
        $this->oops("<b>MySQL Query fail:</b> $sql"); 
        return 0; 
    } 
     
    $this->affected_rows = @mysql_affected_rows($this->link_id); 
 
    return $this->query_id; 
} 
 
 
 
public function query_first($query_string){ 
    $query_id = $this->query($query_string); 
    $out = $this->fetch($query_id); 
    $this->free_result($query_id); 
    return $out; 
} 
 
public function fetch($query_id=-1){ 
    // retrieve row 
    if ($query_id!=-1){ 
        $this->query_id=$query_id; 
    } 
 
    if (isset($this->query_id)){ 
        $record = @mysql_fetch_assoc($this->query_id); 
    }else{ 
        $this->oops("Invalid query_id: <b>$this->query_id</b>. Records could not be fetched."); 
    } 
 
    return $record; 
} 
 
 
 
public function fetch_array($sql){ 
    $query_id = $this->query($sql); 
    $out = array(); 
 
    while ($row = $this->fetch($query_id)){ 
        $out[] = $row; 
    } 
 
    $this->free_result($query_id); 
    return $out; 
} 
 
 
public function update($table, $data, $where='1'){ 
    $q="UPDATE `$table` SET "; 
 
    foreach($data as $key=>$val){ 
        if(strtolower($val)=='null') $q.= "`$key` = NULL, "; 
        elseif(strtolower($val)=='now()') $q.= "`$key` = NOW(), "; 
        elseif(preg_match("/^increment\((\-?\d+)\)$/i",$val,$m)) $q.= "`$key` = `$key` + $m[1], ";  
        else $q.= "`$key`='".$this->escape($val)."', "; 
    } 
 
    $q = rtrim($q, ', ') . ' WHERE '.$where.';'; 
 
    return $this->query($q); 
} 
 
public function insert($table, $data){ 
    $q="INSERT INTO `$table` "; 
    $v=''; $n=''; 
 
    foreach($data as $key=>$val){ 
        $n.="`$key`, "; 
        if(strtolower($val)=='null') $v.="NULL, "; 
        elseif(strtolower($val)=='now()') $v.="NOW(), "; 
        else $v.= "'".$this->escape($val)."', "; 
    } 
 
    $q .= "(". rtrim($n, ', ') .") VALUES (". rtrim($v, ', ') .");"; 
 
    if($this->query($q)){ 
        return mysql_insert_id($this->link_id); 
    } 
    else return false; 
 
} 
 
 
 
private function free_result($query_id=-1){ 
    if ($query_id!=-1){ 
        $this->query_id=$query_id; 
    } 
    if($this->query_id!=0 && !@mysql_free_result($this->query_id)){ 
        $this->oops("Result ID: <b>$this->query_id</b> could not be freed."); 
    } 
} 
 
private function oops($msg=''){ 
    if(!empty($this->link_id)){ 
        $this->error = mysql_error($this->link_id); 
    } 
    else{ 
        $this->error = mysql_error(); 
        $msg="<b>WARNING:</b> No link_id found. Likely not be connected to database.<br />$msg"; 
    } 
 
     
    if(!$this->debug) return; 
    ?> 
        <table align="center" border="1" cellspacing="0" style="background:white;color:black;width:80%;"> 
        <tr><th colspan=2>Database Error</th></tr> 
        <tr><td align="right" valign="top">Message:</td><td><?php echo $msg; ?></td></tr> 
        <?php if(!empty($this->error)) echo '<tr><td align="right" valign="top" nowrap>MySQL Error:</td><td>'.$this->error.'</td></tr>'; ?> 
        <tr><td align="right">Date:</td><td><?php echo date("l, F j, Y \a\\t g:i:s A"); ?></td></tr> 
        <?php if(!empty($_SERVER['REQUEST_URI'])) echo '<tr><td align="right">Script:</td><td><a href="'.$_SERVER['REQUEST_URI'].'">'.$_SERVER['REQUEST_URI'].'</a></td></tr>'; ?> 
        <?php if(!empty($_SERVER['HTTP_REFERER'])) echo '<tr><td align="right">Referer:</td><td><a href="'.$_SERVER['HTTP_REFERER'].'">'.$_SERVER['HTTP_REFERER'].'</a></td></tr>'; ?> 
        </table> 
    <?php 
} 
 
 
}