Datenbank class

[Ghost21]

Hallo liebes Forum.
Nach langer Zeit habe ich mich wieder dran gesetzt mich mit PHP zu beschäftigen und bin hier auf ein Fehler gestoßen. Ich habe schon etwas gegoogle't allerdings seh ich mein Fehler nicht ganz * *
Meine Klasse

class datenbank
    {

            private $connection = Null;
            private $result = Null;
            private $counter = Null;


                public function __construct($host=NUll, $user=NUll, $pw=NUll)
                {

                    if($this->connection == Null)
                    {
                        $this->connection = mysqli_connect($host, $user, $pw);
                    }
                    else
                    {
                        $this->connection = mysql_connect($host, $user, $pw);
                    }

                }

                public function db_auswahl($db)
                {

                    if($this->connection == true)
                    {
                        if(is_resource($this->connection))
                        mysqli_select_db($db, $this->connection);
                    }
                    else
                    {
                        if(is_resource($this->connection))
                        mysql_select_db($db, $this->connection);
                    }

                }

                public function ausgeben($query)
                {

                    $this->result = mysqli_query($this->connection, $query);
                    [B]$fetch_row = mysqli_fetch_row($this->result);[/B]}
                    $this->counter = NUll;
                }

                public function schließe_db()
                {
                    if(is_resource($this->connection))
                    {
                        mysqli_close($this->connection);
                    }
                }


    } 

und so rufe ich die Klasse auf

    var_dump($db = new datenbank("host","user","Passwort")); 

Hier stelle ich Die Verbindung zur Datenbank auf. Hier wird mir auch ein Array angezeigt, die Verbindung steht.

    var_dump($db->db_auswahl("tutorial")); 

Hier wähle ich meine Datenbank aus. Schon hier bekomm ich eine Fehlermeldung.
Warning: mysqli_fetch_row() expects parameter 1 to be mysqli_result, boolean given in C:\xampp\htdocs\Class\Datenbank.php on line 45

    var_dump($db->ausgeben("SELECT * FROM `spiele`")); 

&'nd hier wird mir nur NULL angezeigt.

Hinweiße, wie ich auf mein Fehler stoße wäre sehr nett.
Ich danke schon mal im voraus für eure Hilfreichen antworten.

[VPh]

Hallo,

http://php.net/manual/de/mysqli.error.php
Das würde ich ein bisschen mit untermischen.

Hier wähle ich meine Datenbank aus. Schon hier bekomm ich eine Fehlermeldung.

Bei der Datenbankauswahl? Und wie lautet die Fehlermeldung?

                    if($this->connection == true) 
                    { 
                        if(is_resource($this->connection)) 
                        mysqli_select_db($db, $this->connection); 
                    } 
                    else 
                    { 
                        if(is_resource($this->connection)) 
                        mysql_select_db($db, $this->connection); 
                    } 

Da vermischst du mysql_* und mysqli_*

[Ghost21]

Wie soll ich's genau beschrieben.
Also mir wird da einfach NULL angezeigt. Egal ob ich eine Datenbank auswähle die Existiert oder nicht.
Was ja wiederrrum heißt, dass es die Verbindung nicht herstellen kann. Allerdings steht die Datenbank verbindung also versteh ich den Fehler da irgendwie nicht.

PS: habe die Connection nun auch mysqli_error hinzugefügt.

$this->connection = mysqli_connect($host, $user, $pw) or die ("<br/>" . mysqli_error); 

Hier werden mir auch keine Fehler angezeigt

[monolith]

var_dump($db->db_auswahl("tutorial")); 

1. Was soll var_dump() dort bringen? Die Methode gibt nichts zurück also gibt var_dump() immer null aus.

2.

if(is_resource($this->connection)) 

Wird nicht zu true auch wenn eine Verbindung etabliert wurde. is_resource() unterstützt mySQLi nicht (bzw. eine mySQLi Verbindung ist ein Objekt und keine Ressource.)
Ich weiß auch nicht was diese Bedingung bringen soll, lass sie einfach weg.

3. Die Syntax für mysqli_select_db ist:

bool mysqli_select_db ( mysqli $link , string $dbname )

Du hast die Parameter vertauscht.

BTW: Hat es einen Grund, warum du zum Auswählen der Datenbank eine eigene Methode baust? Das könnte man auch direkt im Konstruktor mit einbauen:

public function __construct($host = null, $user = null, $pw = null, $db = null) 
{ 
    if (! $this->connection) 
    { 
        $this->connection = mysqli_connect($host, $user, $pw, $db); 
        ... 

4.

var_dump($db->ausgeben("SELECT * FROM `spiele`")); 

Auch das gibt nichts aus, da die Methode ausgeben() keinen Rückgabewert liefert. Füge der Methode hinzu:

return $fetch_row; 

[Ghost21]

monolith.
Ok, vielen Dank. Wusste nicht. Welche möglichkeit, zu schauen ob die Verbindung steht außer is_resource() gibt es sonst?
&'nd zu 3 vielen Dank, werde dies dan gleich mal umschreiben (:

[monolith]

Welche möglichkeit, zu schauen ob die Verbindung steht außer is_resource() gibt es sonst?

Falscher Ansatz, meines Wissens nach. Im Konstruktor schaust du ob es einen Fehler beim Aufbau der DB-Verbindung gibt. Falls ja setzt du $this->connection auf null und wirfst eine exception.

if (mysqli_connect_error()) {
    $this->connection = null;
    throw new Exception('Connection error: '.mysqli_connect_error());
} 

Dann kannst du einfach immer prüfen...
- ob eine Verbindung existiert:

if ($this->connection) 

- oder nicht:

if (! $this->connection) 

Sprich die Idee ist: Nur wenn der Verbindungsaufbau gelingt hinterlege ich in $this->connection überhaupt etwas. D. h. wenn sich dort ein Objekt befindet - was man mit if ($this->connection) überprüfen kann - kann man davon ausgehen, dass eine Verbindung existiert.

[Ghost21]

Ok. Ja vielen herzlichen dank für den hinweiß. Werde es sofort ausprobieren & umschreiben.

[tr0y]

Das was du da bauen willst ist ein Datenbank Adapter: https://gist.github.com/Golpha/8446061

[hausl]

Was ist der Sinn von sowas?

[tr0y]

Was ist der Sinn von sowas?

http://de.wikipedia.org/wiki/Inversion_of_Control
http://de.wikipedia.org/wiki/Lazy_Loading

[monolith]

Lazy Loading in dem Sinne, dass "global" nur eine einzige Verbindung aufgebaut wird?

[tr0y]

Lazy Loading in dem Sinne, dass "global" nur eine einzige Verbindung aufgebaut wird?

Lazy Loading in dem Sinne das die Verbindung nur dann Aufgebaut wird wenn sie benötigt wird und das auch nur einmal.

[monolith]

Stimmt, gar nicht gesehen dass da 'ne Closure mit verwendet wird die die Verbindung erst auf Anfrage etabliert.

[Ghost21]

Zum Abschluss hätte ich eventuell noch eine frage.
Ich habe die Klasse jetzt etwas um strukturiert, dass die Verbindung nicht im _-constructor aufgebaut wird sondern eine Methode mit der ich die Verbindung aufbaue.
Meine frage nun, kann/wie bekomm ich die Verbindungsdaten in nem array rein ?
also z.b.

<?PHP

class Database
{

private $db-co = array();

public function __construct(db-co['host'] . . .)
{

...

}
?>

Den so Funktioniert es ja nicht, da ich ja keine " [ " zeichen dadrin haben darf.

Sorry, dass die Frage noch etwas Amateurhaft ist
Oder eventuell ein "Stichwort" wonach ich googlen kann wäre auch sehr Hilfreich.

[monolith]

Na einfach so und als Parameter übergibst du halt einen Array:

public function __construct(array $dbCon) {
    $this->dbCon = $dbCon;
} 

Instanziierung kann dann so erfolgen:

$db = new Database(array('host' => 'localhost', 'user' => 'username', 'password' => 'password'));