Semana 1 #2

agus-germi · 2023-11-21T23:11:50Z

Primer PR. Faltan Test09 y Test12.

Co-authored-by: SebaKrag <skraglievich@fi.uba.ar> Co-authored-by: mariagalindez <mgalindez@fi.uba.ar> Co-authored-by: SairBaretto <gbaretto@fi.uba.ar>

Co-authored-by: SebaKrag skraglievich@fi.uba.ar

Co-authored-by: SebaKrag <skraglievich@fi.uba.ar>

Co-authored-by: Sebakrag <skraglievich@fi.uba.ar> Co-authored-by: mariagalindez <mgalindez@fi.uba.ar>

…tests. Co-authored-by: Sebakrag <skraglievich@fi.uba.ar> Co-authored-by: mariagalindez <mgalindez@fi.uba.ar> Co-authored-by: fnpratto <fnpratto@gmail.com> Co-authored-by: SairBarreto <gbarreto@fi.uba.ar>

Co-authored-by: Sebakrag <sebaskrag@gmail.com> Co-authored-by: mariagalindez <mgalindez@fi.uba.ar> Co-authored-by: fnpratto <fnpratto@gmail.com> Co-authored-by: SairBarreto <gbarreto@fi.uba.ar>

Co-authored-by: agus-germi <agerminario@fi.uba.ar> Co-authored-by: mariagalindez <mgalindez@fi.uba.ar>

Co-authored-by: agus-germi <agerminario@fi.uba.ar> Co-authored-by: mariagalindez <mgalindez@fi.uba.ar> Co-authored-by: SairBarreto <gbarreto@fi.uba.ar> Co-authored-by: fnpratto <fnpratto@gmail.com>

Co-authored-by: mariagalindez <mgalindez@fi.uba.ar> Co-authored-by: SairBarreto <gbarreto@fi.uba.ar> Co-authored-by: Sebakrag <sebaskrag@gmail.com>

Co-authored-by: mariagalindez <mgalindez@fi.uba.ar> Co-authored-by: SairBarreto <gbarreto@fi.uba.ar>

Co-authored-by: mariagalindez <mgalindez@fi.uba.ar> Co-authored-by: SairBarreto <gbarreto@fi.uba.ar> Co-authored-by: agus-germi <agerminario@fi.uba.ar> Co-authored-by: fnpratto <fnpratto@gmail.ar>

…Falta implementar test 09 y 12. Co-authored-by: mariagalindez <mgalindez@fi.uba.ar> Co-authored-by: SairBarreto <gbarreto@fi.uba.ar> Co-authored-by: agus-germi <agerminario@fi.uba.ar> Co-authored-by: fnpratto <fnpratto@gmail.ar>

…a clase Jugador.

Co-authored-by: mariagalindez <mgalindez@fi.uba.ar>

Co-authored-by: mariagalindez <mgalindez@fi.uba.ar> Co-authored-by: fnpratto <fnpratto@gmail.com> Co-authored-by: Sebakrag <sebaskrag@gmail.com>

codecov-commenter · 2023-11-21T23:13:14Z

Welcome to Codecov 🎉

Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.

Thanks for integrating Codecov - We've got you covered ☂️

Co-authored-by: Sebakrag <sebaskrag@gmail.com> Co-authored-by: agus-germi <agerminario@fi.uba.ar> Co-authored-by: SairBarreto <gbarreto@fi.uba.ar>

Co-authored-by: agus-germi <agerminario@fi.uba.ar> Co-authored-by: Sebakrag <sebaskrag@gmail.com> Co-authored-by: SairBarreto <gbarreto@fi.uba.ar>

Co-authored-by: mariagalindez <mgalindez@fi.uba.ar> Co-authored-by: fnpratto <fnpratto@gmail.com> Co-authored-by: Sebakrag <sebaskrag@gmail.com>

Co-authored-by: mariagalindez <mgalindez@fi.uba.ar> Co-authored-by: Sebakrag <sebaskrag@gmail.com>

alejovillores · 2023-11-23T02:29:21Z

src/main/java/edu/fiuba/algo3/modelo/Gladiador.java

+    public void recibirAtaque(){
+        this.disminuirEnergia(this.equipamiento.recibirAtaque()); //ROMPE ENCAPSULAMIENTO
+    }


Se puede evitar el getter de la siguiente forma

public void recibirAtaque(){ this.energia = this.equipamiento.recibirAtaque(this.energia)); }

Genial Ale! Ese código pertenece a Gladiador verdad?

Obvia la pregunta, acabo de ver el path de arriba jajaja

alejovillores · 2023-11-23T02:31:52Z

src/main/java/edu/fiuba/algo3/modelo/Gladiador.java

+        this.disminuirEnergia(this.equipamiento.recibirAtaque()); //ROMPE ENCAPSULAMIENTO
+    }
+
+    public void recibirDanio(int danio) { this.disminuirEnergia(danio); }


Que diferencia hay entre recibirDanio y recibirAtaque si al recibir el ataque se produce un daño?

Tenes toda la razón, es un error que tenemos visto y creemos que encontramos una posible solución. En cuanto comencemos con el refactor va a ser lo primero en arreglar. Gracias por la observación

De hecho, desarrollamos la idea de la solución en un comentario que está al final del archivo de la clase Jugador.java.
Si llegas a leerlo y nos queres anticipar si es un buen planteo y puede llegar a solucionar lo que nos corregiste, nos ayudaría a ahorrar mucho tiempo 🙌🏽

alejovillores

El decorador le falta una vuelta de rosca incluso por como esta el modelo se puede cambiar por un Strategy .

En la reunion vamos a ver en profundidad los comentarios y como arreglarlos

alejovillores · 2023-11-23T02:32:57Z

src/main/java/edu/fiuba/algo3/modelo/Gladiador.java

+    private void disminuirEnergia(int energia) {
+        this.energia -= energia;
+    }
+
+    private void aumentarEnergia(int energia) {
+        this.energia += energia;
+    }


No les veo mucha utilidad la verdad

alejovillores · 2023-11-23T02:34:17Z

src/main/java/edu/fiuba/algo3/modelo/Jugador.java

+public class Jugador {
+    private Gladiador gladiador;
+    private int turnos;
+    private int casillaActual;


Esto debe pasar a ser private Casilla casillaActual;

alejovillores · 2023-11-23T02:35:11Z

src/main/java/edu/fiuba/algo3/modelo/Jugador.java

+    public Jugador(Gladiador gladiador) {
+        this.gladiador = gladiador;
+        this.turnos = 0;
+        this.casillaActual = 0;


public Jugador(Gladiador gladiador, Casilla casillaInicial) { this.gladiador = gladiador; this.turnos = 0; this.casillaActual =casillaInicial;

En el código más reciente introdujimos este cambio. De alguna manera el código sobre el que hiciste esta corrección quedó es viejo. Me pregunto si habremos hecho mal el push al PR 🤔

Para que se haga un push al PR se tiene que hacer un push en la rama semana-1

Perfecto! Mañana a la mañana lo hacemos entonces. Debemos haberlo hecho a la rama Decorator y no nos dimos cuenta 🫣

alejovillores · 2023-11-23T02:39:28Z

src/main/java/edu/fiuba/algo3/modelo/Jugador.java

+    private int tirarDado() {
+        final int CARAS_DADO = 6;
+
+        if (this.gladiador.tieneEnergia()) {
+            Random random = new Random();
+            return random.nextInt(CARAS_DADO) + 1;
+        }
+        return 0;


Esto tiene que estar encapsulado en una entidad que simule el dado, de manera que se pueda mockear y no depender de una clase que puede violar OC como Random

alejovillores · 2023-11-23T02:41:31Z

src/main/java/edu/fiuba/algo3/modelo/Jugador.java

+    private void avanzar(int cantidad) {
+        this.casillaActual += cantidad;
+    }


La nueva casilla deberia ser obtenida a partir de que la casilla actual conoce a sus vecinos

alejovillores · 2023-11-23T02:43:05Z

src/main/java/edu/fiuba/algo3/modelo/afectantes/Bacanal.java

+    private static final int CANTIDAD_COPAS_TOTAL = 6;
+    private static final int ENERGIA_POR_COPA = 4;
+
+    private Random random = new Random();


encapsularlo en un objeto Dado o ProvedorRandom, x

alejovillores · 2023-11-23T02:50:12Z

src/main/java/edu/fiuba/algo3/modelo/afectantes/MejorarEquipamiento.java

+    public void afectar(Jugador jugador) {
+        jugador.mejorarEquipamiento();
+    }


Esta clase debería tener mas responsabilidad, porque sino todos los afectantes simplemente delegan o hacen poquitas cosas

public void afectar(Jugador jugador) { jugador.mejorarEquipamiento(this); } public Equipamiento mejorarEquipamiento(Casco casco) //... asi para los equipamientos//

alejovillores · 2023-11-23T02:50:44Z

src/main/java/edu/fiuba/algo3/modelo/afectantes/Lesion.java

+public class Lesion {
+    //el turno siguiente no avanza
+}


Buscar manera de que haga algo

alejovillores · 2023-11-23T02:52:21Z

src/main/java/edu/fiuba/algo3/modelo/seniorities/SemiSenior.java

+    public int aumentarEnergia(){
+        return AUMENTO_ENERGIA;
+    }


public int aumentarEnergia(int energiaActual){ return energiaActual + AUMENTO_ENERGIA; }

alejovillores · 2023-11-23T02:53:02Z

src/main/java/edu/fiuba/algo3/modelo/seniorities/Senior.java

+    public Seniority ascender(int turno) {
+        return this;
+    }


checkear los turnos...

alejovillores · 2023-11-23T03:05:08Z

src/main/java/edu/fiuba/algo3/modelo/Jugador.java

+// public void recibirImpacto(Afectante afectante) {
+//      this.gladiador.recibirImpacto(afectante);
+//}
+// Con este metodo reemplazariamos los metodos de:
+//                      mejorarEquipamiento()
+//                      recibirAtaque()
+//                      recibirDanio()
+//                      recibirEnergia()


Me parece mejor.

alejovillores · 2023-11-23T03:06:16Z

src/main/java/edu/fiuba/algo3/modelo/Jugador.java

+// public void recibirImpacto(Lesion lesion) {
+//      // PROBLEMA con la propuesta de cambio:
+//      // Aqui el Jugador en el siguiente turno no avanza.
+//      // Esta logica quizas es conveniente que este en Jugador por el hecho de que un
+//      // jugador tiene la cantidad de turnos.
+// }


Habria que buscar la manera de impedir que el jugador avance

Co-authored-by: mariagalindez <mgalindez@fi.uba.ar> Co-authored-by: SairBarreto <gbarreto@fi.uba.ar> Co-authored-by: agus-germi <agerminario@fi.uba.ar> Co-authored-by: fnpratto <fnpratto@gmail.com>

Co-authored-by: mariagalindez <mgalindez@fi.uba.ar> Co-authored-by: fnpratto <fnpratto@gmail.com> Co-authored-by: Sebakrag <sebaskrag@gmail.com> Co-authored-by: SairBarreto <gbarreto@fi.uba.ar>

agus-germi and others added 23 commits November 16, 2023 16:59

Creamos clase jugador

8a1b961

Creamos clase jugador

9f6091d

probando

e9b42ad

Probando co-author.

3c8ecdd

Co-authored-by: SebaKrag <skraglievich@fi.uba.ar> Co-authored-by: mariagalindez <mgalindez@fi.uba.ar> Co-authored-by: SairBaretto <gbaretto@fi.uba.ar>

Probando co-author, parte2.

9c3a56f

Co-authored-by: SebaKrag skraglievich@fi.uba.ar

Probando co-author. again

4ff59ab

Co-authored-by: SebaKrag <skraglievich@fi.uba.ar>

Probando co-author.Again again.

c153daa

Co-authored-by: Sebakrag <skraglievich@fi.uba.ar> Co-authored-by: mariagalindez <mgalindez@fi.uba.ar>

Añadimos Seniority/Equipamiento/Jugador/Gladiador.Nombramos Primeros …

ec89e74

…tests. Co-authored-by: Sebakrag <skraglievich@fi.uba.ar> Co-authored-by: mariagalindez <mgalindez@fi.uba.ar> Co-authored-by: fnpratto <fnpratto@gmail.com> Co-authored-by: SairBarreto <gbarreto@fi.uba.ar>

Añadimos clases concretas de Equipamiento.

b2ca6a0

Co-authored-by: Sebakrag <sebaskrag@gmail.com> Co-authored-by: mariagalindez <mgalindez@fi.uba.ar> Co-authored-by: fnpratto <fnpratto@gmail.com> Co-authored-by: SairBarreto <gbarreto@fi.uba.ar>

Agregamos atributo casilla a Jugador.

76cd139

Co-authored-by: agus-germi <agerminario@fi.uba.ar> Co-authored-by: mariagalindez <mgalindez@fi.uba.ar>

Primer caso de uso (a medias).

b243489

Co-authored-by: agus-germi <agerminario@fi.uba.ar> Co-authored-by: mariagalindez <mgalindez@fi.uba.ar> Co-authored-by: SairBarreto <gbarreto@fi.uba.ar> Co-authored-by: fnpratto <fnpratto@gmail.com>

feat: Caso de uso uno y dos terminados.

0c79ef4

Co-authored-by: mariagalindez <mgalindez@fi.uba.ar> Co-authored-by: SairBarreto <gbarreto@fi.uba.ar> Co-authored-by: Sebakrag <sebaskrag@gmail.com>

feat: Agregamos clase Fiera e interfaz Afectante. Caso de uso 3.

8bbb71f

Co-authored-by: mariagalindez <mgalindez@fi.uba.ar> Co-authored-by: SairBarreto <gbarreto@fi.uba.ar>

Agregamos test 04.

dee620a

Co-authored-by: mariagalindez <mgalindez@fi.uba.ar> Co-authored-by: SairBarreto <gbarreto@fi.uba.ar>

Agregamos una constante.

f3f2724

Agregamos test 06, 07 y 08.

9a7b7be

Co-authored-by: mariagalindez <mgalindez@fi.uba.ar> Co-authored-by: SairBarreto <gbarreto@fi.uba.ar> Co-authored-by: agus-germi <agerminario@fi.uba.ar> Co-authored-by: fnpratto <fnpratto@gmail.ar>

Ordeno el codigo. Agrego un comentario con propuesta de refactor en l…

0de2b3c

…a clase Jugador.

feat: Cambiamos atributos de publicos a privados. Modificamos test 01.

e9bfb47

Co-authored-by: mariagalindez <mgalindez@fi.uba.ar>

feat: Agregamos tests Novato. (a chequear)

20d1e77

Co-authored-by: mariagalindez <mgalindez@fi.uba.ar>

feat: Agregamos tests Seniorities.

477e95f

Co-authored-by: mariagalindez <mgalindez@fi.uba.ar>

feat: Actualizamos.

1cc3132

Co-authored-by: mariagalindez <mgalindez@fi.uba.ar> Co-authored-by: fnpratto <fnpratto@gmail.com> Co-authored-by: Sebakrag <sebaskrag@gmail.com>

Merge branch 'Decorator' into semana-1

dcc6322

alejovillores self-requested a review November 21, 2023 23:13

alejovillores assigned fnpratto, mariagalindez, agus-germi, Sebakrag and SairBarreto Nov 21, 2023

alejovillores added the 📦entrega Etiqueta entrega semanal label Nov 21, 2023

mariagalindez and others added 4 commits November 22, 2023 14:12

feat: Agregamos test 09 con la logica de celdas y tablero

e3a3714

Co-authored-by: Sebakrag <sebaskrag@gmail.com> Co-authored-by: agus-germi <agerminario@fi.uba.ar> Co-authored-by: SairBarreto <gbarreto@fi.uba.ar>

fix: corrección de bug.

a0f75ea

Co-authored-by: agus-germi <agerminario@fi.uba.ar> Co-authored-by: Sebakrag <sebaskrag@gmail.com> Co-authored-by: SairBarreto <gbarreto@fi.uba.ar>

feat: agregamos Test12. A chequear.

d74e856

Co-authored-by: mariagalindez <mgalindez@fi.uba.ar> Co-authored-by: fnpratto <fnpratto@gmail.com> Co-authored-by: Sebakrag <sebaskrag@gmail.com>

feat: primeras implementaciones de Juego.

ed2c888

Co-authored-by: mariagalindez <mgalindez@fi.uba.ar> Co-authored-by: Sebakrag <sebaskrag@gmail.com>

alejovillores reviewed Nov 23, 2023

View reviewed changes

fnpratto and others added 4 commits November 23, 2023 09:05

Merge branch 'Decorator' into semana-1

0a73367

Comenzamos Primer Refactor.

770b583

Co-authored-by: mariagalindez <mgalindez@fi.uba.ar> Co-authored-by: SairBarreto <gbarreto@fi.uba.ar> Co-authored-by: agus-germi <agerminario@fi.uba.ar> Co-authored-by: fnpratto <fnpratto@gmail.com>

GRAN refactor

6090b2b

Co-authored-by: mariagalindez <mgalindez@fi.uba.ar> Co-authored-by: fnpratto <fnpratto@gmail.com> Co-authored-by: Sebakrag <sebaskrag@gmail.com> Co-authored-by: SairBarreto <gbarreto@fi.uba.ar>

fix : pruebas

e7e4161

Co-authored-by: mariagalindez <mgalindez@fi.uba.ar> Co-authored-by: fnpratto <fnpratto@gmail.com> Co-authored-by: Sebakrag <sebaskrag@gmail.com> Co-authored-by: SairBarreto <gbarreto@fi.uba.ar>

alejovillores added the ✅aprobado Semana aprobada label Nov 23, 2023

alejovillores merged commit 4baaf3d into master Nov 23, 2023
8 checks passed

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Semana 1 #2

Semana 1 #2

agus-germi commented Nov 21, 2023

codecov-commenter commented Nov 21, 2023

alejovillores Nov 23, 2023

Sebakrag Nov 23, 2023

Sebakrag Nov 23, 2023

alejovillores Nov 23, 2023

Sebakrag Nov 23, 2023

Sebakrag Nov 23, 2023

alejovillores left a comment

alejovillores Nov 23, 2023

alejovillores Nov 23, 2023

alejovillores Nov 23, 2023

Sebakrag Nov 23, 2023 •

edited

Loading

alejovillores Nov 23, 2023

Sebakrag Nov 23, 2023

alejovillores Nov 23, 2023

alejovillores Nov 23, 2023

alejovillores Nov 23, 2023

alejovillores Nov 23, 2023

alejovillores Nov 23, 2023

alejovillores Nov 23, 2023

alejovillores Nov 23, 2023

alejovillores Nov 23, 2023

alejovillores Nov 23, 2023

Semana 1 #2

Semana 1 #2

Conversation

agus-germi commented Nov 21, 2023

codecov-commenter commented Nov 21, 2023

Welcome to Codecov 🎉

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

alejovillores left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Sebakrag Nov 23, 2023 • edited Loading

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Sebakrag Nov 23, 2023 •

edited

Loading