Discussion :

[MagretsDeCanard]

Bonjour à tous,

J'ai commencé à apprendre Java avec le livre : "Le livre de java premier langage". Il ne me reste plus que les parties sur les bibliothèques graphiques (swing et AWT) ainsi que Android à voir...

Mais avant j'aimerai recevoir quelques conseils sur mon code, sa qualité, des pistes d'améliorations...

Mon code vous provoquera mal de tête et peut être même vomissement, mais vos conseils me permettront certainement de m'améliorer !

Donc si vous avez 5 min pour me faire une "revue de code" je vous en remercie d'avance !



Petite explication du projet :

C'est un système de gestion de compte, on peut ajouter des comptes et leurs créer des "lignes comptables".

Un nombre "infini" de compte peut être créé, chaque compte peut sauvegarder 10 lignes comptables.

Les compte peuvent être supprimés et affichés avec leurs lignes comptables.



Voici mon code pour le projet proposé dans le livre : https://github.com/magrets/ProjetLivreV2.git


Les points dont je suis sur d'avoir besoin de conseil :

- problème avec mes Scanner :

J'ai créé comme dans le livre un Scanner dans chaque fonction ou j'en utilise un. Je sais pas comment faire pour tous les fermer...

- L'utilisation de git :

Je suis en train d'apprendre, quand faites vous un commit (nouvelle fonction, nouvelle class...) ?




Si vous avez également quelques conseils concernant des choses à voir par la suite (pattern, structure de code, bibliothèque etc).

En vous remerciant,

Magrets.

[Mat.M]

salut dans le fichier CompteEpargne.java le do..while m'inquiète un peu.
Vaut mieux éviter..

Ensuite on fait un commit lorsqu'on est certain que le code source n'est pas à modifier en général.
Il faut faire deux branches, une branche de code "stabilisé" et une autre branche de code qui est sujet à modification.

[MagretsDeCanard]

Bonjour, merci d'avoir prit le temps de me répondre.

Je ne comprend pas pourquoi ma boucle pose problème :/ Est ce que vous pourriez m'en dire un peu plus ?

Concernant le versionnage je prend note et je vais m'y mettre !

Encore merci,
Magrets.

[Mat.M]

bonsoir c'est pas que ça pose problème c'est que la boucle risque de tourner indéfiniment.
Il faut donc traiter une condition de sortie

[MagretsDeCanard]

Mais ma boucle a une condition de sortie, je suis désolé je ne comprends pas bien le problème... :/

Code :

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

1
2
3
4
5
6
7
8
9
10
private double controlTaux(){
        double tmpdouble;
 
        do{
            System.out.println("Quel est le taux du compte épargne :");
            tmpdouble = Double.parseDouble(Sc.myScanner.next());
        }while(tmpdouble < 0.0);
 
        return tmpdouble;
}

Merci pour la modifications sur mon message

[Rony Rauzduel]

En fait lorsque Mat.M parle de condition de sortie de la boucle do ... while, il veux te dire que telle que la condition de ta boucle est conçue, il n'y a pas de condition de sortie à proprement parlé.

Une condition de sortie explicite serait le code suivant :

Code Java :

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
 
private double controlTaux(){
        double tmpdouble;
        boolean sortie = Boolean.TRUE;  
        int compteur = 0;        
 
        do{
            System.out.println("Quel est le taux du compte épargne :");
            tmpdouble = Double.parseDouble(Sc.myScanner.next());
            if (tmpdouble < 0.0) {
                sortie = Boolean.FALSE;
            }
            compteur++;
        }while(sortie && compteur < 2);
 
        return tmpdouble;
}

Il faut créer une variable compteur, qui comme son nom l'indique va incrémenter un compteur du nombre de saisie de taux de compte épargne.
Si tu ne fais pas cela la boucle va continuer sans fin à demander la saisie des taux d'épargne au client. Tu souhaites un taux d'épargne et pas 2, 3, 4, ... n taux d'épargne.

Rony Rauzduel