[Débutant] incrémentation du nom d'une variable dans un for

Salut à tous,

voilà ce que je cherche à faire :

Code:

---

1 2 3 4 5 6 7 8

public Garage1(String nomGarage, Voiture v1, Voiture v2, Voiture v3) { this.nomGarage = nomGarage; this.tabVoiture = new Voiture [3]; for(int i=1 ; i<=3 ; i++) { this.tabVoiture[i-1] = v+i; } }

---

Je souhaite donc que le nom de mes variables (v1, v2 et v3) s'incrémente automatiquement grâce à mon autre variable "i".

Bien sur au départ j'avais fait comme ça (et ça fonctionnait très bien !!!) :

Code:

---

1 2 3 4 5 6 7 8

public Garage1(String nomGarage, Voiture v1, Voiture v2, Voiture v3) { this.nomGarage = nomGarage; this.tabVoiture = new Voiture [3]; this.tabVoiture[0] = v1; this.tabVoiture[1] = v2; this.tabVoiture[2] = v3; } }

---

Mais bon, cherchant à optimiser mon code et sachant que le nombre de variable "vX" peut augmenter à l'avenir ... une boucle s'avérait nécessaire.


Merci de m'aider.

@+ Gôm

Eeeeet BONNES FEEEEEETES !!! :D

Ce que tu cherches à faire est impossible...
Si le nombre de voitures vient à augmenter, tu ne vas pas pouvoir gérer tous les cas possibles, et créer autant de constructeurs qu'il y a de possibilités..
Pourquoi ne pas prévoir de constructeur qui prenne en entrée un tableau ou une liste de voitures ? Tu laisses à l'appelant le soin de construire la donnée et il te suffit de la stocker telle quelle (ou presque) :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

public Garage1(String nomGarage, Voiture[] voitures) { this.nomGarage = nomGarage; this.voitures = voitures.clone(); } }   public Garage1(String nomGarage, List voitures) { this.nomGarage = nomGarage; this.voitures = (Voiture[]) voitures.toArray(new Voiture[voitures.size()]); } }

---

On ne peut pas aller chercher les variables de cette manière.

Tu fait une ArrayList, c'est mieux.

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24

/* Déclarations */ static int NB = 3; Voiture [] tabVoiture; ArrayList saListe;   /* Pour créer la liste : */ void creer() { saListe = new ArrayList(); for (int i=0;i<NB;i++) { Voiture taVoiture = new Voiture(...); //remplir ta voiture laListe.add(taVoiture); }//for }//creer()   /* Pour lire la liste : */ void lire() { tabVoiture = new Voiture [NB]; for (int i=0;i<NB;i++) tabVoiture[i]=(Voiture)laListe.get(i); }//lire()

---

C'est impossible d'incrémenter le nom d'une variable en java.
Par contre si tu utilise Java 5.0 tu peux utiliser l'ellipse qui te permet d'avoir un nombre variable de voiture et de les récupérer sous la forme d'un tableau donc tu n'aurais plus qu'à faire :

Code:

---

1 2 3 4

public Garage1(String nomGarage, Voiture... voitures) { this.nomGarage = nomGarage; this.tabVoiture = voitures; }

---

Exemple d'appels possible de ce constructeur:

Code:

---

1 2	Garage1 g = new Garage1 ("garage", v1);

---

ou

Code:

---

1 2	Garage1 g = new Garage1 ("garage", v1, v2);

---

ou

Code:

---

1 2	Garage1 g = new Garage1 ("garage", v1, v2, v3);

---

etc...

Tout ça avec un seul constructeur! :wink:

Wouuuuuaah ! 8O

J'étais entrain de répondre à herve91, qui, je pensais m'avait répondu très rapidement ... et que vois-je ! TROIS réponses en tout !!!

Merci merci :hola: Vraiment developpez.com ... ça cartonne ! :bravo:


Merci pour toutes vos réponses, mais je ne suis pas sur de pouvoir toutes les assimilées correctement ! :?

OK pour la liste chaînée, je pensais m'orienter dans cette voie et de toutes façons c'est ce qui m'est demandé par la suite dans l'exercice de mon cours ! :wink:

J'essaye vos propositions et je reposte après manger !

Bon appétit à tous !

Et merci encore ! :oops:

Gôm

Et bien de rien :P

Citation:

---

Envoyé par BiMouXeTTe

Code:

---

1 2 3 4 5 6 7 8

/* Pour lire la liste : */ void lire() { tabVoiture = new Voiture [NB]; for (int i=0;i<NB;i++) tabVoiture[i]=(Voiture)laListe.get(i); }//lire()

---

---

Serait-il possible que quelqu'un m'explique cette méthode ? Et plus particulièrement :

Code:

---

1 2	tabVoiture[i]=(Voiture)laListe.get(i);

---

Merci d'avance.

Gôm

Citation:

---

Envoyé par ghohm

Code:

---

1 2	tabVoiture[i]=(Voiture)laListe.get(i);

---

---

... signifie dans la "case" i de mon tableau je met le i éme élément de ma liste.

Avec le jdk 1.5 tu peut faire comme ça :

Code:

---

1 2 3 4 5 6 7 8 9 10

public Garage1(String nomGarage, Voiture ...voitures) { this.nomGarage = nomGarage; List<Voiture> liste = new ArrayList<Voiture>(); for(Voiture v : voitures) { liste.add(v); } this.tabVoiture = liste.toArray(new Voiture[liste.size()]); }

---

Ainsi tu n'as pas à te préoccuper du nombre de voitures.

OK merci ça j'avais réussi à comprendre. Ce que je ne comprenais pas, c'était :

Code:

---

(Voiture)

---

si j'ai bien compris l'erreur de compilation générée par NetBeans, si je ne le mets pas, c'est que ça sert à spécifier le type d'objet contenu dans ma liste chaînée. Corrigez moi si je me trompe.

J'ai essayé comme ceci (mais ça fonctionne pas !) :cry:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

public void Affiche() {   Voiture [] tmpTab = new Voiture [listeChainee.size()];   if (this.listeChainee.isEmpty()) System.out.print("Ce garage NE contient PAS de voiture(s) :\n"); else { System.out.print("Le garage "+this.nomGarage+" contient "+this.listeChainee.size()+" voitures :\n"); for (int i=0 ; i<=listeChainee.size() ; i++) { tmpTab[i] = (Voiture)this.listeChainee.get(i); System.out.print(tmpTab[i]); } } }

---

Voici l'erreur retournée :

Citation:

---

Envoyé par Debugger Netbeans

Le garage Saroteau contient 3 voitures :
Exception in thread "main" java.lang.IndexOutOfBoundsException: Index: 3, Size: 3
at java.util.LinkedList.entry(LinkedList.java:368)
at java.util.LinkedList.get(LinkedList.java:313)
at Garage2.Affiche(Garage2.java:40)
at Test1.main(Test1.java:35)
Voiture@10655ddVoiture@ef5502Voiture@b61fd1
Java Result: 1
debug:
BUILD SUCCESSFUL (total time: 0 seconds)

---

Citation:

---

Envoyé par ghohm

J'ai essayé comme ceci (mais ça fonctionne pas !) :cry:

Code:

---

1 2 3

... for (int i=0 ; i<=listeChainee.size() ; i++) ...

---

---

ce n'est pas :
i <= listeChainee.size()
mais :
i < listeChainee.size()

Le (Voiture) est un cast pour transtyper l'objet en voiture.

Une ArrayList contient des objets.

Tu n'aurais pas fait for (int i=0;i<=3;i++) par hasard ?

Il faut for (int i=0;i<3;i++) sans le = puisque la taille de ta liste fait 3 mais les index vont de 0 à 2.

Citation:

---

Envoyé par ghohm

OK merci ça j'avais réussi à comprendre. Ce que je ne comprenais pas, c'était :

Code:

---

(Voiture)

---

si j'ai bien compris l'erreur de compilation générée par NetBeans, si je ne le mets pas, c'est que ça sert à spécifier le type d'objet contenu dans ma liste chaînée. Corrigez moi si je me trompe.

---

Oui tout à fait :wink: J'avais mal ciblé ma réponse désolé :oops:

Il faut utiliser les generics (jdk 1.5).

Citation:

---

Envoyé par ghohm

OK merci ça j'avais réussi à comprendre. Ce que je ne comprenais pas, c'était :

Code:

---

(Voiture)

---

si j'ai bien compris l'erreur de compilation générée par NetBeans, si je ne le mets pas, c'est que ça sert à spécifier le type d'objet contenu dans ma liste chaînée. Corrigez moi si je me trompe.

---

Exactement. La méthode de la liste te retourne un objet. Et toi tu veux le mettre dans une variable de type Voiture. Le compilateur n'apprécie donc pas. Mais en mettant (Voiture), tu rassures le compilateur en lui disant que l'objet retourné sera une Voiture.
Le compilateur te fait alors confiance.

Maintenant, si tu travaille avec le JDK 5, tu pourrais utiliser les Génériques, pour éviter ce genre de code.

Citation:

---

J'ai essayé comme ceci (mais ça fonctionne pas !) :cry:

Code:

---

1 2 3 4 5 6 7

... for (int i=0 ; i<=listeChainee.size() ; i++) { tmpTab[i] = (Voiture)this.listeChainee.get(i); System.out.print(tmpTab[i]); } } }

---

---

en changeant

Code:

---

for (int i=0 ; i<=listeChainee.size() ; i++) {

---

par

Code:

---

for (int i=0 ; i<listeChainee.size() ; i++) {

---

cela ira beaucoup mieux :-)

Vincent

Heu t'as mis la meme chose pour avant et apres lol

Effectivement c'est mieux ... mais pas top !!! :mrgreen:

Je pense que ça vient de mon constructeur, mais bon ... Je me permets de vous poster l'ensemble des éléments "sensibles" de mon code :

Code:

---

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

import java.util.LinkedList;   public class Garage2 {   protected String nomGarage; protected LinkedList listeChainee;   public Garage2(String nomGarage, Voiture v1, Voiture v2, Voiture v3) { this.nomGarage = nomGarage; listeChainee = new LinkedList(); this.listeChainee.add(v1); this.listeChainee.add(v2); this.listeChainee.add(v3); }   public void Affiche() {   Voiture [] tmpTab = new Voiture [listeChainee.size()];   if (this.listeChainee.isEmpty()) System.out.print("Ce garage NE contient PAS de voiture(s) :\n"); else { System.out.print("Le garage "+this.nomGarage+" contient "+this.listeChainee.size()+" voitures :\n");   for (int i=0 ; i<listeChainee.size() ; i++) { tmpTab[i] = (Voiture)this.listeChainee.get(i); System.out.print("Voiture "+(i+1)+" : \n"+tmpTab[i]+"\n\n"); } } } }

---

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23

public class Test1 {   public static void main(String[] args) { Voiture car_1 = new Voiture("Renault", "Mégane", 13900.85, 2004); car_1.Affiche();   Voiture car_2 = new Voiture("Peugeot", "407", 16900.85, 2005); car_2.Affiche();   Voiture car_3 = new Voiture("Citroën", "C6", 28900.85, 2006); car_3.Affiche();   Garage1 concess_1 = new Garage1("Dabireau", car_1, car_2, car_3); concess_1.Affiche();   Voiture [] tabTempVoiture= {car_1, car_2, car_3}; Garage1 g1=new Garage1("Dabireau_bis", tabTempVoiture); g1.Affiche();   Garage2 concess_2 = new Garage2("Saroteau", car_1, car_2, car_3); concess_2.Affiche(); } }

---

Mais ca marche maintenant non ?

Citation:

---

Envoyé par BiMouXeTTe

Mais ca marche maintenant non ?

---

Oui ça fonctionne ... enfin ça génère pas d'erreur !!!

Sinon question affichage :

Citation:

---

Le garage Saroteau contient 3 voitures :
Voiture 1 :
Voiture@10655dd

Voiture 2 :
Voiture@ef5502

Voiture 3 :
Voiture@b61fd1

debug:
BUILD SUCCESSFUL (total time: 0 seconds)

---

Redéfinis la méthode toString() de la classe Voiture pour afficher les informations qui t'intéressent.