Discussion :

[ebh1979]

Bonjour, je voudrais avoir un avis pour savoir si ma conception est bonne ou non. Ca fait un bail que je n’ai pas programmé donc excusez moi si mes questions sont stupides. J’essaie de faire un programme de Location Immobilière.
Le propriétaire peut être une personne physique ou une société. Le locataire aussi.
J’ai crée deux interfaces IProprietaire et ILocataire et mes classes sont :

Code :

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

1
2
3
4
5
6
7
Personne implements IProprietaire, ILocataire{	
             private boolean islocataire;
	private boolean isProprietaire;} 
 
Societe implements IProprietaire, ILocataire{
	private boolean islocataire;
	private boolean isProprietaire;}

Je stocke tous ces contacts crées dans une ArrayList mais comment je peux savoir en demandant le prochain élément de la liste si c’est une Personne ou une Societe ??. En fait j’ai des problèmes de Cast. Je pourrais créer une classe unique avec des variables d’états isPersonne,isSociete mais ça me semble pas la bonne solutions.
Vos suggestions sont les bienvenus
Merci d’avance.

[Sir Coco]

Bonjour,

tu peux utiliser le mot clé instanceof

Exemple :

Code :

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

1
2
3
4
5
6
ILocataire locataire = listeLocataire.get(1);
if (locataire instanceof Personne) {
// Traiter le cas de la personne
} else if (locataire instanceof Societe) {
// Traiter le cas de la société
}

Voilà,
A+

[ebh1979]

Merci je m'etais embrouille dans mes classes avec les instances et tout.
en fait voila la classe que j'utilise pour extraire le Proprietaire avec l'id donnee:

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 IProprietaire getProprietaire(int id){
   Iterator it = liste.iterator();
   while (it.hasNext()){
     IProprietaire prop = (IProprietaire) it.next();
     if (prop.isProprietaire()) // Assurance pour savoir si c'est un proprietaire
     {
     if (prop.getId() == id)
     {
        if (prop instanceof Personne)
        {
          Personne pers = (Personne) prop;
          return pers;
        }
       if (prop instanceof Societe)
       {
         Societe soc = (Societe) prop;
         return soc;
       }
    }
 }
}
return null;
}

ca marche mais ca me semble un peu lourd. Je veux dire sinon niveau conception est-ce qu'il vaudrait mieux creer quatre classes LocatairePersonne, LocataireSociete, ProprietairePersonne, ProprietaireSociete?

[merill]

niveau conception, l'idéal est de ne mettre acun switch, elseif, pour determiner un triatement différencié.
Si l'héritage permet de redéfinir des méthodes c'est justement pour éviter cela:

Code :

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

1
2
3
4
5
6
7
8
9
 
interface I{
void traintement();
}
 
class A implements I{
void traitement(){
  print("A");
}

Apres, là où on peut t'aider, c'est la meileur manière d'implémenter ton opération dans le modèle objet. Pour cela, dit-nous ce que tu veux faire avec tes 4 interfaces, etc.

[ebh1979]

En fait mes soucis sont surtout au niveau de la conception et tu traitements.
J'emploie beaucoup de Cast et je sais pas si c'est bien ou non?

j'utilise presque pas de polymorphisme et meme les interfaces que j'ai cree je peux m'en separer puisque dans mes classes j'ai les booleans isProprietaire, isLocataire. c'est pour ca que je me demande si ca ne serait pas mieux de creer quatre classe :

LocatairePersonne implements ILocataire{}
LocataireSociete implements ILocataire{}
ProprietairePersonne implements IProprietaire{}
ProprietaireSociete implements IProprietaire{}

ca deviendra un peu plus complique mais au moins ca sera plus clair.

aussi ca m'evitera d'instancier des variables que j'utilise pas :
ma classe actuelle est :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
public class Societe implements IProprietaire, ILocataire
{
	private final int id;
	//
	private boolean islocataire;
	private boolean isProprietaire;
	private boolean isFournisseur;
	//
	private ArrayList<Local> listeProprietes = new ArrayList<Local>();
	private ArrayList<Local> listeLoues = new ArrayList<Local>();
}

cad si Societe isProprietaire l'Arraylist ListeLoues( cad liste des locaux loues par une Societe ) n'est jamais utilises.

[merill]

re
je dirais même:

LocatairePersonne implements ILocataire, Personne{}
LocataireSociete implements ILocataire, Societe{}
ProprietairePersonne implements IProprietaire, Personne{}
ProprietaireSociete implements IProprietaire, Societe{}

et essaye de virer les booleen islocataire & isProprietaire. En principe, ca n'a pas d'utilité.
Apres, il faudrait plus de détails sur où tu stoocke tout ca, pour faire quoi.

courage

[visiwi]

Une toute petite question :

A quoi te serve tes 4 interfaces (ILocataire, Personne, Societe et IProprietaire) alors que tu as 4 classes (LocatairePersonne, LocataireSociete, ProprietairePersonne et ProprietaireSociete) qui permettent d'identifier chacun et donc de définir les propriétés et comportements adéquat ?

Me semble-t-il, à rien.

ne serait-ce point :
1 classe par exemple : Entite
et 4 interfaces ou classes Locataire, Proprietaire, Societe et Personne ?