Discussion :

[R1pToR]

Bonjour bonjour,

C'est re-moi pour la dernière fois promis juré
Bon, j'ai encore un petit souci au niveau d'un if/else :

Voila, en fait je récupère un champ vide (input type="text") et malgré mes tests de 'nullité', je passe au travers (début du problème au else en gras) :

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
try {
          LivraisonActionForm resultatsDemandesValidationActionForm = (LivraisonActionForm) actionForm;
          LivraisonDelegate livraisonDelegate = new LivraisonDelegate();
          LivraisonDto[] livraisonDto;

          // Test de la valeur de la checkbox
          String checkBox = httpServletRequest.getParameter("allDemandes");

    if(checkBox != null && checkBox.matches("on"))
             {
                livraisonDto = livraisonDelegate.livraisonFindAllWFV();
                resultatsDemandesValidationActionForm.setlivListCollection(Arrays.asList(livraisonDto));
             }

      else  {
               String temp = httpServletRequest.getParameter("numero");
               
               System.out.println("voici temp:"+temp+".");
               // Résultat : voici temp:.
               
               if(temp != "" && temp != null && temp != " "){
                 // Malgré les trois tests, le debug passe dans le if
                 
                 int num = Integer.parseInt(temp);
                 Integer numero = new Integer(num);
               if(numero != null ){
                livraisonDto = livraisonDelegate.livraisonFindByNumeroWFV(numero);
                resultatsDemandesValidationActionForm.setlivListCollection(Arrays.asList(livraisonDto));
                 }
               }
              else{
                                
             String sDate = httpServletRequest.getParameter("dateChoisie");
             if(sDate != ""){
             SimpleDateFormat formatter = new SimpleDateFormat("dd-MM-yyyy");
             Date date = formatter.parse(sDate);
             resultatsDemandesValidationActionForm.setlivListCollection
            (Arrays.asList(livraisonDelegate.livraisonFindByDateLivWFV(date)));
                 }
               }
           }
     }

[OButterlin]

Tu veux dire que tu passes toujours dans else ?

[R1pToR]

Nan, ça, ça fonctionne (cf. le topic If/Else).

J'ai pas mis en évidence le if concerné (désolé ).
C'est ce passage :

Code :

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

1
2
3
4
5
6
7
8
System.out.println("voici temp:"+temp+".");
               // Résultat : voici temp:.
               
               if(temp != "" && temp != null && temp != " "){
                 // Malgré les trois tests, le debug passe dans le if
                 
                 int num = Integer.parseInt(temp);
                 Integer numero = new Integer(num);

Le problème étant que lorsqu'il retourne un temp null, il ne doit pas essayer de le faire le parseInt (normal sinon plantage). Le if est donc là pour vérifier que le temp n'est pas null, et malgré les trois conditions, le contrôleur ne voit pas quand temp est null ....

C'est plus clair ?

Edit : En fait, ça serait plus logique de mettre :

Code :

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

 if(temp != "" || temp != null || temp != " ")

Mais ça ne fonctionne pas non plus ...

[zorm]

c'est un string, donc essayes avec les regex de tester que ce que tu as comme valeur dans ta chaine de caractère est bien un entier

la regex est simple dans ce cas, ca sera un truc du genre "^\\d*$"

[OButterlin]

Ca c'est normal qu'il ne voit pas qu'il est NULL, il s'est planté avant
Il faut faire ce test (et dans cet ordre) :

Code :

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

1
2
3
4
5

if ( temp != null && temp.trim().length() > 0)
{
...la il y a quelque chose...
}

On ne test pas la valeur d'un String avec == mais avec equals(...)
Dans le permier cas, tu testes si les 2 chaînes sont les mêmes (même adresse), pas leur contenu

[R1pToR]

Yeeeaaaaah !

Merki beaucoup !!! Sa fonctionne !
En effet, je testait pas comme il fallait, mais maintenant ça tourne !

Voici le code final, et encore merci ! (ouf, à 1/4 d'h de la démo ! chui sauvé ... ou presque !)

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
 else  {
               String temp = httpServletRequest.getParameter("numero");
 
               System.out.println("voici temp:"+temp+".");
               // Résultat : voici temp:.
 
               if(temp != null && temp.trim().length() > 0){
                 // Malgré les trois tests, le debug passe dans le if
 
                 int num = Integer.parseInt(temp);
                 Integer numero = new Integer(num);
                 livraisonDto = livraisonDelegate.livraisonFindByNumeroWFV(numero);
                 resultatsDemandesValidationActionForm.setlivListCollection(Arrays.asList(livraisonDto));
                 }
                else{
               String sDate = httpServletRequest.getParameter("dateChoisie");         
               if(sDate != null && sDate.trim().length() > 0){
               SimpleDateFormat formatter = new SimpleDateFormat("dd-MM-yyyy");
               Date date = formatter.parse(sDate);
               resultatsDemandesValidationActionForm.setlivListCollection
                   (Arrays.asList(livraisonDelegate.livraisonFindByDateLivWFV(
                   date)));
             }
                 }
               }

[zorm]

Bonne chance alors, c'est pour ta soutenance de fin de stage d'IUT?

[R1pToR]

Oui, mais c'est passé !

Ils m'ont gardé en CDD pendant un mois mais aujourd'hui c'est mon dernier jour, donc le boss voulait voir l'appli.

Apparement, il est content , donc je vais moins vous embêter à partir d'aujourd'hui.

En tout cas merci pour toutes les réponses fournis depuis 3mois. R1pToR vous salue !