Conseil de conception

**malgache** · 22/04/2013, 09h32

Bonjour,

Je voudrai juste avoir un conseil sur la conception d'un menu

En faite dans la première boîte où j'étais pour savoir quel onglet du menu était sélectionné (sachant que le header était dans un fichier .php pour qu'il soit commun à toutes les pages) on créer une variable par onglet et on se retrouvais avec un truc de ce style

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
 
<?php
$page01 = "";
$page02 = "selected";
$page03 = "";
 
<ul>
     <li class="<?php echo $page01;?>">onglet 1</li>
     <li class="<?php echo $page02;?>">onglet 2</li>
     <li class="<?php echo $page03;?>">onglet 3</li>
</ul>
?>

Dans la nouvelle boîte, leur menu est légèrement différent. Il y a une seule variable et il y a des vérifs pour chaque onglet

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
 
<?php
$onglet_actif = 2;
 
 
<ul>
     <li class="<?php if($onglet_actif == 1) echo 'selected';?>">onglet 1</li>
     <li class="<?php if($onglet_actif == 2) echo 'selected';?>">onglet 2</li>
     <li class="<?php if($onglet_actif == 3) echo 'selected';?>">onglet 3</li>
</ul>
?>

J'arrive pas trop à savoir quelle manière est la meilleure niveau conception.
Si vous pouvez m'éclairer ...

Merci d'avance.

**andry.aime** · 22/04/2013, 09h47

Bonjour,

Perso, je préfère la 2ème méthode, tu n'as qu'une seule variable pour gérer la sélection, facile à manipuler. Tu n'as qu'à changer seulement sa valeur, or qu'avec la première méthode, tu dois modifier les 3 variables à chaque changement.

A+.

**rawsrc** · 22/04/2013, 09h54

Salut,

A vrai dire, je doute qu'il y ait une recommandation au niveau conceptuel, je parlerai plus d'une préférence.
Personnellement, je préfère la 1ère méthode mais avec un tableau derrière au lieu de 15 variables. Vu que c'est généralement très redondant, tu peux factoriser grandement le code en passant un tableau de menus et le menu actif comme ceci :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
function renderTabs($menus, $active)
{
    $n             = count($menus);
    $tabs          = array_fill(1, $n, '');
    $tabs[$active] = 'selected';
 
    echo '<ul>';
 
    for ($i = 1 ; $i <= $n ; ++$i)
    {
        echo
<<<HTML
    <li class="{$tabs[$i]}">{$menus[$i]}</li>
HTML;
    }
 
    echo '</ul>';
}
 
renderTabs(array(1 => 'menu 1', 'menu 2', 'menu 3'), 2);

**malgache** · 22/04/2013, 10h10

Merci pour vos réponses.

Je pense que je vais opter pour le 2ème méthode. Je vais éviter de créer des fonctions pour ça, puis au moins je reste dans le fonctionnement du menu qui était déjà en place.

Encore merci

**Benjamin Delespierre** · 22/04/2013, 10h33

La deuxième méthode est évidement la plus propre car elle évite la création de variables inutiles.

Je serais toi, j'utiliserai la syntaxe alternative comme ceci:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
<?php
 
// définition des pages du site
$sitemap = array(
    'menu' => array(
        '/' => 'Accueil',
        '/about' => 'A propos',
        '/articles' => 'Articles',
    ),
);
 
// page courante
$request_uri = parse_url( $_SERVER['REQUEST_URI'] );
 
?>
<ul class="menu">
    <? foreach ($sitemap['menu'] as $url => $name): ?>
    <li <? if ($url == $request_uri['path']): ?>class="selected"<? endif ?>>
        <a href="<?=$url?>"><?=$name?></a>
    </li>
    <? endforeach ?>
</ul>

Conseil de conception

Langage PHP

Vue hybride

Discussions similaires

Partager

Partager