juansalinas-xyz/mercantilandina

Hello and welcome to my profile!

I am a Senior Backend Engineer with 10+ years of experience.

For the last 3 years, I have been developing RESTful APIs using Java, Kotlin, NodeJs and Python programming languages.

I am hands on in all stages of the software development cycle, from planning and design to testing and deployment.

My tasks involve building microservices, analyzing algorithms, altering code, fixing bugs, brainstorming ideas or integrating new systems.

Skills:

• Java
• Kotlin
• NodeJs
• Python
• Spring Boot
• RESTful
• GraphQL
• SQL
• NoSQL
• Blockchain (Algorand)
• Maven
• Gradle
• AWS
• Scrum
• Kanban

Feel free to reach out via DM if you have an interesting job opportunity.

Thanks for visiting my profile.

mercantilandina's Issues

Feedback Mercantil andina

Problemas detectados
• Falta implementar el caso de uso de aplicar descuento.
• Producto se pedía con clave de tipo UUID.
• No tiene test de aceptación
• Se utilizan las Entidades como objetos DTOs de los controladores. Ejemplo com.ma.pedidos.controller.PedidoController#createPedido
• Se utiliza Double para los importes.
• Producto no tiene validaciones de los atributos.

Cambios sugeridos
• Crear test de aceptación quitando los Mocks de los servicios y probando todas las capas. Aquí es conveniente crear un set de datos de prueba.
• Usar el patrón ObjectMother para generar ejemplos de prueba. Artículo
• Utilizar Objetos DTOs como pedidos y respuestas en los controladores.
• Utilizar BigDecimal para los importes. Usar float crea problemas de redondeo. Articulo
• Utilizar ControllerAdvice para formatear una respuesta en caso de error. Así quedan más limpios los controladores. Tutorial
• En com.ma.pedidos.controller.PedidoController#createPedido. Llevar el control de producto al servicio de Pedido. No exponer lógica de negocio en los controladores.
• En com.ma.pedidos.controller.ProductoController#deleteProduct También llevar este control al servicio.
• Los métodos de los servicios debería tener lenguaje de negocio. Por ejemplo com.ma.pedidos.service.ProductService#hasPedidosAssociated debería estar en castellano y llamarse “tienePedidosAsignados”.