implement and test method setDenominator#7
Conversation
|
|
||
|
|
||
| if (denominator != 0) { | ||
| this.denominator = denominator; | ||
| } | ||
|
|
There was a problem hiding this comment.
Attention : La fraction doit être simplifiée en cas de modifications du dénominateur
| f.setDenominator(9); | ||
| assertEquals(f.getDenominator(),9); |
There was a problem hiding this comment.
Pourquoi ce changement?
There was a problem hiding this comment.
j'ai remarqué que mon test echouait chaque fois car je n'avais la bonne valeur au denominateur .
f = new Fraction(2,3);
f.setDenominator(4);
assertEquals(f.getDenominator(),2); //ici le dénominateur est différent
//Test the case of the null denominator
//Method fails silently, i.e. no change to the fraction
f = new Fraction(1,2); // ceci manquait
f.setDenominator(0);
assertEquals(f.getDenominator(),2);
There was a problem hiding this comment.
Je ne suis pas d'accord avec le premier changement : Lorsqu'on change la fraction 2/3 en 2/4, elle doit être simplifiée et devenir 1/2, d'où le assertEquals(dénominateur, 2).
2 participants
cette méthode permet d'assigner une valeur au dénominateur d'une fraction et reste inchangé lorsque cette valeur est nulle