Discussion :

[ghohm]

Salut à tous,

voilà ce que je cherche à faire :

Code :

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

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 :

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

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

[herve91]

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 :

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

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()]);
        } 
    }

[BiM]

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

Tu fait une ArrayList, c'est mieux.

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
/* 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()

[Deadpool]

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 :

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

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

Exemple d'appels possible de ce constructeur:

Code :

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

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

ou

Code :

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

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

ou

Code :

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

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

etc...

Tout ça avec un seul constructeur!

[ghohm]

Wouuuuuaah !

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 Vraiment developpez.com ... ça cartonne !


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 !

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

Bon appétit à tous !

Et merci encore !

Gôm

[BiM]

Et bien de rien

[ghohm]

Code :

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

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 :

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

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

Merci d'avance.

Gôm

[gailuris]

Code :

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

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.

[le y@m's]

Avec le jdk 1.5 tu peut faire comme ça :

Code :

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

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.

[ghohm]

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

Code :

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

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

Code :

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

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 :

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)

[le y@m's]

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

Code :

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

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

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

[BiM]

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.

[gailuris]

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

Code :

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

(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 J'avais mal ciblé ma réponse désolé

[le y@m's]

Il faut utiliser les generics (jdk 1.5).

[vbrabant]

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

Code :

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

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

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

Code :

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

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 :

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

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

par

Code :

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

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

cela ira beaucoup mieux :-)

Vincent

[BiM]

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

[ghohm]

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

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 :

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

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

[BiM]

Mais ca marche maintenant non ?

[ghohm]

Mais ca marche maintenant non ?

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

Sinon question affichage :

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)

[herve91]

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