MySQL Verbindungsprobleme über OOP

[LostHope.]

Hallo allerseits,
ich habe Probleme mit meinem Projekt und zwar möchte ich in OOP etwas größeres und komplexeres Programmieren. Derzeit bin ich aber noch am Dazulernen, wenn ihr gute Seiten darüber habt könnt ihr sie gerne posten(bekommt 10 Thanks ).

Hier ist der Code, es wird nichts angezeigt(Problemzone While-Schleife).
Ohne While Schleife bekomme ich meine Ergebnisse.

PHP Code:

<?php

/**
*
* Database Connect
*
**/

$host         = "localhost";
$user         = "root";
$password     = "";
$database     = "database";
$table        = "post";

class MySQL
{
    public  $connection = FALSE;
    private $host;
    private $user;
    private $password;
    private $database;
    
    function connectDB($host,$user,$password)
    {
        $this->connect = mysql_connect(
            $host,
            $user,
            $password
            ) 
                or die (mysql_error());
    }
    
    function selectDB($database)
    {
        mysql_select_db($database)
            or die (mysql_error());
    }
    
    function fetchRow($table)
    {
        $query = mysql_query("SELECT * FROM $table") 
            or die (mysql_error());
        while($row = mysql_fetch_assoc($query))
        {
            $id = $row['id'];
            $content = $row['content'];
        }
    }
}

$MySQL = new MySQL;
$MySQL->connectDB($host,$user,$password);
$MySQL->selectDB($database);
$MySQL->fetchRow(post);

echo $id;
echo $content;

?>

[banjo1]

Beschäftige dich mal mit dem Gültigkeitsbereich von Variablen (local, global).

PHP Code:

    function fetchRow($table)
    {
        $query = mysql_query("SELECT * FROM $table") 
            or die (mysql_error());
        while($row = mysql_fetch_assoc($query))
        {
            $id = $row['id']; //nur in deiner function gültig, $id wird bei jedem Durchlauf überschrieben
            $content = $row['content']; //nur in deiner function gültig, $id wird bei jedem Durchlauf überschrieben
        }
    } 


Quick-and-dirty:

PHP Code:

    function fetchRow($table)
    {
        $query = mysql_query("SELECT * FROM $table") 
            or die (mysql_error());
        while($row = mysql_fetch_assoc($query))
        {
            global $id, $content;    
            $id = $row['id']; //global gültig, $id wird bei jedem Durchlauf überschrieben
            $content = $row['content']; //global gültig, $id wird bei jedem Durchlauf überschrieben
        }
    } 


[Bot_interesierter]

        while($row = mysql_fetch_assoc($query))

ist auch noch falsch, = ist in php kein Vergleichsoperator sondern wird für Zuweisungen verwandt, richtig wäre hier == oder besser noch === zu verwenden.

[banjo1]

Originally Posted by Bot_interesierter

        while($row = mysql_fetch_assoc($query))

ist auch noch falsch, = ist in php kein Vergleichsoperator sondern wird für Zuweisungen verwandt, richtig wäre hier == oder besser noch === zu verwenden.

Schonmal daran gedacht, dass es dort nichts verglichen wird?!

[Bot_interesierter]

Originally Posted by banjo1

Schonmal daran gedacht, dass es dort nichts verglichen wird?!

Ja da hast du recht.

[silon]

-

[hen!]

Was du machen könntest währe z. B. folgendes:

PHP Code:

    function fetchRow($table)
    {
        $qryData = array();
        $query = mysql_query("SELECT * FROM $table") 
            or die (mysql_error());
        $x=0;
        while($row = mysql_fetch_assoc($query))
        {
            $qryData['result'][] = $row;
            $++;
        }
        $qryData['count'] = $x;
        return $qryData;
    } 


PHP Code:

var_dump($MySQL->fetchRow($post)); 


Damit würdest du die Daten inkl. Anzahl der Datensätze ausgeben. Ob dies nun einen tatsächlichen Nutzen bringt (doppeltes durchlaufen beim weiterverarbeiten der Daten) is fraglich. Neben dem Hauch objektorientierung kann man da nicht viel entdecken. Guck dir mal Mysqli an.

Hinzu kommt, dass du *-Selects machst und das ist nicht performant. Du solltest den kompletten Query übergeben.