Discussion :

[DiR95]

Bonsoir, ayant lu sur de nombreux forum de nombreuses solutions, je n'ai pas su l'appliquer dans mon cas.
Voici le bout de code en question

Code :

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

1
2
3
4
5
6
7
8
 for(int i=0;i<=Herbes.size();i++){
		if(Herbes.get(1) != null){
 
		herbe = (Herbe) Herbes.get(i);
		}
		graphics2d.drawImage(herbe.getImage(), herbe.getX(), herbe.getY(), null);
 
	}

Exception in thread "AWT-EventQueue-0" java.lang.IndexOutOfBoundsException: Index: 3, Size: 3
at java.util.ArrayList.rangeCheck(Unknown Source)
at java.util.ArrayList.get(Unknown Source)

Merci de votre futur aide

[joel.drigo]

Salut,

Premièrement, les index des éléments d'une List vont de 0 à size()-1. Ta boucle for(int i=0;i<=Herbes.size();i++) fait que i peut être égal à Herbes.size(), et ça fait donc que herbe = (Herbe) Herbes.get(i); cause une IndexOutOfBoundsException (qui signifie dépassement d'index).

Ensuite ton test if(Herbes.get(1) != null), à priori, devrait être if(Herbes.get(i) != null). Sinon, au pire tu auras des NullPointerException, au mieux ça va dessiner plusieurs fois la même image.

Ensuite, je ne sais pas où tu déclares Herbe, mais il y a un petit problème de logique : si l'un des éléments de la liste est null (enfin l'élément d'index 1 si tu conserves ton test), on affichera le dernier mis dans herbe. Sans parler du fait qu'il vaut mieux déclarer une variable là où on l'utilise, à commencer par limiter sa portée. Pour peu que la variable soit un attribut de la classe, et tu vas avoir un lien sur l'instance, ce qui risque de poser un problèmes de libération de ressources (l'image) éventuellement.

En plus, je ne sais pas pourquoi tu castes en Herbe ? Est-ce parce que tu aurais déclaré ta List sans paramètre : List Herbes; au lieu de List<Herbe> herbes. Si elle est bien déclarée comme ça, inutile de faire le cast.

Accesoirement, les conventions Java demandent à ce que les noms de variables commencent par une minuscules.

Donc en résumé

Code :

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

1
2
3
4
5
6
for(int i=0;i<herbes.size();i++) {
    Herbe herbe = herbes.get(i); 
    if (herbe != null){
 	graphics2d.drawImage(herbe.getImage(), herbe.getX(), herbe.getY(), null);
    }
}

Tu peux même faire :

Code :

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

1
2
3
4
5
for(Herbe herbe : herbes) {
    if (herbe != null){
 	graphics2d.drawImage(herbe.getImage(), herbe.getX(), herbe.getY(), null);
    }
}

c'est encore plus simple, plus de risque de dépassement d'index, et une variable à portée limitée.

Enfin, si tu peux éviter de mettre des null dans ta liste, et ce sera encore mieux : plus besoin de parcourir et tester des éléments inutilement dans le thread graphique.

Le code deviendrait :

Code :

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

1
2
3
for(Herbe herbe : herbes) {
 	graphics2d.drawImage(herbe.getImage(), herbe.getX(), herbe.getY(), null);
}

[bouye]

A partir du JDK8, il est possible de faire quelque chose du genre :

Code :

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

1
2
3
herbes.stream()
    .filter(herbe -> herbe != null)
    .foreach(herbe -> graphics2d.drawImage(herbe.getImage(), herbe.getX(), herbe.getY(), null));