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

Atividade de casa_Semana 05 #18

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

Atividade de casa_Semana 05 #18

wants to merge 6 commits into from

Conversation

Lupaes
Copy link

@Lupaes Lupaes commented Jul 4, 2024

Como?

  • Adicionei pastas no Main : Readme.py e sobre cada etapa do código.
  • Na pasta Readme.py descrevo passo a passo a minha lógica para o desenvolvimento do código.
  • Nas pastas de cada etapa do Código do Colab se encontra também link do código

• O código foi todo desenvolvido pela plataforma do Colab.

#Comentários finais 💪🏽


• Achei bem desafiador esse exercício.🔥

• O resultado foi muito legal, foi muito emocionante ver o código.
• Ansiosa para o próximo projeto.

Copy link
Collaborator

@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 Luana!

Fiz uma revisão de código (code review) para você e espero que você possa revisar seu projeto e colocar posteriormente mais coisas. Futuramente, se quiser complementar mais o sistema, você pode pensar em encapsular as funcionalidades, dando mais especifidade a função e dando responsabilidades únicas para ela.

Vi que você usou 5 arquivos diferentes para fazer o projeto, mas era necessário apenas 1 arquivo do projeto (já disponibilizado) + 1 arquivo do dataset (já disponibilizado) para fazer essa tarefa. Você pode fazer o commit de um único arquivo diversas vezes conforme você o altera, não precisa estar em arquivos diferentes.

Boa sorte e sucesso na sua jornada! ♥
Parabéns pelo trabalho!

return codigo_opcao

def incluir_nova_aluna():
pass
Copy link
Collaborator

Choose a reason for hiding this comment

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

Em Python, a instrução pass é usada como um placeholder. Ela é útil em situações onde o código precisa de uma declaração sintaticamente, mas você ainda não quer (ou não precisa) escrever o código funcional. Neste caso, ao implementar a função, poderia ter removido.

Sugestão:

Suggested change
pass

lista_presenca = eval(input("Presença da aluna(entre colchetes, ex: [True, False, False]): "))
nota_participacao = float(input("Participação da aluna: "))
try:
notas = obter_notas()
Copy link
Collaborator

Choose a reason for hiding this comment

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

A função obter_notas não está nesse arquivo, e nem foi importado. O projeto deve ser feito de forma que funcione como um todo, não é uma tarefa de funcionalidades. Neste caso, o programa quebra ao tentar chamar esta função.

except ValueError:
print("Entrada inválida. Por favor, insira um número válido.")

#salvar_dados_aluna(nome, turma, sobrenome, notas, lista_presenca, nota_participacao)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Como esta função está comentada, então não está salvando os dados no dicionário

Comment on lines +59 to +60
for nome in dataset:
print(f"Nome: {nome}")
Copy link
Collaborator

Choose a reason for hiding this comment

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

Para que tenhamos os nomes formatados e não imprimir com os parenteses, necessitamos trabalhar com a tupla:

Suggested change
for nome in dataset:
print(f"Nome: {nome}")
for nome, sobrenome in dataset.keys():
print(f'Nome: {nome} {sobrenome}')

Comment on lines +64 to +71
print("Inserir os seguintes dados: ")
nome = input("Nome da aluna: ")
sobrenome = input("Sobrenome da aluna: ")
qtd_faltas = dataset[(nome, sobrenome)]["Presença"].count(False)
if qtd_faltas > 2:
print ("Reprovada")
else:
print ("Aprovada")
Copy link
Collaborator

Choose a reason for hiding this comment

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

Nesta função, tínhamos o seguinte requisito funcional:
Consultar a quantidade de faltas de uma aluna

  • O sistema deve permitir ao usuário buscar as faltas de uma aluna específica.
  • O usuário deve informar o nome completo da aluna para realizar a consulta.
  • Após a consulta, o sistema deve exibir a quantidade de faltas da aluna

Por isso, era necessário imprimir as faltas da aluna, ao contrário do que foi implementado.

print("Inserir os seguintes dados: ")
nome = input("Nome da aluna: ")
sobrenome = input("Sobrenome da aluna: ")
quantidade_notas = input("Quantidade de notas: ")
Copy link
Collaborator

Choose a reason for hiding this comment

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

Neste ponto, é esperado que já tenhamos as notas da aluna, esta linha acaba sendo desnecessária. Pois não utilizamos a informação de quantidade de notas em nenhum outro local.

Comment on lines +85 to +88
quantidade_notas = input("Quantidade de notas: ")
nota1 = float(input("Qual é a nota: ?"))
nota2 = float(input("Qual é a nota: ?"))
nota3 = float(input("Qual é a nota: ?"))
Copy link
Collaborator

Choose a reason for hiding this comment

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

Estamos recebendo os valores, e não usando esses dados. Esse trecho de código era esperado estar na função obter_notas que não foi implementado neste arquivo.

nota3 = float(input("Qual é a nota: ?"))
notas = dataset[(nome, sobrenome)]["Notas"]
media_das_notas = float(sum(notas)/len(notas))
print(float(media_das_notas))
Copy link
Collaborator

Choose a reason for hiding this comment

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

Procure imprimir mensagens de apoio para que fique claro para o usuário do que se trata este número.

Comment on lines +95 to +96
elif media_das_notas < 5:
status = "Recuperação"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Neste projeto não era necessário implementar nenhum fluxo de recuperaçã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