提问者:小点点

如何改进我的排队系统-Discord.py


我创建了这个音乐机器人,我想改进我的队列命令,因为它目前不是很实用。每次我排队一首歌时,我都必须使用播放命令来播放它,但我想自动播放下一首歌,将队列命令实现到播放命令中也很酷,但我不知道如何做到这一点。你能帮忙吗??

youtube_dl.utils.bug_reports_message = lambda: ''

ytdl_format_options = {
    'format': 'bestaudio/best',
    'outtmpl': '%(extractor)s-%(id)s-%(title)s.%(ext)s',
    'restrictfilenames': True,
    'noplaylist': True,
    'nocheckcertificate': True,
    'ignoreerrors': False,
    'logtostderr': False,
    'quiet': True,
    'no_warnings': True,
    'default_search': 'auto',
    'source_address': '0.0.0.0' # bind to ipv4 since ipv6 addresses cause issues sometimes
}

ffmpeg_options = {
    'options': '-vn'
}

ytdl = youtube_dl.YoutubeDL(ytdl_format_options)

class YTDLSource(discord.PCMVolumeTransformer):
    def __init__(self, source, *, data, volume=0.5):
        super().__init__(source, volume)

        self.data = data

        self.title = data.get('title')
        self.url = data.get('url')

    @classmethod
    async def from_url(cls, url, *, loop=None, stream=False):
        loop = loop or asyncio.get_event_loop()
        data = await loop.run_in_executor(None, lambda: ytdl.extract_info(url, download=not stream))

        if 'entries' in data:
            # take first item from a playlist
            data = data['entries'][0]

        filename = data['url'] if stream else ytdl.prepare_filename(data)
        return cls(discord.FFmpegPCMAudio(filename, **ffmpeg_options), data=data)

queue = []


@client.command(name='queue', help='This command adds a song to the queue')
async def queue_(ctx, url):
    global queue

    queue.append(url)
    await ctx.send(f'`{url}` added to queue!')



@client.command(name='play', help='This command plays songs')
async def play(ctx):
    global queue

    server = ctx.message.guild
    voice_channel = server.voice_client

    async with ctx.typing():
        player = await YTDLSource.from_url(queue[0], loop=client.loop, stream=True)
        voice_channel.play(player, after=lambda e: print('Player error: %s' % e) if e else None)

    await ctx.send('**Now playing:** {}'.format(player.title))
    del(queue[0])

编辑:所以我试图做这样的事情,但它不起作用,当我尝试使用!播放时,一首歌正在播放,它不把它放在队列中,它说:ClientException:已经在播放音频。当歌曲完成时,它说:TypeError: next()缺少2个必需的位置参数:“队列”和“歌曲”这是代码:

queue = []

def next(client, queue, song):
    if len(queue)>0:
        new_song= queue[0]
        del queue[0]
        play(client, queue, new_song)

@bot.command()
async def join (ctx):
    member = ctx.author
    if not ctx.message.author.voice:
        await ctx.send(f"{member.mention} You are not connected to a voice channel  ❌")
    else:
        channel=ctx.message.author.voice.channel
        await channel.connect()


@bot.command(help="This command plays a song.")
async def play (ctx,*args):
    server = ctx.message.guild
    voice_channel= server.voice_client
    url=""
    for word in args:
        url+=word
        url+=''

    async with ctx.typing():
        player = await YTDLSource.from_url(url ,loop=bot.loop, stream=True)
        queue.append(player)
        voice_channel.play (player, after=lambda e: next(ctx))
        
    await ctx.send(f"**Now playing:** {player.title}")

共1个答案

匿名用户

在您的异步play函数中,您有以下代码行:

voice_channel.play(player, after=lambda e: print('Player error: %s' % e) if e else None)

如您所见,有一个参数可以用来在机器人完成向语音通道播放音频后预形成一个动作。我建议您使用在=参数之后更改代码以递归播放下一首歌曲,而不是像现在这样使用它。您可以通过将lambda中的当前print语句更改为再次异步调用play函数来做到这一点。

我还想指出,虽然学习如何使用del()函数对于学习如何优化代码非常有用,但这并不是您想要实现它的方式。在这种情况下,您应该改为. pop()值以将其从队列中取出(像这样:queue.pop(0)。您还应该知道弹出值会返回值,因此您可以直接将其实现到YTDLSource的第一个参数中。from_url()函数)。

我能提供的最后一点信息是你为你的队列系统编写了一个类。显然,这实际上不会改变你需要做什么来解决你的问题,但是学习如何编写OOP对象和类将在未来对你有所帮助,而且它还使编写代码变得更容易,因为它变得更有条理。

总结一下:

  • 首先删除del(queue[0]),取而代之的是将from_url方法中的queue[0]参数值与que. pop(0)交换。
  • 最理想的情况是,您应该使用Queue类更改整个队列数组,因为您可以非常轻松地向其添加方法和其他有用的包装函数,但这不是必须的。
  • 最后,代替lambda中的print语句(在. play()方法的参数之后),再次异步调用play函数。(例如,像so-lambda e:await play(ctx)