Discussion :

[Invité]

Bonjour !
J'aimerai savoir, pour comparer une classe qui gère des comptes en banque, comment je pourrais comparer l'argent de deux comptes différents.
J'ai essayé ceci

Code :

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

1
2
3
4
    public void comparer ( Banque autreCompte ) {
        double comparer = this.getSolde() - autreCompte.getSolde();
        System.out.println("Vous avez " + comparer + "€ de différence");
    }

Merci d'avance !

[joel.drigo]

Salut,

Quel est le problème au juste ?

[Invité]

Salut,

Quel est le problème au juste ?

Je voudrais savoir comment on pouvait comparer deux objects car le code que j'ai essayé ne fonctionne pas et je trouve pas comment faire

[joel.drigo]

le code que j'ai essayé ne fonctionne pas et je trouve pas comment faire

C'est-à-dire ne fonctionne pas ? Même si ce n'est pas la meilleure façon de procéder (il vaut mieux isoler les parties traitements et affichages), ton code compile à priori (à moins que la méthode getSolde() n'existe pas, ou ne retourne pas de double, ou autre... vu que je n'ai qu'un extrait de code, je peux faire des milliers de conjectures ça ne servirait à rien), s'exécute et son résultat s'apparente à une certaine forme de comparaison (afficher la différence des soldes de deux comptes).

Sans plus d'informations impossible de comprendre ce qui à ton sens ne fonctionne pas.

[Invité]

C'est-à-dire ne fonctionne pas ? Même si ce n'est pas la meilleure façon de procéder (il vaut mieux isoler les parties traitements et affichages), ton code compile à priori (à moins que la méthode getSolde() n'existe pas, ou ne retourne pas de double, ou autre... vu que je n'ai qu'un extrait de code, je peux faire des milliers de conjectures ça ne servirait à rien), s'exécute et son résultat s'apparente à une certaine forme de comparaison (afficher la différence des soldes de deux comptes).

Sans plus d'informations impossible de comprendre ce qui à ton sens ne fonctionne pas.

Oui, je comprends.
J'ai pas plus de code pour l'instant car je suis sur les ordis de mon ecole;
En gros le code c'est :
* crée une classe Banque avec paramètre solde
* crée un (je sais plus le nom) getSolde()
* crée un (idem) qui permet d'afficher la comparaison entre 2 objets de Banque
Le code cité plus haut me donne 0 alors que les soldes sont de 250 et 230 pour les objets.

Je vous mes le code complet ce soir si vous ne comprenait pas très bien maintenant

Merci

[joel.drigo]

Le code cité plus haut me donne 0 alors que les soldes sont de 250 et 230 pour les objets.

On peut déjà commencer par le vérifier en ajoutant System.out.println("solde = " + this.getSolde() + " autre = " + autreCompte.getSolde()).
Le problème est peut-être dans le paramètre autreCompte. Quel est le code de l'appel de la méthode comparer() ?

[Invité]

On peut déjà commencer par le vérifier en ajoutant System.out.println("solde = " + this.getSolde() + " autre = " + autreCompte.getSolde()).
Le problème est peut-être dans le paramètre autreCompte. Quel est le code de l'appel de la méthode comparer() ?

Ton code fonctionne pas
Si je mes 150 de solde sur compte et 200 sur compte1, il m'affiche que sur les deux comptes il y a 200 soldes
Voilà le code du fichier banque.java

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
package com.company;
 
public class Banque {
 
    public static double solde;
 
 
    public Banque ( double sld ) { //Constructeur avec choix soldre
        solde = sld;
    }
    public Banque(){ //Constructeur par default
        solde = 0;
    }
 
 
    public void deposer ( int prix ) {
        solde += prix;
    }
 
    public void retirer ( int prix ) {
        solde -= prix;
    }
 
    public static double getSolde () {
        return solde;
    }
 
    public void afficherSolde () {
        System.out.println("Vous avez : " + solde + "€ sur votre compte." );
    }
 
    public void comparer ( Banque autreCompte ) {
        System.out.println("solde = " + this.getSolde() + " autre = " + autreCompte.getSolde());
    }
}

Et le code du fichier main :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
package com.company;
 
public class Main {
 
    public static void main(String[] args) {
	Banque compte = new Banque(200);
	Banque compte2 = new Banque(125);
 
 
	compte.comparer(compte2);
    }
}

Et ça me donne :

[joel.drigo]

Mon code, comme tu dis, il faut l'ajouter à ta méthode compare() pour afficher les soldes dont tu dis que leur différence donne 0. Il fonctionne très bien et tu en donnes même la preuve dans le résultat qu'on voit à la fin de ton dernier message (que j'aurais préféré en texte plutôt qu'en image). C'est ce que je voulais savoir : la différence de 125 et de 125 donne bien 0. CQFD.

Le problème est ailleurs :

Code :

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

 public static double solde;

Une variable static est partagée entre toutes les instances de la classe. Avec solde en static, tous les comptes ont forcément le même solde. Supprime ce mot static.

Il te faudra le supprimer aussi dans la déclaration de getSolde().

Par ailleurs, c'est plutôt dangereux que la variable solde soit public. Cela veut dire que n'importe quelle classe peut la modifier n'importe quand et comment : tu ne pourras pas l'empêcher en ajoutant des contrôles dans les mutateurs et si tu veux pouvoir gérer des notions de limites de découvert, tu ne pourras pas.

[Invité]

Merciii ça marche nickel !!!