[nox]

Hi,

nach langer pause in sachen php lass ich mich dann doch nochmal hier blicken und muss direkt eine Frage loswerden.

Zu meinem Problem:

PHP-Code:

function getUserOnline($params)
 {
    global $db;

    $sql = "SELECT `userid`
            FROM `" . DBPREFIX . "sessions`";

    $result = $db->query($sql);
    return $db->num_rows($result);

 }

$Smarty->register_function('getuseronline', 'getUserOnline'); 


Funktioniert genau wie es soll, zumindest fast. Ich arbeite nur sehr ser ungern mit globals auf on. Eigentlich nie! Das wollte ich auch nicht ändern jedoch weiss ich nicht wie ich das Object "$db" anders in die funktion reinbekommen sollte.

Für alle die noch nicht wissen was ich meine hier ein kleines Beispiel:

PHP-Code:

function getUserOnline($params)
 {
    $sql = "SELECT `userid`
            FROM `" . DBPREFIX . "sessions`";

    $result = $db->query($sql);
    return $db->num_rows($result);

 }

$Smarty->register_function('getuseronline', 'getUserOnline'); 


somit müsste ich ja das Object ($db) irgendwie bei der Funktion "register_function" mit übergeben, jedoch möchte ich es nicht im template lösen!

Schlechtes Beispiel:

PHP-Code:

function getUserOnline($params)
 {
    $db = $params['db'];

    $sql = "SELECT `userid`
            FROM `" . DBPREFIX . "sessions`";

    $result = $db->query($sql);
    return $db->num_rows($result);

 }

$Smarty->register_function('getuseronline', 'getUserOnline'); 


dazugehörige Template:

Online: {getuseronline db=$db}

Habs jetzt nicht getestet ob so überhaupt funktioniert aber ich denke doch schon.

Also vielleicht jemand einen kleinen Tipp für mich ?

Danke schonmal im vorraus!

Gruß nox

[Bleistift]

Du machst etwas grundlegendes falsch... Im Template SQL-Anfragen zu machen ist ziemlich unschön. Das Template sollte eigentlich schon alle Daten haben. Folglich würde ich dir empfehlen diese Anfrage im PHP-Script zu implementieren, welches dann dein Template aufruft.

[nox]

Zitat von Bleistift

Du machst etwas grundlegendes falsch... Im Template SQL-Anfragen zu machen ist ziemlich unschön. Das Template sollte eigentlich schon alle Daten haben. Folglich würde ich dir empfehlen diese Anfrage im PHP-Script zu implementieren, welches dann dein Template aufruft.

ich mache keine sql abfragen im template das sind die phpdatein im hintergrund!

[Bleistift]

Zitat von Lordnox

ich mache keine sql abfragen im template das sind die phpdatein im hintergrund!

Aber du rufst im Template eine Funktion auf, die einen SQL-Query sendet...

[nox]

stimmt, gut jetzt weiss ich was du meinst, das problem ist, das es Modular aufgebaut sein soll, daher finde ich es sehr unpraktisch alle funktionen auszufürhren und mit $smarty->assign zu übergeben obwohl diese funktion in dem Template evtl. garnicht gebraucht wird!

Wenn du eine bessere Idee hast immer her damit, bin für alles offen!

Gruß nox

[Waq]

- getUserOnline() direkt zusammen mit $db zu verpacken geht nicht, da es in PHP keine Closures gibt

- in einem Objekt geht es schon
Rohbau:

PHP-Code:

class GetUserOnline {
  private $db;
  public function userOnline() {
    $this->db->...;
  }
}
$getUserOnline = new GetUserOnline($db);
$smarty->register_function("getuseronline",array(&$getUserOnline,'userOnline'));