Use Discord Structures

I see your code, that you’re not using discord structure to instantiate the Player for each guild. Rather you’re having a collection of players that is saved to the client. I think that’s not appropriate, to use efficiently. Because it requires to pass the <Guild>.id to each method we want to access, to access the player of that particular guild.

One more thing, why every method of the class Player requires us to pass a message object. That’s really inappropriate and redundant.

[skdhg]

Discord.js Slash Commands are available from Discord.js v13 and current Discord Player version targets Discord.js v12. We will have a clean rewrite targeting Discord.js v13 with many breaking changes. It’s not a good idea to use this lib (which currently is for v12) with discord.js master (v13) anyways

[Majoramari]

I don’t see a point of using Discord Structures in this library (users will do it if they need). However, we can improve the Player where you do not need to pass message to every methods 🤔

using a message prevent you from using slash commands so this need to change