Discussion :

[Orang-]

Bonjour,

Développant un modèle MVC très simple et je l'espère très classique, je cherche à éviter que le visiteur ne tombe sur un page blanche s'il s'amuse avec les requêtes GET ou qu'une erreur se soit glissée.
Voici le script du routeur :

Code php :

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
require_once(".." . DIRECTORY_SEPARATOR . "public" . DIRECTORY_SEPARATOR . "config". DIRECTORY_SEPARATOR ."config.php");
require_once("router_configuration.php");
if($_GET){
	foreach ($routes AS $controller => $actions) {
		foreach ($actions as $action) {
			if (in_array($_GET['controller'], array_keys($routes))) {
				if (isset($_GET['controller']) && $_GET['controller'] == $controller) {
					require_once(".." . DIRECTORY_SEPARATOR . "controller" . DIRECTORY_SEPARATOR . $controller . '.php');
					if (isset($_GET['action']) && $_GET['action'] == $action) {
						$controller = ucfirst($controller);
						$obj = new $controller;
						$obj->$action();
					}
				}
			} else {
				header('location:' . RACINE_SITE);
			}
		}
	}
}

Et le fichier de configuration :

Code php :

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

1
2
3
4
$routes = [
	'controller_users' => ['connexion','deconnexion','profil','maj','mdpPerdu','inscription'],
	'controller_salles' => ['index','reservation','recherche','reservationDetails','modiPosts','supriPosts']
]

J'arrive à faire en sorte que lorsque le visiteur modifie la valeur du GET du controller il soit redirigé vers la page d'accueil. En revanche, la même opération avec l'action ne fonctionne pas, le visiteur tombe sur une page blanche en tapant volontairement 'inddex' au lieu de index par exemple :

Code php :

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

if (in_array($_GET['controller'], array_keys($routes)) && in_array($_GET['action'], $actions) ) {

Par avance, merci pour toute aide, même une remarque sur la qualité de ce code

[Orang-]

Bonjour
Pas même une petite idée ?

[Spartacusply]

Redirige le vers une page spéciale (type 404.php), et retourne un code d'erreur 404.

[Orang-]

Bonne idée, merci
Mais je cherche justement à savoir comment.

[Spartacusply]

Ben il suffit de créer un "else" à ton if :

Code :

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

1
2
3
4
5
if (in_array($_GET['controller'], array_keys($routes)) && in_array($_GET['action'], $actions) ) {
 //tout va bien, exécution du code...
} else {
  //page n'existe pas, redirection vers 404
}

[Orang-]

Merci à toi, pour l'avoir essayé cela ne fonctionne pas. Avec une telle condition la redirection est systématique

Code php :

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
if($_GET){
	$i=0;
	foreach ($routes AS $controller => $actions) {
		foreach ($actions as $action) {
			if (in_array($_GET['controller'], array_keys($routes)) && in_array($_GET['action'], $actions)) {
				if (isset($_GET['controller']) && $_GET['controller'] == $controller) {
					require_once(".." . DIRECTORY_SEPARATOR . "controller" . DIRECTORY_SEPARATOR . $controller . '.php');
					if (isset($_GET['action']) && $_GET['action'] == $action) {
						$controller = ucfirst($controller);
						$obj = new $controller;
						$obj->$action();
					}
				}
			} else {
				header('location:' . RACINE_SITE);
			}
		}
	}
}

[Spartacusply]

Pourquoi passes-tu par des boucles ?

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
if($_GET){
	$i=0;
			if (in_array($_GET['controller'], array_keys($routes)) && in_array($_GET['action'], $actions)) {
				if (isset($_GET['controller']) && $_GET['controller'] == $controller) {
					require_once(".." . DIRECTORY_SEPARATOR . "controller" . DIRECTORY_SEPARATOR . $controller . '.php');
					if (isset($_GET['action']) && $_GET['action'] == $action) {
						$controller = ucfirst($controller);
						$obj = new $controller;
						$obj->$action();
					}
				}
			} else {
				header('location:' . RACINE_SITE);
			}
}

[Orang-]

Chacune des deux boucles parcourent le tableau $route en fonction du niveau auquel elle interviennent.
Effectivement, ce serait une bonne idée de s'en passer le plus possible en faisant quelque chose du genre :

Code php :

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
	if($_GET){
		$i=0;
		for ($i=0;$i<sizeof($routes);$i++) {
			if (in_array($_GET['controller'], array_keys($routes)) && in_array($_GET['action'], $routes[array_keys($routes)[$i]])) {
				if (isset($_GET['controller']) && $_GET['controller'] == $controller) {
					require_once(".." . DIRECTORY_SEPARATOR . "controller" . DIRECTORY_SEPARATOR . $controller . '.php');
					if (isset($_GET['action']) && $_GET['action'] == $action) {
						$controller = ucfirst($controller);
						$obj = new $controller;
						$obj->$action();
					}
				}
			} else {
				//header('location:' . RACINE_SITE);
			}
		}

Mais là encore, cela ne fonctionne pas : la combinaison controller+action ne donne lieu à aucun contenu voulu.

[Celira]

Récapitulons : Tu veux vérifier que le contrôleur demandé est dans la liste des contrôleurs et que l'action demandée est dans la liste des actions du contrôleur, avant de déclencher l'action. C'est bien ça ?

[Spartacusply]

Je comprend pas ta boucle puisque de toutes façons il y a qu'un seul routage possible pour ta route.

Je te propose une autre approche bien plus simple sans boucle :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
$routeOk = false;
if (array_key_exist($_GET['controller'],$routes)) {
	$actions = $routes[$_GET['controller']];
	if (in_array($_GET['action'],$actions)) {
		$action = $_GET['action'];
		$routeOk = true;
	}
} 
 
if ($routeOk) {
	//on continue la requête
} else {
	//route inexistante
}

[Orang-]

Désolé pour le temps pris avant réponse. Merci pour la tienne
Effectivement, une condition est bien plus bien plus élégante qu'une boucle ! À titre indicatif, voici ce que cela donne :

Code php :

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
23
$routeOk = false;
if($_GET){
	if (array_key_exists($_GET['controller'],$routes)) {
		$controller = $_GET['controller'];
		$action = $routes[$_GET['controller']];
			if(in_array($_GET['action'],$action)){
				$action = $_GET['action'];
				$routeOk = true;
			}
	}
	if ($routeOk) {
		require_once(".." . DIRECTORY_SEPARATOR . "controller" . DIRECTORY_SEPARATOR . $controller . '.php');
		$controller = ucfirst($controller);
		$obj = new $controller;
		$obj->$action();
	} else {
		header('location:' . RACINE_SITE);
	}
} else {
	require_once("..".DIRECTORY_SEPARATOR."controller".DIRECTORY_SEPARATOR."controller_salles.php");
	$obj = new Controller_salles();
	$obj->index();
}