Discussion :

[mehdikwa]

Bonjour, j'ai une exception de type outofbound alors que j'instancie mon objet après chaque iteration. quelqu'un pourrait m'aider svp

Code :

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

1
2
3
4
5
 
		int i = 0 ;
		String type="";
		while(itr.hasNext()){
				type = infoCompte.getInformationSolde().getDetailCompte().get(i).getType();

Exception:

Code :

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

1
2
3
4
 
java.lang.IndexOutOfBoundsException: Index: 1, Size: 1
	at java.util.ArrayList.RangeCheck(ArrayList.java:547)
	at java.util.ArrayList.get(ArrayList.java:322)

[adiGuba]

Salut,



Il n'y a rien de bizarre si tu as cette erreur c'est que tu accèdes à un index incorrect.


Le code que tu donnes est tronqué et ne permet pas de voir grand chose, si ce n'est que tu mêles un accès par itérateur avec un accès via index (ce qui peut être casse-gueule).


a++

[joel.drigo]

Salut,

Cette erreur signifie que tu essayes de récupérer un élément en deuxième position (i=1, les indices allant de 0 à nbElement-1) dans une liste qui ne contient qu'un seul élément.

[mehdikwa]

Merci pour vos retours, mais je penses que type ne sera pas instancié une nouvelle fois s'il n'y a pas de nouvelles itérations sur l'objet detailCompte. L'erreur en fait dans l'exception pointe sur la ligne

Code :

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

1
2
 
Iterator itr = infoCompte.getInformationSolde().getDetailCompte().iterator();

voila mon code

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
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
 
 
int i = 0 ;
String type="";
while(itr.hasNext()){
Iterator itr = infoCompte.getInformationSolde().getDetailCompte().iterator();
  if (type.equals("Euros"))
				{
					try {
						balance = Double.parseDouble(infoCompte.getInformationSolde().getDetailCompte().get(i).getMontant());
						String date = infoCompte.getInformationSolde().getDetailCompte().get(i).getDateFinValidite();
						date = date.substring(beginIndex, endIndex);
						endValDate = sf.parse(date);
						createBalance( ConsumptionConstants.BALANCE_TYPE_INITIAL, UnitCode.EURO, balance, 0, 0, null,
								endValDate, false );
 
					} catch (ParseException e) {
						// TODO Auto-generated catch block
						e.printStackTrace();
					}
			else if (type.equals("Secondes")){
					try {
						balance = Double.parseDouble(rihelper.getWsriService().getInfoAccountWithOptions(applicationId, msisdn).getInformationSoldeWithOptions().getListeComptesWithOptions().get(i).getCompte().getMontant());						
						String date = rihelper.getWsriService().getInfoAccount(applicationId, msisdn).getInformationSolde().getDetailCompte().get(i).getDateFinValidite();
						date = date.substring(beginIndex, endIndex);
						endValDate = sf.parse(date);
						createBalance( ConsumptionConstants.USAGE_TIME, UnitCode.SECOND, balance, 0, 0, null,
								endValDate, false );
 
					} catch (ParseException e) {
						// TODO Auto-generated catch block
						e.printStackTrace();
					}
				i++;
				itr.next();

[joel.drigo]

Merci pour vos retours, mais je penses que type ne sera pas instancié une nouvelle fois s'il n'y a pas de nouvelles itérations sur l'objet detailCompte. L'erreur en fait dans l'exception pointe sur la ligne

Code :

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

1
2
 
Iterator itr = infoCompte.getInformationSolde().getDetailCompte().iterator();

Pas possible parce que ton message dit :

 
java.lang.IndexOutOfBoundsException: Index: 1, Size: 1
	at java.util.ArrayList.RangeCheck(ArrayList.java:547)
	at java.util.ArrayList.get(ArrayList.java:322)

Donc, l'erreur est lors d'un appel de get(int) sur une ArrayList, et il n'y en pas dans la ligne que tu indiques. L'erreur se passe probablement dans une ligne de ce type :

Code :

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

balance = Double.parseDouble(infoCompte.getInformationSolde().getDetailCompte().get(i).getMontant());

Par ailleurs, comme le dit @adiGuba, il semble que tu mélanges itérateurs et parcours de liste par l'indice. En tout cas, d'un coté tu itères par l'Iterator itr, sans utiliser les valeurs pour chaque itération, et de l'autre tu accèdes à une liste sur laquelle tu itères via un indice (int i), que tu incrémentes, pour chaque itération sur itr. C'est étrange, mais bon, ça va fonctionner (parce que c'est la même liste sur laquelle tu itères de 2 manières différentes en même temps). Le problème c'est que tu utilise ce même indice pour accèder à d'autres listes, par exemple :

Code :

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

balance = Double.parseDouble(rihelper.getWsriService().getInfoAccountWithOptions(applicationId, msisdn).getInformationSoldeWithOptions().getListeComptesWithOptions().get(i).getCompte().getMontant());

Peut être que fonctionnellement il est garanti que cette liste ait la même taille que la liste sur laquelle tu itères, mais c'est prendre un risque d'erreur, car rien ne peut le garantir à priori. D'ailleurs, tu obtiens une exception, c'est donc que ce n'est pas le cas (soit c'est une erreur d'utiliser le même indice parce que les listes n'ont pas la même taille, soit la liste n'a pas la bonne taille à cause d'une erreur).

Tu as le numéro de la ligne ou se passe l'erreur : 547. Tu dois pouvoir trouver quelle liste tu accèdes avec un indice erroné.

[jeffray03]

tu fais 2 fois :

Code :

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

itr.next();

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
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
int i = 0 ;
String type="";
while(itr.hasNext()){
Iterator itr = infoCompte.getInformationSolde().getDetailCompte().iterator();
  if (type.equals("Euros"))
				{
					try {
						balance = Double.parseDouble(infoCompte.getInformationSolde().getDetailCompte().get(i).getMontant());
						String date = infoCompte.getInformationSolde().getDetailCompte().get(i).getDateFinValidite();
						date = date.substring(beginIndex, endIndex);
						endValDate = sf.parse(date);
						createBalance( ConsumptionConstants.BALANCE_TYPE_INITIAL, UnitCode.EURO, balance, 0, 0, null,
								endValDate, false );
 
					} catch (ParseException e) {
						// TODO Auto-generated catch block
						e.printStackTrace();
					}
			else if (type.equals("Secondes")){
					try {
						balance = Double.parseDouble(rihelper.getWsriService().getInfoAccountWithOptions(applicationId, msisdn).getInformationSoldeWithOptions().getListeComptesWithOptions().get(i).getCompte().getMontant());						
						String date = rihelper.getWsriService().getInfoAccount(applicationId, msisdn).getInformationSolde().getDetailCompte().get(i).getDateFinValidite();
						date = date.substring(beginIndex, endIndex);
						endValDate = sf.parse(date);
						createBalance( ConsumptionConstants.USAGE_TIME, UnitCode.SECOND, balance, 0, 0, null,
								endValDate, false );
 
					} catch (ParseException e) {
						// TODO Auto-generated catch block
						e.printStackTrace();
					}
				i++;
				itr.next();

[joel.drigo]

tu fais 2 fois :

Code :

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

itr.next();

Code :

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

1
2
3
4
5
6
int i = 0 ;
String type="";
while(itr.hasNext()){
Iterator itr = infoCompte.getInformationSolde().getDetailCompte().iterator();
  ...
				itr.next();

Oui, euh, non, pas tout à fait. J'avais pas vu l'inversion entre le while et la déclaration : mais je pense que c'est une erreur de recopie (!). Sinon ça ne compilerait même pas, et donc pas de risque de IndexOutOfBoundsException !

Mais il n'y a pas 2 itr.next().

Code :

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

1
2
3
4
5
6
7
8
9
int i = 0 ;
Iterator itr = liste.iterator();
while(itr.hasNext()){
  ...
  liste.get(i);
  ...
  i++;
  iter.next();
}

itères sur liste et l'indice est correct à chaque fois. Ce n'est pas peut être pas le plus simple, mais ça fonctionne.

[mehdikwa]

Merci joel.drigo en effet c'etait un melange entre un iterateur et un index, en modifiant ma boucle à

Code :

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

1
2
3
 
			int itr = infoCompte.getInformationSolde().getDetailCompte().size();
			for(int i=0; i<itr; i++){

Ca a marché.

Merci à tous pour votre aide.