Problem mit Konstruktor

[maramiro]

Hallo
habe ein kleines Problem in einer PHP Klasse.
class Aufgaben extends Conn
{
public function __construct()
{

}
function uebersicht()
{
$query = "SELECT aufgabenID, titel, beschreibung, prio, DATE_FORMAT(beginn,'%d.%m.%Y') AS beginn, DATE_FORMAT(termin,'%d.%m.%Y') AS termin FROM ri_aufgaben ORDER BY termin DESC";
$result = $this->query($query);
if (mysql_num_rows($result) > 0)
{
echo "TEST";
}
}

}
Wenn der Konstruktor vorhanden ist, kommt eine MySql Query Fehlermeldung, als wäre der Select nicht korrekt.
Kaum ist der Konstruktor ausgeklammert klappt alles. An was kann das liegen?
Danke für die Hilfe

[AmicaNoctis]

Hallo,

das liegt bestimmt daran, dass in diesem Falle der Konstruktor der Elternklasse ausgeblendet wird. Ohne Konstruktor wird der Elternkonstruktor benutzt und alles ist in Ordnung. Mit Konstruktor muss dieser mit [FONT="Courier New"]parent::__construct()[/FONT] den Elternkonstruktor explizit aufrufen.

Gruß,

Amica

[maramiro]

Gelöst

Super, genau das war's. Wieder was gelernt. Vielen Dank Amica.
Gruss maramiro

[unset]

Da aber eh nix im Konstruktor steht, kannst du den in der Kindklasse auch einfach weglassen …*dann wird automatisch der nächst-höhere verwendet.

[maramiro]

Der Konstruktor soll später nicht leer sein. Trotzdem Danke für den Tipp. So langsam verstehe ich den Zusammenhang der Konstruktoren in vererbten Klassen.

[combie]

class Aufgaben extends Conn 

Da lauert ein schwerer Design Fehler.

"Aufgaben" ist ein Datenmmodell und "Conn" doch sicher eine Datebankverbindung. Oder?

"extends" bilden immer nur "ist eine" Beziehungen aus.
Hier also: Aufgaben ist eine Datenbank Verbindung.
Für mich hört sich das falsch an.

Besser:
Übergib dem Datenmodell das, was es benötigt, per Konstruktor oder Setter.

[unset]

Jedenfalls sofern man mit Models arbeitet –*was man tun sollte aber nicht muss.

[maramiro]

Mal schauen ob ich das verstanden habe (?):

class Conn
{
    var $connect;
    var $res;
    var $host = DB_HOST;
    var $user = DB_USER;
    var $password = DB_PW;
    var $dbname = DB_DATABASE;
    public function __construct ()
    {
        if(!$this->connect = mysql_connect($this->host, $this->user, $this->password))
        {
            echo "Verbindung zu Mysql fehlgeschlagen...";
        }
        //return $this->conn;
        if (!mysql_select_db($this->dbname, $this->connect))
        {
            echo "Datenbank konnte nicht verbunden werden...";
        }
    }
    function query($sql){
        if (!$this->res = mysql_query($sql, $this->connect))
        {
            echo "Query konnte nicht gesendet werden...";
        }
        return $this->res;
    }
} 

class Aufgaben
{
    public function __construct()
    {
         
    }
    function uebersicht()
    {
        $conn = new Conn();
        $query = "SELECT aufgabenID, titel, beschreibung, prio, DATE_FORMAT(beginn,'%d.%m.%Y') AS beginn, DATE_FORMAT(termin,'%d.%m.%Y') AS termin FROM ri_aufgaben ORDER BY termin DESC";
        $result = $conn->query($query);
        if (mysql_num_rows($result) > 0)
        {
            echo 'TEST';    
        }
    }
    
} 

[unset]

Bitte benutze die PHP-Tags des Forums, andernfalls ist der Quelltext so gut wie unlesbar.

[AmicaNoctis]

Hallo,

bitte ändere deinen Beitrag und benutze die PHP-Tags für den PHP-Quelltext.

Die Connection würde ich als Membervariable speichern und im Konstruktor instanziieren und nicht in jeder Methode einzeln.

Gruß,

Amica