valentinarau / dds-monedero

Monedero

Contexto

Este repositorio contiene el código de un monedero virtual, al que podemos agregarle y quitarle dinero, a través de los métodos Monedero.sacar y Monedero.poner, respectivamente. Pero hay algunos problemas: por un lado el código no está muy bien testeado, y por el otro, hay numeros code smells.

Consigna

Tenés seis tareas:

1. 🍴 Hacé un repositorio usando este template (presionando desde Github el botón use this template)
2. ⬇️ Descargalo y construí el proyecto, utilizando maven
3. 👃 Identificá y anotá todos los code smells que encuentres
4. 🧪 Agregá los tests faltantes y mejorá los existentes.
   • 👀 Ojo: ¡un test sin ningún tipo de aserción está incompleto!
5. ⛑️ Corregí smells, de a un commit por vez.
6. ⬆️ Subí todos los cambios a tu fork
7. 🐛 Activá los issues de Github desde https://github.com/TU_GITHUB_USERNAME/dds-monedero-java8/settings así podemos darte nuestras devoluciones

Tecnologías usadas

• Java 8.
• JUnit 5. ⚠️ La versión 5 de JUnit es la más nueva del framework y presenta algunas diferencias respecto a la versión "clásica" (JUnit 4). Para mayores detalles, ver:
  • Apunte de herramientas
  • Entrada de Blog (en inglés)
  • Entrada de Blog (en español)
• Maven 3.3 o superior

Solucion

Code Smells:

1. Duplicated code en Cuenta poner y sacar

   if (cuanto <= 0) { 
   throw new MontoNegativoException(cuanto + ": el monto a ingresar debe ser un valor positivo");
   }

2. Numeros Magicos en Cuenta:

double limite = 1000 - montoExtraidoHoy;

if (getMovimientos().stream().filter(movimiento -> movimiento.isDeposito()).count() >= 3) {

3. Long Method en Cuenta poner y sacar
4. Message Chain en Cuenta getMontoExtraidoA

movimiento.getFecha().equals(fecha)

5. Misplaced Method en Movimiento agregateA

  public void agregateA(Cuenta cuenta) {
    cuenta.setSaldo(calcularValor(cuenta));
    cuenta.agregarMovimiento(fecha, monto, esDeposito);
  }

6. Type Test y Misplaced Method en Movimiento calcularValor

  public double calcularValor(Cuenta cuenta) {
    if (esDeposito) { // type test
      return cuenta.getSaldo() + getMonto();
    } else {
      return cuenta.getSaldo() - getMonto();
    }
  }