PHP - La mejor práctica para pasar variables para incluir archivos

Question

Esto es más de un '¿Cómo debería I?' en lugar de '¿Cómo do I?' pregunta.

En general, ¿cuál es la mejor manera de pasar variables a un archivo incluido?

Por ejemplo, supongamos que estoy usando un fragmento para un menú, y quiero que uno de los elementos del menú (el actual) tenga un cierto nombre de clase (esta es una muestra muy genérica, no es algo que yo soy en realidad usando):

<?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> 

Llamaré al 'menú.inc.php' anterior. obviamente está buscando una variable (int) llamada $active_index para determinar a qué enlace dar la clase '.active'.

así que podría simplemente definir $active_index antes de llamar al include, pero esto me parece una mala práctica ya que tal vez una variable de ese nombre podría haber sido definida para algo más anterior y una parte posterior del script todavía está buscando eso.

o - se puede utilizar una ruta absoluta y añadir variables utilizando una cadena de consulta (include 'menu.inc.php?active_index=1'), pero de nuevo que parece una mala idea ya que es posible que tenga la 'verdadera' $_GET dentro de cualquier include dada.

o - se puede empezar cada incluía archivo con ob_start y volver ob_end_clean(), a continuación, utilizar algo como esto para conseguir el retorno:

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

pero de nuevo esto parece tener una serie de desventajas (tener que reestructurar toda Incluya sus archivos de acuerdo con las funciones ob y una declaración return).

Answer 1

I como un objeto para este, como se describe in this MVC stack post. En un archivo llamado 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"); 
    } 
    } 

Definir $ active_link dentro del método vista contiene el alcance de la variable $ a active_link dentro del método. A continuación, llame a este código:

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

Sin embargo, soy casi nuevo para MVC en PHP, y doy la bienvenida al reproche.

Answer 2

Personalmente, simplemente definiría $active_index antes de incluir, o, siendo tal vez un poco más en línea con buenas prácticas de codificación, haga que la generación del menú sea una función con un parámetro $active_index.

Por ejemplo (valga el lío que sigue):

<?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> 
} 

y:

<?php // mypage.php 
generate_menu(0); 
?> 
blah blah content goes here