Discussion :

[supaneo]

Bonjour,

Est-ce vous savez remplir la classe en pièce jointe ?
J'ai fini de la remplir mais ça ne fonctionne pas...

En pièce jointe, la classe à compléter puis celle que j'ai complétée.

Pouvez-vous me dire où j'ai faux svp ?
Ps : Je n'ai pas encore rempli la classe toString vu que le reste ne marche pas pour l'instant.

Merci beaucoup!

[eulbobo]

Donc ca ne "compile" pas

Code :

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

1
2
3
4
5
6
 
 if (croute < MIN_CROUTE || croute > MAX_CROUTE) {
            CROUTES[CROUTE_DEFAUT] = croute;
        } else {
            CROUTES[croute] = croute;
        }

Ici, tu as CROUTES[], un tableau qui contient des chaines de caractères, et croute qui est un entier qui correspond à l'indice de la croute à utiliser dans ton tableau

Dans ton constructeur, du définis des chaines de caractères dont 'c' qui je pense correspond à la valeur à afficher de la croute
Donc il faudrait faire plutôt comme ça

Code :

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

1
2
3
4
5
6
 
if (croute < MIN_CROUTE || croute > MAX_CROUTE) {
            c = CROUTES[CROUTE_DEFAUT];
        } else {
        	c = CROUTES[croute];
        }

Mais le soucis, c'est que les variables que tu définis au niveau de ton constructeur ne sont pas "visibles" ailleurs : elles n'existent que quand tu instancies ton objet.
Pour pouvoir les utiliser ailleurs, il faudrait les "sortir" du constructeur pour qu'elles deviennent visible par ton instance !

[supaneo]

Merci!

Donc pour le problème de compilation, je m'étais rendu compte après l'envoi du message que j'avais fait une erreur sur croute.

Suite à ton commentaire, j'ai sorti mes déclarations String du constructeur et je les ai placées dans la classe pour qu'ils soient visibles partout.
Par contre, quand je crée l'objet et que je l'inspecte, tout est à null

[supaneo]

J'ai ajouté en dernier mon nouveau code si vous voulez voir

[professeur shadoko]

pourquoi ne pas utiliser des énumérés pour les accessoires comme garniture, viande, etc.... ?
(par ailleurs les noms des variables d'instances ne sont pas "parlants" et ça c'est un vrai défaut)

[eulbobo]

Par contre, quand je crée l'objet et que je l'inspecte, tout est à null

Tu as 3 constructeurs dans ta classe
- constructeur par défaut, qui appelle le constructeur avec 4 paramètres avec les valeurs par défaut
- constructeur avec 4 paramètres QUI NE FAIT RIEN
- constructeur avec 5 paramètres qui fait quelque chose

D'où ma question : tu le crée comment ton objet?

[supaneo]

Bon j'en suis rendu à ça pour ma classe Pizza
ps: Je n'ai pas le droit d'ajouter d'autres constructeurs et je dois répondre aux instructions au-dessus de chaque méthode, constructeur, getter, setter , etc

J'arrive pas à afficher correctement la pizza en l'appelant d'un programme testerPizza. Plus particulièrement, la partie extraViande et la partie extraGarniture...
Une idée de l'endroit où j'ai faux ?

[joel.drigo]

Salut,

Je suppose que pour ton test, tu utilises ton constructeur à 6 arguments.

Regarde d'un peu plus près la fin du constructeur :

Code :

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

1
2
3
4
5
6
7
8
9
10
if (extraViande < MIN_VIANDE || extraViande > MAX_VIANDE) {
            this.extraViande = EXTRA_DEFAUT;
} else {
            this.extraViande = viande;
}
if (extraGarniture < MIN_GARN || extraGarniture > MAX_GARN) {
            this.extraGarniture = EXTRA_DEFAUT;
} else {
            this.extraGarniture = garniture;
}

Ce que j'ai surligné devrait t'interpeller...