Discussion :

[july]

Bonjour à tous,
Sur mon site les utilisateurs peuvent se loguer et j'aimerai qu'ils puissent se "deloguer".
Le problème c'est que j'ai une erreur : java pointer null exception

Code :

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

1
2
3
4
5
6
7
 
private HttpSession session;
HttpSession session = null;
this.session = request.getSession(false);
if(this.session!=null){
				this.session.invalidate();
			}

Et il ne veut pas la méthode invalidate().
Ou peut etre faut-il utiliser une autre fonction pour fermer une session ?
Merci par avance

[fabszn]

Hello,

Ou est ce que tu appelles ce bout de code?

Pourrais tu montrer l'ensemble du code?

[july]

C'est dans une servlet que j'appelle lorsque l'utilisateur clique sur "se deloguer"
Voila le code de la servlet entière.

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
49
50
51
52
53
 
public class UnloginServlet extends HttpServlet
{
	private DispatchManager mngdispatch;
	private HttpSession session;
 
	/**
	 * Initialisation du dispatcher et de la base de données.
	 */
	public void init(ServletConfig conf)throws ServletException
	{
	    super.init(conf);
	    // On initialise le dispatch qui nous permettra de rediriger la réponse vers le fichier JSP
	    mngdispatch = new DispatchManager(this);
	}
 
	/**
	 * Traitement dans le cas d’une requête GET
	 * Fermeture de la connexion d'un utilisateur
	 */
	public void doGet(HttpServletRequest request, HttpServletResponse response)
		throws ServletException, IOException
	{
		doPost(request, response);
	}
 
	/**
	 * Traitement dans le cas d’une requête POST
	 * Fermeture de la connexion d'un utilisateur
	 */
	public void doPost(HttpServletRequest request, HttpServletResponse response)
	throws ServletException, IOException
	{
		HttpSession session = null;
		try {
			// On récupère la session de l'utilisateur
			this.session = request.getSession(false);
			// On la met à nul
			if(this.session!=null){
				this.session.invalidate();
			}
			//On renvoi l'utilisateur vers la page de login
			request.setAttribute ("titre", "Veuillez vous authentifier avant d'effectuer une action");
			mngdispatch.setDest("/index.jsp");
			mngdispatch.send(request,response);		
		}
		catch(Exception e){
			System.out.println(e);
			e.printStackTrace();
		}
	}
 
}

Merci !

[fabszn]

Hello,

// On récupère la session de l'utilisateur
this.session = request.getSession(false);
// On la met à nul
if(this.session!=null){
this.session.invalidate();

En mettant this devant l'objet session, tu fais reference à celui qui est declaré au niveau de ta servlet

Code :

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

1
2
3
4
5
public class UnloginServlet extends HttpServlet
{
   private DispatchManager mngdispatch;
   private HttpSession session;
...

Et pas à l'objet que tu initialises dans ta méthode doPost:

HttpSession session = null;
// On récupère la session de l'utilisateur
this.session = request.getSession(false);
// On la met à nul
if(this.session!=null){
this.session.invalidate();

Donc remplace ton code par :

HttpSession session = null;
// On récupère la session de l'utilisateur
session = request.getSession(false);
// On la met à nul
if(session!=null){
session.invalidate();

Aussi , une regle importante!! il ne faut jamais declarer de variable membre dans un servlet (variable d'un porté global à la servlet) cela peut introduire des comportements bizarre... car je crois qu'une servlet n'est pas declarée ThreadSafe...

J'espere que cela t'aidera , si tu as des questions , n'hesite pas!!

[july]

Ca marche parfaitement !
Merci beaucoup je n'y avait pas du tout pensé !!