palindromes回文函数报错原因咨询及代码排查请求

问题描述 #

需要实现palindromes函数判断字符串是否为回文（返回True），同时编写main函数让用户输入单词，直到输入回文为止。当前代码运行时，输入非回文单词会触发逻辑不严谨的问题。

原代码 #

def palindromes(user_word):
    og_word = list(user_word)
    inverse_word = []
    for i in range(len(og_word)-1, -1, -1):
        inverse_word.append(og_word[i])
    if og_word == inverse_word:
        return True

def main():
    while True:
        user_word = input('Please type in a palindrome: ')
        if palindromes(user_word):
            print(f"{user_word} is a palindrome!")
            break
        else:
            print("that wasn't a palindrome")
            
main()

错误原因 #

palindromes函数仅在字符串是回文时返回True，当输入非回文字符串时，函数没有明确的返回语句，Python会默认返回None。虽然if判断中None会被视为False，程序能进入else分支，但这不符合函数设计规范，且可能引发潜在逻辑问题。

修复方案 #

方案1：补全返回逻辑 #

给palindromes函数添加非回文场景的明确返回值False：

def palindromes(user_word):
    og_word = list(user_word)
    inverse_word = []
    for i in range(len(og_word)-1, -1, -1):
        inverse_word.append(og_word[i])
    if og_word == inverse_word:
        return True
    # 明确返回False
    return False

def main():
    while True:
        user_word = input('请输入一个回文字符串: ')
        if palindromes(user_word):
            print(f"{user_word} 是回文！")
            break
        else:
            print("这不是回文，请重新输入")
            
main()

方案2：简化代码实现 #

利用Python字符串切片特性直接反转字符串，代码更简洁高效：

def palindromes(user_word):
    # 字符串切片[::-1]直接生成反转后的字符串
    return user_word == user_word[::-1]

def main():
    while True:
        user_word = input('请输入一个回文字符串: ')
        if palindromes(user_word):
            print(f"{user_word} 是回文！")
            break
        else:
            print("这不是回文，请重新输入")
            
main()

内容的提问来源于stack exchange，提问作者Python Beginner