globale Variablen & oo-Ansatz

[Timo Trallala]

Hi Leutz,

ich muss mich jetzt mal an euch wenden. Habe an anderer Stelle mal ne Codestrecke mit einer Frage dazu veröffentlicht, und bekam von allen Seiten nur Buh-Rufe: Globale Variablen...das macht man nicht, schlechter Stil...die müssen weg!

BSP:

error_reporting (E_ALL);

ini_set("display_errors", TRUE);

class functions {

    function getMainCat($cat)
        {
        global $lang;
        $db1 = new dbconn(host, user, pass, name);

        if (empty($cat)) {
            return $lang["leftcontent"]["navigation"];
        } else {
            $sql = "select * from " . tbl_catalog . " where cat_id='$cat'";
            $result = $db1->query($sql);

            while ($data = mysql_fetch_assoc($result))
                if ($data["cat_parent_id"]) {
                    return $this->getMainCat($data["cat_parent_id"]);
                } else {
                    return $data["cat_description"];
                }
        }
        } 

nun muss ich aber auf z.b. auf diverse Sprachpakete zurückgreifen. Die befinden sich in einem Ordner lang:
BSP:

$lang["secondary_nav"]["Warenkorb"] = "Warenkorb";
$lang["secondary_nav"]["Konto"] = "Mein Konto";
$lang["secondary_nav"]["Versandkosten"] = "Versandkosten"; 

Aber auch in der config befinden sich best. Array-Vars auf die u.U. zurückgegriffen werden muss. Die hab ich bis dato immer als globale Variablen eingebunden.
Antwort/ Verbesserungsvorschläge habe ich von diesen Nörglern nicht bekommen.
Vielleicht kann mir hier jemand sagen, warum meine Vorgehensweise soooooooo böse & schlecht ist und vor allem, wie ich es anders machen könnte

mfG
derTrallala

[mrhappiness]

Schlecht ist es, weil du dir bspw. die Möglichkeit nimmst, $lang mal in $language_array umzubenennen ohne gleich an tausenden Stellen Anpassungen vornehmen zu müssen.

Besser machen könntest du es, indem du das Array $lang in ein Attribut der Klasse schreibst und innerhalb der Klasse dann mit diesem Attribut arbeitest.

[Timo Trallala]

Besser machen könntest du es, indem du das Array $lang in ein Attribut der Klasse schreibst und innerhalb der Klasse dann mit diesem Attribut arbeitest. [/B]

nur dass ich dich nicht missverstehe:

class functions  {
          $lang=array();
... 

und dann global $lang entfernen?

[mrhappiness]

ja, und irgendwo bevor du mit der Klasse arbeitest einmal

$instanz_der_klasse->lang = $lang; 

[prego]

ich glaube er meint....

class foo {
  private $lang;

  function __constructor($lang){
    $this->lang = $lang;
  }

  function bar(){
    echo $lang['bar'];
  }

}



$foo = new foo($language_array); 

EDIT:

---

tststs... während dem schreiben kommt der hapiness daher

---

[Timo Trallala]

na denn danke!

[Timo Trallala]

Original geschrieben von prego
ich glaube er meint....

class foo {
  private $lang; 

gibts private nicht erst ab php 5?

[prego]

__constructor auch...

//PHP5
class foo {
  private $lang;

  function __constructor($lang){
    $this->lang = $lang;
  }

  function bar(){
    echo $this->lang['bar'];
  }

}

//PHP4
class foo {
  var $lang;

  function foo($lang){
    $this->lang = $lang;
  }

  function bar(){
    echo $this->lang['bar'];
  }

} 

EDIT:

---

Sorry, flüchtigkeitsfehler - danke mrhappiness

---

[mrhappiness]

Wenn du jetzt noch $lang['bar'] durch $this->lang['bar'] ersetzt, ist alles in Ordnung...

[prego]

natürlich

[Timo Trallala]

nun auszugsweise siehts dann so aus bei mir:

error_reporting (E_ALL);

ini_set("display_errors", TRUE);

class functions {
    var $lang;

    function functions($lang)
        {
        $this->lang = $lang;
        }

    function getMainCat($cat)
        {
        $db1 = new dbconn(host, user, pass, name);

        if (empty($cat)) {
            return $this->lang["leftcontent"]["navigation"];
        } else {
            $sql = "select * from " . tbl_catalog . " where cat_id='$cat'";
            $result = $db1->query($sql);

            while ($data = mysql_fetch_assoc($result))
                if ($data["cat_parent_id"]) {
                    return $this->getMainCat($data["cat_parent_id"]);
                } else {
                    return $data["cat_description"];
                }
        }
        }
}

;
/* Instanz der Klasse wird gebildet*/
$functs = new functions($lang); 

leider kommts zur fehlermeldung:
otice: Undefined variable: lang in D:\...\src\db_functions.php on line 1543
dies ist letzte Zeile

[Timo Trallala]

nimm alles zurück!

funktioniert einwandfrei!

D-A-N-K-E