Discussion :

[Iphelias]

Bonjour à tous,

Voila, je me pose une question de conception.

Voici un exemple pour illustrer mon problème.

Code :

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

1
2
3
4
5
6
7
8
9
 
public class Client {
    private Compte moncompte;
 
    public Client() {
        moncompte=new Compte();
    }
[...]    
}

A l'utilisation je sais que ces 2 solutions sont possibles. Quel est le meilleur choix. Que nous impose le Génie logiciel?

public void AjouterCompte(Client cli){
Compte leCompte = new Compte();
[...]
cli.setMonCompte(leCompte);
}

ou

public void AjouterCompte(Client cli){
Compte leCompte = cli.getMonCompte();
[...]
}

Pour ma part j'ai une préférence pour la première solution car on peut retrouver facilement l'association Client / Compte via la méthode setMonCompte(). Si par exemple l'application devient important et que de nombreuses personnes doivent faire de la maintenance. Mais je me trompe peut être.

Donc j'attends votre avis

[Catsoumoto]

Bonjour,


Pour moi si tu fait un fonction AjouterCompte faudrai que la classe soit comme cela :

Code :

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

1
2
3
4
5
6
7
8
9
10
 
public class Client {
    private ArrayList<Compte> mescomptes;
 
    public Client() {
        mescomptes = new ArrayList<Compte>;
        mescomptes.add(new Compte());
    }
[...]    
}

Une liste de compte, quand le client est cree un compte est cree dans la liste.

Ensuite pour la fonction ajouterCompte elle devrai etre dans la class Client et ajouter un compte a la liste:

Code :

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

1
2
3
4
 
public void AjouterCompte(){
    this.mescomptes.add(new Compte());
}

Si j'ai bien comprit ce que tu ve faire voila comment je le ferai.

[Iphelias]

Je me suis peut être mal exprimé, mais ma méthode "ajouterCompte" n'est pas une méthode de l'objet Client ou Compte.

la question que je me pose c'est : "faire un .getCompte" et travailler sur l'objet instancié dans le constructeur ou faire un "new Compte()" et un "setCompte()" par la suite.

[Sharcoux]

Ben les deux sont très différents... L'un te fait partir d'une instance de monCompte vierge, définie par le constructeur alors que l'autre (getCompte) te fait partir d'une instance de compte déjà créée et qui a pu évoluer

[aelmalki]

A mon avis, et de point de vue évolution du code source, il vaut mieux dissocier l'instanciation du Compte de la classe Client, il se peut fans le future que le ou les Compte Client soit récupérer par un service, ou de la base de données ou un fichier XML...

[diskdiet]

Je me suis peut être mal exprimé, mais ma méthode "ajouterCompte" n'est pas une méthode de l'objet Client ou Compte.

la question que je me pose c'est : "faire un .getCompte" et travailler sur l'objet instancié dans le constructeur ou faire un "new Compte()" et un "setCompte()" par la suite.

En pratique on utilise des getters et des setters

[Sharcoux]

En pratique on utilise des getters et des setters

Sauf dans le constructeur : il est tout à fait toléré d'initialiser une variable à ce moment. La question est plutôt pour moi : est-ce que le processus d'initialisation doit être séparé du processus de modification ?

[Catsoumoto]

Moi je pense qu'il est mieu de recuperer l'instance de l'objet et de la modifier, que d'ecraser l'ancienne instance par une nouvelle.