[PHP][OOP]class werkt niet

[Spruit_elf]

Ik ben een beetje aan het expirimenteren met OOP en php ed en nu probeerde ik een database class werkend te krijgen

mysqldb_obj_inc.php

PHP:

<?php
class mysqldb {

    //  set up the object
    var $host;
    var $db;
    var $user;
    var $password;
    var $sql;
    var $numrows;
    var $connection;
    
    //  Property Get & Set
    function gethost() {
        return $this->host;
    }
    function sethost($req_host) {
        $this->host = $req_host;
    // ff geknipt in de Get & Set functies omdat ze allemaal ongeveer het zelfde zijn als bovenstaande

    //  Magic functions
    function mysqldb() {
        global $mysqlhost, $mysqldatabase, $mysqlusername, $mysqlpassword;
        $this->sethost($mysqlhost);
        $this->setdb($mysqldatabase);
        $this->setuser($mysqlusername);
        $this->setpassword($mysqlpassword);
        $this->setconnection(0);
    }
    //  Methods
    function openconnection() {
        $this->dblink = mysql_connect($this->host, $this->user, $this->password);
        if ($this->connection == 1) {
            $this->db = mysql_select_db($this->db);
            $this->setconnection(1);
        } else {
            $this->setconnection(0);
            return false;
        }
        return true;
    }
    function closeconnection() {
        if ($this->dbconnection = 1) {
            mysql_close($this->dblink);
        }
    }
    function selectquery() {
        if ($this->connection == 0)
            $this->openconnection();
        $this->qry = mysql_query($this->sql);
        if (!$this->qry) {
            echo('[Error:] mysql_query failed');
            return false;
        } else {
            $this->numrows = mysql_num_rows($this->qry);
            if ($this->numrows > 0) {
                for($x = 0; $x > $this->numrows; $x++) {
                    $this->result[$x] = mysql_fetch_array($this->qry);
                }
            } else {
                echo('[Error:] mysql_num_rows failed');
                return false;
            }
            return true;
        }
    }
}
?>

dit is de betreffende class die word gebruikt in

test.php

PHP:

<?
include('include/mysqldb_obj_inc.php');
include('include/conf_inc.php');
$db0 = new mysqldb();
$db0->openconnection();
$db0->setsql("SELECT nid,title FROM news_news ORDER BY nid ASC");
$db0->selectquery();
for ($x = 0; $x < count($db0->result); $x++) {
    echo('<li><a href="?page=home#nid' . $db0->result[$x]['nid'] . '">' . $db0->result[$x]['title'] . '</a></li>');
}
    
echo($db0->result[1]->title);
echo($db0->connection);
echo($db0->result);
echo($db0->qry);
?>

het enige wat ik hier uit krijg is [Error:] mysql_query failed, toch kan ik niet vinden waarom die failed voor zover ik kan zien klopt alles
ben al een hele dag bezig om uit te vinden wat er niet klopt en of alle variabelen wel kloppen, vandaar die echo's aan het eind.

het ligt niet aan de database, die klopt en werkt en daar zit info in.
kan iemand me verder helpen?

[elevator]

Voeg even mysql_error() toe en print je SQL string uit in de "echo mysql_failed" functie - dan heb je denk ik een beter inzicht in wat er mis gaat

[PrisonerOfPain]

Waar heb je setsql gedefinieerd?

[Spruit_elf]

elevator schreef op 25 juli 2004 @ 18:33:
Voeg even mysql_error() toe en print je SQL string uit in de "echo mysql_failed" functie - dan heb je denk ik een beter inzicht in wat er mis gaat

1046: No Database Selected is wat hij geeft al ik

PHP:

echo mysql_errno($this->dblink) . ': ' . mysql_error($this->dblink);

achter de foutmeldingen zet heel vreemd want zover ik kan zien klopt het wel in dat gedeelte
aangezien echo($db0->db); wel de goede database geeft
* Spruit_elf gaat weer ff testen

PrisonerOfPain schreef op 25 juli 2004 @ 18:35:
Waar heb je setsql gedefinieerd?

zie comment in get set deel, die heb ik geknipt, hij werkt precies als sethost

PHP:

    function setsql($req_sql) {
        $this->sql = $req_sql;
    }

[ Voor 54% gewijzigd door Spruit_elf op 25-07-2004 18:55 ]

[Verwijderd]

PHP:

//    Methods 
    function openconnection() { 
        $this->dblink = mysql_connect($this->host, $this->user, $this->password); 
        if ($this->connection == 1) { 
            $this->db = mysql_select_db($this->db); 
            $this->setconnection(1); 
        } else { 
            $this->setconnection(0); 
            return false; 
        } 
        return true; 
    }

Connection heeft in eerste iig niet de waarde 1. Ofwel je maakt connectie met een database server en omdat connection geen 1 is selecteer je nooit een database.

Vervang $this->connection == 1 eens met $this->dblink. Op die manier kijk of er een geldige mysql connectie tot stand is gekomen. Dan zou alles moeten werken.

[Spruit_elf]

Verwijderd schreef op 25 juli 2004 @ 19:20:
[...

ik heb idd ff de open en close connection functies iets beter gemaakt

PHP:

    function openconnection() {
        $this->dblink = mysql_connect($this->host, $this->user, $this->password)
            or die ('[Error:] mysql_connect failed<br />' . mysql_errno($this->dblink) . ': ' . mysql_error($this->dblink) . '<br />');
        mysql_select_db($this->db)
            or die ('[Error:] mysql_select_db failed<br />' . mysql_errno($this->dblink) . ': ' . mysql_error($this->dblink). '<br />');
        return true;
    }
    function closeconnection() {
        mysql_close($this->dblink);
        return true;
    }

nu geeft ie aleen helemaal niets meer, zelfs niet een fout melding

[ Voor 16% gewijzigd door Spruit_elf op 25-07-2004 19:26 ]

[Apollo_Futurae]

PHP:

    function openconnection() {
        $this->dblink = mysql_connect($this->host, $this->user, $this->password);
        if ($this->connection == 1) {
            $this->db = mysql_select_db($this->db);
            $this->setconnection(1);
        } else {
            $this->setconnection(0);
            return false;
        }
        return true;
    }

Als ik het goed begrijp doet deze functie het volgende:
Maak een verbinding met de database.
Kijk vervolgens of er al een verbinding was (dat is toch wat $this->connection aangeeft?); als dat zo is, selecteer dan de juiste database en zet connection op 1 (maar die stond daar al op ); als dat niet zo is, zet dan connection op 0 (maar laat de verbinding intact, zonder een database te selecteren).

Trouwens, mysql_select_db geeft true terug als het selecteren gelukt is en anders false. Is het de bedoeling dat dat wordt opgeslagen in $this->db? Het lijkt me niet.

Waarschijnlijk gebeurt in test.php dit:
Je roept openconnection aan. De verbinding wordt gemaakt, maar $this->connection staat nog op 0 (zo is hij geïnitialiseerd), dus wordt er geen database geselecteerd en wordt $this->connection op 0 gezet.
Later roep je selectquery aan. Deze ziet dat $this->connection op 0 nul staat en roept daarom nogmaals openconnection aan, maar dat doet precies hetzelfde als hierboven: er wordt een verbinding gemaakt, maar geen database geselecteerd. Nu voert selectquery de ingestelde query uit. Dit veroorzaakt een fout, want er is geen database ingesteld.

Als je er nu eens zoiets van maakt:

PHP:

    function openconnection() {
        if ($this->getconnection()) {
            $this->closeconnection();
        }
        $this->dblink = mysql_connect($this->host, $this->user, $this->password);
        if ($this->dblink !== false) {
            if (mysql_select_db($this->db)) {
                $this->setconnection(true);
            } else {
                echo(mysql_error());
                $this->setconnection(false);
            }
        } else {
            echo(mysql_error());
            $this->setconnection(false);
        }
        return($this->getconnection());
    }

Nog over deze functie:

PHP:

    function closeconnection() {
        if ($this->dbconnection = 1) {
            mysql_close($this->dblink);
        }
    }

Ik denk dat dit moet zijn:

PHP:

    function closeconnection() {
        if ($this->getconnection()) {
            mysql_close($this->dblink);
            $this->setconnection(false);
        }
    }

Beetje laat

[ Voor 8% gewijzigd door Apollo_Futurae op 25-07-2004 19:52 . Reden: Beetje laat B) ]

[Apollo_Futurae]

mrcactus schreef op 25 juli 2004 @ 19:24:

PHP:

$this->dblink = mysql_connect($this->host, $this->user, $this->password)
            or die ('[Error:] mysql_connect failed<br />' . mysql_errno($this->dblink) . ': ' . mysql_error($this->dblink) . '<br />');

Je gebruikt nu $this->dblink (nl. in mysql_errno en mysql_error) voordat er iets aan is toegewezen.

$this->dblink is trouwens niet gedeclareerd.

Heb je ergens een error_reporting(E_ALL) staan (of vergelijkbaar)?

[Spruit_elf]

ok heb heb het nu veranderd in dit

PHP:

<?php
class mysqldb {

    //  set up the object
    var $host;
    var $db;
    var $user;
    var $password;
    var $sql;
    var $numrows;
    var $dblink;
    var $result;
    var $qry;
    
    //  Property Get & Set
    function gethost() {
        return $this->host;
    }
    function sethost($req_host) {
        $this->host = $req_host;
    }
// ff geknipt in de Get & Set functies omdat ze allemaal ongeveer het zelfde zijn als bovenstaande,
// alle vars hebben een get en set functie zoals bovenstaande

    
    
    //  Magic functions
    function mysqldb() {
        global $mysqlhost, $mysqldatabase, $mysqlusername, $mysqlpassword;
        $this->sethost($mysqlhost);
        $this->setdb($mysqldatabase);
        $this->setuser($mysqlusername);
        $this->setpassword($mysqlpassword);
    }
    //  Methods
    function openconnection() {
        if ($this->dblink) {
            return true;
        } else {
            $this->dblink = mysql_connect($this->host, $this->user, $this->password);
            if ($this->dblink) {
                mysql_select_db($this->db)
                or die ('[Error:] mysql_select_db failed<br />' . mysql_errno($this->dblink) . ': ' . mysql_error($this->dblink). '<br />');
            } else { 
                echo('[Error:] mysql_connect failed<br />' . mysql_errno($this->dblink) . ': ' . mysql_error($this->dblink) . '<br />');
            }
        }
    }
    function closeconnection() {
        if (!$this->dblink) {
            mysql_close($this->dblink);
        } else {
            echo('[Error:] Connection already closed');
        }
        return true;
    }
    function selectquery() {
        if ($this->dblink)
            $this->openconnection();
        $this->qry = mysql_query($this->sql);
        if (!$this->qry) {
            echo('[Error:] mysql_query failed' . mysql_errno($this->dblink) . ': ' . mysql_error($this->dblink). '<br />');
            return false;
        } else {
            $this->numrows = mysql_num_rows($this->qry);
            if ($this->numrows > 0) {
                for($x = 0; $x > $this->numrows; $x++) {
                    $this->result[$x] = mysql_fetch_array($this->qry);
                    echo('a');
                }
                echo('b');
            } else {
                echo('[Error:] mysql_num_rows failed<br />' . mysql_errno($this->dblink) . ': ' . mysql_error($this->dblink). '<br />');
                return false;
            }
            return true;
        }
    }
}
?>

en

PHP:

<?
error_reporting(E_ALL);
include('include/mysqldb_obj_inc.php');
include('include/conf_inc.php');
$db0 = new mysqldb();
$db0->openconnection();
$db0->setsql("SELECT nid,title FROM news_news ORDER BY nid ASC");
$db0->selectquery();
for ($x = 0; $x < count($db0->result); $x++) {
    echo('<li><a href="?page=home#nid' . $db0->result[$x]['nid'] . '">' .     $db0->result[$x]['title'] . '</a></li>');
}
echo(implode(',',get_object_vars($db0)));
?>

de complete output is nu

blocalhost,database,user,password,SELECT nid,title FROM news_news ORDER BY nid ASC,2,Resource id #4,,Resource id #6

database user en password zijn, offcourse ff vranderd

de eerste b voor localhost hoort niet bij de var maar die komt van de echo('b'); die tussen de selectquery staat

het lijkt mij nu alsof php de for loop overslaat, maar ik zie geen reden waarom die dat zou doen

[ Voor 38% gewijzigd door Spruit_elf op 25-07-2004 20:20 ]

[PrisonerOfPain]

$x > $this->numrows zal de eerste keer al false opleveren omdat $x 0 is, $x < $this->numrows niet

[Spruit_elf]

....
* Spruit_elf krijgt rode kleur
damn wat een klote fout
tnx nu werkt het

[Apollo_Futurae]

mrcactus schreef op 25 juli 2004 @ 20:17:

PHP:

$this->numrows = mysql_num_rows($this->qry);
if ($this->numrows > 0) {
    for($x = 0; $x > $this->numrows; $x++) {
        $this->result[$x] = mysql_fetch_array($this->qry);
        echo('a');
    }
    echo('b');
} else {
    echo('[Error:] mysql_num_rows failed<br />' . mysql_errno($this->dblink) . ': ' . mysql_error($this->dblink). '<br />');
    return false;
}

Als mysql_num_rows 0 geeft, betekent dat dat de query nul rijen opgeleverd heeft, niet dat er iets mis is gegaan.

Je zou het ook zo kunnen doen:

PHP:

$this->result = array();
while (false !== ($row = mysql_fetch_array($this->dblink))) {
    $this->result[] = $row;
}

offtopic:
Als je getters en setters niet gebruikt, hoef je ze ook niet te maken .