Skip to main content
1 de 2
hkotsubo
  • 66,9mil
  • 25
  • 87
  • 170

O primeiro problema (não atualiza o número da série) acontece porque você não está incrementando o contador i dentro do loop.

O segundo problema é resolvido verificando-se o tamanho da string digitada, através da função len.

O terceiro problema (mostrar as séries) acontece por causa deste loop:

for i in series:
    print("{}: {}".format(i, series))

A variável series é uma lista que contém todas as séries. Ao fazer for i in series, a cada iteração do loop, a variável i contém o nome de uma das séries. Mas você imprime o nome (a variável i) e toda a lista (a variável series), por isso a saída está do jeito que você indicou.

Outros detalhes para arrumar:

  • a variável series começa como uma lista contendo uma string vazia (series = ['']). Mas não há motivo para isso, basta atribuir o valor [] para que a lista comece vazia
  • a função input já retorna uma string, não é necessário usar str(input(...))
  • se quer iterar pelos elementos da lista juntamente com o respectivo índice, basta usar enumerate

O código final fica assim:

nome = ''
while not nome:
    nome = input("Digite seu nome: ").strip()

print("{}, quero lembrar que não é permitido digitar apenas uma letra".format(nome))
print("Deseja responder a pergunta? ")
resposta = input("S - Sim | N - Não: ").strip()[0]

i = 1
series = []
while resposta.lower() == 's':
    while True:
        serie = input("Digite a sua {}a série favorita: ".format(i)).strip()
        if len(serie) <= 1:
            print('Nome deve ter mais de uma letra')
        else:
            break # nome OK, sai do while True

    # o while True acima já garante que o nome estará preenchido
    print("Nome da série: {}".format(serie))
    series.append(serie)
    i += 1

    print("Deseja digitar mais alguma série, {}?".format(nome))
    resposta = input("S - Sim | N - Não: ").strip()[0]

if resposta.lower() == 'n':
    print("{}, Bye!".format(nome))

for i, serie in enumerate(series):
    print("{}: {}".format(i + 1, serie))

Outros detalhes:

  • usei strip() para remover os espaços do início e fim das strings retornadas por input (não é estritamente necessário, use se quiser)
  • o loop while True já garante que o nome da série deve estar preenchido, então não preciso testar o nome antes de imprimi-lo
  • para verificar a opção, eu uso resposta.lower() == 's' - pois lower() transforma a string em minúsculas. Mas se quiser, também pode fazer while resposta in ('S', 's'):.
  • while not nome é o mesmo que while nome == ""

Outra opção é quebrar cada parte do código em funções (uma para ler o nome da série, outra para as opções, etc), assim o código fica um pouco mais organizado:

def ler_opcao(mensagem, nome=None):
    if nome:
        print('{}, {}? '.format(mensagem, nome))
    else:
        print('{}? '.format(mensagem))
    # já retorna a opção como letra minúscula
    return input("S - Sim | N - Não: ").strip()[0].lower()


def ler_nome_serie(i):
    while True:
        serie = input("Digite a sua {}a série favorita: ".format(i)).strip()
        if len(serie) <= 1:
            print('Nome deve ter mais de uma letra')
        else:
            return serie  # nome OK, retorna o nome da série


nome = ''
while not nome:
    nome = input("Digite seu nome: ").strip()

print("{}, quero lembrar que não é permitido digitar apenas uma letra".format(nome))
resposta = ler_opcao("Deseja responder a pergunta")

i = 1
series = []
while resposta == 's':
    serie = ler_nome_serie(i)
    print("Nome da série: {}".format(serie))
    series.append(serie)
    i += 1

    resposta = ler_opcao("Deseja digitar mais alguma série", nome)

if resposta == 'n':
    print("{}, Bye!".format(nome))

for i, serie in enumerate(series):
    print("{}: {}".format(i + 1, serie))

Outro detalhe é o if resposta == 'n':. O while só verifica se a opção é s. Então se a pessoa digitar, por exemplo, x, ele também sai do while, mas depois não entra no if (pois não foi digitado n). Se quiser que sempre imprima esta mensagem, basta retirar o if.

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