Uma das principais dicas está na resposta do @nosklo: você está repetindo código sem necessidade, sendo que poderia usar um loop (é para isso que os loops servem, para executar alguma coisa repetidas vezes).
Outro detalhe é que tem variáveis a mais que não seriam necessárias. Por exemplo, quando você faz isso:
numero_aleatorio= random.randint(1,10)
x = numero_aleatorio
Você cria o numero_aleatorio
e depois só usa ele para atribuir seu valor em x
. Não tem porque criar duas variáveis para isso, use somente uma delas. E no caso, eu escolheria usar somente numero_aleatorio
, por ser um nome melhor do que x
, já que indica exatamente o que aquela variável significa.
Isso vale para outras variáveis também. y
e z
são nomes "genéricos" demais e quem vê o código não consegue perceber o que elas significam (a menos que leia o código todo, claro). Pode parecer um detalhe bobo, mas dar nomes significativos para tudo (variáveis, funções, classes, etc) torna todo o processo de programar mais fácil, pois você não precisa ficar toda hora lembrando "O que é esse x
mesmo?" (já se você ler numero_aleatorio
, já sabe imediatamente o que ele significa).
Dito isso, a solução proposta pelo @nosklo já simplifica bastante o seu código. Veja como você pode chamar int(input(...))
diretamente, já guardando o valor numérico na variável, e veja também como o while
já elimina a necessidade de repetir o código, e a verificação do valor digitado é simplificada com apenas um if
/else
dentro de outro while
.
Eu ainda proponho mais algumas modificações. Se quiser, nem precisa da variável para guardar a opção de sair ou não do jogo, pois também dá para fazer assim:
while True:
# lê a tentativa, verifica se está certa, etc
....
# se a opção não for "s", sai do "while True"
if input("Você quer continuar jogando? (s/n)") != 's':
break
O while True
indica que o loop vai rodar indefinidamente. Mas se a opção digitada for diferente de 's'
, é chamado o break
, que interrompe o while
.
A menos que você vá usar a opção digitada para mais alguma coisa, não precisa guardá-la em uma variável. Mas se você quiser usar uma variável para isso, chame-a de opcao
ou algum outro nome significativo, em vez de z
ou qualquer outro nome "genérico" que não diz nada sobre a sua função no código.
Outra sugestão é verificar o valor digitado. Você está pegando um valor digitado com input
e em seguida usando int
para tentar converter para um número. Só que o usuário pode digitar qualquer coisa, inclusive algo que não é um número, então seria interessante verificar isso também, ficando com um código mais ou menos assim:
while True:
try:
# lê o valor e tenta converter para número
tentativa = int(input("Adivinhe o número:"))
break # se chegou aqui, é porque int() funcionou
except ValueError:
# se int() falhou, é porque não foi digitado um número, então tente novamente
print("Digite um número válido")
Se o int()
falhar (porque não foi digitado um número), ocorrerá um ValueError
, e neste caso, o código dentro do except
será executado. No caso, eu informo o que aconteceu, e como estou dentro de um while
, será lido outro número.
Mas como no código estamos lendo a tentativa duas vezes (uma antes de verificar se o número foi adivinhado, e outra depois, caso não tenha sido), é melhor encapsular toda esta lógica de ler o número em uma função. O código final fica assim:
import random
# função que lê o número e retorna o valor lido
def ler_numero(mensagem):
while True:
try:
# lê o número e retorna o valor
return int(input(mensagem))
except ValueError:
# se não foi digitado um número, int() lança um ValueError
print("Digite um número válido")
while True:
numero_aleatorio = random.randint(1, 10)
tentativa = ler_numero("Advinhe o numero:")
while tentativa != numero_aleatorio:
if tentativa < numero_aleatorio:
print("Numero muito baixo")
else:
print("Numero muito alto")
tentativa = ler_numero("Advinhe o numero:")
print("Voce advinhou!")
# se a opção não for "s", sai do "while True"
if input("Você quer continuar jogando? (s/n)") != 's':
break
Repare que dentro da função ler_numero
eu usei return
em vez de break
. É que se int()
funcionar (ou seja, foi digitado um número e este foi convertido corretamente), eu já posso retornar o seu valor imediatamente (não precisa ter um break
para sair do while
e depois ter um return
).
Já se o int()
falhar, eu caio no bloco except
, a mensagem informa que o valor digitado não é um número e o while
continua executando, pedindo que seja digitado outro número.