Discussion :

[ghostdu69]

Bonsoir à tous,

Tout d'abord je tiens à m'excuser si je me trompe de section mais je suis nouveau sur ce forum.

Je vous expose donc mon problème :

J'ai une classe mère nommé Compte qui possède deux classes filles "CompteCourant" et "LivretEpargne". Utilisant la sérialisation plutôt qu'une base de donnée (contrainte personnelle, j'aime me faire du mal -_-'), j'ai du stocker les classes filles dans une ArrayList de Compte. Bon jusque là pas de soucis et comme ça j'écris tout dans un seul fichier et je récupère tout correctement avec les bons formats de classes. Bref. Les choses se compliquent quand je veux créer la fiche d'un compte. Soit lire les données d'un compte de l'une ou de l'autre des classes filles (la classe mère n'a pas de constructeur).

Voici le code qui fait mal aux yeux :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
while (!trouve && i < f.getListeCompte().size()) {
 
            if (numCompte == f.getListeCompte().get(i).getNumeroCompte()) {
                trouve = true;
                if(f.getListeCompte().get(i).getClass().getName().equals("gestioncb.CompteCourant")){
                    CompteCourant leCompte = f.getListeCompte().get(i);
                }
            }else{
                i++;
            }
        }

CompteCourant leCompte = f.getListeCompte().get(i) est vraiment pas conseillé et de toute façon NetBeans n'en veut pas même si j'ai bien vérifié qu'il s'agissait d'un compteCourant au préalable.

Une solution serait de passer ma fenêtre dans ma classe mère Compte et de la rendre Abstract et de créer une méthode qui remplirait les champs de la fenêtre. Ainsi j'utiliserai le polymorphisme.... Mais est-ce propre ?

Quelqu'un peut-il venir à mon secours ?

[ghostdu69]

Bon du coup je pense avoir trouvé une réponse :

Dans ma boucle je récupère un Compte que j'envoie a ma méthode remplirFiche qui est définit comme suit :

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
 
public void RemplirChamps(Compte leCompte) {
 
        if (leCompte.getClass().getName().equals("gestioncb.CompteCourant")) {
            CompteCourant ceCompte = (CompteCourant) leCompte;
            TF_CoutCarte.setText(ceCompte.getCoutCarte() +"");
            TF_CoutMensuel.setText(ceCompte.getCoutMensuel() +"");
            TF_typeCompte.setText("Compte Courant");
        } else {
            livretEpargne ceCompte = (livretEpargne) leCompte;
            TF_Plafond.setText(ceCompte.getPlafond() +"");
            TF_interet.setText(ceCompte.getInteretAnnuel() +"");
            TF_typeCompte.setText("Livret Epargne");
        }
 
        TF_dateOuverture.setText(leCompte.getDateOuverture().toString());
        TF_numCompte.setText("" + leCompte.getNumeroCompte());
        //TF_solde.setText("");
        TF_titulaire.setText(leCompte.getTitulaire().getNom() + " " + leCompte.getTitulaire().getPrenom());
 
        B_EffectuerOperation.setVisible(true);
        B_ajouterCarte.setVisible(true);
 
 
    }

Que pensez-vous de cette méthode ? Trop cracra ou pas ? En tout cas ça marche.

[verbose]

Tout d'abord, sur le plan technique, lorsqu'on veut vérifier le type d'un objet, il y a le mot-clé instanceof :

Code :

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

1
2
3
4
 
if (leCompte instanceof CompteCourant) {
    CompteCourant ceCompte = (CompteCourant) leCompte;
}

Ensuite, de manière plus générale, il n'est pas très judicieux d'utiliser la serialisation pour persister les données des comptes, car la serialisation ne permet pas d'effectuer les opérations CRUD comme l'autorise l'utilisation d'une base de données.

Et puis pour ton IHM, perso j'aurais fait deux écrans distincts pour les comptes courants et les livrets d'épargne. Ces comptes partagent bien des informations communes, comme la date d'ouverture, le numéro de compte, etc.. Mais certaines informations sont spécifiques comme le taux de rémunération.

[ghostdu69]

Bonjour,

Merci pour ta réponse. Le coup de la sérialisation c'est imposé par mes études car dans un autre projet j'utilise déjà une base de donnée. Sinon pour l'affichage des fiches de comptes je "créer" la fenêtre en fonction du type de compte justement (d'ailleurs merci pour le coup du instance of).

Mais le problème est toujours le même, est-ce "bien" de traiter en fonction du type de classe ou est-ce trop sale ? Et as-tu une alternative ?

[ghostdu69]

Au passage, pour les opérations CRUD je passe par des arrayList comme ça à chaque modification ou suppression je vide le fichier et j'écris l'arrayList sur celui-ci. En passant par le buffer je met moins de 2 secondes pour un fichier de 3.6 Mo donc je suis large.

[Vakhyw]

En fait il y a selon moi un problème de logique d'archi.

Soit tu utilises un ArrayList<Compte> et effectivement tu veux manipuler une collection de "Compte" et alors à toi d'exploiter le polymorphisme comme il se doit pour ne pas avoir besoin d'appel à instanceof. Soit les classes "ComptesCourant" et "LivretEpargne" sont des entites logiques différentes (bien que toutes les deux héritent de "Compte") et alors il te faut 2 ArrayList ; une pour chaque type d'éléments.

A toi de voir ce que tu cherches effectivement à manipuler dans tes ArrayList.


De la même manière tu pourrais très bien avoir encore des sous-classes de "CompteCourant" du genre "CompteCourantClassique" "CompteCourantJeune", ...