Division de tares 2 ELIJAN

ldtg commented 7 years ago

Paso en limpio la lista si hay mas agreguen o comente. La idea es que elijamos cada uno lo que tiene ganas de hacer.

[x] Hay que simplificar los tipos
[x] Y ponerlos en un paquete aparte
[ ] Más documentación y formato
[ ] Completar el diagrama de clase
[x] Las clases del juego
[x] Escribir las nuevas pruebas
[x] Decidirse por una interfaz gráfica
[x] Y empezar a codearla
[x] Ver como vamos a hacer un logger
[x] Y obvio #3 y #4
[x] Si llegamos, un modo contra la PC
[x] Hacer andar el vampiro
[x] Programar clase efectos y agregarla

ldtg commented 7 years ago

Yo elijo las dos ultimas, creo que tengo una idea como seguirlo.

keze87 commented 7 years ago

Dejenme el formato a mi, please. Pd2: No me dejen la documentación a mi.

wjma commented 7 years ago

la documentacion iria mas que nada en los paquetes efectos y estado creo

guido-z commented 7 years ago

¿Qué significa simplificar los tipos? Puedo ocuparme de eso, de documentar, refactorizar y mover los tipos a otro paquete.

2016-11-08 21:26 GMT-03:00 wjma notifications@github.com:

la documentacion iria mas que nada en los paquetes efectos y estado creo

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/keze87/TP2/issues/20#issuecomment-259302586, or mute the thread https://github.com/notifications/unsubscribe-auth/ASdG42Kg5fJY286b5AaWiIaBT8ekj55Dks5q8RMpgaJpZM4KtEQa .

keze87 commented 7 years ago

Para mi (para mi) no tiene mucho sentido esto:

package src.fiuba.algo3.modelo;

import java.util.List;

import src.fiuba.algo3.modelo.ataques.Ataque;

public class AlgoMonFuego extends AlgoMon {

    public AlgoMonFuego(String nombre, int vidaMaxima, List<Ataque> ataques) {
        super(nombre, vidaMaxima, ataques);
        this.tipo = new Fuego();
    }

}

Cuando podría ser esto:

public class AlgoMon{

    public AlgoMon(String nombre, int vidaMaxima, List<Ataque> ataques, Tipo tipo) {

        this.tipo = tipo;

        //Todo lo demás

Si quieren lo cambio, si no va, lo revierto.

guido-z commented 7 years ago

Si bien te ahorrás algunas clases, ahí surge el problema de tener que darle el tipo explícitamente, perdiendo encapsulamiento. No está bueno tener que saber cómo se instancia un algoMon de tipo fuego, por ejemplo, respecto de uno normal o de otro tipo.

2016-11-09 15:23 GMT-03:00 keze87 notifications@github.com:

Para mi (para mi) no tiene mucho sentido esto:

package src.fiuba.algo3.modelo; import java.util.List; import src.fiuba.algo3.modelo.ataques.Ataque; public class AlgoMonFuego extends AlgoMon {
public AlgoMonFuego(String nombre, int vidaMaxima, List<Ataque> ataques) {
    super(nombre, vidaMaxima, ataques);
    this.tipo = new Fuego();
}
}

Cuando podría ser esto:

public class AlgoMon{
public AlgoMon(String nombre, int vidaMaxima, List<Ataque> ataques, Tipo tipo) {

    this.tipo = tipo;

    //Todo lo demás
Si quieren lo cambio, si no va, lo revierto.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/keze87/TP2/issues/20#issuecomment-259486498, or mute the thread https://github.com/notifications/unsubscribe-auth/ASdG41wup3DnNr4J1HYr30PLZTMBk6xZks5q8g-1gaJpZM4KtEQa .

guido-z commented 7 years ago

PD: A la noche me pongo a codear, hoy estuve un poco ocupado.

El 9 de noviembre de 2016, 16:15, Guido Zaglulguido.zaglul@gmail.com escribió:

Si bien te ahorrás algunas clases, ahí surge el problema de tener que darle el tipo explícitamente, perdiendo encapsulamiento. No está bueno tener que saber cómo se instancia un algoMon de tipo fuego, por ejemplo, respecto de uno normal o de otro tipo.

2016-11-09 15:23 GMT-03:00 keze87 notifications@github.com:
Para mi (para mi) no tiene mucho sentido esto:

package src.fiuba.algo3.modelo; import java.util.List; import src.fiuba.algo3.modelo.ataques.Ataque; public class AlgoMonFuego extends AlgoMon {
public AlgoMonFuego(String nombre, int vidaMaxima, List<Ataque> ataques) {
    super(nombre, vidaMaxima, ataques);
    this.tipo = new Fuego();
}
}

Cuando podría ser esto:

public class AlgoMon{
public AlgoMon(String nombre, int vidaMaxima, List<Ataque> ataques, Tipo tipo) {

    this.tipo = tipo;

    //Todo lo demás
Si quieren lo cambio, si no va, lo revierto.

— You are receiving this because you commented. Reply to this email directly, view it on GitHub https://github.com/keze87/TP2/issues/20#issuecomment-259486498, or mute the thread https://github.com/notifications/unsubscribe-auth/ASdG41wup3DnNr4J1HYr30PLZTMBk6xZks5q8g-1gaJpZM4KtEQa .

keze87 / TP2

Division de tares 2 ELIJAN #20