Discussion :

[renarde]

Bonjour tout le monde,

J'ai un URL http://10.10.20.20:8080/ACCES?para1=...a2&para3=data3

Le dernier paramètre est optionnel. Je voudrais faire du traitement différemment selon sa présence. J'ai le code ci-dessous :

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
 
String p1= req.getParameter("para1");
String p2 = req.getParameter("para2");
String p3 = req.getParameter("para3");
 
if(type.equals("type1"))
{
        methode1();
}
else if(type.equals("type2"))
{
        methode(2);
}else if(type==null)
{
      methode3();
                }

Ca marche bien avec les deux premiers cas, mais quand le troisième paramètre est absent http://10.10.20.20:8080/ACCES?para1=data1&para2=data2 J'ai une exception java.lang.NullPointerException

Comment je pourrai faire mon traitement quand il n'y a que deux paramètres ?

Merci à toutes et tous.

Renarde @Nantes

[Sniper37]

On ne vois pas tous le code, mais je suppose que l'objet type est null dans le troisième cas, pour ce là, un conseil quand tu utilise equals vaut mieux faire comme ceci:

Code :

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

1
2
 
if("type1".equals(type))

au lieu de

Code :

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

1
2
 
if(type.equals("type1"))

[renarde]

Merci pour ta reponse. En fait je me suis mal exprimée : dans mon code, le type devrait être mon 3ième paramètre "p3". Et quand l'url ne contient que deux paramètres, p3 est null mais la ligne "if(p3==null)" me donne l'erreur java.lang.NullPointerException.

Je peux faire "type1".equals(p3)" pour les premiers deux cas, mais le 3ième ne marche pas...

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
 
String p1= req.getParameter("para1");
String p2 = req.getParameter("para2");
String p3 = req.getParameter("para3");
 
if(p3.equals("type1"))
{
        methode1();
}
else if(p3.equals("type2"))
{
        methode2();
}else if(p3==null)
{
      methode3();
}

[Sniper37]

vérifie la ligne de l'exception, si tu garde

Code :

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

null==null

ne génère pas de NullPointerException donc le problème n'est pas là..

[renarde]

vérifie la ligne de l'exception, si tu garde

Code :

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

null==null

ne génère pas de NullPointerException donc le problème n'est pas là..

Quand je fais http://10.10.20.20:8080/ACCES?para1=data1&para2=data2 dans mon log il a bien "p3=null", donc je ne comprends pas pourquoi "null==null" qui ne marche pas...

Merci again !

[link256]

tu dois faire un double test.

Code :

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

1
2
3
4
5
 
if((p3!=null)&&(!p3.equals("")))
{
      methode3();
}

[renarde]

tu dois faire un double test.

Code :

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

1
2
3
4
5
 
if((p3!=null)&&(!p3.equals("")))
{
      methode3();
}

Merci,

Mais ce que je voulais c'est quand para3 n'apparait pas dans l'url, donc ma méthode3 s'applique plutôt quand p3 = null, non? Je ne comprends pas ce que tu as proposé et ça ne marche pas...

[link256]

Le code que je t'ai mis te permet d'éviter de tombe sur un nullPointerException
et de contrôles les paramètres que ton application recoit.
A utiliser pour tes 2 premiers test notament afin de voir si un objet n'est pas null avant de tester une de ses propriétés

Pour ton problème inverse le test d'egalité

Code :

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

if(p3==null)

par

Code :

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

 if(null==p3)

De plus ton algo est à l'envers commence par le teste de la nullité avant de savoir si type vaut "type1" ou "type2"

Ce qui explique que ton test ne marche pas car l'erreur que tu as doit surement pointer sur la ligne

Code :

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

if(type.equals("type1")

[renarde]

Super, ça marche !

Je suis contente, merci à tous les deux

[Sniper37]

Quand je fais http://10.10.20.20:8080/ACCES?para1=data1&para2=data2 dans mon log il a bien "p3=null", donc je ne comprends pas pourquoi "null==null" qui ne marche pas...

Merci again !

Donne nous le code exacte, juste un conseil de bonne pratique
faut toujours faire

Code :

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

1
2
3
4
if("chaine".equals(variable)) {
 
// jamais de NullPointerException, même lorsque variable==null 
}

Ensuite si tu fais un main de base pour tester

Code :

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

System.out.println(null == null);

le résultat serait true et pas d'exception donc :

String p1 = null;

Code :

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

1
2
3
4
 
if(p1==null){
 
}

ne génère jamais de NullpointerException, peu etre l'exception est génère dans l'une des méthode.

NB: Cette exception est simple à corriger, avec Eclipse, faut lire bien la trace de l'exception et la cause de l'exception, tu trouvera même un lien sur la ligne source de l'exception.

[OButterlin]

Damned, j'hallucine

C'est quoi ce "p3 == null" ou "null == p3" ?

Il suffit de commencer les tests par p3 == null, comme ceci

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
String p1= req.getParameter("para1");
String p2 = req.getParameter("para2");
String p3 = req.getParameter("para3");
 
if(p3==null)
{
      methode3();
}
else if(p3.equals("type1"))
{
        methode1();
}
else if(p3.equals("type2"))
{
        methode2();
}

Dans le code d'origine

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
 
String p1= req.getParameter("para1");
String p2 = req.getParameter("para2");
String p3 = req.getParameter("para3");
 
if(p3.equals("type1"))
{
        methode1();
}
else if(p3.equals("type2"))
{
        methode2();
}
else if(p3==null)
{
      methode3();
}

Si p3 est null, on ne peut référencer la méthode equals d'où NullPointerException
En traitant ce cas particulier en premier, l'algo doit passer...