Add user to array

Question

I am sure the solution to my problem is something easy that I am overlooking.

What I am trying to do is when a user types !add in discord, the bot will add them into an array (for a 6 man pick up game).

Currently when the first user types !add, it adds them to the array. When the second user does !add, the array does not add the second user and keeps the first user in the array.

The way it should work is user 1 and user 2 are both in the array.

const pugSize = 6; // Maximum amount of players in the PUG
const pugMembers = []; // Array to hold the members in the PUG

function checkPugSize(){
    if (pugMembers.length == 6){
        //TODO Create the two teams
        console.log(`PUG IS FULL: ${pugMembers.length}`);
    }else{
        console.log(`THE PUG IS NOT FULL: ${pugMembers.length}`);
    }
}

function addUserPug(msg){
    // console.log(msg.author);
    // Add user to the pugMembers Array if the array is not full
    if (pugMembers<=6){
        pugMembers.push(msg.author.username);
    }else{ // Create a new pug and pass the user into the array
        console.log("TODO: Create a new pug when current array is filled");
        // createNewPug(msg.author.username);
    }
    msg.channel.send(`${msg.author} added to queue ${pugMembers.length}/6.`); // Mention the user that they are added into the queue
    // msg.reply(' added to queue. ' + `${pugMembers.length}/6`);
    msg.delete()
    .then(msg => console.log(pugMembers))
    .catch(console.error);
}

client.on('ready', () => {
    console.log(`Logged in as ${client.user.tag}!`);
});

client.on('message', msg => {
    if (msg.content == '!size'){
        msg.channel.send(`Current PUG size: ${pugMembers.length}`);
    }
    if (msg.content === '!add'){
        // console.log(msg.author);
        checkPugSize();
        addUserPug(msg);
    }
});

CollinD · Accepted Answer · 2018-12-20 20:29:23Z

1

Use

pugMembers.length <= 6

Your current statement is undefined in terms of behaviour.

edited Dec 20, 2018 at 20:29

CollinD

7,6032 gold badges25 silver badges47 bronze badges

answered Dec 20, 2018 at 20:28

OHHH

1,0614 gold badges16 silver badges36 bronze badges

Sign up to request clarification or add additional context in comments.

1 Comment

user2683183 Over a year ago

I knew it was something stupid. Works now! Ill give you the answer once 5 mins pass :D

Collectives™ on Stack Overflow

Add user to array

1 Answer 1

1 Comment

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

1 Comment

Your Answer

Sign up or log in

Post as a guest

Related