Dá erro de out of range porque a última string é vazia, ou seja, o tamanho dela é zero. Sendo assim, ela não tem nenhum índice (não tem message[0]
, nem message[-1]
, nem qualquer outro valor), e qualquer tentativa de obter algum caractere dela dará erro.
Para solucionar, você pode testar se a string é vazia e retornar True
, caso contrário, retorne o resultado da comparação:
def first_and_last(message):
if not message: # string vazia, retorna True
return True
return message[0] == message[-1]
print(first_and_last("else")) # True
print(first_and_last("tree")) # False
print(first_and_last("")) # True
O if not message
testa se a string é vazia. Isso é possível porque uma string vazia é considerada um valor falso.
Depois, se a string não for vazia, retorne o resultado da comparação do primeiro com o último caractere. Repare que não precisa do if
aqui, pois o resultado da comparação message[0] == message[-1]
já é um booleano e você pode retorná-lo diretamente.
De forma geral, qualquer expressão desta forma:
if condição:
return True
return False
Pode ser substituída por:
return condição
Por isso que em vez de:
if(message[0] == message[-1]):
return True
return False
Eu fiz simplesmente:
return message[0] == message[-1]
Por fim, dá para simplificar as duas condições assim:
def first_and_last(message):
return not message or message[0] == message[-1]
Ou seja, retorna o resultado de not message or message[0] == message[-1]
(string é vazia, ou o primeiro e último caracteres são iguais).
Este caso não dá erro porque o operador or
é um short-circuit operator: se a primeira condição for verdadeira, ele nem verifica a segunda. Então primeiro eu verifico se a string é vazia. Caso ela não seja, aí a segunda condição é verificada. Mas se a string for vazia, a expressão já retorna True
, não correndo o risco de dar o erro de out of bounds.