Skip to content

[René Navarro] DAPT#65

Open
Reneferna wants to merge 1 commit intota-data-mexpt:masterfrom
Reneferna:master
Open

[René Navarro] DAPT#65
Reneferna wants to merge 1 commit intota-data-mexpt:masterfrom
Reneferna:master

Conversation

@Reneferna
Copy link
Copy Markdown

No description provided.

@ta-data-mexpt
Copy link
Copy Markdown
Owner

Hola René, buen trabajo!

Acá te dejo un par de comentarios:

  • Me gustó la lógica que seguiste en la lista del super aunque igual podríamos hacerlo un poco más directo, algo como, `'Grocery list: '+ ', '.join([food.lower() for food in food_list if food.lower().startswith('b')==True]) + '.'
  • Al calcular el área del círculo definiste el área como 2 * (pi * radio^2), lo veremos más adelante pero la idea era que definieras el cálculo dentro de la función area (la que se define con def) y para eso es importante respetar la identación.
  • El conteo del poema está muy bien implementado, solo falló un poco la limpieza por ejemplo convertir en minúscula, eliminar puntuación, etc.
  • Felicidades por el bonus ;D

Por cierto, también se debía entregar el Challenge 2 jeje

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