Discussion :

[moimp]

Bonjour à tous,
J'ai le code suivant:

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
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
switch($page)
{
	/*
	case 1:
	case 2:
	case 3:
	case 4:
	case 5:
		require_once VIEW_PAGE.'userMenu.php';
	*/
	case 1:		// contact list
		require_once VIEW_PAGE.'userMenu.php';
		require_once VIEW_PAGE.'list.php';
		break;
	case 2:		// maincard
		require_once VIEW_PAGE.'userMenu.php';
		require_once VIEW_PAGE.'cardForm.php';
		break;
	case 3:		// new family's member
		require_once VIEW_PAGE.'userMenu.php';
		require_once VIEW_PAGE.'editMember.php';
		break;
	case 4:		// edit address
		require_once VIEW_PAGE.'userMenu.php';
		require_once VIEW_PAGE.'editAddress.php';
		break;
	case 5:		// edit member
		require_once VIEW_PAGE.'userMenu.php';
		require_once VIEW_PAGE.'editMember.php';
		break;
	case 101:	// login
	case 102:	// demo
		require_once DIR_CONTROLLER.'login.php';
		break;
	case 110:	// forms treatment
		require_once DIR_CONTROLLER.'cardFormHandler.php';
		break;
	case 190:	// logout
		unset($_SESSION);
		unset($_COOKIE['code']);
		$page = 0;
		//break;
	default:
		require_once VIEW_PAGE.'homeMenu.php';
		require_once VIEW_PAGE.'connect.php';
		require_once VIEW_PAGE.'home.php';
		break;
}

Dans ce code, afin de ne pas répéter l'instruction require_once VIEW_PAGE.'userMenu.php';, j'ai essayé d'insérer le code en commentaire. Je ne comprend pas pourquoi le cas 1 est toujours exécuté alors que page vaut entre 2 et 5.

[Séb.]

1 seul case est exécuté (le premier qui vérifie la condition) jusqu'au premier break rencontré.

Si ma réponse ne te suffit pas montre-nous un cas concret à exécuter.

[cavo789]

Bonsoir

Puisque tu as mis un break pour chaque proposition, il est impossible que switch exécute plus d'une option.

Un conseil : rends ton code lisible (self explained en anglais). Qu'est-ce que 1 ? Qu'est-ce que 2 ? Qu'est-ce que 101 ? Là tu écris en commentaire "login" du coup pourquoi ne pas utiliser la valeur login et non un code arbitraire 101 ?

Bonne soirée

[moimp]

Il s'agit du routeur de la page index. La variable testé est un numéro de page vers lequel la redirection doit se faire. Le code en commentaire ne comporte pas de break pour être exécuté avant et en plus des cas suivants pour éviter d'avoir à répéter le require_once pour les cas 1 à 5 qui sont les vues et qui nécessitent toutes le menu principal (d'où le require).
Les cas au-delà de 100 sont des contrôleurs. Ils ne posent aucune difficultés et ne sont pas concernés par ce code en commentaire.

[cavo789]

Ne trouves-tu pas que "case 'login'" est bien plus parlant que "case 101"? C'est ça mon propos, n'utilise pas le code 101 mais un texte qui décrit exactement ce que tu fais / veux.

Au moins "pire", tu pourrais définir une constante LOGIN_PAGE=101 et du coup "case LOGIN_PAGE" serait aussi quelque chose de parlant.

[rv26t]

Bonjour,

... Le code en commentaire ne comporte pas de break pour être exécuté avant et en plus des cas suivants pour éviter d'avoir à répéter le require_once pour les cas 1 à 5 qui sont les vues et qui nécessitent toutes le menu principal (d'où le require)...

Comme l'indique Séb. , tu ne peux pas procéder de la sorte.
Si tu ne veux pas écrire 5 fois ta ligne de code require_once VIEW_PAGE.'userMenu.php'; , tu dois la traiter avant le switch (la doc décrit très bien le principe avec la break) ; par exemple avec la méthode in_array.

Ce qui te donnerait

Code :

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

1
2
3
4
5
6
7
8
9
if (in_array($page, array(1,2,3,4,5))) { require_once VIEW_PAGE.'userMenu.php'; }
switch($page){
	case 1:		// contact list
		require_once VIEW_PAGE.'list.php';
		break;
	case 2:		// maincard
		require_once VIEW_PAGE.'cardForm.php';
		break;
...

La remarque de cavo789 sur les constantes permet une compréhension et une maintenance plus facile.

Bonne programmation.

[laurentSc]

Bonjour,

la suggestion de rv26t est séduisante, mais je pense qu'il manque des case dans le in_array : if (in_array($page, array(1,2,3,4,5,101,102,110))). Mais comment traiter le case default ?

La remarque de cavo789 sur les constantes permet une compréhension et une maintenance plus facile.

Tout à fait d'accord.

[moimp]

Merci à tous pour vos éclaircissements et vos conseils.

[rv26t]

la suggestion de rv26t est séduisante, mais je pense qu'il manque des case dans le in_array : if (in_array($page, array(1,2,3,4,5,101,102,110))). Mais comment traiter le case default ?

Non, non.
Relis la demande

... pour éviter d'avoir à répéter le require_once pour les cas 1 à 5 ...
Les cas au-delà de 100 sont des contrôleurs. Ils ne posent aucune difficultés ...

le if (in_array($page, array(1,2,3,4,5))) traite les cas de 1 à 5 comme demandé, et seulement ceux-la.

Bonne soirée

[laurentSc]

T'as raison, j'ai lu trop vite.