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
.