Skip to main content
2 de 3
adicionou 38 caracteres ao conteúdo
hkotsubo
  • 66,9mil
  • 25
  • 87
  • 170

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.

hkotsubo
  • 66,9mil
  • 25
  • 87
  • 170