Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactorización del código #5

Open
RafaelAybar opened this issue Sep 27, 2020 · 4 comments · Fixed by #12
Open

Refactorización del código #5

RafaelAybar opened this issue Sep 27, 2020 · 4 comments · Fixed by #12
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers Hacktoberfest help wanted Extra attention is needed
Projects

Comments

@RafaelAybar
Copy link
Owner

Actualmente no se valida la entrada más allá de comprobar espacios en blanco, no se comprueban palabras reservadas como None, etc..

Además, tampoco se utilizan métodos o elementos de Python 3 que permitan tener un código legible

@RafaelAybar RafaelAybar added enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed Hacktoberfest labels Sep 27, 2020
@cmaureir
Copy link

El manejo de los parámetros del script se podría realizar con el módulo argparse (documentación en español)

Respecto a la estructura del código, quizás se podrían mirar otros proyectos o tutoriales al respecto. Personalmente me gustan los consejos de https://docs.python-guide.org/writing/structure/

@vmjelicic
Copy link
Contributor

vmjelicic commented Oct 6, 2020

Buenas, me gustaría colaborar con esta issue. De momento puedo revisar el punto de generar un código más legible.

@RafaelAybar
Copy link
Owner Author

Buenas, me gustaría poder colaborar con esta issue

Encantado de tener tu feedback, si puedes, edita tu comentario con lo que te gustaría hacer, para no tener esfuerzo duplicado.

¡Gracias!

@MaikaRG
Copy link
Contributor

MaikaRG commented Oct 16, 2020

Hay que modificar el código porque los elementos de la lista empiezan en '0'
Habría que cambiar la funcion 'calculamedia' en base a este parámetro.

@RafaelAybar RafaelAybar reopened this Oct 17, 2020
@RafaelAybar RafaelAybar added this to To do in QA-Empleo Sep 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers Hacktoberfest help wanted Extra attention is needed
Projects
Development

Successfully merging a pull request may close this issue.

4 participants