tomas-teston / iwvg.swc.tomasteston Goto Github PK

En el método "isEquivalent" de la clase "Fraction" que has creado, en la parte del return, métele paréntesis para que no de problemas.

Así: ( numerator * fraction.getDenominator() ) == ( denominator * fraction.getNumerator() );
Quedaría más legible.

Info: Sacar el valor de una posición

Hito: 23/09/2018

No modifiques el método "fullName", no te hace falta y no se pide.

Info: Fracción propia

Hito: 30/09/2018

Info: Presentar el nombre en mayúsculas

Hito: 23/09/2018

Info: Modificar coordenadas

Hito: 23/09/2018

El método "format" que creas no hace falta.

Info: Fracción equivalente

Hito: 30/09/2018

En la issue se te especifica que crees una fracción llamada equivalentFraction que será con la que compararás con fraction. Ej: assertTrue(Fraction.isEquivalent(equivalentFraction)).

Esto aplica para los tres test que has creado.

Si te fijas, en la wiki se detalla que el método "nameToUpperCase" devuelve "void", y en tu caso estás devolviendo un String. (el único cometido de esta función es convertir el nombre a mayúsculas, nada más).

Info: Fracción impropia

Hito: 30/09/2018

No te hace falta comprobar si la variable "name" está vacía o no, ya que su único cometido es pasarlo a mayúsculas y no te daría error si está vacía.

El método isNotEmpty es prescindible, por lo que elimínalo.