Connectie class

<?php 
class db { 
    function connect() { 
        $this->conn = mysql_connect("localhost", "youri_game", "......"); 
        mysql_select_db("youri_game", $this->conn); 
    } 
    function query($sql) { 
        $this->connect(); 
        return mysql_query($sql, $this->conn); 
    } 


    function fetch_object($query) { 
        return mysql_fetch_object($query); 
    } 


} 


$db         = new db; 
$dbres      = $db->query("SELECT * FROM `Users`"); 
$result     = $db->fetch_object($dbres); 
?>

try that

Ik denk alleen dat het niet veilig is.
Want zover ik zie maak je bij elke query een connectie aan waardoor je site straks ontzettend traag gaat lopen.

Ook kan je het makkelijker maken voor je zelf door de functie fetch_object groter te maken door:
dit toe te voegen:
$sql = $this->query($query);
return mysql_fetch_object($sql);

En dan kan je het zo oproepen:
$db = new db;
$result = $db->fetch_object("SELECT * FROM users");

sowieso is ie atm nite echt veilig hij is niet beveiligd tegen sql injectie's etc, en atm is het nog niet echt een class aangezien het nu een hoopje functions die bij elkaar gegooit zijn

Beveilig hem inderdaad tegen sql intjectie en het is al helemaal niet slim om je config gegevens hier zo maar vrij te geven. Neem aan dat je ze op je site zelf al hebt aangepast?

"youri_game", "......"

Ook vind ik de reactie van nielsB zeer geslaagd dit is natuurlijk altijd een mooiere manier. (zet het in het vervolg wel ff in code tags:)

Suc6 met je site verder:)

<?php 
class db {


    protected static $_connection;


    protected function connect() {
        if (self::$_connection)
            return;


        self::$_connection = mysql_connect("localhost", "youri_game", "......"); 
        mysql_select_db("youri_game", self::$_connection); 
    } 
    public function query($sql) { 
        self::$_connection OR $this->connect(); 
        return mysql_query($sql, self::$_connection); 
    } 


    public function fetch_object($query) { 
        return mysql_fetch_object($query); 
    } 


}

Zoiets is al beter...

zie je het "static" voor $_connection?

Dat zorgt er voor dat die waarde statisch is waardoor het niet per instantie van dat object verschillend kan zijn...
db::$_connection zou ook kunnen, maar self is de statische versie van $this.

connect() is nu protected en kan alleen binnen de class (of een van zijn children) aangeroepen worden...
Je hoeft het ook niet meer zelf aan te roepen...

$this->connect() wordt alleen uitgevoerd als self::$conn een of andere vorm van false is (of NULL).
Ook zit er in MySQL::connect() nog een check...