Bonjour,

dans le cadre d'un tp je dois réaliser une simulation de distributeur automatique de billets, tout en gérant le nombre de billets dans le distributeur

Voici ce que j'ai fait :
http://pastebin.com/W5g6mfGD

ca ne marche pas, comme vous vous en doutez c'est la fonction distribue qui pose problème, je ne vois pas où est-ce qu'il faut utiliser la fonction Math.max

Si quelqu'un pourrait m'aider svp !

-Ton test dans vide() est à l'envers
-Tu peux dire à ton prof que les méthodes java commencent par des minuscules par convention.
-Ne pas utiliser "Boolean" mais "boolean"
- Il faut que tu testes si le montant qu'on veut retirer est divisible par 10 à l'aide d'un modulo.

Pour la distribution, il y a plusieurs solutions.

perso j'aurai

fait un truc du style

int resteADistribuer = somme;
int maxNb50 = this.nbBillets50;
int nb50 = 0;

while( resteADistribuer > 50 && maxNb50 > 0 )
{
maxNb50 - 1;
resteADistribuer -= 50;
nb50++;
}

Tu peux même factoriser cela sous forme de fonction.
Pourquoi ne pas décrémenter directement le membre de la classe? Parce que faut d'algo plus sophistiqué tu ne sais si à la fin tu auras assez de billets de 20 ou de 10 pour pouvoir distribuer.

Autre souci, il se peut que tu puisses pas livrer
2x50 + 10
mais que tu puisses livrer 50 + 2x20 + 2x10 mais vu le niveau de l'exercice je doute qu'on vous demande de gérer ça de façon sophistiquée.

le test n'est pas à l'envers c'est bien écrit dans le tp que Vide() renvoie true s'il reste des billets et false sinon

Pour ta solution en fait on décrémente les attributs de la classe tout à la fin de la distribution ?

Demander que la méthode "vide" retourne "true" si le distributeur ne l'est pas est un non sens d'un point de vue conceptuel.
En Java, on ne met pas de majuscules aux méthodes, c'est pas du C# ou du C++ Microsoft.
Mettre les types de billets en attribut de classe est également un piètre choix au niveau de la conception : si le programme a besoin d'être utilisé pour un autre pays ? On doit tout réécrire ?
Au passage, rien que cette ligne "if (somme>(nbBillets50*50+nbBillets20*20+nbBillets10*
10))"
devrait inciter à utiliser une classe Billet plutôt que des valeurs magiques.

Bon, passons.

Déjà, tu devrais vérifier que ta somme est bien développable en produits de tes valeurs de billets (Si l'utilisateur demande 7€ ?).
D'un point de vue pratique, tu devrais, à l'instar des vrais distributeurs, veiller à distribuer les coupures pour optimiser tes stocks.

en voyant le titre, je dirais Licence1

Toujours bloqué ? Perso j'ai du temps si tu as besoin

L'auteur est banni

Si quelqu'un connait la solution je suis prenant, j'arrive pas à trouver le pb de son code.