Discussion :

[mesken]

Je suis débutant et je fais un TP simple selon MVC appélé ProjetTest, qui consiste a affcher première une page d'accueil avec 4 liens :

CLASSES
ETUDIANTS
ENSEIGNANTS
INSCRIPTIONS

Lorsque l'utilisateur clique sur l'un des liens, il lui ouvre le formulaire correspondant et dans chacun de ces formulaires, se trouve le lien Retour au Menu Principal
Ce que j'ai fait marche normalement mais j'ai un soucis d'optimisation.
Dans chacun de ces liens de ma page index.jsp qui contient ce Menu Principal, je fais ceci :

Code :

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

1
2
3
4
<td><a href="/ProjetTest/ServletControl?id=classes">CLASSES</a> </td>
<td><a href="/ProjetTest/ServletControl?id=etudiant"> ETUDIANTS</a></td>
<td><a href="/ProjetTest/ServletControl?id=inscriptions"> INSCRIPTIONS</a></td>
<td><a href="/ProjetTest/ServletControl?id=enseignants"> ENSEIGNANTS</a></td>

Et dans le doGet de ma Servlet, j'ai les 5 IF suivants :

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
if(id.equals("classes")){
	RequestDispatcher disp = request.getRequestDispatcher("/WEB-INF/vues/classes.jsp");
	disp.forward(request, response);
}
if(id.equals("etudiant")){
             RequestDispatcher disp = request.getRequestDispatcher("/WEB-INF/vues/etudiant.jsp");
	disp.forward(request, response);
}
if(id.equals("inscriptions")){
	RequestDispatcher disp = request.getRequestDispatcher("/WEB-INF/vues/inscription.jsp");
	disp.forward(request, response);
}
if(id.equals("enseignants")){
	RequestDispatcher disp = request.getRequestDispatcher("/WEB-INF/vues/enseignant.jsp");
	disp.forward(request, response);
}
if(id.equals("retourMenu")){
	RequestDispatcher disp = request.getRequestDispatcher("index.jsp");
	disp.forward(request, response);
}

ça marche sans problème, mais ma question est la suivante :

Est-ce à dire que si j'ai 20 formulaires avec peut-être 50 liens dans mon application, j'aurais à faire 50 IF ? N'est-ce pas lourd ? N'y-a-t-il pas un autre moyen plus léger ?

Merci

[OButterlin]

Il y a la possibilité de passer par une map, du coup, ton code ressemblerait à 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
23
24
25
26
27
28
 
public class MaServlet extends HttpServlet
{
    private static final long serialVersionUID = 1;
    private Map<String, String> map;
 
    @Override
    public void init(ServletConfig config) throws ServletException
    {
        super.init(config);
        map = new HashMap<String, String>();
        map.put("classes", "/WEB-INF/vues/classes.jsp");
        map.put("etudiant", "/WEB-INF/vues/etudiant.jsp");
        map.put("inscriptions", "/WEB-INF/vues/inscription.jsp");
        map.put("enseignants", "/WEB-INF/vues/enseignant.jsp");
    }
 
 
    @Override
    public void service(HttpServletRequest request, HttpServletResponse response) throws IOException
    {
        String id = request.getParameter("id");
        String page = map.get(id);
        ...
        request.getRequestDispatcher(page).forward(request, response);
 
    }
}

[mesken]

Merci bcp, ça marche et je préfère cette méthode et là je