Problem mit Konstruktor in eienr PHP Klasse

[Nevada208]

Hallo Zusammen,

ich finde für folgendes Problem einfach nicht den Fehler:

Meine Klasse:

class Artikel extends DB
{
    protected $titel;

    public function Artikel($artikel_nr){
        $lange          = $_SESSION['lange']; // Sprache holen
        $spalte_lange   = "b.artikel_titel_".$lange;
        $sql_artikel    = "select $spalte_lange as titel, a.artikel_nr from dbo_artikel a, dbo_artikel_titel b where a.artikel_nr = b.artikel_nr and a.artikel_nr = '$artikel_nr'";
        $res_artikel    = $this->mysqli->query($sql_artikel);
        $a_artikel      = $res_artikel->fetch_object();
        $this->titel    = $a_artikel->titel;
    }

    public function showArtikelTitel(){
        echo $this->titel;
    }
} 

Ausgabe aus der Seite:

$Artikel = NEW Artikel($artikel_nr);
echo $Artikel->showArtikelTitel(); 

bringt die Fehlermeldung:

Fatal error: Call to a member function query() on a non-object in /home/www/....

Wenn ich die Funktion Artikel($artikel_nr) nicht als Konstruktor definiere und umbenenne in Artikel_($artikel_nr) und so aufrufe

$Artikel = NEW Artikel();
$Artikel->Artikel_($artikel_nr);
echo $Artikel->showArtikelTitel(); 

funktioniert es. Warum geht es nicht wenn ich die Funktion als Konstruktor aufrufe?

Gruß René

[Christian Marquardt]

function __construct($artikel_nr)
http://php.lernenhoch2.de/lernen/php...nitialisieren/

[Falk Prüfer]

Hallo,

...Warum geht es nicht wenn ich die Funktion als Konstruktor aufrufe?

Weil du in deinem Konstruktor etwas wichtiges vergessen hast! Es fehlt der Aufruf des Konstruktors der Elternklasse, in der sicherlich auch die Eigenschaft $this->mysqli initialisiert wird.

class Artikel extends DB
{
    protected $titel;

    public function Artikel($artikel_nr){
        parent::DB();
        $lange          = $_SESSION['lange']; // Sprache holen
        $spalte_lange   = "b.artikel_titel_".$lange;
        ... 

oder besser:

class Artikel extends DB
{
    protected $titel;

    public function __construct($artikel_nr){
        parent::__construct();
        $lange          = $_SESSION['lange']; // Sprache holen
        $spalte_lange   = "b.artikel_titel_".$lange;
        ... 

Gruß Falk

Anmerkung: Ich halte es für keine gute Idee eine Artikelklasse von einer Datenbankklasse abzuerben.

[Nevada208]

Hallo Falk,

Anmerkung: Ich halte es für keine gute Idee eine Artikelklasse von einer Datenbankklasse abzuerben.

wie macht man es richtig?

Gruß René

[Falk Prüfer]

Hallo,

...wie macht man es richtig?

Datenbankklassen sind meist nach dem Single-Instance- oder Singleton-Pattern angelegt. Für einen Artikel ist das denkbar ungünstig, da hier durchaus die gleichzeitige Existenz mehrerer Instanzen denkbar ist. Erstelle also eine einfache Artikelklasse oder erbe sie von einer einfachen Basis-Objekt-Klasse ab und gib ihr eine Eigenschaft für die DB-Instanz. Ich kenne jetzt die konkret von dir verwendete DB-Klasse nicht, aber sollte es sich um ein Singleton handeln, dann wäre sowas denkbar:

<?php

class DBObject
{
  protected $db = null;

  public function __construct()
  {
    $this->db = DB::getInstance();
  }
}

class Artikel extends DBObject
{
  protected $titel;

  public function __construct($artikel_nr)
  {
    parent::__construct();
    $lange          = $_SESSION['lange']; // Sprache holen
    $spalte_lange   = "b.artikel_titel_".$lange;  // Hier sollte unbedingt noch eine Maskierung erfolgen!!!
    $sql_artikel    =
     "select $spalte_lange as titel, a.artikel_nr
      from dbo_artikel a, dbo_artikel_titel b
      where a.artikel_nr = b.artikel_nr and a.artikel_nr = '$artikel_nr'"; // Hier sollte unbedingt noch eine Maskierung erfolgen!!!
    $res_artikel    = $this->db->mysqli->query($sql_artikel);
    ...
  }
  ...
}

?>

Ist neben der Artikel-Klasse keine weitere Klasse mit DB-Zugriff geplant, so kann man natürlich auch auf die DBObject-Klasse verzichten und die Eigenschaft $db direkt in die Artikel-Klasse packen.

Gruß Falk

[Nevada208]

Hallo Falk,

danke für Deine Antwort. Was meinst Du mit Maskierung?

Gruß René

[Falk Prüfer]

Was meinst Du mit Maskierung?

==> http://de.wikipedia.org/wiki/SQL-Injection

[Nevada208]

Hallo Falk, danke für die Info.