Skip to content

[Lab-Data-Vikings] Alejandro Vidal#120

Open
Alexvidalcor wants to merge 5 commits intoironhack-datalabs:masterfrom
Alexvidalcor:lab-vikings
Open

[Lab-Data-Vikings] Alejandro Vidal#120
Alexvidalcor wants to merge 5 commits intoironhack-datalabs:masterfrom
Alexvidalcor:lab-vikings

Conversation

@Alexvidalcor
Copy link
Copy Markdown

No description provided.

@Alexvidalcor Alexvidalcor changed the title Ejercicio Terminado menos Bonus [Lab-Data-Vikings] Alejandro Vidal Jan 18, 2020
@Gori89
Copy link
Copy Markdown

Gori89 commented Jan 20, 2020

Buen trabajo , todo perfecto.
Solo dos comentarios de buenas prácticas:

  • El método attack no varía de la clase soldado en la clase Vikigos ni saxons. Por lo tanto, no es necesario que lo vuelvas a definir. Estas clases lo heredan.

  • Al poner la batalla en el mismo fichero y llamar al método battle(), simpre que se importa el fichero se ejecuta una batalla. Habría sido mejor ponerlo en un fichero a parte.

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