Python Todo程序if分支逻辑异常求助：所有输入均触发add指令

你的核心问题是条件判断的语法错误，导致第一个add分支始终被触发。比如if 'add' or 'Add' in user_input这种写法中，'add'本身是一个非空字符串，在布尔判断里会被视为True，所以整个条件永远为真，不管输入什么内容都会执行add分支。

修复步骤： #

1. 修正条件判断逻辑：
   • 不要单独写'add'，而是要明确判断该字符串是否属于指令部分；更高效的方式是将输入统一转为小写，再判断指令，避免大小写问题。
   • 用split(maxsplit=1)拆分输入的指令和内容，确保指令判断准确，不会因为输入包含多个单词而误判。
2. 增加健壮性处理：添加文件不存在、无效数字输入等异常情况的处理，避免程序崩溃。

修正后的完整代码： #

while True:
    user_input = input("Type add, show, edit, completed or exit: ").strip()
    if not user_input:
        print("Please enter a valid command.")
        continue
    
    # 拆分指令和参数，maxsplit=1确保只分割一次
    parts = user_input.split(maxsplit=1)
    command = parts[0].lower()
    content = parts[1] if len(parts) > 1 else ""

    if command == 'add':
        if not content:
            print("Please enter a todo item after 'add'.")
            continue
        todo = content + '\n'
        # 处理文件不存在的情况
        try:
            with open('files/todos.txt', 'r') as file:
                todos = file.readlines()
        except FileNotFoundError:
            todos = []
        
        todos.append(todo)
        with open('files/todos.txt', 'w') as file:
            file.writelines(todos)

    elif command == 'show':
        try:
            with open('files/todos.txt', 'r') as file:
                todos = file.readlines()
        except FileNotFoundError:
            todos = []
        
        if not todos:
            print("No todos in the list.")
            continue
        for index, item in enumerate(todos):
            item = item.strip('\n')
            print(f"{index+1}. {item}")

    elif command == 'edit':
        try:
            with open('files/todos.txt', 'r') as file:
                todos = file.readlines()
        except FileNotFoundError:
            print("No todos to edit.")
            continue
        
        if not todos:
            print("No todos in the list.")
            continue
        try:
            number = int(input("Enter the number of todo to edit: "))
            if number < 1 or number > len(todos):
                print(f"Please enter a number between 1 and {len(todos)}.")
                continue
        except ValueError:
            print("Please enter a valid number.")
            continue
        
        new_todo = input("Enter the new todo: ").strip()
        if not new_todo:
            print("Todo content cannot be empty.")
            continue
        todos[number - 1] = new_todo + "\n"
        with open('files/todos.txt', 'w') as file:
            file.writelines(todos)

    elif command == 'completed':
        try:
            with open('files/todos.txt', 'r') as file:
                todos = file.readlines()
        except FileNotFoundError:
            print("No todos to mark as completed.")
            continue
        
        if not todos:
            print("No todos in the list.")
            continue
        try:
            number = int(input("Enter the number of todo that is completed: "))
            if number < 1 or number > len(todos):
                print(f"Please enter a number between 1 and {len(todos)}.")
                continue
        except ValueError:
            print("Please enter a valid number.")
            continue
        
        index = number - 1
        todo_to_remove = todos[index].strip('\n')
        todos.pop(index)
        with open('files/todos.txt', 'w') as file:
            file.writelines(todos)
        print(f"Todo '{todo_to_remove}' is removed from the list")

    elif command == 'exit':
        break
    else:
        print("Unknown Command. Valid commands: add, show, edit, completed, exit")

print("Bye")

关键改动说明： #

• 指令拆分：用split(maxsplit=1)将输入拆分为指令和内容，确保show Play chess这类错误输入会被识别为show指令，而非误触发add分支。
• 统一小写判断：将指令转为小写，支持Add、SHOW等大小写混合的输入。
• 异常与边界处理：捕获文件不存在的异常，校验输入数字的合法性，处理空输入情况，提升程序稳定性和用户体验。

内容的提问来源于stack exchange，提问作者5P33DC0R3