Pedido de Cambio: computando-patentes - Los Apila Bits

[Noorman999]

Buenas! Resolviendo ejercicios para practicar y comparando con los resueltos en el repo, se me ocurrio una forma mas limpia y mas corta de resolver el método 'getConversor' de la clase Conversor.

Metodo como esta Actualmente:

 public static Conversor getConversor(Patente patente) {
    Conversor conversor;
    if (patente.valor.length() == 6) {
        conversor = new Conversor1995(patente);
    } else {
        conversor = new Conversor2016(patente);
    }
    return conversor;
 }

Cambio Sugerido:

public static Conversor getConversor(Patente patente) {
    return patente.valor.length() == 6 ? new Conversor1995(patente) : new Conversor2016(patente);
}

Ejercicio

Clase

Se que no es un gran cambio, pero es mas fácil de entenderlo a primera vista.

Saludos!

[delucas]

No me termino de decidir si el cambio es mejor, o peor. Recurro al juicio de @Federico-G @LucasMPL y @hlanzillotta

[Federico-G]

Me parece más claro, pero a la vez estaría bueno que todo el repo este manejado de una manera, que sea siempre la misma. Podría ser ésta

[stale[bot]]

Este PR se marcó automáticamente como abandonado porque no ha tenido actividad recientemente. Será cerrado si no se aplica ninguna medida opuesta. Gracias por las contribuciones.

[stale[bot]]

Lamentablemente este PR se debió cerrar por falta de actividad.