Skip to content

[Lab-data-vikings] Eli M-Montalvo#117

Open
E2811 wants to merge 6 commits intoironhack-datalabs:masterfrom
E2811:lab-data-vikings
Open

[Lab-data-vikings] Eli M-Montalvo#117
E2811 wants to merge 6 commits intoironhack-datalabs:masterfrom
E2811:lab-data-vikings

Conversation

@E2811
Copy link
Copy Markdown

@E2811 E2811 commented Jan 18, 2020

Me ha gustado mucho! 🕺 🚀
meme1510668146gen

@Gori89
Copy link
Copy Markdown

Gori89 commented Jan 20, 2020

Muy buen trabajo, y la guerra esta muy bien.
El código esta bien, solo un par de comentarios de buenas braxis.

  • Al definir el constructor de Saxons, como no cambia nada con respecto al de Soldier, no hacía falta que lo definieras.
class Saxon(Soldier):
    def __init__(self,health,strength):
        super().__init__(health,strength)
  • Cuando des nombres a las variable, intenta que sea autoexplicativo. A la vuelta de una semana serña complicado que te acuerdes de que contiene cada variable sin tener que mirar el código.
saxon_atacado = random.choice(self.saxonArmy)
saxon_atacado2 = saxon_atacado.receiveDamage(strength_viking)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants