Discussion :

[flora806]

Bonjour,
j'ai une méthode qui me permet de me renvoyer la valeur d'un entier qui est placée à la 3ème case d'une liste.
Mais j'ai toujours une erreur concernant le type d'objet que doit retourner cette méthode "this method must return a result of type int" , pourtant j'ai bien mis return à la fin de ce code pour "valeur" qui est de type int
Est-ce que vous auriez une explication?

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
public static int getRV(int V, int utilisateur) {
	int[] RVList = new int[0];
	int valeur = 0;
 
	for (int[] dataLine : data) {
		if (dataLine[3] == V) {
			int[] newList = new int[RVList.length + 1];
 
			for (int i = 0; i < RVList.length; i++) {
				valeur = Integer.valueOf(dataLine[3]);
			}
		}
		return valeur;
	}
}

Merci!!!

[nolofinwe]

je vais peut etre dire une bétise mais je crois que :

Code :

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

1
2
 
valeur = Integer.valueOf(dataLine[3]);

retourne un objet et non pas un int. ==> à vérifier

edit:
alors j'ai vérifier et effectivement tu renvois un objet mais ca n'est pas la l'erreur ;

valueOf(int i)
Returns a Integer instance representing the specified int value.

source: http://java.sun.com/j2se/1.5.0/docs/api/
deux choses : premierement dataline[3] est déja un int donc :

Code :

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

valeur = dataline[3]

devrai suffire
deuxiemement ton return est a l'interieur de ton for donc si ta liste est vide tu ne retourne jamais de valeur d'ou ton erreur...

voila l'analyse que j'ai faite mais je peux me tromper...

[keub51]

Imagine le cas où ta liste est vide.
Dans ce cas on ne passe pas dans la boucle for :

Code :

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

for (int[] dataLine : data)

Donc on ne retourne rien.

Ajoute un "return -1" par exemple en dehors de la boucle.

[Benouze]

Bonjour,

Navré mais je ne comprends vraiment pas ton code...surtout les boucles.

Pour récupérer le 3ème élement d'un tableau tu dois :
- vérifier que le tableau contient bien au moins 3 éléments
- récupérer le 3ème élément
- éventuellement le formater

A noter que l'index d'un tableau commence à 0. Pour récupérer le 3ème élément il faut faire tab[2].

Ton pb de compil vient du fait que le return peut ne pas être atteint.

[flora806]

Bonjour,

Navré mais je ne comprends vraiment pas ton code...surtout les boucles.

Pour récupérer le 3ème élement d'un tableau tu dois :
- vérifier que le tableau contient bien au moins 3 éléments
- récupérer le 3ème élément
- éventuellement le formater

A noter que l'index d'un tableau commence à 0. Pour récupérer le 3ème élément il faut faire tab[2].

Ton pb de compil vient du fait que le return peut ne pas être atteint.

----------
En fait ma classe contient déja au début une autre méthode qui permet de parser un fichier et de créer (data = new int [items.size()][3], j'ai utilisé aussi un Tokenizer pour distinguer entre les 3 items (int V = Integer.parseInt(splitter.nextToken());...)
et là ce que je veux c juste une simple méthode qui me retourne la valeur de l'entier V.

[flora806]

Si ça retourne un objet, je dois faire comment pour que "valeur" récupére l'entier de la 3ème case de la liste?
T'as une idée?

[flora806]

je viens de mettre valeur = dataLine[3], j'ai toujours la même erreur, donc ça vient probablement du cas où on ne retourne rien.
je peux l'exprimer comment le return -1

[Benouze]

Tu n'as qu'à déplacer ton return :

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
public static int getRV(int V, int utilisateur) {
	int[] RVList = new int[0];
	int valeur = 0;
 
	for (int[] dataLine : data) {
		if (dataLine[3] == V) {
			int[] newList = new int[RVList.length + 1];
 
			for (int i = 0; i < RVList.length; i++) {
				valeur = Integer.valueOf(dataLine[3]);
			}
		}
		// return valeur;
	}
     return valeur;
}

A noter que je ne comprend toujours pas l'intérêt de tes 2 boucles for...ni celui de ton code en fait

Si ton but c'est seulement de retourner le 3ème élément d'un tableau un simple :

Code :

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

return Integer.valueOf(dataLine[2]);

suffit...

[nolofinwe]

une petite question tu veux retourner que la valeur a la fin de ta boucle? ou toutes les valeurs que prendra ta variable "valeur" dans la boucle ?
dans le premier cas benouze a raison...

je suis pas sur d'avoir été clair la

[flora806]

En fait je veux récupérer toutes les valeurs que peut prendre "valeur" lors de l'exécution de la boucle.