aide programmation débutant

Version imprimable

Bonjour,
débutant en java je voudrais résoudre ce petit sujet, je l'ai commencé mais rencontre des problemes. Pourriez vous me dire si je suis bien parti pour l'instant.
Merci d'avance.

Ecrire la classe abstraite Vehicule
Attributs :
* nbPlace
* prix
* volumeChargement

Ecrire la classe Voiture héritant de Vehicule
Attribut : booleen : diesel
* Le constructeur verifiera que nbPlace >= 4 (sinon nbPlace sera mis à 4) et volumeChargement<=100 (sinon volumeChargement sera mis à 100)
* Une surcharge des méthodes equals et toString
* Ecrire les accesseurs et mutateurs

Ecrire la classe Garage permettant de gérer une liste de Véhicule(s)
* Ajouter un Véhicule
* Supprimer un Véhicule
* Afficher tous les Véhicules

Voici la classe abstraite Vehicule:
Code:

1 2 3 4 5 6 7 public abstract class Vehicule { private int nbPlace; private int prix; private int volumeChargement; }
La classe Voiture heritant de Vehicule
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 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 public class Voiture { private boolean diesel; private int place; private int prix; private int chargement; public Voiture () { place=0; prix=0; chargement=0; diesel=false; } public Voiture (boolean d, int p, int pr, int c) { diesel=d; place=p; prix=pr; chargement=c; } public int Voiture(int p, int v) { if (place>=4) { return (p=place); } else { return p=4; } if(chargement<=100) { return v=chargement; } else { return v=100; } } public int getnbPlace() { return place; } }
Voilà ce que j'ai fait pour l'instant, petit probleme dans public int voiture avec les if.
encore Merci d'avance.

14/06/2009, 13h59
michel.di

tu pourrais avoir une enum pour dire si c'est un essence, un diesel, s'il est au gpl
tu peux ajouter la plaque d'immatriculation au véhicule et deux véhicules sont égaux s'ils ont la même plaque

par contre la voiture n'hérite pas de vehicule ici !
faut mettre public class Voiture extends Vehicule

c'est quoi de ce public int Voiture ?
14/06/2009, 14h12
azerty0125

et comment faire
Le constructeur verifiera que nbPlace >= 4 (sinon nbPlace sera mis à 4) et volumeChargement<=100 (sinon volumeChargement sera mis à 100)

le public int voiture c'était pour essayer Le constructeur verifiera que nbPlace >= 4 (sinon nbPlace sera mis à 4) et volumeChargement<=100 (sinon volumeChargement sera mis à 100)

teste ça directement dans ton constructeur alors :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
 
 
 public Voiture (boolean d)
    {
        diesel=d;
    }
 
 public Voiture (boolean d, int p)
    {
        this(d);
        if(p>4) place = 4; else place=p;
    }
 
 public Voiture (boolean d, int p, int pr, int c)
    {
        diesel=d;
        if(p>4) place = 4; else place=p;
        prix=pr;
        if (v>100) v=100; else chargement=c;
    }

Après tu peux faire tous les constructeurs possibles!

Pour l'enum tu peux écrire

Code:

1
2
3
4
 
public enum typesVoitures{
		DIESEL,ESSENCE,GPL;
	}

14/06/2009, 15h52
azerty0125

Merci bien, cela est correct, je ne sais pas pourquoi je voulais faire autrement.

les accesseurs sont bien:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
 
public int getplace()
   {
        return place;
   }
 
   public int getprix()
   {
        return prix;
   }
 
        public int getchargement()
        {
            return chargement;
        }

et les mutateurs la meme chose mais avec set à la place de get.

14/06/2009, 16h04
michel.di

oui ceci est correct.
Tu veux coder quoi au final.
N'hésite pas a demander quand tu auras codé le garage! ;)
14/06/2009, 16h09
azerty0125

en fait, je vais coder toutes les classes qui sont demandée dans l'énoncé que j'ai mis au début.
c'est à dire la classe abstraite Vehicule
la classe voiture heritant de vehicule
la classe Garage permettant de gérer une liste de Véhicule(s)

Je vais essayer de termine toute les classes. une fois celle ci terminée pourrais je les mettres en ligne afin d'avoir une correction?
Encore Merci
14/06/2009, 16h13
michel.di

oui pas de soucis je veux bien te les corriger!

Voici mes trois classes. Dans la seconde il me manque la surcharge de methode equals et dans la derniere classe j'ai un probleme dans ajouter et supprimer avec les whiles. Merci de l'aide

Code:

1
2
3
4
5
6
public abstract class Vehicule
{
    private int nbPlace;
    private int prix;
    private int volumeChargement;
}

la classe voiture qui herite de Vehicule

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
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
 
public class Voiture extends Vehicule
 
{
    // instance variables - replace the example below with your own
    private boolean diesel;
    private int place;
    private int prix;
    private int volumeChargement;
 
    /**
     * Constructor for objects of class Voiture
     */
 
    public Voiture ()
    {
        place=0;
        prix=0;
        volumeChargement=0;
        diesel=false;
    }
 
    public Voiture (boolean d, int p, int pr, int v)
    {
        diesel=d;
        if (p>=4) 
        {place=4;}
        else 
        {place=p;}
        prix=pr;
        if (v<100) 
        {volumeChargement=100;}
        else 
        {volumeChargement=v;}
 
     }
 
    public boolean getdiesel()
    {
        return diesel;
    }
 
    public int getplace()
   {
        return place;
   }
 
   public int getprix()
   {
        return prix;
   }
 
        public int getvolumeChargement()
        {
            return volumeChargement;
        }
 
        public void setdiesel( boolean d)
        {
            diesel=d;
        }
 
        public void setplace (int p)
        {
            place=p;
        }
        public void setprix (int pr)
        {
            prix=pr;
        }
        public void setvolumeChargement(int c)
        {
            volumeChargement=c;
        }
 
         public int getVehicule ()
        {
            int x = getplace() + getprix() + getvolumeChargement();
            return x;
        }
 
        public String toString()
        {
            String s = new String ();
 
                s = s+getdiesel()+" "+getplace()+" " +getprix()+ " " + getvolumeChargement();
 
            return s;
        }
 
        public void affiche()
        {
            System.out.print( "diesel: " + getdiesel()+", "+"nombres de places: " +getplace()+", " +"prix: "+getprix()+ ", " +"Volume: " +getvolumeChargement());
        }
 
}

la classe Garage permettant de gérer une liste de Véhicule(s)

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
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
 
public class Garage 
{
    // instance variables - replace the example below with your own
    int nbVoiture;
   // boolean diesel;
    public Voiture[] lesVoitures;
 
    public Garage()
    {
        // initialise instance variables
 
        lesVoitures=new Voiture[100];
    }
 
 public void getVoiture(Voiture v)
    {
         int x =v.getplace()+  v.getprix() + v.getvolumeChargement();
    }
 
    public void ajouter (Voiture v)
    {
        int p= v.getplace();
        int pr=v.getprix();
        int c=v.getvolumeChargement();
 
        if(nbVoiture==0)
        {
            lesVoitures[nbVoiture]=v;
            nbVoiture++;
        }
 
        else
        { 
            int i=0;
            int j=0;
            while(i<nbVoiture && getVoiture(Voiture v ).equals(lesVoitures[i].getVehicule())!=true)
            {
                i++;
            }
        }
}
    public void affiche ()
    {
        for(int i = 0; i<nbVoiture;i++)
        {
            if (lesVoitures[i] !=null)
            {
               {
                    System.out.print(lesVoitures[i].toString());
                }
            }
        }
    }
 
    public void Supprimer (Voiture v)
    {
        int i=0;
        int j=0;
        int p=v.getplace();
        int pr=v.getprix();
        int c=v.getvolumeChargement();
    while (i<nbVoiture && p.equals(lesVoitures[i].getVoiture(Voiture v))!=true)
        {
            i++;
        }
        for (j=i; j<nbVoiture-1; j++)
        {
            lesVoitures[j] = lesVoitures[j+1];
        }
        lesVoitures[j]=null;
        nbVoiture--;
    }
}

je ne comprend pas trop ce que tu fais.
Pourquoi avoir un getVoiture dans la classe Voiture qui retourne un int en plus?!
dans Garage, tu devrais mettre une liste de voiture ce serait plus simple a gérer

en gros tu aurais :

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
 
import java.util.LinkedList;
import java.util.List;
 
 
public class Garage {
 
	//liste de voitures representant le garage
	private List<Voiture> voitures;
 
	//le conctructeur ne fait que creer la liste
	public Garage() {
		this.voitures = new LinkedList<Voiture>();
	}
 
	public void ajouterVoiture(Voiture v){
		this.voitures.add(v);
	}
 
	public void supprimerVoiture(Voiture v){
		this.voitures.remove(v);
	}
 
	public void afficherGarage(){
		for(Voiture v : this.voitures){
			System.out.println(v.toString());
		}
	}
}

14/06/2009, 18h15
michel.di

attend je vais réécrire tes 3 classes et documentant pour que tu vois ce qui ne va pas!
14/06/2009, 18h19
azerty0125

mais je ne me suis jamais servi de linkedlist, j'ai deja vu arraylist, on peut rien faire avec arraylist.

alors voila le résultat, tu as 4 classes, tes 3 d'avant+ l'enumération

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
 
 
public enum TypeVehicule {
	DIESEL,ESSENCE,GPL;
 
	@Override
	/**
         * permet d'afficher le type de vehicule
         */
	public String toString() {
		switch(this){
		case DIESEL : return "diesel";
		case ESSENCE : return "essence";
		case GPL : return "gpl";
		default : return "error";
		}
	}
}

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
 
public abstract class Vehicule
{
	//protected signifie que tes classes filles heritent des attributs
    protected int nbPlace;
    protected int prix;
    protected int volumeChargement;
    protected TypeVehicule type;
 
    //tu n'auras pas a reecrire tout le code pour tout les vehicules differents
 
    public String toString() {
		String s = new String();
 
		s = s + this.type.toString() + " " + this.nbPlace + " " + this.prix + " "
				+ this.volumeChargement;
 
		return s;
	}
 
	protected void affiche() {
		System.out.print(this.toString());
	}
 
}

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
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
 
public class Voiture extends Vehicule
 
{
 
	/**
         * Constructor for objects of class Voiture
         */
 
	public Voiture() {
		// mets bien tes this. a chaque fois c'est plus lisible
		this.nbPlace = 0;
		this.prix = 0;
		this.volumeChargement = 0;
	}
 
	//utilisation de l'enumeration, comme ca tu peux ajouter de nouveau 
	//types de vehicules sans tout reecrire
	public Voiture(TypeVehicule t, int p, int pr, int v) {
		this.type = t;
		if (p >= 4) {
			this.nbPlace = 4;
		} else {
			this.nbPlace = p;
		}
		this.prix = pr;
		if (v < 100) {
			this.volumeChargement = 100;
		} else {
			this.volumeChargement = v;
		}
 
	}
 
	public TypeVehicule getTypeVehicule() {
		return this.type;
	}
 
	public int getplace() {
		return this.nbPlace;
	}
 
	public int getprix() {
		return this.prix;
	}
 
	public int getvolumeChargement() {
		return this.volumeChargement;
	}
 
	public void changerTypeVehicule(TypeVehicule tv) {
		this.type = tv;
	}
 
	public void setPlace(int p) {
		this.nbPlace = p;
	}
 
	public void setPrix(int pr) {
		this.prix = pr;
	}
 
	public void setVolumeChargement(int c) {
		this.volumeChargement = c;
	}
 
 
 
 
}

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
 
import java.util.LinkedList;
import java.util.List;
 
 
public class Garage {
 
	//liste de voitures representant le garage
	//plus facile a utiliser qu'un tableau
	private List<Voiture> voitures;
 
	//le conctructeur ne fait que creer la liste
	public Garage() {
		this.voitures = new LinkedList<Voiture>();
	}
 
	public void ajouterVoiture(Voiture v){
		this.voitures.add(v);
	}
 
	public void supprimerVoiture(Voiture v){
		this.voitures.remove(v);
	}
 
	public void afficherGarage(){
		for(Voiture v : this.voitures){
			System.out.println(v.toString());
		}
	}
}

voila dis moi ce que tu ne comprends pas! ;)

14/06/2009, 18h35
azerty0125

je ne me suis jamais servis de linkedlist, mais les tableaux oui.
merci d'avoir documenter, cela me permet de comprendre plus facilement ce que tu as fait.
14/06/2009, 18h43
michel.di

si tu veux gérer ça avec des tableaux ça va étre bien chiant pour gérer l'ajout et la suppression de voitures.
alors que la les listes gérent tout toutes seules!
si tu veux tu peux remplacer LinkedList par ArrayList!
14/06/2009, 18h47
azerty0125

Ok, oui je voulais gerer avec des tableaux, mais comme tu dis c'est bien chiant je vais donc utiliser arraylist. Merci encore
14/06/2009, 18h51
michel.di

l'ajout ne poserai pas de problème dans le tableau mais a chaque suppression tu devrais décaler toutes les voitures dans le tableau!
si tu as d'autres questions n'hésite pas
14/06/2009, 19h00
azerty0125

il serait possible que tu me montre ce que ça donnerai avec un tableau? si ça te derange, je comprendrai et n'hesite pas à le dire.

je demande ça car dans la formation ou je suis on utilise les tableau, et vu que j'ai un tp controle bientot.

ok je vais refaire ça avec les tableaux alors! ;)

ça devrait étre a peu près ça!

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
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
 
 
public class Garage {
 
	// liste de voitures representant le garage
	// plus facile a utiliser qu'un tableau
	private Voiture[] voitures;
	// indice pour savoir ou placer la voiture dans le tableau
	private int current;
 
	// le conctructeur ne fait que creer la liste
	public Garage() {
		this.voitures = new Voiture[100];
		this.current = 0;
	}
 
	public void ajouterVoiture(Voiture v) {
		if (current < 99) {
			this.voitures[current] = v;
			//une voiture de plus
			this.current++;
		} else {
			System.out.println("le garage est plein");
		}
	}
 
	public void supprimerVoiture(Voiture v) {
		//on parcours tout le tableau jusque l'indice courant(derniere voiture)
		for(int i = 0; i<current;i++){
			if(this.voitures[i] == v){
				//il s'agit de la derniere voiture, il n'y a donc rien a decaler
				if(i==current){
					this.voitures[i]=null;
				}
				else{
					//sinon decalage de toutes les voitures suivantes
					for(int j = i;j<current;j++){
						this.voitures[j] = this.voitures[j+1];
					}
 
				}
				//il y a une voiture de moins
				this.current--;
			}
		}
	}
 
	public void afficherGarage() {
		for (Voiture v : this.voitures) {
			System.out.println(v.toString());
		}
	}
}

14/06/2009, 19h31
azerty0125

je te remercie beaucoup, cela va me permettre de comprendre mieux, ça me fait une bonne revision pour le tp controle.
désolé de t'avoir fait travailler un dimanche, encore merci.
14/06/2009, 19h32
michel.di

pas de soucis, si ça m'aurait embêté je ne l'aurai pas fait! ;)
si tu as d'autres questions n'hésite pas!
14/06/2009, 19h36
azerty0125
Ok, je te remercie, car ce n'ai pas toujours facile de trouver de l'aide.

j'aurais une petite question encore: que signie les deux points dans le for?
Code:

1 2 3 4 5 public void afficherGarage() { for (Voiture v : this.voitures) { System.out.println(v.toString()); } }
puis j'ai une erreur lorsque je souhaite affiche une voiture une fois kel a été mise dans garage.
l'erreur retournée est dans la classe garage dans la fonction afficher à System.out.println c'est NullPointExeption : null.
cela signifie qu'il faut mettre v à null?
15/06/2009, 13h05
bigbear19

Salut,

Les : correspondent simplement à la nouvelle syntaxe des boucles for dispo depuis Java 1.5.

Ton erreur provient du fait que ton tableau est initialisé à 100 cases dans le constructeur. Or si ton tableau n'est pas rempli quand tu appelles la méthode afficherGarage, certaines cases contiendront un pointeur null.
Tu peux rajouter le test if (v != null) dans la boucle for

ou alors fais ça!

Code:

1
2
3
4
5
6
7
 
public void afficherGarage() {
                //tu n'affichera que les voitures dans le garage et non tout le tableau
		for (int i = 0;i <= this.current;i++) {
			System.out.println(this.voitures[i].toString());
		}
	}

15/06/2009, 14h24
azerty0125

merci pour votre aide. Il n'est pas toujours facile de trouver une aide sur la toile.
15/06/2009, 15h17
Djobird
Alors je vais me permettre de corriger le code de départ, parce que si c'est pour un cours, et que tu ne fais que relire le code de michel.di, tu risques de zapper des trucs importants.

Déjà en effet ta classe Voiture n'hérite pas de la classe Véhicule. Donc pour cela comme te l'as fais remarquer michel.di, il te faut faire

Code:

public class Voiture extends Vehicule

Ensuite, quand tu fais de l'héritage les classes fille (dans notre cas voiture) herite des méthodes et attributs de leur classe mère ( dans notre cas véhicule). Donc quand tu vas définir ta classe Voiture, il ne faut pas redéfinir les attributs hérités (dans ton cas place, prix et volume). Seul le booléen doit être dans la classe Voiture (et pas dans Véhicule c'est pas ce qui est demandé !).

Mais pour que cet héritage soit possible, il faut que la visibilité de la variable le permette.
Tu as 3 visibilités différentes :
public : la variable est visible par tout le monde
protected : la variable est visible uniquement par la classe la possédant, et par les classes filles.
private : la variable est visible uniquement par la classe la possédant.
Or toi tu as mis tes variables de Vehicule en privé, donc quand en effectuant l'héritage la classe Voiture ne pourrait en bénéficier.

Enfin les conctructeurs sont faux, en tout cas au moins celui par défaut.
Je comprend que le nombre de place est au minimum de 4, et la charge au maximum de 100, et vous vous mettez l'inverse.
D'autre part, en admettant que cela soit bien le contraire, le constructeur par défaut est erroné puisque soit le nombre de place soit la charge sont à 0.
Vous auriez pu éviter ce problème en ayant le constructeur par défaut :
Code:

1 2 3 4 public Voiture() { this(4,0,0); }
Cela va appeler le constructeur Voiture(place, charge, prix) avec les arguments 4,0,0. L'intérêt est que le tests de validité de place et de charge seront fait à un seul endroit. En admettant que demain le nombre de place mini (ou maxi selon la logique) change, il suffit de changer la valeur dans le constructeur effectuant le test.
15/06/2009, 18h14
michel.di

effectivement j'aurai du détailler la visibilité des attributs. :)

je lui ai juste écrit le plus gros constructeurs avec tous les paramètres possibles, à lui de voir s'il veut en mettre d'autres!

Citation:
Envoyé par michel.di
alors voila le résultat, tu as 4 classes, tes 3 d'avant+ l'enumération
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 public enum TypeVehicule { DIESEL,ESSENCE,GPL; @Override /** * permet d'afficher le type de vehicule */ public String toString() { switch(this){ case DIESEL : return "diesel"; case ESSENCE : return "essence"; case GPL : return "gpl"; default : return "error"; } } }
bizarre ce toString() ?
1) la cas default ne peux pas se produire
2) la méthode toString par défaut rend le nom de l'instance (DIESEL par exemple) ... bon à limite tu passes en caractères minuscules

16/06/2009, 14h48
michel.di

le default est obligatoire sinon le compilo rale que la méthode ne retourne rien dans un autre cas que ces 3 la!
j'ai réécris le toString pour plus de clarté pour azerty0125 qui débute en Java