PHP Klassen Aufbau

MyXoToD

Hallo liebes Forum. Ich hab mich dazu entschlossen meine Seite mal Objektorientiert zu programmieren. Hab aber sofort zu beginn die ersten Schwierigkeite.

Hier mal mein geplanter Aufbau:

index.php

PHP

error_reporting(E_ALL);


include('includes/classes/myxotod.class.php');


$myxotod = new myxotod;

myxotod.class.php

PHP

class myxotod {
    public function myxotod() {
        include('includes/cfg/config.cfg.php');
        include('includes/classes/datenbank.class.php');
        $db = new database;
    }
}

Erste Frage: Ist diese Art von Aufbau sinnvoll oder sollte ich das ganze anders machen? Mein problem hierbei ist folgendes:

In der config.cfg.php ist ein Array, dieses heißt $config. Hier stehen unter anderem die Zugangsdaten zur Datenbank drin. Leider komme ich an diese in der datenbank.class.php nicht dran. In der datenbank Klasse versuche ich folgendes:

global $config;
$this->host = $config['host'];

In $config['host'] scheint aber nichts drin zu stehen.
Kann mir hier jemand weiterhelfen? Vielen Dank

jojo87

Irgendwie macht deine Herangehensweise keinen Sinn.
Warum legst du die Konfiguration nicht im DB-Object ab?

PHP

include "config.php";
require_once "database.class.php";
$db = new Database();
$db->setConfig($config);

Achso: includes in Funktionsaufrufen: Großes Pfui!

MyXoToD

Vielen Dank für die Antwort. Leider komme ich immernoch nicht ganz weiter.

myxotod.class.php:

PHP

class myxotod {
    public function myxotod() {
        require('includes/cfg/config.cfg.php');
        require('includes/classes/datenbank.class.php');
        $db = new datenbank;
        $db->setConfig($config);
    }
}

datenbank.class.php:

PHP

public function datenbank() {
        $config = $this->config;
        $this->host = $config['host'];
        $this->username = $config['username'];
        $this->password = $config['password'];
        $this->datenbank = $config['datenbank'];
        $this->verbindung = @mysql_connect($this->host, $this->username, $this->password);
        if(!$this->verbindung) {
            $this->error(4);
        } else {
            if(!@mysql_select_db($this->datenbank, $this->verbindung)) {
                $this->error(5);
            } else {
                $this->connected = true;
                mysql_query("SET NAMES 'utf8'", $this->verbindung);
                mysql_query("SET CHARACTER SET 'utf8'", $this->verbindung);
                $this->query_counter = 0;
            }
        }
    }

    public function setConfig($config) {
        $this->config = $config;
    }

Alles anzeigen

Leider kommt der Inhalt des $config-Arrays nicht in der Datenbankklasse an
Wieso sollte ich lieber require statt include verwenden?

jojo87

wollen wir wetten, dass in der Klasse myxotod die Variable $config gar nicht existiert?
Für Klassen sollte es require_once sein, da das Fehlen einer Klasse meist zum Abbruch der Applikation führen soll.

Tobse

Das kommt nicht an, weil du im Konstruktor auf die config zugreiffen willst, die du erst danach über setConfig übergibst. Der Konstruktor wird aber schon bei

PHP

$db=new datenbank;
// btw, new datenbank; geht zwar, halte ich aber für unsauber
$db=new datenbank();

ausgeführt.
Schau dir mal OOP in PHP an, hier: http://php.net/manual/de/language.oop5.php

Zitat von jojo87

Achso: includes in Funktionsaufrufen: Großes Pfui!

Wieso? Wenn

PHP

function ladeIrgendwas() {
    include("iwas_mit_infos.php");
    $auswerten=$IFNO;
}

schreibe, sind die inhalte von iwas_mit_infos.php nur in der Funktion verfügbar - kann doch sein, dass das durchaus im interesse liegt?

MyXoToD

jojo87: Im Konstruktor der Klasse myxotod kann ich $config ohne Probleme ausgeben. Es steht auch das drin, was drin stehen soll.
Tobse: Hab den Klasse jetzt mal mit "()" erstellt. Das wird aber an meinem Problem leider nichts ändern^^ Ob man jetzt mit include oder require am besten Dinge einbindet sei auch mal so dahingestellt. Ich hab hier eigentlich gerade ein anderes Problem

Tobse

Hast du meinen Post auch ganz gelesen?
Probier mal das:

PHP

$db=new datenbank($config);
// ....
class datenbank {
          public function __construct($config) {
         $this->host = $config['host'];
                 $this->username = $config['username'];
                 $this->password = $config['password'];
                 $this->datenbank = $config['datenbank'];
                 $this->verbindung = @mysql_connect($this->host, $this->username,                   $this->password);
                 if(!$this->verbindung) {
                          $this->error(4);
                 } else {
              if(!@mysql_select_db($this->datenbank, $this->verbindung)) {
                              $this->error(5);
                     } else {
                              $this->connected = true;
                              mysql_query("SET NAMES 'utf8'", $this->verbindung);
                              mysql_query("SET CHARACTER SET 'utf8'", $this->verbindung);
                              $this->query_counter = 0;
                      }
             }
    }

Alles anzeigen

The User

__construct benutzen… include in einem Konstruktor macht keinen Sinn, die Objekterzeugung soll irgendwelche Ausgaben erzeugen? Oder sollen Funktionen importiert werden? Beides macht keinen Sinn. Zum Importieren benutze autoloading und require_once. Was soll ein myxotod sein?

Tobse

Häh? Was der Konstruktor von datenbank machen soll ist doch ganz klar - eine verbindung zur datenbank aufbauen. Wer redet da von includen und ausgeben?

Unregistriert

Was soll das bringen:

PHP

$this->username = $config['username'];
$this->password = $config['password'];
$this->datenbank = $config['datenbank'];

Das Umschreiben auf eine andere Variable ist sinnlos, genauso wie sich die Daten überhaupt zu merken.

The User

Zitat von Santos

PHP

class myxotod {
    public function myxotod() {
        include('includes/cfg/config.cfg.php');
        include('includes/classes/datenbank.class.php');
        $db = new database;
    }
}

Da wird ja wohl included. Fürs zweite include machst du lieber Autoloading und fürs erste nimm lieber ne ini-Datei o.ä. Was mit der Variable $db geschieht, ist ungewiss…

@Unregistriert
Das ist kein Umschreiben, er benutzt halt ein assoziatives Array zur Parameterübergabe.

Tobse
Mach lieber Parameter $user, $pw etc.

Tobse

Ich wollte damit eig. Sein problem beheben. aber bevor irgendwer eine klasse schreibt, die die mysql_* funktionen verpakt, sollte man eher gleich

PHP

$db=new mysqli($host, $user, $passwort, $datenbankname);
// FTW!

Unregistriert

[QUOTE=The User;382071Das ist kein Umschreiben[/QUOTE]

PHP

$config = $this->config;
$this->host = $config['host'];
$this->username = $config['username'];
$this->password = $config['password'];
$this->datenbank = $config['datenbank'];

Was soll das sonst sein? Da ist jede Zeile überflüssig, beim mysql_connect auf $this->config['host'], etc. zugreifen und fertig.

MyXoToD

Vielen Dank für eure Antworten. Das "umschreiben" mach ich deswegen, weil ich es dort halt eben gleichmäßig haben will. Da mein Internet gerade nicht richtig will werde ich morgen nochmal weiterprobieren mit euren Tipps. Danke bisher.

MyXoToD

Ich habe das ganze jetzt mal abgeändert. Läuft soweit auch. Hab es jetzt mal so gemacht, wie Tobse es eine Seite zuvor vorgeschlagen hat.

myxotod.class.php

PHP

class myxotod {
// ...
    private $config;


    public function myxotod() {
        // Konfiguration einbinden
        require_once('includes/cfg/config.cfg.php');
        $this->config = $config;

        // Datenbankklasse einbinden und erstellen
        require_once('includes/classes/datenbank.class.php');
        $db = new datenbank($this->config);
    }
// ...
}

Alles anzeigen

datenbank.class.php

PHP

class datenbank {
// ...
    public function datenbank($config) {
        $this->host = $config['host'];
        $this->username = $config['username'];
        $this->password = $config['password'];
        $this->datenbank = $config['datenbank'];
        $this->verbindung = @mysql_connect($this->host, $this->username, $this->password);
        if(!$this->verbindung) {
            $this->error(4);
        } else {
            if(!@mysql_select_db($this->datenbank, $this->verbindung)) {
                $this->error(5);
            } else {
                $this->connected = true;
                mysql_query("SET NAMES 'utf8'", $this->verbindung);
                mysql_query("SET CHARACTER SET 'utf8'", $this->verbindung);
                $this->query_counter = 0;
            }
        }
    }
// ...
}

Alles anzeigen

Vielen dank für die Informationen und Tipps. Was ein myxotod ist? So nenne ich mich sonst, wie man an meiner Webseite erkennen kann^^

Tobse

Dad heisst diene seite wird vonneiner instanz von dir selbst errechnet? Programmiertechnisch kein problem, aber semantisch. Z.B könnte man bei Joomla schreiben (soweit ich weiss nutzt joomla oop eher wenig):

PHP

$cms=new JoomlaCMS();
$cms->seiteLaden();

MyXoToD

Meine Webseite programmiere ich immer selbst aber ich arbeite nebenbei noch mit vielen anderen Systemen. Dort wird als Name der "Haupt"-Klasse auch meistens der Name des Systems verwendet. Zumbeispiel Etomite oder ModX.
Das was ich hier jetzt gerade programmiere soll ja nur unter meiner Seite laufen. Falls ich etwas programmieren würde, was andere auch benutzen könnten, würde ich die Namenswahl anders machen.

Tobse

Ja, mir gings nur drum, dass du das prinzip verwtehst. Eer Sinn von OOP ist ja, im Programmcode eine möglichst gute Nachbildung der (theoretischen) Relität zu haben. Das hilft beim nachvollziehen und beim finden von Fehlern. Aber mach wie du es für richtig hälst, ist ja dein programm ;).

P.S.: Noch ein Bsp:

PHP

$forum=new vBulletin("Forum-Hilfe");
while ($forum->hasUnawnseredThred()==true) {
    $thread=$forum->getnextUnAwnseredThread();
    while(!$thread->solved()) {
        $thread->post(createAwnser($thread->getLastPosting()));
    }
}
echo "alles palleti";

MyXoToD

Komme zu meiner nächsten Frage, werde deswegen jetzt keinen neuen Thread aufmachen, da ich mir sicher bin, dass ihr mir schnell helfen könnt. Versteh nicht ganz wieso es nicht klappt.

Struktur:
index.php
-myxotod.class.php
--cache.class.php

In der Cache Klasse prüfe ich ob der Ordner "/tmp/cache" existiert. Wenn neun soll dieser erstellt werden.

PHP

class Cache {


    public $cachePath;


    //Cache-Pfad speichern
    function Cache() {
        $this->cachePath = $_SERVER['DOCUMENT_ROOT']."tmp/cache/.";
        if (!is_dir(dirname($this->cachePath))) {
            if (!mkdir(dirname($this->cachePath), 0777)) {
                $this->error(1);
            }
        }
    }
// ...

Alles anzeigen

Leider bekomme ich bei dem ganzen hier folgende Fehlermeldung:
"Warning: mkdir() [function.mkdir]: No such file or directory in /www/htdocs/user/root/includes/classes/cache.class.php on line 11"

Folgendes steht in dirname($this->cachePath):
"/www/htdocs/user/root/tmp/cache"

Hat jemand eine Idee was hier falsch ist?

Tobse

Du kannst kein verzeichniss "cache" in "tmp" erstellen, wenn "tmp" nicht existiert. Ergo

PHP

mkdir("tmp");
mkdir("tmp/cache");

BTW 1.: Zum Cachen von variablen/objekten besser sowas wie APC verwenden.
BTW 2.: Ich weiss nicht, wie sicher $_SERVER['DOCUMENT_ROOT'] ist und wie es durch verändern des Requests verändert werden kann, lass es besser weg. Mit relativen Pfaden klappt das auch.

Tags