TP1-G2

Resultado: Aprobado con obs.

 

La entrega está muy bien, les pasamos algunas observaciones y después les contamos como sigue esto.

 

1.      Todo tiene Javadoc eso hace que los ayudantes seamos muy muy felices!

2.      Por otro lado... falta documentación. Ármense un documento con algún diagrama y la explicación de porque tomaron ciertas decisiones. Por ejemplo, no veo nada con respecto al administrador, donde entiendo porque esta así?
También nos gustaría ver un diagrama de clases de la aplicación que vayan actualizando a medida que tocan las cosas.

 

3.      No suban los .classpath y .project al repositorio. La idea es que el código sea agnóstico de la ide, para eso también está maven.

4.            public void cancelarSubscripcion(Usuario usuario) {
                  if (fecha.before(Calendar.getInstance())) return;
Cuidado con esto, es muy peligroso que por algún motivo un método no haga lo que tiene que hacer y no de visibilidad hacia afuera. En este caso yo les diría que si no se puede hacer lo que hay que hacer, lancen una excepción.

5.      Subscripcion, Evento, Usuario no respetan el contrato de equals y hashcode. Acá explica un poco http://ignorante.wordpress.com/2007/10/11/calculo-correcto-del-hashcode-en-java/
El eclipse (otras ides también) tiene la posibilidad de generar estos 2 métodos de forma automática Source -> Generate hashCode and equals.
Si tienen dudas sobre porque esto está mal y como solucionarlo, pregunten que les explicamos con más detalle.

6.      Muy bien los Test, a mejorar, traten de no poner todo en una sola clase que se llame AppTest, traten de dividir por algún criterio (funcionalidad, entidades, etc) y armar varias clases.
Otra cosa buena de los tests es que con mvn package corren todos y funcionan (aunque los tire en orden aleatorio). Esto quiere decir que los tests son independientes entre sí =)

7.      Muy fino, el log queda más prolijo en Evento.addComentario que en el constructor de Comentario

La entrega está aprobada, pero para la próxima entrega queremos que:

a)      Hagan algún tipo de documentación

b)      Arreglen los métodos que “no funcionan” pero no tiran excepciones.

c)      Arreglen los hashcodes y equals.

 

Saludos a todos, cualquier consulta o crítica constructiva la pueden hacer por mail.

Comments