Discussion :

[bleach1234]

Bonjour,

J'aimerais savoir ce que vous me conseillez en terme de performances. Vaut-il mieux effectuer des try/catch ou des conditions ?

Exemple de code :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
List<String> liste = new ArrayList<String>();
if(liste.size() > 0) {
   System.out.println(liste.get(0));
}
if(liste.size() > 1) {
   System.out.println(liste.get(1));
}
if(liste.size() > 2) {
   System.out.println(liste.get(2));
}
...

Code :

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

1
2
3
4
5
6
7
8
 
List<String> liste = new ArrayList<String>();
try {
   System.out.println(liste.get(0));
   System.out.println(liste.get(1));
   System.out.println(liste.get(2));
   ...
} catch(IndexOutOfBoundsException e) {}

[CyberChouan]

Sans hésiter des conditions.

La levée d'une exception est très gourmande en ressources.

Maintenant, dans tous les cas, ton code d'exemple est bizarrement fait, et si tu as un cas similaire dans ton programme, tu devrais peut-être plutôt revoir la conception.

[CheryBen]

Et si ta liste peut contenir 50 élément, tu vas faire 50 blocs if ?
Ceci permet d'obtenir le même résultat, quelque soit la longueur de la liste :

Code :

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

1
2
3
for (int i = 0 ; i < liste.size() ; i++) {
     System.out.println(liste.get(i));
}

[bleach1234]

Non, oui c'était juste un exemple (bête, j'admets )

Dans ce cas là, je ne vois plus trop quand utiliser les try/catch, car finalement on peut toujours s'en sortir avec les conditions . . .

[adiGuba]

Salut,

bleach1234 Ton code est assez "bizarre" il faudrait savoir ce que tu veux faire précisément...

CyberChouan Je dirais plutôt que cela dépend des cas !
Un exception est "lourde" mais seulement lorsqu'elle est généré (et elle comporte alors un stacktrace très utile). Lorsque tout se passe bien cela n'entraine aucune perte de performance (cf sur mon blog : Exception et Performance)

Donc on a deux cas distincts qui dépend de la source de l'erreur :

• Soit elle peut être "fréquente" et peu arriver souvent. Dans ce cas remonter une exception peu en effet être assez lourd (surtout s'il s'agit d'un code appelé intensivement dans une boucle par exemple).
• Soit il s'agit d'un cas qui ne devrait pas survenir pendant le déroulement du programme (ou très rarement), et dans ce cas on évite ainsi de vérifier trop souvent une condition qui doit être vrai

CheryBen Si l'objectif est de parcourir la liste, il est quand préférable d'utiliser un Iterator qui permet le meilleurs compromis dans la plupart des cas :

Code :

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

1
2
3
for (String str : liste) {
     System.out.println(str);
}

L'accès par index ne doit se faire que si on est sûr de travailler avec une ArrayList et pourrait être catastrophique en terme de performance sur une LinkedList par exemple. Si vraiment on cherche la petite bête pour une optimisation maximum, il faudrait vérifier la présence de l'interface RandomAccess :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
if (list instanceof RandomAccess) {
     // L'accès par index est plus rapide :
     for (int i = 0 ; i < liste.size() ; i++) {
          System.out.println(liste.get(i));
     }
} else {
     // L'accès par iterator est plus rapide :
     for (String str : liste) {
          System.out.println(str);
     }
}

Mais bon je pense qu'il faut manipuler des listes énormes pour que cela soit différentiable...


a++

[bleach1234]

Ok, je vois un peu mieux l'utilisation des try/catch.

Merci pour vos réponses

[CyberChouan]

J'ai déjà lu ton (très bon) article, adiGuba.

Mais en parlant de l'utilisation des Exceptions pour faire une vérification par rapport à un test "if", je pensais plutôt à des codes comme ça (que j'ai déjà vus), plus qu'à une absence volontaire de tests:

Code :

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

1
2
3
4
5
6
7
public static boolean isNumeric(String chaine) {
   try {
      Integer.parseInt(chaine);
      return true;
   } catch (Exception e) {
      return false;
   }