Discussion :

[koyot3]

Bonjour,

j'ai un probleme de recuperation de type. je m'explique :
je crée un tableau de type Employe. Or Employe peut etre un commercial ou un employé horaire.

Employe[] employes = new Employe[5];

employes[0]=new Employe("Poiret", new EmployeHoraire(42));
employes[1]=new Employe("Burot", new EmployeHoraire(30));
employes[2]=new Employe("Dupont", new Commercial(10000));
employes[3]=new Employe("Ravier", new Commercial(20000));
employes[4]=new Employe("Terrice", new Commercial(15000));

or pour un calcul, je voudrais pouvoir a partir de employes[i] savoir comment faire pour connaitre si cet employé est un commercial ou non ....
j'ai essayé avec les "instanceof" mais ca ne marche pas ...

merci d'avance pour votre aide

[verbose]

j'ai essayé avec les "instanceof" mais ca ne marche pas ...

C'est pourtant bien la technique qu'il faut utiliser. Peux-tu nous montrer comment tu t'y prends ?

[koyot3]

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

if (employes[i] instanceof Commercial){
salaireChef+=employes[i].calSalaire();
}
}

"incompatible conditional operand types Employe and Commercial"
voila ce que m'indique eclipse.....

[Sanguko]

Ton problème, c'est que Commercial ne derive pas d'Employe

[koyot3]

j'aurais du faire par héritage .....

donc la, ya pas de solution possible ?

[in]

Ben si tu n'a pas de relation extends ou implements entre les deux types d'objet, comment veux tu que java devine que ces objets sont équivalents ??

[kerinel]

D'après ce que tu as écrit, tu crées un employé en lui passant un paramètre supplémentaire de type EmployeCommercial ou EmployeHoraire.
pour utiliser instanceof tu dois dériver la classe Employe en EmployeCommercial, EmployeHoraire etc...
Quelque chose comme ceci:

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
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
 
*********un premier fichier, Employe
public class Employe{  //eventuellement abstract 
 
     String nom;
     //autre variable commune à tous les employés (adresse etc...)    
 
     public Employe(String nom){
          this.nom=nom;
          //autre truc à faire
     }
 
     public void uneMethodeCommuneaTousLesEmployes(){
         //faire un truc commun à tous les employés
     }
 
}
 
***********Autre fichier EmployeHoraire
 
public Class EmployeHoraire extends Employe{
 
         public int nombreDHeure;
         //autres variables particulières à un employé horaire
 
        public EmployeHoraire(String nom,int nbH){
             super(nom);
             nombreDHeure=bbH;
             //autre choses particulières à un employé horaire
        }
 
        public void uneMethodePourlesEmployeHoraire(){
              //faire un truc spécifique au employés horaires
       }
 
}
 
************Autre fichier EmployeCommercial
 
public Class EmployeCommercial extends Employe{
 
         public float pourcentage;
         //autres variables particulières à un employé commercial
 
        public EmployeCommercial(String nom,float perc){
             super(nom);
             pourcentage=perc;
             //autre choses particulières à un employé commercial
        }
 
        public void uneMethodePourlesEmployeCommercial(){
              //faire un truc spécifique aux commerciaux
       }
 
}
 
***************Fichier pour utiliser
//déclaration
Employe[] employes = new Employe[5];
 
employes[0]=new EmployeHoraire("Poiret", 42);
employes[1]=new EmployeHoraire("Burot", 30);
employes[2]=new EmployeCommercial("Dupont", 7.5);
employes[3]=new EmployeCommercial("Ravier", 8.4);
employes[4]=new EmployeHoraire("Terrice",20);
 
...
//utilisation
for (int i=0;i<employes.length;i++){
      if employes[i].instanceof(EmployeCommercial.class){
                   //faire le traitement ad-hoc
      }
}

Dans ton cas tu as peut être moyen de récupérer le second paramètre passé lors de la construction, mais tu perds l'avantage de l'héritage et ça me semble un peu "pas très clean" .

Bon code,
kerinel

Edit: télescopage pendant que je postais... Pourquoi sans héritage ? Alors que c'est justement l'un des exemples types de l'héritage. En plus tu dois faire de l'héritage quelque part car ton constructeur de Employé accepte (String, Commercial) ou (String, Horaire). A moins d'avoir définis deux constructeurs. Mais dans ce cas si un jour tu veux créer des cadres, ou des employés "à la tache" il va falloir refaire encore d'autres constructeurs...

Si vraiment tu veux faire sans héritage alors pourquoi faire une classe Commerciale et une classe Horaire. dans ce cas tu crées des objets inutiles et le fait que l'employé soit un commercial ou un horaire est géré uniquement par une variables de employé. éventuellement tu déclares des variables statiques
public static final int EMPLOYE_HORAIRE et public static final EMPLOYE_COMMERCIAL .

Bon code,
kerinel

[verbose]

Si tu as ce message d'erreur, cela signifie que les classes EmployeHoraire et Commercial n'héritent pas de Employe. Lorsque tu ecris que les Employés peuvent être des employé horaire ou des commerciaux, de quelle façon implémentes-tu cela dans ton code ?

[koyot3]

employes[4]=new Employe("Terrice", new Commercial(15000));

j'ai un employé de type commercial dans cette case...
je pense rajouter un parametre dans Employe pour reperer le type d'employé...

je précise que le but est d'essayer de le faire sans héritage .....

[verbose]

Tu t'y prends bizarrement, m'enfin, c'est ton code

Ce que tu peux faire est de rajouter un getter qui te permet de récupérer l'attribut correspondant au type de l'employé, puis tu vérifie le type de cet attribut. M'enfin, c'est vraiment tiré par les cheveux.