Isso aqui:
if None in classe1 or classe2 or classe3:
Não faz o que você pensa que faz. Na verdade, o que você tem são 3 condições:
None in classe1
: isso verifica se None
está na lista classe1
(ou seja, se None
foi inserido anteriormente na lista). Mas como você só insere números, o None
nunca estará na lista, então essa condição sempre será falsa
classe2
: sim, uma lista (e qualquer outro valor) pode ser usada em um contexto booleano. No caso, se a lista for vazia, é considerado um valor falso, e se não for vazia, é considerado verdadeiro.
classe3
: idem
Portanto, a primeira condição sempre será falsa, mas se classe2
ou classe3
tiverem pelo menos um elemento, serão consideradas True
e entrará no if
, já que uma expressão que tem várias condições com or
é considerada verdadeira se qualquer uma das condições for verdadeira (imprima as listas depois deste if
e veja você mesmo que se classe2
ou classe3
não estiver vazia, sempre é adicionado um zero).
E é por isso que ele não entra no if
quando as listas classe2
e classe3
são vazias: porque uma lista vazia é avaliada como False
, e como já dito, a primeira condição também é falsa porque None
não foi adicionado em classe1
(então todas as condições são falsas e não entra no if
).
Mas sinceramente, isso que você pensou em fazer é - na minha opinião - uma grande gambiarra. Se a ideia é retornar zero caso a lista seja vazia, não tem porque adicionar um elemento artificialmente só para não dar erro. Eu acho mais simples verificar se a lista é vazia:
n = int(input())
classe1 = []
classe2 = []
classe3 = []
for _ in range(n):
velocidade = int(input())
if velocidade < 10:
classe1.append(velocidade)
elif velocidade < 20:
classe2.append(velocidade)
else:
classe3.append(velocidade)
def maior(lista):
if len(lista) == 0: # se a lista é vazia, retorna zero
return 0
return max(lista) # lista não é vazia, retorna o max()
print(maior(classe1), maior(classe2), maior(classe3))
Criei uma função para facilitar: ela verifica se a lista é vazia (se for, retorna zero), e se não for, usa max
. Isso é melhor do que adicionar um elemento a mais. Claro que também podemos discutir se faz sentido retornar um valor artifical qualquer (pois e se for digitado somente zeros, como você vai saber se o zero é o maior valor que foi de fato digitado ou se caiu no caso de lista vazia? Talvez seja o caso de não retornar nada e imprimir uma mensagem de erro, por exemplo), mas enfim, inserir um valor artificial "só para o max
não dar erro" me parece uma opção pior.
Removi a lista listaVelocidade
, pois parece que não estava sendo usada para nada (apenas para guardar as velocidades, para depois só serem inseridas em uma das outras listas, então me parece meio redundante; eu achei mais simples inserir o valor direto nas respectivas listas).
E criei as listas apenas uma vez, antes do for
. Você estava criando elas dentro do for
, ou seja, a cada iteração elas eram recriadas novamente, o que não faz sentido. Crie uma vez fora do loop e dentro dele apenas insira os elementos.
Repare também nos if
's: se não entrou no if velocidade < 10
é porque o valor com certeza é maior ou igual a 10, então não precisa testar isso de novo no elif
.
E se não entrou no elif velocidade < 20
, é porque com certeza é maior ou igual a 20, então não precisa testar de novo no else
:
if velocidade < 10:
# se entrou aqui, é porque é menor que 10
classe1.append(velocidade)
# se não entrou no if acima, é porque com certeza é >= 10, então não precisa testar de novo no elif
elif velocidade < 20:
classe2.append(velocidade)
# se não entrou no if nem no elif, é porque com certeza é >= 20, então não precisa testar de novo
else:
classe3.append(velocidade)