Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor de mtax #19

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Refactor de mtax #19

wants to merge 3 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Nov 3, 2018

Refactor: Hacer más legible el código de mtax

  • Renombrar X_Tax para cumplir con las convenciones de Java
  • Renombrar las propiedades tasaOCuota y trasladad de XTax para homologar el idioma de nombramiento
  • Cambiar 4 espacios por tab para facilitar las identaciones en el código
  • Renombrar la variable cont de MTax para ayudar a identificar la función de la variable más rápido
  • Renombrar la variable map_taxs para cumplir con las convenciones de Java

Tipo: Refactor

@ghost
Copy link
Author

ghost commented Nov 3, 2018

Me podrían proporcionar el código fuente de la interface Constant y el código del método TaxsByListId.
Adicional, podrían proporcionarme datos de entrada para el método validate, la salida esperada para el conjunto de datos dado y, de ser posible, proporcionar también el conjunto de características necesarias que debe de tener una tasa. Esto para asegurarme de no romper nada.

@benoror
Copy link
Contributor

benoror commented Nov 5, 2018

Hola @rogerdavila, gracias por tu aportación.

La interface Constant es irrelevante para este ejercicio, no te preocupes por eso.

El método TaxsByListId accede a la base de datos, puedes asumir que regresa una List de N objetos X_Tax (podrías usar esta info para armar un "Mock" y testearlo con JUnit 😉 )

En resumen, podrías agregar más cambios de fondo y no sólo de forma al código. Igualmente estaría bien si agregaras pruebas unitarias significativas. Ojalá lo puedas hacer antes de la siguiente entrevista. Saludos!

RIDavila and others added 3 commits November 6, 2018 10:34
- Renombrar X_Tax para cumplir con las convenciones de Java
- Renombrar las propiedades tasaOCuota y trasladad de XTax para homologar el idioma de nombramiento
- Cambiar 4 espacios por tab para facilitar las identaciones en el código
- Renombrar la variable cont de MTax para ayudar a identificar la funciÃn
  de la variable má rádo
- Renombrar la variable map_taxs para cumplir con las convenciones de Java
- Remover cÃdigo comentarizado

Tipo: Refactor
- Cambiar condiciones de comparación List.size >= 0 por !List.isEmpty para hacer mas legible
- Cambiar contador de Tasas no locales por una bandera de tipo boolean
- Encapsular la funcionalidad para setear la propiedad created de un XTax en un método nuevo
- Cambiar HashMap de tipo String, XTax por String, Date debido a que no se requiere el objeto completo en el mapa
- Se agregan unit tests para los posibles escenarios
- Se agrega Mockito para simular la consulta a BD

Tipo: Refactor
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants