Está abusando do elif
. Não é porque tem várias condições que tudo deve ser agrupada. Tem que fazer isso de acordo com a necessidade.
A validação de cada nota deve ser independente, não faz sentido colocar um elif
ali, a não ser para complicar o código, até poderia de uma outra forma mas complica muito o código e ele ficaria menos legível. O mais correto quando um dado inválido é entrado é encerrar. Se fosse em uma função bastaria usar um return
, se fosse em um loop, poderia usar um continue
ou até um break
em algumas situações. Como está no script puro deve sair com exit()
.
A média só deve ser calculada quando tem dados válidos e continuou a execução.
N]ao é necessário verificar se a média é até 10 porque se os dados estão válidos ela foi calculada até o limite correto.
E finalmente não precisa verificar nada na reprovação porque é o oposto da aprovação, basta um else
simples.
import sys
n1 = int(input('digite nota 1: '))
n2 = int(input('digite nota 2: '))
if n1 > 10:
print('{} não é uma nota válida!'.format(n1))
sys.exit(0)
if n2 > 10:
print('{} não é uma nota válida!'.format(n2))
sys.exit(0)
media = (n1 + n2) / 2
if media >= 7:
print('Você foi aprovado com média {}'.format(media))
else:
print('Você foi reprovado com média {}'.format(media))
Veja funcionando no ideone. E no repl.it. Também coloquei no GitHub para referência futura.
Dá para fazer do jeito que estava fazendo sem forçar a saída? Dá, mas a lógica ficaria confusa demais para uma pessoa experiente, imagina para uma pessoa começando. E correria o risco de aprender de um jeito confuso para o resto da vida. Fora que seria menos eficiente, ainda que não seja algo relevante para esse caso poderia ser um dia e já acostuma fazer mais correto.