Bonjour,

je souhaite savoir si il y a possibilité de simplifier ou d'optimiser ce petit code. J'ai tenté plusieurs solution en vain. 

	$templateRoot = 'templates/pages/';

	if (isset($_GET['page'])){
		
		$templatePage = $_GET['page'];
		
		include($templateRoot.((is_file($templateRoot.$templatePage.'.php')) ? $templatePage : '404').'.php');
		
		if($_GET['page'] != 'results_input') {echo $view-> render('templates/layouts/primary.php');}	
		
	}else{
		
		$templatePage = 'home';
		
		include($templateRoot.((is_file($templateRoot.$templatePage.'.php')) ? $templatePage : '404').'.php');
		
		echo $view-> render('templates/layouts/primary.php');
	}	

salut,

tu utilises les mêmes formules dans ta condition, tu n'as donc qu'une data qui peut changer.

c'est possible de le réduire ; perso, je le ramène à 4 lignes.

$templateRoot = 'templates/pages/';
 
if(isset($_GET['page']) && in_array($_GET['page'], array('results_input'))) {
    $templatePage = $_GET['page'];
} else {
    $templatePage = 'home';
}
     
include($templateRoot.((is_file($templateRoot.$templatePage.'.php')) ? $templatePage : '404').'.php');
echo $view-> render('templates/layouts/primary.php');

Attention dans sa première condition il y a une seconde condition à ne pas ignorer. Le code ci-dessus est adapté pour les multiples conditions à venir dans l'évolution de ton système. Par contre, c'est bizarre que les deux echo renvoient vers primary.php, mais on ne connais pas ton architecture, après.

En théorie via ton modèle de vérification (avant dernière ligne dans mon code) tu seras en mesure de tout réduire en.. 3-4 lignes, en l'appliquant à mon premier IF. Attention après, d'un point de vu personnel, il s'agit d'une optimisation du nombre de ligne, pas des performances ! Il est donc mieux d'organiser ton code comme tu le souhaites. Facilites toi la vie ! 

-
Edité par RPGBookOnline 10 décembre 2018 à 15:43:52

Réduit au max, ça donne un truc comme ça :

<?php
    $templateRoot = 'templates/pages/';

    $templatePage = (isset($_GET['page'])) ? $_GET['page'] : 'home';
    include($templateRoot.((is_file($templateRoot.$templatePage.'.php')) ? $templatePage : '404').'.php');
    echo ($templatePage != 'results_input') ? $view-> render('templates/layouts/primary.php') : '';

mais en effet, réduction du nombre de lignes ne veut pas dire amélioration des perfs. 

-
Edité par rga 10 décembre 2018 à 15:45:15

@sagaff : La dernière ligne du code de @rga peut aussi inclure in_array() dans le cas ou tu as plusieurs pages (autre que "results_input") qui viendraient s'incrémenter à ta condition. C'est maintenant à toi de voir quelle solution te convient le mieux ! 

-
Edité par RPGBookOnline 10 décembre 2018 à 16:13:11

if($_GET['page'] != 'results_input') {echo $view-> render('templates/layouts/primary.php');}

donc en prenant ta condition et en gardant le in_array(), il faut :

if(isset($_GET['page']) && !in_array($_GET['page'], array('results_input'))) {

ce qui doit empêcher la page results_input de s'afficher.

En effet ! 

A vouloir aller trop vite, j'ai oublié d'inverser la condition sur l'in_array(). Ma faute.