Exercice sur l'héritage

**khaled_prg** · 30/01/2012, 11h44

Voici mes essais pour la question :

Code :

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

1
2
3
4
5
6
7
for(int i =0;i < ensembleVolailles.length;i++)
            if(ensembleVolailles[i] != null){
                if(ensembleVolailles[i].getType().equals("Poulet"))
                    prixTotalPoulet += ensembleVolailles[i].prixJour;
                if(ensembleVolailles[i].getType().equals("Canard"))
                    prixTotalCanard += ensembleVolailles[i].prixJour;
            }

Quelles sont vos suggestions ?

**Mako 5013** · 30/01/2012, 11h58

Envoyé par khaled_prg

Quelles sont vos suggestions ?

Bah tu n'as plus qu'à additionner tes deux sous-totaux, et faire quelques petits tests (sur le même principe que pour la seconde question) histoire de vérifier que cela fonctionne...

Mako.

**khaled_prg** · 30/01/2012, 16h41

Bonjour !

Selon les règles du forum on ne doit pas poser une question non accompagnée d'un essai, par contre si on fait un essai on a le droit à une réponse (ou une idée).
Donc, j’espère que les participants qui se donnent la peine-avec toute la reconnaissance qu'ils méritent-veuillent bien apporter des idées qui contribuent à élucider la solution évitant autant que possible les ambiguïtés et l’imprécision, On est dans le domaine scientifique et il va de soi donc que la précision aide à orienter le chercheur à déterminer ses failles.

Pour le moment voici un nouveau essai :

Volaille.Java

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 abstract class Volaille {
    int ID;
    double Poids;
    public Volaille(double P, int id){
        Poids = P;
        ID = id;
    }
    @Override
    public String toString(){
        return "Poids : "+Poids+", N° : "+ID;
    }
    public abstract String getType();
    public abstract double getPrixJour();
}

Poulet.Java

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
public class Poulet extends Volaille{
    //On suppose que le poids d'abattage égal cinq kilo.
    final static double POIDS_ABAT = 5.00;
    static double prixJour;
    public Poulet(double p, int id, double pj){
        super(p,id);
        prixJour = pj;
    }
    public void SetPrixJour(double PrixPoulet){
         prixJour = PrixPoulet;
    }
    public double getPrixJour(){
        return prixJour;
    }
    @Override
    public String toString(){
        return super.toString() +", Prix du Jour : "+prixJour+", Poids d'abattage : "+POIDS_ABAT;
    }
    public String getType(){
        return "Poulet";
    }
}

Canard.Java

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
public class Canard extends Volaille{
    //On suppose que le poids d'abattage égal quatre kilo.
    final static double POIDS_ABAT = 4.00;
    static double prixJour;
    public Canard(double p, int id, double pj){
        super(p,id);
        prixJour = pj;
    }
    public void SetPrixJour(double prixCanard){
         prixJour = prixCanard;
    }
    public double getPrixJour(){
        return prixJour;
    }
    @Override
    public String toString(){
        return super.toString() +"Prix du Jour : "+prixJour+", Poids d'abattage : "+POIDS_ABAT;
    }
    public String getType(){
        return "Canard";
    }
}

EnsembleVolailles.Java

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
public class EnsembleVolailles{
    final static short NBR_MAX = 200;
    static int prixTotalPoulet,prixTotalCanard;
    static Volaille ensembleVolailles[] = new Volaille[NBR_MAX];
    public static double PrixTotal(){
        for(int i =0;i < ensembleVolailles.length;i++)
            if(ensembleVolailles[i] != null){
                if(ensembleVolailles[i].getType().equals("Poulet"))
                    prixTotalPoulet += ensembleVolailles[i].getPrixJour();
                if(ensembleVolailles[i].getType().equals("Canard"))
                    prixTotalCanard += ensembleVolailles[i].getPrixJour();
            }
        return prixTotalPoulet + prixTotalCanard;
    }
    public static void main(String args[]){
        Poulet Plt_1 = new Poulet(0.25,18,0.30);
        Canard Cnrd_1 = new Canard(0.30,21,0.33);
        Poulet Plt_2 = new Poulet(0.19,36,0.24);
        Canard Cnrd_2 = new Canard(0.18,42,0.2);
        Poulet Plt_3 = new Poulet(0.07,25,0.17);
        ensembleVolailles[0] = Plt_1;
        ensembleVolailles[1] = Cnrd_1;
        ensembleVolailles[2] = Plt_2;
        ensembleVolailles[3] = Cnrd_2;
        ensembleVolailles[4] = Plt_3;
        double prix = PrixTotal();
        System.out.println("Le prix total = "+prix);
    }
}

lorsque j'ai testé la classe EnsembleVolailles on m'a afficher :
Le prix total = 0.0
pour quoi le prix total apparait-il nul ??
Merci.

**tchize_** · 30/01/2012, 17h20

1) prix du jour étant une variable statique: on ne la change pas dans un constructeur (erreur de design)
2) prix du jour étant statique, les méthode qui retournent / changent cette valeur devraient être statique
3) prix du jour est un double < 1 (cf ton main). Tu somme les prix sur des entiers. 0 +0.25 arrondi à un entier, ça fait 0. ajoute y 0.25 au tour suivant, c'est toujours 0. Donc on ne mélange pas les pommes et les poires, reste cohérent sur ton type de données.

**Mako 5013** · 30/01/2012, 17h32

Envoyé par khaled_prg

Selon les règles du forum on ne doit pas poser une question non accompagnée d'un essai, par contre si on fait un essai on a le droit à une réponse (ou une idée).
Donc, j’espère que les participants qui se donnent la peine-avec toute la reconnaissance qu'ils méritent-veuillent bien apporter des idées qui contribuent à élucider la solution évitant autant que possible les ambiguïtés et l’imprécision, On est dans le domaine scientifique et il va de soi donc que la précision aide à orienter le chercheur à déterminer ses failles.

Ouais, enfin là, cela fait presque une semaine et une dizaine de messages que j'essaie d'aider. Et quant à parler de précisions alors que l'on arrive simplement en disant "ça marche pas" ou bien "des idées ?", je trouve cela légèrement malvenu.

Surtout qu'en l'occurrence, si tu avais testé un tant soit peu (ajout de quelques System.out par-ci par-là), tu te serais vite rendu compte de ton erreur.

Sinon, +1 pour les remarques/conseils de tchize_.

Mako.

**khaled_prg** · 31/01/2012, 10h08

1) prix du jour étant une variable statique: on ne la change pas dans un constructeur (erreur de design)
2) prix du jour étant statique, les méthode qui retournent / changent cette valeur devraient être statique
3) prix du jour est un double < 1 (cf ton main). Tu somme les prix sur des entiers. 0 +0.25 arrondi à un entier, ça fait 0. ajoute y 0.25 au tour suivant, c'est toujours 0. Donc on ne mélange pas les pommes et les poires, reste cohérent sur ton type de données.

Vos 3 remarques sont extrêmement pertinentes, la 1ére et la 3éme me sont très claires. Mais avec la 2éme j'ai eu les difficultés suivantes en l’appliquant :

D'abord, la méthode abstraite peut-elle être statique ??
j'ai essayé de rendre la méthode getPrixJour abstraite mais je me suis buté sur une erreur (cf l'image) :
http://sub5.rofof.com/img3/01ajvgp30.png
supposons que la méthode getPrixJour soit abstraite, il apparaitra une erreur dans la classe EnsembleVolailles parce qu’on l'a utilisée avec un objet (ensembleVolailles[i]) même si on l'utilise avec la classe mère Volaille l'erreur continue.

Mako.

Vos conseils et vos remarques m'ont servi beaucoup, je vous en remercie infiniment. Mon impatience s'inscrit dans le cadre de l'utilisation optimale du temps, je m'en excuse.

**tchize_** · 31/01/2012, 12h59

une méthode statique n'a pas a être abstraite. Votre champ est statique, donc votre méthode doit être statique, donc elle ne doit pas être abstraite. Le reste du code doit en découler.

**khaled_prg** · 31/01/2012, 15h10

puisque je vais appliquer getPrixJour sur une des cases de tableau dont le type est Volaille, alors cette méthode doit se présenter dans la classe mère (c'est obligatoire qu'elle soit abstraite), sinon Netbeans m'affiche l'erreur suivante :
http://sub5.rofof.com/img3/01uytsf31.png

c'est quoi la solution alors ?

**tchize_** · 31/01/2012, 15h37

L'erreur c'est dans le design, pourquoi as-tu besoin de l'appeler "sur l'instance". Ceci marche tout aussi bien si la méthode est statique.

Code :

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

1
2
3
4
5
6
7
8
for(int i =0;i < ensembleVolailles.length;i++)
            if(ensembleVolailles[i] != null){
                if(ensembleVolailles[i].getType().equals("Poulet"))
                    prixTotalPoulet += Poulet.getPrixJour();
                if(ensembleVolailles[i].getType().equals("Canard"))
                    prixTotalCanard += Canard.getPrixJour();
            }
        return prixTotalPoulet + prixTotalCanard;

**khaled_prg** · 31/01/2012, 17h03

L'erreur c'est dans le design, pourquoi as-tu besoin de l'appeler "sur l'instance". Ceci marche tout aussi bien si la méthode est statique.

Code :

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

1
2
3
4
5
6
7
8
for(int i =0;i < ensembleVolailles.length;i++)
            if(ensembleVolailles[i] != null){
                if(ensembleVolailles[i].getType().equals("Poulet"))
                    prixTotalPoulet += Poulet.getPrixJour();
                if(ensembleVolailles[i].getType().equals("Canard"))
                    prixTotalCanard += Canard.getPrixJour();
            }
        return prixTotalPoulet + prixTotalCanard;

C'est bien juste mais :

c'est un cas particulier, en effet si on ajoute un autre animal qui hérite la classe mère Volaille (L’oie par exemple) le prix total ne la comportera pas !!, et c'est un problème : (

Ensuite, si on suppose que pour chaque animal nouveau il faudrait un test nouveau ce ne serait pas pratique tant au niveau de la conception que de l'optimisation !!, on aurait :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
static double prixTotal = 0, prixTotalPoulet = 0, prixTotalCanard = 0, prixTotalOie = 0 ....
for(int i =0;i < ensembleVolailles.length;i++)
            if(ensembleVolailles[i] != null){
                if(ensembleVolailles[i].getType().equals("Poulet"))
                    prixTotalPoulet += Poulet.getPrixJour();
                if(ensembleVolailles[i].getType().equals("Canard"))
                    prixTotalCanard += Canard.getPrixJour();
                if(ensembleVolailles[i].getType().equals("Oie"))
                    prixTotalOie += Oie.getPrixJour();
                if......
            }
        return prixTotalPoulet + prixTotalCanard + prixTotalOie + ........ ;

**tchize_** · 31/01/2012, 18h04

1) oui et le code ne compilera pas, ce qui est l'effet voulu quand une méthode doit exister mais n'existe pas
2) le problème est plus à chercher dans ta conception là, ton code de départ n'a déjà pas de sens. Si ta variable est statique, les méthodes qui l'utilisent telle quelles doivent être statique. Si t'as besoin que l'information soit liée à l'instance, alors elle n'a pas à être statique!

**khaled_prg** · 31/01/2012, 18h42

Pour le moment j'ai deux choix les deux ont des avantages et des inconvénient :

Le premier : c'est que je déclare la méthode getPrixJour comme static (et non pas abstraite), son avantage c'est que la valeur de la variable statique va être retournée par une méthode static mais son inconvénient c'est le fait qu'elle ne peut nous affichée que le prix total de Poulet/Canard, pour les autres animaux il faut qu'on écrit pour chacun d'entre eux un nouveau test (si on a 10 animaux le test prendra 20 ligne !!!).
Le deuxième : je déclare la méthode en tant qu'une méthode abstraite (et non static), son avantage c'est que le test se fera sur un animal, la vérification sur le type de volaille se fait automatiquement .. mais son inconvénient c'est que la valeur de la variable statique prixJour va être retournée par une méthode static.

lequel de choix est-il préférable ?
Merci.

**Robin56** · 31/01/2012, 19h29

Envoyé par khaled_prg

lequel de choix est-il préférable ?
Merci.

Je n'ai pas tout suivis et bien compris pourquoi tu utilises des statiques dans ton cas. Moi ce que je vois c'est que toute ta volaille (Poulet, Canard) dispose d'un attribut "prixJour" et d'un getter associé. Du coup moi je ne m'embêterais pas avec abstract, static, .. Je ferais ça :

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
// Déclaration
public abstract class Volaille {
  private double prixJour;
  public double getPrixJour() {
    return prixJour;
  }
}
 
// Utilisation
public double PrixTotal(){
  double total = 0;
  for(int i =0;i < ensembleVolailles.length;i++)
    total += ensembleVolailles[i].getPrixJour();
  }
  return total;
}

Moi je dis ça parce que je vois que ta distinction Canard, etc... n'est pas utile durant ton calcul puisque tu te contentes du prix total.

**khaled_prg** · 01/02/2012, 00h23

Je n'ai pas tout suivis et bien compris pourquoi tu utilises des statiques dans ton cas

il semble que vous n'avez pas compris, peut être parce que vous n'avez pas lu !!, suivez cette phrase :

le prix est le même pour tous les individus de la même espèce.

La variable prixJour existe une seule fois dans chaque classe alors elle doit obligatoirement être statique.

Merci.

**tchize_** · 01/02/2012, 00h40

khaled, veuillez restez courtois s'il vous plait.

Quand au message il précise bien, effectivement, le même pour tout espèce, mais ça n'implique pas nécessairement d'utiliser un champ statique, vu qu'il existe plusieurs espèces justement. Le comportement (prix du jour) est présent dans tous les animaux, mais chaque sous classe en a un différent, ce qui est incompatible avec un champ statique. Mais c'est aussi (tous les animaux de la même espece on le même) incompatible avec un paramètre direct à passer au constructeur (comme vous le faites actuellement)

Dans votre cas, on peux représenter, par exemple, cette contraite comme ceci si on ignore la contrainte "change chaque jour":

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
public abstract class Volaille {
  private double prixJour;
  public Volaille(double prixJour){this.prixJour=prixJour;}
  public abstract double getPrixJour() {
    return prixJour;
  }
}
public class Poulet {
   public Poulet(double p, int id){
      super(15.20); // Le poulet est à 15,20€, mais du coup on sait pas configurer
   }
}

Sinon, en prenant "change chaque jour" mais en considérant des poulet/ canard indépendant du jour d'enregistrement (autrement dit leur prix n'est pas fixe à leur abattage mais évolue dans le temps)

Code :

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

1
2
3
4
5
6
7
8
 
public class PrixDuJour{
    private static Map<String,Double> prix  = new HashMap<String,Double>();
    public static void setPrix(String type, Double prix){
       prix.put(type,prix);
    }
    public Double getPrix(String type){return prix.get(type);}
}

et après initialisation correct des prix du jour

Code :

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

1
2
3
4
5
6
7
8
 
public double PrixTotal(){
  double total = 0;
  for(int i =0;i < ensembleVolailles.length;i++)
    total += PrixDuJour.getPrix(ensembleVolailles[i].getType());
  }
  return total;
}

**khaled_prg** · 01/02/2012, 01h28

khaled, veuillez restez courtois s'il vous plait.

je ne pense pas avoir manqué quelqu'un de respect si vous faites allusion à mon message à Roben pour qui j'ai beaucoup de respect je lui ai montré tout simplement le passage qu'il a avoué lui même ne pas suivre assez. Je ne suis pourtant pas exempt des maladresses qui peuvent survenir inconsciemment si cela arrive je m'en excuse.

pour ce qui est de ce code j ne l'ai pas bien compris :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
public abstract class Volaille {
  private double prixJour;
  public Volaille(double prixJour){this.prixJour=prixJour;}
  public abstract double getPrixJour() {
    return prixJour;
  }
}
public class Poulet {
   public Poulet(double p, int id){
      super(15.20); // Le poulet est à 15,20€, mais du coup on sait pas configurer
   }
}

Comment écrire :

Code :

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

public Volaille(double prixJour){this.prixJour=prixJour;}

alors que vous avez dit :

incompatible avec un paramètre direct à passer au constructeur

**tchize_** · 01/02/2012, 12h25

Volaille est abstrait, ce sont les fils qui n'ont pas le prix du jour en paramètre. L'exemple du poulet montre bien que c'est la classe poulet qui fixe le prix du jour, et pas l'appelant de new Poulet()

**khaled_prg** · 01/02/2012, 16h15

Deux remarques sur ce code s'il vous plait :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
public abstract class Volaille {
  private double prixJour;
  public Volaille(double prixJour){this.prixJour=prixJour;}
  public abstract double getPrixJour() {
    return prixJour;
  }
}
public class Poulet {
   public Poulet(double p, int id){
      super(15.20); // Le poulet est à 15,20€, mais du coup on sait pas configurer
   }
}

Comment vous avez pu utiliser l'opérateur super alors que le Poulet (selon votre code) n'a pas de classe mère ??
Comme la variable prixJour est la même pour tous les individus de la même espèce donc elle existe une seule fois alors, elle est évidemment incompatible avec un paramètre direct à passer au constructeur. Qu'on la place dans le constructeur de la sous-classe ou de la classe mère on en revient au même resultat !! on y recourt à chaque fois à la création d'un objet pour en changer le prix

**tchize_** · 01/02/2012, 17h14

1) oubli

2) la valeur 15.20 est dans le code et donc bien unique. Poulet respecte donc bien la contrainte. Volaille, par contre, ne respecte pas la contrainte "la même valeur pour tous" Mais en même temps on a pas la même valeur pour toutes les Volailles, donc c'est logique.

**khaled_prg** · 01/02/2012, 20h01

puisque le poids el l'id sont commun entre tous les volailles donc elles doivent faire partie de paramètres du constructeur de la classe mère.
la méthode getPrixJour est abstraite donc elle doit être non définie.
Selon votre code, si on crée un nouveau objet de la classe Poulet, le prix du jour sera 15.20 $ et si on change la valeur du prix du jour avec la méthode setPrixJour et on crée un autre objet de Poulet, la valeur de prix deviendra 15.20 $ donc c'est pas logique quand même !!
puisque la variable prixJour est la même pour tous les Poulet, donc il faut la lier à la classe et non pas à l'instance ! ce qui veux dire que sa valeur ne doit absolument pas être changée dans le constructeur.

Exercice sur l'héritage

avec Java

Discussions similaires

Partager

Partager