search

zuldare / IWVG.SwC.JaimeHernandez

Práctica de Software Colaborativo
0 stars 0 forks source link

Issue#1 - Clase Fraction: división de dos fracciones #1

Closed zuldare closed 6 years ago

zuldare commented 6 years ago

Se pide incluir el método dividir que permita la división de dos fracciones. Se le pasará por parámetro la fracción por la cual se quiere dividir la primera.

Un ejemplo de esta operación podría ser el siguiente:

image

Tiempo estimado : 1 día

Para más información Ver wiki

GuillermoSanz commented 6 years ago

fixed

zuldare commented 6 years ago

Según Sonarcloud hay alguna cosa que debería cambiarse. En la clase Fraction en el método dividir está devolviendo un aviso de 'smell code'. Esto se produce porque estás calculando resultado, guardandolo y posteriormente devolviendo ese resultado. Sonar pide que se haga en una instrucción. El cambio sería esto

Fraction franctionResult = new Fraction(this.numerator fractionB.denominator, this.denominator fractionB.numerator) return franctionResult;

cambiarlo por

return new Fraction(this.numerator fractionB.denominator, this.denominator fractionB.numerator)

Reabro incidencia.

GuillermoSanz commented 6 years ago

smell code solved