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

Projeto guiado - Steffany Peres #36

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

mayumisngk
Copy link
Collaborator

No description provided.

Copy link
Collaborator Author

@mayumisngk mayumisngk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oi Steffany!

Fiz uma revisão de código (code review) para você e espero que você possa revisar seu projeto e colocar posteriormente mais coisas. Vi que seu projeto não está funcional e você implementou as funções pela metade.

Espero que possa revisitar o seu projeto futuramente, por hora, ele está a desejar. As funções que foram implementadas, estão com revisão de código nas respectivas partes.

Boa sorte e sucesso na sua jornada! ♥

Comment on lines +41 to +44
nome_completo = input("\nDigite o nome completo da nova aluna: ")
sobrenome = input("Digite o sobrenome da aluna: ")
turma = input("Digite a turma da aluna: ")

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Não está recebendo as notas, lista de presença e nota de participação.

Comment on lines +52 to +53
for idx, aluna in enumerate(dataset ):
print(f"{idx}. {aluna['nome']} {aluna['sobrenome']} - Turma: {aluna['turma']}")
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Implementar de forma mais simples ajuda na legibilidade do código. Colocar variáveis com nomes intuitivos ajuda no processo:

Sugestão:

Suggested change
for idx, aluna in enumerate(dataset ):
print(f"{idx}. {aluna['nome']} {aluna['sobrenome']} - Turma: {aluna['turma']}")
for nome, sobrenome in dataset.keys():
print(f'- {nome} {sobrenome}')

Comment on lines +62 to +63
qtd_faltas = dataset[(nome, sobrenome)] ["presença"].count(False)
print((f"A quantidade de faltas de {nome} {sobrenome} são {qtd_faltas[0]}, {qtd_faltas[1]}, {qtd_faltas[2]} "))
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Este trecho de código apresenta alguns problemas de lógica e formatação que precisam ser corrigidos para funcionar corretamente.

  • A variável qtd_faltas armazena o número total de faltas, e não uma lista.
  • O uso de qtd_faltas[0], qtd_faltas[1], qtd_faltas[2] está incorreto, pois qtd_faltas é um inteiro representando a quantidade total de faltas, e não uma lista.
  • Há excessos de parenteses na linha 63.
    Sugestão:
Suggested change
qtd_faltas = dataset[(nome, sobrenome)] ["presença"].count(False)
print((f"A quantidade de faltas de {nome} {sobrenome} são {qtd_faltas[0]}, {qtd_faltas[1]}, {qtd_faltas[2]} "))
qtd_faltas = dataset[(nome, sobrenome)] ["presença"].count(False)
print(f"A quantidade de faltas de {nome} {sobrenome} são {qtd_faltas} ")

Comment on lines +70 to +71
valor_notas = dataset[(nome, sobrenome )] ["notas"]
print(f"As três notas de {nome} {sobrenome} é {valor_notas}. ")
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Boa implementação :)

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