Où dois-je déclarer mon "Model" ?

Version imprimable

Bonjour tout le monde,

J'ai réalisé une petit application web pour comprendre le principe du MVC.
En fait, j'ai le code suivant dans mon fichier (vue.jsp)
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 <% String L=(String) request.getAttribute("login"); String P=(String) request.getAttribute("pass"); %> <!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd"> <html> <head> <meta http-equiv="Content-Type" content="text/html; charset=ISO-8859-1"> <title>Insert title here</title> </head> <body> <%-- Forward --%> <h3>Votre Login est: <% out.println(L); %> </h3> <h3>Votre Pass est: <%=P %></h3> </body> </html>
Ok ça fonctionne mais ce n'est pas propre !
Mon code Java ne doit pat être déclaré dans ce fichier (Vue.jsp) afin de respecter le modèle MVC.
Où dois-je mettre mon code java pour respecter le modèle MVC ?

M= ?
V= Vues.jsp
C= MonServlet.java

Merci d'avance,

Yves

PS: Pour information, ce petit projet/tuto est réalisé sous Eclipse Kelper J2EE (Dynamic Web Project).

14/12/2014, 08h33
Cafeinoman

Pour le model, la base c'est un simple POJO avec, dans ton cas deux propriétés de type string. Après, un modele est normalement persisté en base. Ça nécessite donc une deuxième classe (à minima) qui s'occupe de connecter ton appli à une base de donner et d'y persister ton POJO.

Mais fait un tour dans les tutos, il y en a pas mal sur le sujet...

Bonjour,

Merci pour ta réponse, j'ai regardé le POJO à première vue, c'est une classe toute simple.
Hélas, je n'arrive pas à l'implémenter dans ce petit tutorial.

Pouvez-vous m'aider, le code est ci-dessous ?

D'avance, tout grand MERCI,

Yves

FirstServlet.java:

Code:

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
package web;
 
import java.io.IOException;
import javax.servlet.ServletException;
import javax.servlet.http.HttpServlet;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;
 
public class FirstServlet extends HttpServlet{
/**
	 * 
	 */
	private static final long serialVersionUID = 1L;
 
@Override
protected void doGet(HttpServletRequest request, HttpServletResponse response)
		throws ServletException, IOException {
    	String l=request.getParameter("login");
    	String p =request.getParameter("pass");
    	request.setAttribute("login", l);
    	request.setAttribute("pass", p);
    	// Forward
    	 request.getRequestDispatcher("Vue.jsp").forward(request, response);
    	}
 
@Override
	protected void doPost(HttpServletRequest request, HttpServletResponse response)
			throws ServletException, IOException {
	doGet(request, response);
	}
 
}

Web.xml:

Code:

1
2
3
4
5
6
7
8
9
10
11
<?xml version="1.0" encoding="UTF-8"?>
<web-app xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns="http://java.sun.com/xml/ns/javaee" xsi:schemaLocation="http://java.sun.com/xml/ns/javaee http://java.sun.com/xml/ns/javaee/web-app_3_0.xsd" id="WebApp_ID" version="3.0">
  <servlet>
    <servlet-name>cs</servlet-name>
    <servlet-class>web.FirstServlet</servlet-class>
  </servlet>
  <servlet-mapping>
    <servlet-name>cs</servlet-name>
    <url-pattern>*.php</url-pattern>
  </servlet-mapping>
</web-app>

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
<!DOCTYPE html>
<html>
<head>
<meta charset="ISO-8859-1">
<title>Insert title here</title>
</head>
<body>
<form action="control.php" method="post" >
 Login:<input type="text" name="login"><br/>
 Pass:<input type="password" name="pass"><br/>
 <input type="submit" value="OK">
 </form>
</body>
</html>

Vue.jsp:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
<%
String L=(String) request.getAttribute("login");
String P=(String) request.getAttribute("pass");
%>
<!DOCTYPE html PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd">
<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=ISO-8859-1">
<title>Insert title here</title>
</head>
<body>
<%-- 	Forward --%>
	<h3>Votre Login est: <% out.println(L); %>	</h3>
	<h3>Votre Pass est: <%=P %></h3>
</body>
</html>

16/12/2014, 14h11
OButterlin

Citation:

Envoyé par Cafeinoman

Pour le model, la base c'est un simple POJO avec, dans ton cas deux propriétés de type string. Après, un modele est normalement persisté en base. Ça nécessite donc une deuxième classe (à minima) qui s'occupe de connecter ton appli à une base de donner et d'y persister ton POJO.

Euh, pas vraiment... Le "Model" au sens MVC n'est rien d'autre qu'un bean représentant les données de la "Vue" (la JSP ici).
La persistence n'a rien à voir avec MVC, c'est juste un découpage logique en fonction des responsabilités de chaque élément pour la gestion d'une page "web".
En d'autres termes :
- M représente les données utilisées par la page (simple Bean Java) et placé dans le scope voulu (request, session, application)
- V représente la page JSP qui utilise et/ou affiche ces données (JSP le plus souvent, xhtml pour JSF2)
- C représente le contrôleur qui répond à une invocation de la page ou autre (une servlet ou assimilée en fonction du framework utilisé)

Ensuite, pour une application type CRUD, effectivement il faudra voir l'aspect extraction/sauvegarde de la base de données. Généralement, on place ces actions dans une couche dédiée à base d'EJB ou de classes gérées par un framework comme Spring.
16/12/2014, 14h21
vandeyy

Bonjour,

Merci pour la réponse, est-ce que mon "model" est-il correct car j'ai l'impression que je ne devrais pas y avoir du code java dans ma page JSP ?

Merci d'avance,

Yves

Dans ton exemple, ton modèle devrait être comme ceci

Code:

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
 
package mon.package;
 
public class LoginBean
{
    private String user;
    private String password;
 
    public Modele()
    {
        super();
    }
 
    public String getUser()
    {
        return user;
    }
 
    public void setUser(String user)
    {
        this.user = user;
    }
 
    public String getPassword()
    {
        return password;
    }
 
    public void setPassword(String password)
    {
        this.password = password;
    }
}

Dans ta page JSP, tu devrais le référencer par

Code:

1
2
 
<jsp:useBean id="loginBean" scope="request" class="mon.package.LoginBean"/>

et l'utiliser par

Code:

1
2
3
4
5
6
 
<jsp:getProperty name="loginBean" property="user"/>
 
ou
 
<%=loginBean.getUser()%>

(bon, je suis un peu rouillé avec ces anciennes méthodes)

16/12/2014, 15h46
vandeyy
Merci bcp pour les explications, je comprends bcp mieux.
Mais, pour le résultat, je reçois des valeurs "NULL" pour mon login et password.
Il manque surement quelque chose, sûrement dans le fichier "FirstServlet.java" ?!

Pourtant, j'ai essayé les deux:
Code:

1 2 3 <jsp:getProperty name="loginBean" property="user"/> ou <%=loginBean.getUser()%>
dans mon fichier Vue.jsp

Au fait, j'ai remplacé
Code:

1 2 3 4 5 6 7 8 9 10 11 public Modele() { super(); } PAR public LoginBean() { super(); }
Merci

Yves
16/12/2014, 16h17
OButterlin
Effectivement, j'ai commencé par créer la classe Modele et ensuite j'ai renommé en LoginBean en oubliant le constructeur :aie:

Les valeurs "NULL" sont normales, pour éviter ce problème, tu n'as qu'à initialiser les 2 propriétés dans LoginBean
Code:

1 2 3 private String user = ""; private String password = "";
Tu appelles la page directement ou tu passes par une servlet ?

Où dois-je déclarer mon "Model" (MVC) ?

Je passe par un servlet -> FirstServlet.java (voir dans mes messages précédents)

J'ai essayé de changer mon servlet (voir code ci-dessous) mais je ne reçois pas de résultat (plus de NULL mais rien)

FirstServlet.java:

Code:

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
package web;
 
import java.io.IOException;
import javax.servlet.ServletException;
import javax.servlet.http.HttpServlet;
import javax.servlet.http.HttpServletRequest;
import javax.servlet.http.HttpServletResponse;
 
public class FirstServlet extends HttpServlet{
/**
	 * 
	 */
	private static final long serialVersionUID = 1L;
 
@Override
protected void doGet(HttpServletRequest request, HttpServletResponse response)
		throws ServletException, IOException {
    	LoginBean lb = new LoginBean();
    	lb.setUser(request.getParameter("login"));
    	lb.setPassword(request.getParameter("pass"));
    	// Forward
    	 request.getRequestDispatcher("Vue.jsp").forward(request, response);
    	}
 
@Override
	protected void doPost(HttpServletRequest request, HttpServletResponse response)
			throws ServletException, IOException {
	doGet(request, response);
	}
 
}

MErci

Yves

16/12/2014, 17h08
OButterlin
Il manque juste un tout petit truc, tu n'enregistres pas ton bean dans le scope request.
Il suffit de rajouter ceci
Code:

1 2 request.setAttribute("loginBean", lb);
Ceci dit, il faudrait modifier tes 2 getter dans la classe pour éviter de renvoyer NULL la première fois que tu afficheras la page
Code:

1 2 3 4 5 6 7 8 9 10 public String getUser() { return user == null ? "" : user; } public String getPassword() { return password == null ? "" : password; }
16/12/2014, 17h45
vandeyy
Merci beaucoup cela fonctionne correctement, j'ai mon application orientée MVC, ouf !

J'ai initialisé mes variables par:
Code:

1 2 private String user = ""; private String password = "";
Donc, si je comprends bien, je ne suis pas obligé de redéfinir mes getters (test si null) ?

Encore un tout grand MERCI,

Yves
17/12/2014, 08h44
OButterlin

Dans la mesure où tu fais une affectation directe de ces champs par request.getParameter(...), il se pourrait qu'il n'y ait rien dans le request et du coup, tu affecterais null aux attributs... d'où la modification du getter (on pourrait le faire dans le setter plutôt que le getter)
17/12/2014, 10h47
vandeyy

Exact, Merci.

Yves