如何实现Discord机器人战斗命令的无重复代码循环回合

看起来你现在的代码重复了相同的询问逻辑，这确实不利于后续维护和功能扩展。我们可以通过封装重复逻辑为函数+循环控制回合流程的方式来解决问题，同时能完美实现多回合战斗直到一方战败的需求。

核心思路拆解 #

• 把「询问玩家行动」这一重复逻辑抽成独立可复用的函数，每次调用只需传入玩家信息即可
• 用while循环驱动回合流程，每次循环对应一个完整回合，直到某玩家战败或投降才终止循环
• 维护玩家的核心状态（比如生命值），每次处理完行动后更新状态并判断战斗是否结束

完整实现示例 #

首先我们可以定义一个简单的Player类来存储玩家的关键状态，方便后续管理和扩展：

import discord
from discord.ext import commands

bot = commands.Bot(command_prefix="!", intents=discord.Intents.all())

class Player:
    def __init__(self, user, hp=100):
        self.user = user  # 存储Discord用户对象
        self.nickname = user.display_name
        self.hp = hp  # 玩家初始生命值，可根据需求调整

# 封装询问玩家行动的复用函数
async def get_player_choice(ctx, player):
    # 发送行动选择提示
    await ctx.send(f"{player.nickname}, what will you do now? OPTIONS: Fight, Defense, Items, Surrender")
    # 定义消息检查规则：必须是目标玩家在当前频道发送的消息
    def check_msg(m):
        return m.channel == ctx.channel and m.author == player.user
    # 等待玩家回复
    player_response = await bot.wait_for('message', check=check_msg)
    # 返回标准化的行动指令（小写方便后续判断）
    return player_response.content.strip().lower()

# 战斗命令主逻辑
@bot.command(name="startfight")
async def initiate_battle(ctx, opponent: discord.User):
    # 初始化两名玩家：发起命令的用户为Player1，被@的用户为Player2
    player1 = Player(ctx.author)
    player2 = Player(opponent)
    
    turn_number = 1
    # 战斗循环：只要两名玩家都存活，就继续回合
    while player1.hp > 0 and player2.hp > 0:
        await ctx.send(f"-- TURN {turn_number} --")
        
        # 获取并处理Player1的行动
        action_p1 = await get_player_choice(ctx, player1)
        if action_p1 == "fight":
            player2.hp -= 25
            await ctx.send(f"{player1.nickname} attacks! {player2.nickname} loses 25 HP. Current HP: {player2.hp}")
        elif action_p1 == "defense":
            await ctx.send(f"{player1.nickname} takes a defensive stance! Reduces incoming damage next round.")
        elif action_p1 == "surrender":
            await ctx.send(f"{player1.nickname} surrendered! {player2.nickname} wins the battle!")
            return
        # 可在这里补充Items等其他行动的处理逻辑
        
        # 检查Player2是否已战败
        if player2.hp <= 0:
            await ctx.send(f"{player2.nickname} has been defeated! {player1.nickname} is the winner!")
            return
        
        # 获取并处理Player2的行动
        action_p2 = await get_player_choice(ctx, player2)
        if action_p2 == "fight":
            player1.hp -= 25
            await ctx.send(f"{player2.nickname} attacks! {player1.nickname} loses 25 HP. Current HP: {player1.hp}")
        elif action_p2 == "defense":
            await ctx.send(f"{player2.nickname} takes a defensive stance! Reduces incoming damage next round.")
        elif action_p2 == "surrender":
            await ctx.send(f"{player2.nickname} surrendered! {player1.nickname} wins the battle!")
            return
        # 可在这里补充Items等其他行动的处理逻辑
        
        # 检查Player1是否已战败
        if player1.hp <= 0:
            await ctx.send(f"{player1.nickname} has been defeated! {player2.nickname} is the winner!")
            return
        
        # 回合数递增，进入下一轮
        turn_number += 1

bot.run("YOUR_BOT_TOKEN")

关键细节说明 #

1. 函数封装的优势：get_player_choice函数把发送提示、等待回复的逻辑打包，彻底消除了代码重复。后续如果要修改行动选项或提示语，只需要修改这一处即可，非常便于维护。
2. 循环控制逻辑：while循环的终止条件是「任意玩家生命值归0」，只要满足条件就会跳出循环并宣布战斗结果，完全符合你的多回合需求。
3. 状态管理：用Player类统一管理玩家的生命值、昵称等状态，比零散的变量更清晰，后续扩展攻击力、防御力等属性也会更方便。
4. 行动扩展性：示例中简化了战斗逻辑，你可以根据需求自由扩展——比如让Defense真正实现减伤效果、Items对应不同的道具功能等。

这样改造后，你的代码不仅解决了重复问题，还能流畅实现多回合战斗的完整流程，后续功能迭代也会轻松很多。

内容的提问来源于stack exchange，提问作者Chins man