Discussion :

[sinda1]

bonjour,

s'il vous plait,j'ai réalisé un code java de la classe comptebancaire,et je veux votre aide et vos remarques sur ce code en sachant que je suis une débutante en java
merci d'avance


voici le code:

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
public class Comptebancaire
{
private String nom;
private int solde;
private String prenom;
private int numerodecompte;
/** permet de sortir de l'argent   */
public void retrait (int montant)
{
solde=solde-montant;
}
/**
     * Permet d'ajouter de l'argent
     * param montant
     */
public void poser( int montant)
{
solde=solde+montant;
} 
 
  public Comptebancaire(int numero, String nom, String prenom,int y)
       {
       	this. numerodecompte= numero;
        this.nom = nom;
        this.prenom = prenom;
        this.solde=y;
} 
 
 
 
  /** return Nom du client */ public String getNom() {
        return nom;
    }
 
  /** return Numéro de compte */
    public int numerodecompte() {
        return numerodecompte;
    }
 
  /** return nom de client  */
    public String getPrenom() {
        return prenom;
    }
 
 /** return Solde présent sur le compte */
 public int getsolde(){
        return solde;
    }
 public void transferer(int montant,int x,int y){
x=solde-montant;
y=solde+montant;
 
}
 
}

[tchize_]

coté conventions:
-> essayer de respecter une logique dans votre indentation, c'est un peu compliqué à suivre là
-> La classe devrait s'appeler CompteBancaire

Coté erreur, je vois tout d'abord:

Code :

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

1
2
3
4
5
 public void transferer(int montant,int x,int y){
x=solde-montant;
y=solde+montant;
 
}

Cette méthode n'a pas de sens. Tous les paramètres en java sont passé par copie. Votre méthode ne fait donc rien puisque les modifications sur x et y seront perdues dès la fin de la méthode.

[Drawingrom]

C'est sympa de commenter du code.

Ici :

Code :

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

solde=solde-montant;

Ajoutez des espaces pour la lisibilité :

Code :

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

solde = solde - montant;

Ou mieux :

Code :

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

solde -= montant;

Là :

Code :

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

public int numerodecompte() {

C'est peu cohérent avec vos autres accesseurs :

Code :

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

public int getNumeroDeCompte() {

D'autre part, vos commentaires JavaDoc n'apportent aucune information :

Code :

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

1
2
/** permet de sortir de l'argent   */
public void retrait (int montant)

J'imagine bien que dans une classe CompteBancaire, la méthode retrait permet de retirer de l'argent. Vos noms de méthodes et de paramètres sont suffisamment expressifs.

Sinon comme l'a dit tchize_, essayer de garder une cohérence dans votre style de code et de suivre les conventions.

[OsmosIT]

Bonjour,

j'aurais tendance également à valider vos paramètres. Dans le cas d'un dépôt, il est illogique de pouvoir passer un nombre négatif. Même remarque dans le cas d'un retrait.

Bonne journée!

Kevin Renson