PHP - Best Practice für die Übergabe von Variablen Include-Dateien

Dies ist mehr eine "Wie sollte ich? "anstatt" Wie tun ich?' Frage.

In der Regel, was als der beste Weg zur übergabe von Variablen an eine Datei enthalten?

Zum Beispiel, sagen wir, ich bin mit einem fragment für ein Menü, und wollen eines der Menüelemente aus (das aktuelle) eine bestimmte Klasse name (dies ist eine sehr generische Probe - nicht etwas, was ich bin tatsächlich mit):

<?php
$links = array(
    array('text' => 'home', 'href' => 'home.php'),
    array('text' => 'about', 'href' => 'about.php'),
    array('text' => 'contact', 'href' => 'contact.php') 
);
?>
<ul>
<?php for($i = 0; $i < 3; $i++) : 
    $link = $links[$i];
    $is_active = ($i == $active_index);
?>
    <li><a <?=($is_active ? 'class="active"' : '')?> href="<?=$link['href']?>"><?=$link['text']?></a></li>
<?php endfor; ?>
</ul>

ich rufe die oben 'menu.inc.php'. offensichtlich sieht es für eine variable (int) aufgerufen $active_index zu bestimmen, die einen link zu geben '.active' - Klasse.

so könnte man einfach definieren $active_index vor dem Aufruf die gehören, aber dies scheint wie eine schlechte Praxis, um mir da vielleicht eine variable mit diesem Namen könnte die definiert worden sind, etwas anderes zu früheren und einen späteren Teil des Skriptes ist immer noch auf der Suche für Sie.

oder - Sie können einen absoluten Pfad und fügen Sie Variablen mit einem querystring (include 'menu.inc.php?active_index=1'), aber das scheint wie eine schlechte Idee, da brauchst du die 'echte' $_GET innerhalb einer bestimmten include.

oder - Sie können beginnen, jedes enthaltene Datei mit ob_start und zurück ob_end_clean(), dann verwenden Sie etwas wie diese zu bekommen, die Rückkehr:

function load_view($file, $variables){
  extract($variables);
  return include($file);
}
//passed like
<?=load_view('menu.inc.php', array('active_index' => 2))?>

aber auch dies scheint eine Reihe von Nachteilen (mit bis zu restrukturieren, alle Ihre include-Dateien entsprechend mit den ob Funktionen und eine return - Anweisung).

InformationsquelleAutor momo | 2011-02-09
  1. 4

    Ich wie ein Objekt für diese, wie beschrieben in diesem MVC-stack-post. In einer Datei namens viewMenu.class.php, 

    class viewMenu
  {
  private $active_link;

  public function __construct ( $active_link )
    {
    $this->active_link = $active_link; 
    }
  //If the constructor doesn't work for you, try a "set" method.  

  public function view ()
    {
    $active_link = $this->active_link;
    include_once ( "menu.inc.php" );
    }
  }

    Definieren $active_link innerhalb der view-Methode enthält den Geltungsbereich von Variablen von $active_link innerhalb der Methode. Dann nennen Sie diesen code:

    $aViewMenu = new viewMenu( $active_link );
$aViewMenu->view();

    Allerdings bin ich mir fast neue MVC in PHP, und ich begrüße jeden Zweifel erhaben.

    • Dies ist ähnlich zu dem Ansatz im Zend Framework views. Die Steuerung setzt das Modell in der Ansicht (ViewMenu übernimmt sowohl die Steuerung als auch das Modell-Rollen oben) und führt dann einen view Skript über include innerhalb der Ansicht ist render() Methode. Man könnte den gleichen Effekt erzielen, vor, indem er die $active_link lokale variable und es in das mitgelieferte Skript mit $this->active_link.
    InformationsquelleAutor jpwco
  2. 1

    Persönlich würde ich einfach definieren $active_index vor darunter, oder, dass vielleicht ein wenig mehr inline-gute programmierpraktiken, stellen Sie die Menü-Generierung eine Funktion mit einem $active_index parameter.

    Beispielsweise (verzeihen Sie das Chaos, das folgt):

    <?php //menu.php
$links = array(
    array('text' => 'home', 'href' => 'home.php'),
    array('text' => 'about', 'href' => 'about.php'),
    array('text' => 'contact', 'href' => 'contact.php') 
);

function generate_menu($active_index)
{
?>
    <ul>
    <?php
    $linkcount = count($links);
    for($i = 0; $i < $linkcount; $i++)
    {
        $link = $links[$i];
        $is_active = ($i == $active_index);
    ?>
        <li><a <?=($is_active ? 'class="active"' : '')?> href="<?=$link['href']?>"><?=$link['text']?></a></li>
    <?php
    }
    ?>
    </ul>
}

    und:

    <?php //mypage.php
generate_menu(0);
?>
blah blah content goes here
    • danke für die Antwort rfw, aber als ich erwähnte, wie es scheint, schlechte Praxis, wie Sie diese Variablen-Namen würde global sein. Sie können auch verwenden, querystring mit umfasst, wenn die Verwendung von absoluten Pfaden (siehe Beispiel 3 hier: php.net/manual/en/function.include.php), aber aus den oben genannten Gründen denke ich nicht, dass dies die beste Allgemeine Ansatz entweder (da würden Sie dann verloren die ursprünglichen $_GET-Variablen)
    • MoMo vielleicht möchten Sie meine Funktion Ansatz besser?
    • die Funktion Vorgehen hat ein paar Nachteile (siehe original post), und erfordert die Umstrukturierung der 'view' - Dateien - ideal, die view-Dateien sind html-Seiten mit sehr wenig php - gerade genug, um Inhalte (smarty und codeigniter sind gute Beispiele, obwohl jeder von denen hat seine eigenen vor-und Nachteile). auch die, funktionieren wie, die kann ein wenig umständlich beim Umgang mit großen Dateien anzuzeigen und nicht viel tun zu trennen Logik von Inhalten.
    • MoMo: Wenn Sie schauen, um zu tun, vielleicht sollten Sie ein MVC-framework wie CodeIgniter? Obwohl, allgemein gesprochen, für Dinge wie Menüs, Sie sind dynamisch generiert über Funktionen...
    InformationsquelleAutor
Schreibe einen Kommentar