Python中替代if/elif/else语句的方案及语音助手代码优化

Hey there! Great question—dealing with endless elif chains can get messy fast, so let's break down cleaner alternatives for your voice assistant, plus how to refactor your existing code to eliminate redundancy.

1. Dictionary Mapping (Best for Your Current Use Case) #

The most straightforward fix is to map command lists to their corresponding actions using a list of tuples (or a dictionary if you want one-to-one command-to-action mapping). This way, adding new commands only requires adding a new entry to the mapping—no more editing the assistant function's conditionals.

Refactored Code #

First, define separate functions for each action (this keeps your logic modular):

import webbrowser
import random
import datetime

# Assume talkMe is your existing text-to-speech function
def talkMe(message):
    print(message)  # Placeholder for your actual TTS logic

def open_web_browser():
    talkMe("Opening your web browser")
    webbrowser.open("https://www.google.com.tr")

def respond_thanks():
    talkMe("You are welcome")

def open_favorite_site():
    talkMe("Opening your site")
    webbrowser.open("https://www.stackoverflow.com")  # Fixed missing HTTP scheme

def respond_how_are_you():
    msg = ["I am good, you?", "Good", "Not bad"]
    talkMe(random.choice(msg))

def tell_time():
    strTime = datetime.datetime.now().strftime("%H:%M:%S")
    talkMe(f"The time is {strTime} ")

Next, create a mapping of command lists to their actions:

command_mappings = [
    (["open web browser", "web browser", "open browser"], open_web_browser),
    (["thank you", "thank you so much", "thanks"], respond_thanks),
    (["open my favourite web site", "favourite web site", "my best web site"], open_favorite_site),
    (["how are you", "what's up", "how is going"], respond_how_are_you),
    (["whats the time", "the time", "time"], tell_time),
]

Finally, rewrite the assistant function to use this mapping:

def assistant(command):
    for command_list, action in command_mappings:
        if command in command_list:
            action()
            return  # Exit after executing the matching action
    # Handle unrecognized commands
    talkMe("Sorry, I didn't understand that command.")

2. Decorator-Based Command Registration (For Scalability) #

If you plan to add lots of commands or want a more "Pythonic" way to register actions, use decorators. This lets you bind commands directly to their functions without maintaining a separate mapping list.

Example Implementation #

command_registry = {}

def register_command(commands):
    def decorator(func):
        for cmd in commands:
            command_registry[cmd] = func
        return func
    return decorator

@register_command(["open web browser", "web browser", "open browser"])
def open_web_browser():
    talkMe("Opening your web browser")
    webbrowser.open("https://www.google.com.tr")

@register_command(["thank you", "thank you so much", "thanks"])
def respond_thanks():
    talkMe("You are welcome")

# Add other commands with the @register_command decorator...

def assistant(command):
    action = command_registry.get(command)
    if action:
        action()
    else:
        talkMe("Sorry, I didn't understand that command.")

Should You Stick with if/elif/else? #

Only if your command set is tiny (3-4 commands max) and you don't plan to expand it. For your current code (5 command groups) and future expansion, the dictionary or decorator approach is far better:

• Eliminates repetitive elif code
• Makes adding new commands trivial (no need to edit the core assistant logic)
• Keeps your code organized and easier to debug

内容的提问来源于stack exchange，提问作者Can İlgu