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.

[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?

[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]

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

[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.

[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.

[nolofinwe]

alors a ta place je mettrai la boucle principale dans le main et j'appellerai a chaque fois la méthode pour renvoyer une seule valeur

ainsi tu retourne une valeur a chaque fois.

[flora806]

dans le programme principal, ça veut dire que je le mets dans le main ou dans la méthode du parseFile? (sachant que cette dernière méthode est de type void)

[Benouze]

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

Dans ce cas ce n'est pas un int mais un tableau de int que tu dois retourner.
Là où se trouve ton return initial tu remplies ton tableau et à la fin de la méthode (là où moi j'ai mis le return) tu retournes le tableau.

[nolofinwe]

tu met ta premiere boucle dans le main et tu appelle la méthode getRV a chaque itération de la boucle , comme ca tu récupere toutes les valeurs et tu les traitent une par une

edit: la solution de benouze est meilleure j'y avais pas penser

[flora806]

Dans ce cas ce n'est pas un int mais un tableau de int que tu dois retourner.
Là où se trouve ton return initial tu remplies ton tableau et à la fin de la méthode (là où moi j'ai mis le return) tu retournes le tableau.

oui, c'est vrai, ça semble plus logique ta solution.
en fait quand j'ai pensé à la méthode "static int getRV", c'est que à partir d'une autre classe je fais appel à cette méthode pour calculer la différence entre l'entier valeur et un autre entier qui est calculé par une autre méthode.
---------------
int valeur = NomClasse.getRV(.....)
int value2 = NomClasse2.getValue2(..)
......
int Difference = math.abs(value-value2)

donc si je fais un return de tableau pour "valeur", j'aurai un pb de type encore une fois

Merci pour vos conseils et désolé pour mon niveau de débutant

[nolofinwe]

il y a une chose qui me chiffone si tu veux récuperer toutes les valeurs et réaliser un traitement sur chacune d'entre elle je vois pas comment c'est réalisable avec ton code actuel... par contre il est possible de renvoyer un tableau et de réliser ton traitement sur chacune des valeurs du tableau.

au lieu d'appeller directement ta méthode tu appelle les éléments du tableau renvoyé par la méthode..

heu... j'ai été clair ?

Merci pour vos conseils et désolé pour mon niveau de débutant

tu n'as pas a t'excuser on est tous passé par la et pour etre franc je dois etre aussi débutant que toi

[flora806]

il y a une chose qui me chiffone si tu veux récuperer toutes les valeurs et réaliser un traitement sur chacune d'entre elle je vois pas comment c'est réalisable avec ton code actuel... par contre il est possible de renvoyer un tableau et de réliser ton traitement sur chacune des valeurs du tableau.

au lieu d'appeller directement ta méthode tu appelle les éléments du tableau renvoyé par la méthode..

heu... j'ai été clair ?


tu n'as pas a t'excuser on est tous passé par la et pour etre franc je dois etre aussi débutant que toi

-----------

Merci Nayah, mais je n'ai pas bien compris comment je pourrais appeler les éléments du tableau renvoyé par la méthode getRV

[nolofinwe]

-----------

Merci Nayah, mais je n'ai pas bien compris comment je pourrais appeler les éléments du tableau renvoyé par la méthode getRV

et bien je me trompe peut-etre mais disons que tu récupere un tableau x[] via ta méthode, je pense que tu pourra utiliser quelques chose du genre:

Code :

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

1
2
3
4
for (int i=0 ; i<size ; i++){
valeur = x[i];
//traitement de la valeur
}

size = taille de ton tableau

je pense que comme ca tu pourras traiter toutes tes valeurs une par une.

[flora806]

et bien je me trompe peut-etre mais disons que tu récupere un tableau x[] via ta méthode, je pense que tu pourra utiliser quelques chose du genre:

Code :

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

1
2
3
4
for (int i=0 ; i<size ; i++){
valeur = x[i];
//traitement de la valeur
}

size = taille de ton tableau

je pense que comme ca tu pourras traiter toutes tes valeurs une par une.

------------
c'est bon, je n'ai plus l'erreur signalée par le compilateur concernant le "int result"
j'ai mis finalement:

{
int [] RVList = new int[0];
int valeur = 0;

for (int[] dataLine : data)
{
if (RVList.length == 0)
return 0;
else
{
if (dataLine[2] == V)

{

for (int i=0; i < RealVoteList.length; i++)
{
valeur = dataLine[2];
}
}

}

}
return valeur;

}

là j'ai d'autres exceptions "NullPointerException"...c'est parti pour une nouvelle galère en java
Merci à Nayah et à tous!!!

[nolofinwe]

déja il y a quelque chose que je comprends pas :

Code :

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

int [] RVList = new int[0];

ici tu créé un tableau dont la taille est nulle

deuxieme chose:

Code :

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

for (int i=0; i < RealVoteList.length; i++)

ton RealVoteList n'est instancié nulle part , si il est null tu peut avoir un "Nullpointeur exception