[debutant]pb de return dans une méthode

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:roll:
Est-ce que vous auriez une explication?

Code:

---

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!!!

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

Code:

---

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 ;

Citation:

---

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:

---

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

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

Code:

---

for (int[] dataLine : data)

---

Donc on ne retourne rien.

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

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?8O
T'as une idée?

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.

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 -18O

Citation:

---

Envoyé par 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.

---

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

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

Code:

---

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:

---

return Integer.valueOf(dataLine[2]);

---

suffit...

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 :aie:

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

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.

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)

Citation:

---

Envoyé par flora806

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.

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 :aie:

Citation:

---

Envoyé par Benouze

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 fois8O

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

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

Citation:

---

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 ;)

Citation:

---

Envoyé par nayah

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

Citation:

---

Envoyé par flora806

-----------

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:

---

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.

Citation:

---

Envoyé par nayah

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:

---

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:cry:
Merci à Nayah et à tous!!!

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

Code:

---

int [] RVList = new int[0];

---

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

deuxieme chose:

Code:

---

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

Citation:

---

Envoyé par flora806

------------
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:cry:
Merci à Nayah et à tous!!!

---

Donne l'ensemble du code de ta classe histoire d'y voir plus clair parce qu'honnêtement ton code est un peu absurde.

Par exemple là :

Code:

---

1 2 3 4

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

---

tu fais une boucle qui ne sert strictement à rien puisque tu n'utilises pas i à l'intérieur...

là :

Code:

---

1 2 3 4 5 6 7

int [] RVList = new int[0]; int valeur = 0;   for (int[] dataLine : data) { if (RVList.length == 0) return 0;

---

tu crées un tableau de taille 0 et tu regardes ensuite si sa taille vaut 0.

là :

Code:

---

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

---

tu utilises RealVoteList mais qu'est-ce que c'est ? D'où ça sort ? Comment c'est initialisé ?

Bref, donne l'ensemble des attributs, dis à quoi ils servent, comment ils sont initialisés et on pourra t'aider. Là on galère.

Citation:

---

Envoyé par nayah

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

Code:

---

int [] RVList = new int[0];

---

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

deuxieme chose:

Code:

---

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

---

dans ma 1ère méthode du parseFile, j'ai mon:
data = new int [items.size()][3];
et:
int V = Integer.parseInt(splitter.nextToken());
---------------
dans la 2ème méthode getRV:
je pense que RVList n'est pas nécessaire.
là j'ai gardé:
public static int getRealVote (int Vote, int user)
{
int valeur = 0;

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

{
for (int i=0; i < dataLine.length; i++)
{valeur = dataLine[2];}
.........return valeur;
-------------
ça ne donne tjr rien :(
j'ai le NullPointerException...

s'il te plait flora806 je te serai infiniment reconnaissant lorsque tu post des bouts de codes d'utiliser les balises [ CODE] et [ /CODE] qui rendraient ton code beaucoup plus lisible parce que la j'avoue que j'ai du mal a décrypter :roll:

Flora, il y a pas mal d'endroit dans ton code qui pourraient générer des exceptions :

if (dataLine[2] == V) --> si dataLine contient moins de 3 éléments : Exception

for (int i=0; i < RealVoteList.length; i++) --> si RealVoteListe vaut null : Exception

Donne l'ensemble de ton code please.

je pense que ton erreur se produit la :

Code:

---

1 2 3

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

---

quelle garantie as tu que dataLine.length n'est pas nulle ?
de plus comme disais benouze , ta boucle ne sert strictement a rien et tu peux la supprimer:
en fait tu écris un nombre x de fois la meme valeur dans ta variable.

Citation:

---

Envoyé par nayah

s'il te plait flora806 je te serai infiniment reconnaissant lorsque tu post des bouts de codes d'utiliser les balises [ CODE] et [ /CODE] qui rendraient ton code beaucoup plus lisible parce que la j'avoue que j'ai du mal a décrypter :roll:

---

Désolé, j'ai pas fait attention.