Test de null-ité

Version imprimable

Test de null-ité

Bonjour bonjour,

C'est re-moi pour la dernière fois promis juré :lol:
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:

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)));
                 }
               }
           }
     }

19/07/2007, 14h22
OButterlin

Tu veux dire que tu passes toujours dans else ?

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:

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 ? :P

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

Code:

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

Mais ça ne fonctionne pas non plus ...

19/07/2007, 14h33
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*$"
19/07/2007, 14h40
OButterlin
Ca c'est normal qu'il ne voit pas qu'il est NULL, il s'est planté avant:mouarf:
Il faut faire ce test (et dans cet ordre) :
Code:

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

Yeeeaaaaah ! :yaisse2:

Merki beaucoup !!! Sa fonctionne ! :king:
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:

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)));
             }
                 }
               }

19/07/2007, 15h01
zorm

Bonne chance alors, c'est pour ta soutenance de fin de stage d'IUT?
19/07/2007, 15h39
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 ! ;)