Revisiones

Esta página es para poder hacer las anotaciones del control de calidad. La idea es que aquí se publiquen los resultados de las revisiones (fecha de revisión, errores encontrados, etc.) para que puedan ser consultados por las personas interesadas, y para poder tener una referencia para saber qué está hecho y qué queda por hacer.
Se supone que cada uno solamente puede modificar lo que revisa. No vale modificar y poner que tu parte está revisada y correcta. Si queréis que vuestra parte sea revisada, poneos en contacto con quien os tiene que revisar para que lo hagan.

Si una vez revisado tenéis que hacer cambios, no eliminéis los comentarios antiguos: añadid a continuación de los comentarios viejos la nueva fecha y después los comentarios nuevos.

Iteración de Abril

Plan de iteración de Abril

A completar antes del 10 de mayo.

Daniel y Daniel (revisado por Rubén y Beatriz)

Fecha: 20/05/2008
Comentarios:

• Falta pasar a pdf el manual
• Con respecto al cogigo hay muchos fallos con respecto a las reglas establecidas
  • Faltan JavaDocs en InfoModulos y FiltroJar(comentario de clase) de Servidor
  • Faltan JavaDocs en PartidaMentiroso de JuegosServidorMentiroso
  • Faltan JavaDocs en PanelBaseMentiroso y JuegoMentiroso de JuegosClienteMentiroso

Rubén y Beatriz (revisado por Santi y Victor)

Fecha: 12-5-08
Comentarios: Falta JavaDoc en todo.

Santi y Victor (revisado por Ezequiel y Guillermo)

Fecha: 24-05-08
Comentarios: Todo OK. Respecto al código de los proyectos vinculados al Chat.

Ezequiel y Guillermo (revisado por Jonás y Lara)

Fecha: 25/05/2008
Comentarios:

• Pictionary
  • JuegosClientePictionary
    • JuegoPictionary: el atributo _nombreCorto debería ir en mayúsculas; todas las accesoras y mutadoras deben ir al final
    • PanelJuegoPictionary: falta JavaDoc en algunos métodos
    • DConfPictionary: falta JavaDoc en el método ponDescripcionNivel; las accesoras y mutadores deben ir al final
  • JuegosComunPictionary
    • Trazo: en la línea 16 hay declarados dos atributos en la misma línea; los métodos equals y hashcode deberían ir antes de las accesoras
  • JuegosServidorPictionary
    • PartidaPictionary: el atributo _palabras, al ser estático, debe llevar 'st' o si es una constante, ir en mayúsculas

• Hundir la Flota
  • JuegosClienteHundirLaFlota
    • DialogoConfHundirLaFlota: dameConfiguracion es una accesora y debe ir al final
    • JuegoHundirLaFlota: las accesoras deben ir al final
    • PanelHundirLaFlota: hay declarados varios atributos en la misma línea; el nombre del botón del panel (jButton) es poco representativo
  • JuegosComunHundirLaFlota
    • Barco: las accesoras deben ir al final de la definición de la clase
  • JuegosServidorHundirLaFlota
    • ServidorHundirLaFlota: según la nomenclatura que se ha ido utilizando en todos los juegos, esta clase debería llamarse PartidaHundirLaFlota

Jonás y Lara (revisado por Patri y Conrado)

Fecha: 01/06/2008
Comentarios:

• Cliente y Común de la Oca correctos, aunque hay algunos javadocs de los que se crean automáticamente.
• Faltan javadocs en el Servidor de la Oca.
• Cliente, Común y Servidor de las tres en raya correctos.

Patri y Conrado (revisado por Jesús e Isaac)

Fecha: 04/06/2008
Comentarios:
*Variable grupo en JuegoMentiroso.java sin _ delante
*En JuegosComunMentiroso sobra un paquete, llamado IStation.Modulos.Juegos.Compartido.Mentiroso

Jesús e Isaac (revisado por Cristian y Pablo)

Fecha: 20/05/2008
Comentarios:

• Probador Offline
  • Falta bastante Javadoc
  • Hay problemas de alias al pasar los datos en los mensajes entre cliente y servidor

Cristian y Pablo (revisado por Gerardo y Eduardo )

Fecha: 20/05/2008
Comentarios: Casos de uso satisfechos.

Gerardo y Eduardo (revisado por Daniel y Daniel)

Fecha: 12/05/2008
Comentarios:

• No se ha creado la "comisión" de embellecimiento del portal, pero por los comentarios de la gente en las pruebas es posible que sean necesarios pequeños cambios (como por ejemplo que las letras no sean blancas con el fondo del portal que hay actualmente).
• La página de FAQ está hecha, pero tiene carácteres que se ven raro (no se si es por la codificación usada).
• Aún no está claro la distinción de distintos tipos de usuario en el portal.
• El foro está hecho pero no está subido.
• No está hecho un enlace rápido a las partidas de los amigos.

Iteración de Marzo

Plan de iteración de Marzo

A completar antes del 8 de abril.

Daniel y Daniel (revisado por Rubén y Beatriz)

Fecha: 07/04/2008
Comentarios:

• La documentacion esta actualizada
• No se ha creado un frameWork de cartas(Tareas 8.1-8.3)
• No se ha probado el juego(Tarea 10.4)
• No se ha elaborado un manual del juego para el usuario(Tarea 10.5)
• Sigue sin funcionar la salida con enter en el Servidor
• Con respecto al código se han encontrado fallos con respecto a las reglas de control de calidad
  • JuegosClienteMentiroso
    • PanelBase.java
      • Faltan javaDocs(en la declaracion de la clase y algunos parametros)
      • Mala ubicacion de accesores y mutadores.
  • JuegosComunMentiroso
    • DatosPartidaMentiroso.java
      • Faltan javaDocs(en constructoras)
  • JuegosServidorMentiroso
    • PartidaMentiroso.java
      • Faltan javaDocs
  • Servidor
    • Principal.java
      • Faltan javaDocs(no tiene ninguno)
    • InfoModulos.java
      • Faltan javaDocs

Rubén y Beatriz (revisado por Santi y Victor)

Fecha: 21/04/2008
Comentarios: Todo correcto.

Santi y Victor (revisado por Ezequiel y Guillermo)

Fecha: 06/04/2008
Comentarios:

• Módulo de Chat (Servidor)
  • La documentación está actualizada
  • Incumplimiento del estándar de calidad: el proyecto ModuloChat no compila
  • Incumplimiento del estándar de calidad: revisar JavaDoc de ChatDatos y UsuarioChat.
• Portal Web
  • No se ha nombrado un responsable del portal web (o al menos no consta en ningún sitio)
  • No se han llegado a cubrir todos los casos de uso.

Ezequiel y Guillermo (revisado por Jonás y Lara)

Fecha: 21/04/2008
Comentarios: En ModuloJuegos falta el JavaDoc de algunos métodos y no está puesto lo que representan las variables y sus propósito. En InicioModuloJuegos falta JavaDoc. Los demás proyectos están bien.

Jonás y Lara (revisado por Patri y Conrado)

Fecha: 07/04/2008
Comentarios:

• JuegosTableroComun
  • Incumplimiento del estándar de calidad: revisar JavaDoc de InfoFicha.java.
• JuegosClienteOca
  • Incumplimiento del estándar de calidad: revisar JavaDoc de ConfiguraciónOca.java.
  • Incumplimiento del estándar de calidad: revisar nombre de atributos que no empiezan por "_" de ConfiguraciónOca.java.
  • Incumplimiento del estándar de calidad: revisar JavaDoc de JuegoOca.java.
• JuegosComunOca
  • Ok.
• JuegosServidorOca
  • Incumplimiento del estándar de calidad: revisar JavaDoc de PartidaOca.java.

Patri y Conrado (revisado por Jesús e Isaac)

Fecha: 04/06/2008
Comentarios:
*Variable grupo en JuegoMentiroso.java sin _ delante
*En JuegosComunMentiroso sobra un paquete, llamado IStation.Modulos.Juegos.Compartido.Mentiroso

Jesús e Isaac (revisado por Cristian y Pablo)

Fecha: 21/04/2008
Comentarios: Falta Javadoc y que funcione con todos los juegos

Cristian y Pablo (revisado por Gerardo y Eduardo )

Fecha: 22/4/08
Comentarios: Progresa adecuadamente

Gerardo y Eduardo (revisado por Daniel y Daniel)

Fecha: 20/4/08
Comentarios: La base de datos y el portal satisfacen los casos de uso esfecificados en la iteración.
No se ha nombrado "oficialmente" un responsable dentro de quienes desarrollan portal, pero Gerardo actúa como dicha persona dentro del mismo.

Iteración de Enero

Plan de iteración de Enero

Daniel y Daniel (revisado por Rubén y Beatriz)

Fecha: 03/03/2008
Comentarios:

• Las Iteraciones de Dicciembre y Enero estan acabadas y revisadas:
• Con respecto al código se han encontrado fallos con respecto a las reglas de control de calidad
  • Instancia.java
    • La clase no tiene javaDoc(solo la declaracion de la clase)
    • El atributo _loader es estatico, por lo que se deberia llamar _stLoader
    • Mala ubicacion de accesores y mutadores.
  • Main.java
    • El metodo creaInstanciaJars no tiene descripción en el javaDoc
    • Mala ubicacion de accesores.
  • FiltroJar.java
    • Esta correcto

Rubén y Beatriz (revisado por Santi y Victor)

Fecha: 21/02/2008
Comentarios:

• Bien

Santi y Victor (revisado por Ezequiel y Guillermo)

Fecha: 02/03/2008
Comentarios:

• Todo lo que se pedía para la iteración de enero está hecho, salvo lo de concluir el modulo para que funcione en el prototipo. Esto se ha quedado sin hacer, el chat sigue sin estar probado.
• La documentación en la wiki es escasa, e insuficiente para poder saber cómo se usa el módulo.
• En cuanto a lo que sí está implementado:
  • Faltan algunas especificaciones en el JavaDoc en IModuloChat.java y en ICanal.java. Por ejemplo, valores que devuelve en muchas de las funciones. Muchas funciones devuelven enteros que no sabemos qué son, ni cómo se deben interpretar. Habría que especificar el significado de este resultado o cambiar el código para que controle el flujo de errores mediante excepciones.
  • ¿Cuál es el objetivo de la interfaz IChatDatos? Aunque las cosas que implementa sean obvias getters y setters, está bien que se especificaran nombres de los métodos. Sino desde mi punto de vista no tiene sentido definir está inferfaz.
  • En ningún momento debéis hacer uso de la clase ModuloComunicaciones. En su lugar, debéis utilizar la interfaz IModuloComunicaciones, que está en el jar llamado ModuloComunicacionesComun.
  • Para cumplir con las historias de la carga dinámica la idea es que las interfaces que tengan que ser usadas por otros módulos estén en otro proyecto, y generen un JAR aparte. Y ese es el JAR que verán el resto de los módulos, sólo las interfaces.
  • En cuanto a lo de usar String en lugar de int en el método enviarMensaje, ahí ha habido un malentendido bastante grande que habrá que arreglar reuniéndose durante un rato para aclarar todo.

Ezequiel y Guillermo (revisado por Jonás y Lara)

Fecha: 03/03/2008
Comentarios:

• Falta JavaDOC en un par de clases.

Jonás y Lara (revisado por Patri y Conrado)

Fecha: 03/03/2008
Comentarios:

• Revisado: juegoAbstracto.java, CreadorPartidasAbstracto.java, ICliente.java, IComponenteCliente.java, IModuloComunicaciones.java
  • Hay comentarios en 'arameo'.
  • Falta JavaDOC.

Patri y Conrado (revisado por Jesús e Isaac)

Fecha: 07/03/08
Comentarios: revisado y correcto, aunque aún falta algo

Jesús e Isaac (revisado por Cristian y Pablo)

Fecha: 11/03/08
Comentarios:Ya revisado

Cristian y Pablo (revisado por Gerardo y Eduardo )

Fecha: 10/03/08
Comentarios:

• Canal.java: Se crea esta clase que tiene unas pocas funciones, mayormente accedentes y mutadores. No obstante, MODULOCHAT tiene una interfaz llamada ICANAL.JAVA que tiene la mayoría de las funciones implementadas en esta primera clase. La duda es: ¿está hecho así a posta (tal vez para hacer alguna prueba)? ¿Se necesitaba usar y no se quiso/supo incluir la interfaz?
• ChatDatos.java: Exactamente el mismo caso.
• No he encontrado generado el javaDoc.
• Varias funciones no tienen comentario XML.
• La documentación de la wiki me ha parecido algo confusa.

Gerardo y Eduardo (revisado por Daniel y Daniel)

Fecha: 03/03/08
Comentarios:

• No se han añadido interfaces que indiquen cómo se pasa de una otra en el portal.
• Como no hemos podido ver las nuevas interfaces del portal en ejecución, y no están subidas en la sección correspondiente de la wiki, no podemos comprobar que está bien hecho.
• No se ha subido ninguna información acerca del diseño del portal a la wiki, así que no lo podemos evaluar.

Subid todo lo que tengais hecho y podremos empezar, porque no hay nada.