Mudando o nome da variável de n
para preco
talvez fique um pouco mais "didático" para explicar (e também porque dar nomes melhores ajuda na hora de programar):
maiorv = menorv = 0
while True:
preco = float(input('Digite o valor do produto: R$ '))
if preco == 1:
maiorv = menorv = preco
else:
if preco > maiorv:
maiorv = preco
if preco < menorv:
menorv = preco
etc...
Ou seja, se o preço digitado for igual a 1 você faz com que o maior e menor preço sejam 1. Isso está errado porque, no caso em que o 1
seja o último valor digitado (por exemplo, o usuário digita vários valores, depois digita 1
e depois escolhe "N" para não continuar), então tanto o maior quanto o menor serão 1. Isso faz com que todos os valores anteriores sejam ignorados. Esse if
não faz sentido, remova-o.
Outro detalhe é que menorv
começa em zero. Então qualquer valor positivo que for digitado (1, 2, 3, qualquer outro valor positivo) nunca será menor que zero, então nunca entrará no if preco < menorv
. Somente se você digitar um valor negativo, aí ele será menor que zero e entrará neste if
(mas pela lógica do programa, um preço negativo não faria sentido). Então se você só digitar valores positivos, nunca entrará no if
e no fim menorv
será zero.
Sendo assim, uma solução é fazer com que menorv
comece com o maior valor possível, assim qualquer coisa que for digitada será com certeza menor que ele. O "maior valor possível", no caso, é "infinito", que pode ser obtido usando-se float('inf')
.
A mesma lógica vale para maiorv
: ele deve começar com o menor valor possível, assim qualquer coisa que for digitada será com certeza maior que ele (e uma opção é usar o "infinito negativo", que pode ser obtido com float('-inf')
ou -float('inf')
).
E tire o if preco == 1
, como já explicado:
maiorv = float('-inf') # infinito negativo
menorv = float('inf') # infinito
while True:
preco = float(input('Digite o valor do produto: R$ '))
if preco > maiorv:
maiorv = preco
if preco < menorv:
menorv = preco
continuar = ' '
while continuar not in 'SN':
continuar = input('Quer continuar: (S/N) ').upper().strip()[0]
if continuar == 'N':
break
Repare também que input
já retorna uma string, então fazer str(input(...))
é redundante e desnecessário. Faça apenas input(...)
.
Por fim, sugiro que aprenda a fazer o teste de mesa e a debugar o código, que erros de lógica como esse podem ser facilmente detectados.
Uma das respostas guarda os valores em uma lista e depois usa max
e min
para obter o maior e menor.
Também funciona, claro, mas há o detalhe de que cada uma dessas chamadas percorre toda a lista (ou seja, a lista é percorrida duas vezes, uma para verificar o maior, e outra para o menor).
Claro que para uma quantidade pequena de números, a diferença é irrelevante. Mas se você não precisa guardar todos os números digitados, eu não criaria a lista para este caso. E de qualquer forma é interessante saber que existem funções prontas na linguagem.