[PHP/MySQL] Database class

[gyarnoc]

Gegroet,

Ik heb op tweakers gezocht naar dit probleem, maar niks gevonden.. Het leek me nogal simpel, maar snap niet wat ik fout doe...

Ik heb de volgende class:

PHP:

class dbcon {
    var $db;
    var $result;
    var $row;
    function connect()
    {
        $this->$db = mysql_connect($sql_host, $sql_dbuser, $sql_dbpass);
        mysql_select_db($sql_dbname,$this->$db);
    }
    function query($query)
    {
        $this->$result = mysql_query($query,$this->$db) OR die("MySQL Error!");
        $this->$row = mysql_fetch_array($this->$result);
        
        return $this->$row;
    }
}

Gemaakt om (wat ik dacht..) makkelijker queries uit te voeren.. maar als ik nu dit doe:

PHP:

$dbcon = new dbcon;

$row = $dbcon->query("SELECT * FROM config");

Krijg ik, wat ik ook doe, de volgende error: Warning: mysql_query(): supplied argument is not a valid MySQL-Link resource in d:\webserver\www\develop\mainfile.php on line 23
MySQL Error!

Dat is dit stukje: $this->$result = mysql_query($query,$this->$db) OR die("MySQL Error!");

Om toe te geven.. 1e keer dat ik met classes werk in PHP, maar doe ik nu iets principeels fout of kan dit gewoon niet in een class of iets?

Gegroet!!

[Verwijderd]

Ik heb altijd een de fetch_array en de query gesplit in 2 stukken, zeker omdat ik wel meer dan alleen query's uit wil voeren, waar ik ook andere acties op wil uitvoeren dan een fetch_array.

Deze komt uit een project waar ik nou mee bezig ben, nog niet echt optimaal, maar werkt wel redelijk. Ik gebruik KORTE namen, zodat ik niet veel hoef te scripten.
Ik heb liever $db->qu(query) dan $database->query_execute_commando_dingetje(query)

PHP:

    //query's and fetches
    function qu($query) {
        //turn ($this->dbconnected==true AND $ret = mysql_query($query))?$ret:false;
        $ret = mysql_query($query) or die(mysql_error());
        return $ret;
    }
    function fa($data) {
        return ($this->dbconnected==true AND $ret=mysql_fetch_array($data))?$ret:false;
    }
    //end query's and fetches

[Erkens]

ze je error_level eens op E_ALL en je ziet wat je fout doet

$this->$db kan niet, (kan wel maar nu even niet, en het is ranzig) dit moet zijn $this->db, zo ook met al die andere

[gyarnoc]

Had ik eerst ook al gedaan.. maar toen werkte het ook niet, het is dus nu:

PHP:

class dbcon {
    var $db;
    var $result;
    var $row;
    function connect()
    {
    global $sql_host, $sql_dbuser, $sql_dbpass, $sql_dbname;

        $this->db = mysql_connect($sql_host, $sql_dbuser, $sql_dbpass);
        mysql_select_db($sql_dbname,$this->db);
    }
    function query($query)
    {
        $this->result = mysql_query($query,$this->db) OR die("MySQL Error!");
        $this->row = mysql_fetch_array($this->result);
        
        return $this->row;
    }
}

Maar krijg precies dezelfde error.. en de 1e reply lost niet echt mijn probleem op

[LuCarD]

vergeet je niet de connect eerst aan te roepen?

[gyarnoc]

Ik dacht dat de 1e functie de constructor was? Of is dat niet zo..

EDIT: Ah.. opgelost, Ik dacht dat ik ergens had gelezen dat de 1e functie de constructor was.. mjawh.. verkeerd gelezen he..

[ Voor 55% gewijzigd door gyarnoc op 10-10-2005 16:05 ]

[LuCarD]

http://nl2.php.net/manual/nl/language.oop5.decon.php

For backwards compatibility, if PHP 5 cannot find a __construct() function for a given class, it will search for the old-style constructor function, by the name of the class. Effectively, it means that the only case that would have compatibility issues is if the class had a method named __construct() which was used for different semantics.

Dus als je de functie connect hernoemt naar dbcon of __construct dan word hij wel aan geroepen als constructor

[masq]

Zou je niet eerst $dbcon->connect() aanroepen voordat je met queries aan de slag gaat?


[edit]
never mind

[ Voor 17% gewijzigd door masq op 10-10-2005 16:08 ]

[mcB]

1> Doe wat Erkens hierboven al zei, je vars goed aanroepen.
2> Connectparameters instantieren
3> Connect aanroepen
4> Dan pas query doen

[Verwijderd]

Voeg de volgende functie toe om via de constructor connect aan te roepen

PHP:

function dbcon() {
  this->connect();
}

En dan in je code

PHP:

$dbcon = new dbcon();

[ Voor 3% gewijzigd door Verwijderd op 11-10-2005 15:00 ]

[Bosmonster]

Is het sowieso niet wat onzinnig een database class te maken die maar 1 row terug kan geven

[ Voor 74% gewijzigd door Bosmonster op 11-10-2005 15:12 ]

[PhoeniX-]

Ik zou er zo iets van maken:

PHP:

class dbcon {
    var $db;
    var $resultset;
    var $result;

    function dbcon($host, $user, $pass, $db_name){
        $this->db = mysql_connect($host, $user, $pass);
        mysql_select_db($db_name,$this->db);
    }

    function query($query){
        $this->resultset = mysql_query($query,$this->$db) OR die("MySQL Error!");
        $this->result = mysql_fetch_array($this->resultset);
        
        return $this->result;
    }
}

Ik heb je constructor aangepast zodat hij de connectie opbouwt en je kunt nu dus ook een instantie aanmaken met connectieparameters. Dit doe je vervolgens zo (ook wat aangepast uit je originele post ):

PHP:

$dbcon = new dbcon("localhost", "username", "password", "databasenaam");

$row = $dbcon->query("SELECT * FROM config");

Ik heb het niet getest, dus er kan een typootje in zitten .. Maar ik denk dat je het idee wel ziet?