Servlet] Problème avec SendRedirect

Version imprimable

12/10/2011, 21h48
wonay

[JSP/JQuery/Servlet] Problème avec SendRedirect

Bon mon soucis est un peu long a expliquer mais je pense que c'est un soucis tres commun mais je n'ai pas trouvé la solution tout seul.

alors en gros j'ai ma JSP qui affiche un formulaire ,
le bouton est un type "button" pas "submit" ( on verra pourquoi ^^ )(du coup pas de balise FORM)
lorsque l'on clique sa active un JS qui envoie la requete POST a ma servlet,

la servlet fait plein de verification et renvoie des messages derreur ( dans ce cas JS l'affiche dans le formulaire , truc du genre " invalid user " )
ou bien s'il n'y a pas d'erreur la servlet continue les traitement et fait un SENDREDIRECT ,

les message system.out.println saffiche tous dans la console de glassfish mais la page de mon browser ne change pas ... que se passe t il ?

PS: j'ai mis button et pas submit sinon ma servlet est appelé deux fois et c'est la misere.

merci beaucoup d'avance. Si vous avez besoin de codes je peux les poster mais il y en a un paquet.
13/10/2011, 09h10
DevServlet

Citation:

Envoyé par wonay

alors en gros j'ai ma JSP qui affiche un formulaire ,
le bouton est un type "button" pas "submit" ( on verra pourquoi ^^ )(du coup pas de balise FORM) mais il y en a un paquet.

C'est pas parceque t'as pas de bouton submit que tu mettras pas de balise form, le bouton submit a juste la particularité de soumettre tout le formulaire dans lequel il est invoqué en appelant automatiquement l'action serveur, mais par souci (ou même contrainte de de validité du W3C) il te faut tout structurer dans une form.fais nous voir ton code jsp et la servlet appelée.

J'avais tout posté mais cest beaucoup trop de codes, cest stupide. j'ai refait un exemple tout con :

Servlet :

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
34
35
36
37
38
39
40
41
42
43
44
45
46
@Override
    public void doGet(HttpServletRequest request, HttpServletResponse response)
					throws IOException, ServletException{
 
        System.out.println("Page: REINSTALL(GET)");
        Init(request, response);
 
        if(ParameterNumber == 0){
            System.out.println("Init REINSTALL Page");
            System.out.println("Lang = " + session.getAttribute("Lang").toString());
            request.setAttribute("LangCore", LangCore);
            if(InstallerXML.isAlreadySet()){
                System.out.println("Setting Exists.");
                dispatch = request.getRequestDispatcher("ReInstall.jsp");
                dispatch.forward(request, response);
                return;
            } else {
                System.out.println("Setting NOT Exists.");
                response.sendRedirect("INSTALL");
                return;
            }
       }
    }
 
    @Override
    public void doPost(HttpServletRequest request, HttpServletResponse response)
					throws IOException, ServletException{
 
        System.out.println("Page: REINSTALL(POST)");
        Init(request, response);
 
        if(request.getParameter("Reset") != null){
           boolean reset = (request.getParameter("Reset").compareToIgnoreCase("TRUE") == 0);
           System.out.println("Reset setting: " + reset);
           if(reset){
               System.out.println("Delete setting. Success: " + InstallerXML.reset());
               response.sendRedirect("INSTALL");
               return;
           } else {
               System.out.println("Canceled, go to login page.");
               response.sendRedirect("LOGIN");
               return;
           }
        }
        return;
    }

JSP

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
<%@ include file="include/header.jsp" %>
<LINK rel="stylesheet" href="design/ReInstall.css" type="text/css"/>
<script src="js/ReInstall.js"></script>
<%@ include file="include/headPage.jsp" %>
 
 
 
<div class="Content">
<div class="InstallDiv">
    <fieldset>
    <legend>Re<%= ((Lang)request.getAttribute("LangCore")).GetLangVersion("Install") %></legend>   
        <fieldset class="SubBorder">
            <legend class="IDTitle"><%= ((Lang)request.getAttribute("LangCore")).GetLangVersion("DBReset") %></legend>
                <p class="PCenter">
                    <input id="ValidationResetDBButton" type="button" value="OK"/>
                    <input id="CancelResetDBButton" type="button" value="<%= ((Lang)request.getAttribute("LangCore")).GetLangVersion("Cancel") %>"/>
                </p>
                <p class="PCenter"><label class="ERROR" id="result_DB_display">&nbsp;</label></p>
        </fieldset>
    </fieldset>
        </div>
</div>
 
<%@include file="include/footer.jsp" %>

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
34
35
36
$(function(){
 
    $("input#ValidationResetDBButton").click(function(){
       var parameter = "Reset=" + true;
       SendInstallRequest(parameter,function(output){
 
        });
    });
 
    $("input#CancelResetDBButton").click(function(){
       var parameter = "Reset=" + false;
       SendInstallRequest(parameter,function(output){
 
        });
    });
 
});
 
function SendInstallRequest(params,f){
    var http = new XMLHttpRequest();
    var url = "REINSTALL";
 
    http.open("POST", url, false);
 
    http.setRequestHeader("Content-type", "application/x-www-form-urlencoded");
    http.setRequestHeader("Content-length", params.length);
    http.setRequestHeader("Connection", "close");
 
    http.onreadystatechange = function() { //Call a function when the state changes.
            if(http.readyState == 4 && http.status == 200) {
                   f(http.responseText);
            }
    };
 
    http.send(params);
}

et mon log :

La je vais sur ma page REINSTALL

Citation:

INFO: Page: REINSTALL(GET)
INFO: Init REINSTALL Page
INFO: Lang = EN
INFO: Setting Exists.

La je clique sur OK on reset les reglages

Citation:

INFO: Page: REINSTALL(POST)
INFO: Reset setting: true
INFO: Delete setting. Success: true

La Servlet fait un SendRedirect vers INSTALL

Citation:

INFO: Page: INSTALL(GET)
INFO: Init INSTALL Page
INFO: Lang = EN
INFO: Setting NOT Exists.

mais a cette etape la page web n'a absolument pas changé je dois faire un refresh pour relancer le procédé et etre redirigé vers la page correct.

13/10/2011, 14h37
OButterlin

Mais où est le code de la fonction "f" ?
13/10/2011, 15h50
wonay
bah la :
Code:

1 2 3 SendInstallRequest(parameter,function(output){ });
ici j'en ai pas besoin, il y a pas de traitement d'erreur possible c'est juste deux bouttons. OK / Cancel
14/10/2011, 16h56
wonay

Personne ne sait resoudre ce soucis ?
16/10/2011, 04h37
wonay

Je nai toujours pas resolu mon soucis
16/10/2011, 12h42
OButterlin

A la base, c'est le mécanisme AJAX que tu n'as pas compris...
Tu fais un traitement asynchrone et tu voudrais que ta page change... syntaxe error ! L'ajax est fait justement pour ne pas quitter une page.
Ton sendRedirect de la servlet ne sert strictement à rien.
Si tu veux absolument faire les contrôles par AJAX, il faudra traiter le résultat OK et faire la redirection par javascript (window.location.replace(nouvelle URL)).
Ceci dit, je ne vois pas trop l'intérêt... d'habitude, on soumet le formulaire, on test, si pas bon -> on le reaffiche, sinon, on passe à la suite.
16/10/2011, 12h59
wonay

Ok mais ds ce cas comment traité les erreurs de remplissage du formulaire ? Pour les afficher sans rafraichir la page. Et dans le cas dun remplissage correct etre rediriger vers la suite.
17/10/2011, 08h49
OButterlin

Pour les messages d'erreurs, il y a plusieurs façons de faire.
Une solution simple consiste à utiliser un objet que tu places dans le contexte request avant de redemander l'affichage de la page, ta page s'occupant de le mettre en forme.
Sinon, si tout est OK, tu utilises response.sendRedirect(url);
17/10/2011, 10h29
wonay

Mais cette methode implique un rafraichissement de la page... Ce que je ne veux pas ^^
17/10/2011, 11h10
OButterlin

Comment ça un rafraichissement de la page, que veux-tu dire ?
17/10/2011, 11h33
wonay

Peut etre que je nai pas compris ce que tu as dit mais ce que je comprend cest mettre un objet dans le request pour y passer les messages derreur. Mais sans rafraichir la page lobjet a beau changer laffichage reste le meme. Seul le javascript permet ( a ma connaissance ) de modifier laffichage de la page sans rafraichir.
17/10/2011, 14h03
OButterlin

Ben, je ne vois surtout pas où est le problème avec le reaffichage...
Normalement, tu as un bean qui contient les attributs liés à la page jsp.
Ta page jsp utilise cet objet pour construire la réponse.
Que tu affiches 1x ou 10x la page du moment que les informations contenues dans ce bean ne change pas, tu as le même "rendu".
Si on part de l'origine, tu as :
-> affichage de la page (bean vide) -> saisie des données dans le formulaire -> soumission du formulaire -> remplissage du bean -> contrôle :

1) une erreur est détectée, on renvoie le bean pour la même page (qui pourrait contenir une liste d'erreurs)
2) pas d'erreur, on fait le traitement suivant

Il y a quelque chose qui te gène dans cette façon de faire ?
17/10/2011, 17h24
wonay

Mon objectif etait de remplir le formulaire et de pouvoir afficher des messages derreurs en live sans envoyer de requetes au serveur excepter une fois a la fin lorsque le formulaire est duement rempli
17/10/2011, 17h32
OButterlin

Au fur et à mesure de la saisie ?
18/10/2011, 00h11
wonay

Ouaip
18/10/2011, 08h37
OButterlin

Alors rien n'empêche d'avoir un flag dans la page et de faire n appels de méthodes AJAX pour les contrôles.
Ensuite, tout dépend des contraintes de cette page. Si tous les champs doivent être remplis, ou seulement certains, etc...
Détecter le moment où le formulaire doit (ou peut) être soumis n'est pas forcément évident...
Dans tous les cas, ça n'empêche pas d'avoir un contrôle côté serveur qui sera activé lorsque la page sera soumise (ou alors tu masques le bouton de soumission)
18/10/2011, 09h01
DevServlet

Citation:

Envoyé par wonay

Mon objectif etait de remplir le formulaire et de pouvoir afficher des messages derreurs en live sans envoyer de requetes au serveur excepter une fois a la fin lorsque le formulaire est duement rempli

J'ai du mal à capter pourquoi ce post est autant rallongé 8-). S'il veut faire des controles au fur et à mesure des saisies dans son formulaire sans appel serveur, c'est forcement du javascript qui prend la main, au sortir de chaque champ il peut activer un contrôle sur la saisie.t'as des tonnes d’événements js rattachés à chaque champ en fonction de ton besoin.
18/10/2011, 09h08
OButterlin

Citation:

Envoyé par DevServlet

J'ai du mal à capter pourquoi ce post est autant rallongé 8-). S'il veut faire des controles au fur et à mesure des saisies dans son formulaire sans appel serveur, c'est forcement du javascript qui prend la main, au sortir de chaque champ il peut activer un contrôle sur la saisie.t'as des tonnes d’événements js rattachés à chaque champ en fonction de ton besoin.

Sauf qu'avec la ligne que tu viens de marquer, on se rend bien compte que ce n'est pas clair dans sa tête...
18/10/2011, 11h26
wonay

En gros jai un formulaire, au moment ou on valid, on fait une requete au serveur si tout est ok on switch de page sinon on reste sur la meme mais on affiche un ptit message derreur.

Pour faire sa jai mon js qui intercepte le forlulaire, le renvoi au servlet, le servlet lanalyse et renvoie une erreur au js si yen a.
si le js recoi quelque chose il affiche une erreur, sinon la servlet fait un Sendredirect.

Je lai deja fait plusieurs fois et sa fonctionne mais pour cette page des fois il est envoyer deux fois, jai bidouiller et maintenant il est envoyer mais la redirection marche pas.
18/10/2011, 11h56
DevServlet

Citation:

Envoyé par wonay

Pour faire sa jai mon js qui intercepte le forlulaire, le renvoi au servlet, le servlet lanalyse et renvoie une erreur au js si yen a.

Une servlet ne cause pas avec un script js directement, une servlet echange un document html/(ou jsp devant produire un html), la js ne sait communiquer qu'avec ton DOM.
Ceci

Citation:

Envoyé par wonay

Pour faire sa jai mon js qui intercepte le forlulaire, le renvoi au servlet, le servlet lanalyse et renvoie une erreur au js si yen a.
si le js recoi quelque chose il affiche une erreur, sinon la servlet fait un Sendredirect.

n'est pas cohérent avec ceci :

Citation:

Envoyé par wonay

Au fur et à mesure de la saisie ? ==> Ouaip

J'ai l'impression que tu nous menes an bateau là 8-)
18/10/2011, 12h06
wonay

Je post les script de la page ce soir, vous comprendrez peut etre mieu ce que je veux faire.
18/10/2011, 12h22
OButterlin

Citation:

Envoyé par DevServlet

Une servlet ne cause pas avec un script js directement, une servlet echange un document html/(ou jsp devant produire un html), la js ne sait communiquer qu'avec ton DOM.

Ajax est là pour causer avec une servlet depuis le javascript et une servlet n'est pas limitée au html, heureusement d'ailleurs...
18/10/2011, 13h08
wonay

Citation:

Envoyé par OButterlin

Ajax est là pour causer avec une servlet depuis le javascript et une servlet n'est pas limitée au html, heureusement d'ailleurs...

Ah merci ! ^^

Donc voila mon soucis :

j'ai ce formulaire :

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
<%@ include file="include/header.jsp" %>
<LINK rel="stylesheet" href="design/Install.css" type="text/css"/>
<script src="js/Install.js"></script>
<%@ include file="include/headPage.jsp" %>
 
 
 
<div class="Content">
<div class="InstallDiv">
    <fieldset>
    <legend><%= ((Lang)request.getAttribute("LangCore")).GetLangVersion("Install") %></legend>   
        <fieldset class="SubBorder">
            <legend class="IDTitle"><%= ((Lang)request.getAttribute("LangCore")).GetLangVersion("DBTitle") %></legend>
            <form id="ValidationDBForm" method="POST">
            <p><label for="DBHost"><%= ((Lang)request.getAttribute("LangCore")).GetLangVersion("Host") %></label><input type="text" id="DBHost" name="DBHost" /></p>
            <p><label for="DBPort">Port: </label><input type="text" id="DBPort" name="DBPort" /></p>
            <p><label for="DBDBName"><%= ((Lang)request.getAttribute("LangCore")).GetLangVersion("DBName") %></label><input type="text" id="DBDBName" name="DBDBName" /></p>
            <p><label for="DBUserName"><%= ((Lang)request.getAttribute("LangCore")).GetLangVersion("username") %></label><input type="text" id="DBUserName" name="DBUserName" /></p>
            <p><label for="DBPsw"><%= ((Lang)request.getAttribute("LangCore")).GetLangVersion("psw") %></label><input type="password" id="DBPsw" name="DBPsw" /></p>
            <p class="PCenter"><input class="button" id="ValidationDBButton" type="submit" value="OK"/></p>
            <p class="PCenter"><label class="ERROR" id="result_DB_display">&nbsp;</label></p>
            </form>
        </fieldset>
    </fieldset>
        </div>
</div>
 
<%@include file="include/footer.jsp" %>

qui est lié a ce JS

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
$(function(){
 
    $('form#ValidationDBForm').submit(function(){
        $('label#result_display').html("&nbsp;");
        var itsok = true;
        var Host = $('input#DBHost').val();
        var username = $('input#DBUserName').val();
        var psw = $('input#DBPsw').val();
        var port = $('input#DBPort').val();
        var DBName = $('input#DBDBName').val();
 
        var parameter = "DBHost=" + Host + "&DBUserName=" + username + "&DBPsw=" + psw + "&DBPort=" + port + "&DBName=" + DBName;
 
        SendInstallRequest(parameter,function(output){
            $('label#result_display').html("&nbsp;");

ici cest le traitement des erreurs

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
 
            if (output.indexOf("ERROR--")!=-1){
                var outputredif = output.replace("ERROR-- ","").replace("PSW ", "").replace("USER ", "").replace("HOST ", "").replace("PORT ", "").replace("NAME ", "");
 
                if(output.indexOf("USER ") != -1){
                    $('input#DBUserName').focus();
                }
                if(output.indexOf("PSW ") != -1){
                    $('input#DBPsw').focus();
                }
 
                if(output.indexOf("NAME ") != -1){
                    $('input#DBDBName').focus();
                }
 
                if(output.indexOf("PORT ") != -1){
                    $('input#DBPort').focus();
                }
 
                if(output.indexOf("HOST ") != -1){
                    $('input#DBHost').focus();
                }
 
                $('label#result_DB_display').text(outputredif);
 
                itsok = false;
            }
        }); // fin sendrequest
        return itsok;
    });
 
});

et la cest la fonction d'envoi de requete

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
 
function SendInstallRequest(params,f){
    var http = new XMLHttpRequest();
    var url = "INSTALL";
 
    http.open("POST", url, false);
 
    http.setRequestHeader("Content-type", "application/x-www-form-urlencoded");
    http.setRequestHeader("Content-length", params.length);
    http.setRequestHeader("Connection", "close");
 
    http.onreadystatechange = function() { //Call a function when the state changes.
            if(http.readyState == 4 && http.status == 200) {
                   f(http.responseText);
            }
    };
 
    http.send(params);
}
 
runOnLoad(function(){
    $("input#DBHost").focus();
});

et tout ça est managé par la servlet :

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
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
 
    @Override
    public void doGet(HttpServletRequest request, HttpServletResponse response)
					throws IOException, ServletException{
 
        System.out.println("Page: INSTALL(GET)");
        Init(request, response);
 
        if(ParameterNumber == 0){
            System.out.println("Init INSTALL Page");
            System.out.println("Lang = " + session.getAttribute("Lang").toString());
            request.setAttribute("LangCore", LangCore);
            if(InstallerXML.isAlreadySet()){
                System.out.println("Setting Exists.");
                response.sendRedirect("REINSTALL");
                return;
            } else {
                System.out.println("Setting NOT Exists.");
                dispatch = request.getRequestDispatcher("Install.jsp");
                dispatch.forward(request, response);
                return;
            }
       }
    }
 
    @Override
    public void doPost(HttpServletRequest request, HttpServletResponse response)
					throws IOException, ServletException{
 
        System.out.println("Page: INSTALL(POST)");
        Init(request, response);
        if(request.getParameter("DBPort")!= null && request.getParameter("DBHost")!= null && request.getParameter("DBUserName")!= null && request.getParameter("DBPsw")!= null && request.getParameter("DBName")!= null){
            String Host = request.getParameter("DBHost");
            String Username = request.getParameter("DBUserName");
            String Psw = request.getParameter("DBPsw");
            String Port = request.getParameter("DBPort");
            String Name = request.getParameter("DBName");
 
            if(!Security.InputValid(Host)){
                out.print("ERROR-- HOST " + LangCore.GetLangVersion("WrongHost"));
                return;
            }
            if(Security.InputParse(Port).compareToIgnoreCase(Security.INVALID) == 0){
                out.print("ERROR-- PORT " + LangCore.GetLangVersion("WrongPort"));
                return;
            }
            if(!Security.InputValid(Name)){
                out.print("ERROR-- NAME " + LangCore.GetLangVersion("WrongDBName"));
                return;
            }
            if(!Security.InputValid(Username)){
                out.print("ERROR-- USER " + LangCore.GetLangVersion("WrongUser"));
                return;
            }
            if(!Security.InputValid(Psw)){
                out.print("ERROR-- PSW " + LangCore.GetLangVersion("WrongPSW"));
                return;
            }
 
            try {
                InstallerXML.Writer(Username, Psw, Host, Port,Name);
                DBLink.getInstance(new GetSetting(pathInstall), new GetDBStructure(pathRessources)).GlobalDBReset();
            } catch (Exception ex) {
                Logger.getLogger(Installer.class.getName()).log(Level.SEVERE, null, ex);
            }
            response.sendRedirect("LOGIN");
            return;
        }
 
        return;
    }

18/10/2011, 17h41
wonay

et donc des fois a la fin du traitement il reste sur .../INSTALL et une page vierge s'affiche et des fois il est rediriger correctement vers .../LOGIN mais dans ce cas le formulaire a ete envoyé deux fois.

c'est a s'arracher les cheveux !