Discussion :

[LDPDC]

Bonjour,
Je dois vérifier une condition sur le paramètre (de type String) d'une fonction grâce à des fonctions de la classe String. Je dois donc tout d'abord vérifier que le paramètre est non null, puis-je utiliser un unique if ou mieux vaut il en utiliser deux?

Petit "schéma":
Faut il faire:

Code :

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

1
2
3
4
5
6
7
8
maFonction(String maString)
{
  if(maString != null)
   {
     if(maString.length < X)
      [...]
   }
}

ou bien:

Code :

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

1
2
3
4
5
6
7
8
maFonction(String maString)
{
  if(maString != null && maString.length < X)
   {
 
      [...]
   }
}

ma crainte dans la deuxième solution étant que l'ordre des vérifications ne soit pas garanti et mène à une null pointer exception éventuellement.

Merci d'avance,
LDPDC

[Invité]

Salut
En Java, si tu utilises l'opérateur logique && l'évaluation paresseuse va être utilisée. Donc si la première condition est fause, il n'évaluera pas la seconde. Par contre si tu utilises & les deux parties sont toujours évaluées.
Donc dans ton cas avec

Code :

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

1
2
 
if(maString != null && maString.length < X)

tu ne cours aucun risque
A plus

[nouknouk]

Par contre si tu utilises & les deux parties sont toujours évaluées.

Je pense que tu voulais plutôt parler de l'opérateur ||

[Invité]

Je pense que tu voulais plutôt parler de l'opérateur ||

Non...

Un seul signe -> tout est toujours évalué
deux fois l'opérateur -> évaluation paresseuse.
Que ce soit avec |, ||, & ou &&
Donc avec &, les deux côtés sont toujours évalués, tu peux tester

[nouknouk]

ok, sauf que les opérateurs | et & sortent du cadre du 'if' puisque ce sont des opérateurs 'bit à bit' (qui font un calcul et renvoient une valeur) et pas des opérateurs 'logiques'.

Il est donc tout à fait normal que & et | soient systématiquement évalués, au même titre que n'importe quel autre opérateur de calcul (+, -, *, /, etc...).

Ceci étant dit, merci pour la précision, notamment pour l'opérateur || pour lequel je ne pensais pas qu'il faisait également une évaluation paresseuse

[Invité]

Si tu utilises & et | sur un booléen, ca se comporte comme je le dis ci dessus. Ca rentre tout à fait dans le cadre du if et répond à la question posée, voir par exemple http://www1.cs.columbia.edu/~lennox/...s/logical.html
Si tu ne me crois pas fais ce test :

Code :

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

1
2
3
 
String test = null;
if(test != null & test.length() > 0) {}

Tu vas avoir un NullPointerException et pas avec &&