2009C1‎ > ‎Correcciones de entregas‎ > ‎

Corrección - Caripela Libro - Entrega 1 - Grupo 2

Puntos para refactoring:

  • Ojo con el @override. Encontré muchos que no sobreescribian nada. En algunas compilaciones se marca como inválido
  • Ojo con el estilo de programación en java: las variables y los métodos empiezan con minúscula. Se que la convención parece tonta pero es un estandar y usarlo nos aporta bastante información... por ejemplo puedo diferenciar rapidamente un método estático de un objeto (Math.abs != math.abs) . Usar TDD no sería justificación suficiente para generar tantas excepciones. Si bien uds hacer assert de que la validación falle, quizás podrian indicar un tipo de error o simplemente asegurar que falla
  • El uso de excepciones no es el mejor. Piensen si realmente tiene sentido tener un objeto por cada tipo de error diferente que tiene el sistema. Acá el truco es pensar en "a quién le puede interesar diferencias entre un NoExisteUsuarioConfirmado ó NoExisteUsuarioPotencial?). Si le dan una vuelta más van a ver que el cliente de estas excepciones solo quiere... un mensaje. 
  • En Comentario, para qué existe un método estático que llama al constructor? No se está usando mensaje

La entrega está aprobada.
Comments