Actividad 1 - ¿Cómo mejorarías este código de programación? by Diego Martín Sanz https://padlet.com/diegomartinucjc/actividad1_revision_codigo Tecnologías aplicadas en-us 2020-10-25 18:36:23 UTC 2025-10-14 18:02:49 UTC hello@padlet.com https://padlet.net/icons/png/1f4bb.png Revisión de código diegomartinucjc https://padlet.com/diegomartinucjc/actividad1_revision_codigo/wish/859698415 La revisión de código es una práctica común en equipos de desarrollo software que tiene como objetivos, entre otros: 
  • adoptar mejores prácticas aumentando la calidad del código
  • incrementar la detección temprana de errores
  • aumentar el conocimiento general sobre el proyecto
Instrucciones: 
Somos un equipo de desarrolladores trabajando en un mismo proyecto. Uno de los desarrolladores ha terminado de programar una pequeña funcionalidad y el resto del equipo debe revisar su código para decidir si se aprueban, o no, esos cambios.

Cada desarrollador debe:
  1.  añadir un comentario con un aspecto a mejorar en ese código, si nadie lo ha mencionado ya
  2. dar una puntuación de 1 a 5 estrellas a al menos dos comentarios de otros compañeros, donde a mayor número de estrellas, mayor importancia creemos que tiene ese comentario
  3. Opcionalmente se puede discutir cada comentario de código para justificar la puntuación otorgada.

El código a comentar:
]]> 2020-10-25 18:36:33 UTC https://padlet.com/diegomartinucjc/actividad1_revision_codigo/wish/859698415 Pedro diegomartinucjc https://padlet.com/diegomartinucjc/actividad1_revision_codigo/wish/859704969 El nombre de la clase debería ser más significativo, por ejemplo: 
public class EmailNotificator
]]> 2020-10-25 18:41:59 UTC https://padlet.com/diegomartinucjc/actividad1_revision_codigo/wish/859704969 Laura diegomartinucjc https://padlet.com/diegomartinucjc/actividad1_revision_codigo/wish/859705365 El nombre del método 
DoSomething
debería sustituirse por alguno que sea más significativo de la funcionalidad de ese algoritmo, por ejemplo:
SendEmails
]]> 2020-10-25 18:42:25 UTC https://padlet.com/diegomartinucjc/actividad1_revision_codigo/wish/859705365 Inés diegomartinucjc https://padlet.com/diegomartinucjc/actividad1_revision_codigo/wish/859705800 Haz que la dirección de remitente 
noreply@test.com
sea configurable, por si cambia en el futuro
]]> 2020-10-25 18:42:51 UTC https://padlet.com/diegomartinucjc/actividad1_revision_codigo/wish/859705800 Javier diegomartinucjc https://padlet.com/diegomartinucjc/actividad1_revision_codigo/wish/859706165 No crees una instancia del servidor 
SmtpServer
en cada iteración, créalo fuera del bucle.
]]> 2020-10-25 18:43:10 UTC https://padlet.com/diegomartinucjc/actividad1_revision_codigo/wish/859706165 Anexo 1: Revisión de código en el mundo real diegomartinucjc https://padlet.com/diegomartinucjc/actividad1_revision_codigo/wish/859719018 Recomiendo la visualización del siguiente video en el que se muestra una revisión de código en el mundo real con la herramienta GitLab.]]> 2020-10-25 18:54:35 UTC https://padlet.com/diegomartinucjc/actividad1_revision_codigo/wish/859719018