Comentarios entrega #1 about algochess HOT 3 CLOSED

Buenas, como quedó el estado de los dos items que faltan? Además la tarea del README está incompleta, falta el enlace al informe y a los diagramas.

from algochess.

@edjus respecto a lo de atacar, lo estabamos haciendo de esa manera pero no nos gusta como esta quedando. Ahora estabamos por pasar a hacerlo por composicion en vez de crear clases abstractas que hereden, porque sino de todos modos queda repetido el codigo de atacar en atacante y atacanteMovible.

respecto al de moverUnidad de tablero, la forma en la que ya estaba hecha era que uno llama al mover de unidad unidad.mover(posicionDestino. tablero) quien luego usa tablero para llamar a moverUnidad con la posicion destino y la posicion propia de la unidad. No es eso a lo que te referias con esto?:

Otra manera de hacerlo sería directamente mandar el mensaje a unidad:unaUnidad.mover(posicionDestino, tablero), sería una variante del anterior.

from algochess.

• Me parece bien el enfoque que van a tomar.
• No llego a entender a lo que te refieres, pero la idea es que no se haga algo como: tablero.mover(posiciónOrigen, posiciónDestino)
  Si no se use a la unidad, que ya contiene su posición (posiciónOrigen). Si no se entiende del todo lo vemos mañana en la segunda entrega.

from algochess.