GooBot Help Thread

Gooberg · 2015-07-18 22:53:12

This is my first bot, and I'm not much of a "tutorial" kind of guy, so I decided instead of creating alternate threads for each issue, how about just use 1 thread.

#1 - My commands don't work. IDK why. Does not give errors, when someone says .help,

My code:

List<eeplayer> playerList = new List<eeplayer>();

public void handlemsg(object sender, PlayerIOClient.Message m)
{
if (m.Type == "init")
{
say("Connected");
}
if (m.Type == "add")
{
playerList.Add(new eeplayer() { userID = m.GetInt(0), username = m.GetString(1) });
}
if (m.Type == "left")
{
foreach (eeplayer p in playerList)
{
if (p.userID == m.GetInt(0))
{
playerList.Remove(p);
}
}
}
if (m.Type == "say")
{
if (m.GetString(1) == ".help")
{
foreach (eeplayer p in playerList)
{
con.Send("say", "/pm " + p.username + "'.afk' '.unafk'");

if (m.GetString(1) == ".unafk")
{
{
if (p.userID == m.GetInt(0))
{
con.Send("say", "/kill " + p.username);
}
}
if (m.GetString(1) == ".afk")
{
if (p.userID == m.GetInt(0))
{
con.Send("say", "/godon " + p.username);
}
}
}
}
}
}
}
}
}

Jabatheblob1 · 2015-07-18 22:57:42

So i don't understand, are you trying to make a help command? Because right now you have the curly braces in the wrong places, tell me what commands you want, because idk what is in the middle and what is going on with the foreach loop

Era · 2015-07-18 22:59:43

1. if the string is already .help it cannot be .afk or .unafk , they have to be seperate , you could uses switch (m.getstring(1)) for example

edit: you arent sending init2 !! this meens the server won't send you the "add" messages ;
send "init2" in the "init" section

Gooberg · 2015-07-18 23:25:23

Era, thank you for this. I thought that was the issue, but my friend ninjasupeatsninja denied that it was that issue earlier today. (Screw you ninja) I hope it works!

JabatheBlob1: Yes I am trying to create a help function that lists the player's permissions, an afk function that enables godmode on the player and an unafk function that kills the player.

ewoke · 2015-07-18 23:50:52

 if (m.Type == "say")
            {
                if (m.GetString(1) == ".help")//someone says help
                {
                    foreach (eeplayer p in playerList)
                    {
                      con.Send("say", "/pm " + p.username + "'.afk' '.unafk'");//all players get spammed with help command
                        
                        if (m.GetString(1) == ".unafk")//stuff that doesnt really belong in this if statement
                        {//stuff that doesnt really belong in this if statement
                            {//stuff that doesnt really belong in this if statement
                                if (p.userID == m.GetInt(0))//stuff that doesnt really belong in this if statement
                                {//stuff that doesnt really belong in this if statement
                                    con.Send("say", "/kill " + p.username);//stuff that doesnt really belong in this if statement
                                }//stuff that doesnt really belong in this if statement
                            }//stuff that doesnt really belong in this if statement
                            if (m.GetString(1) == ".afk")//stuff that doesnt really belong in this if statement
                            {//stuff that doesnt really belong in this if statement
                                if (p.userID == m.GetInt(0))//stuff that doesnt really belong in this if statement
                                {//stuff that doesnt really belong in this if statement
                                    con.Send("say", "/godon " + p.username);//stuff that doesnt really belong in this if statement
                                }//stuff that doesnt really belong in this if statement
                            } //stuff that doesnt really belong in this if statement
                        }//stuff that doesnt really belong in this if statement
                    }
                }

  
if (m.GetString(1) == ".help")//someone says help
                {
                    foreach (eeplayer p in playerList)
                    {
                              if (p.userID == m.GetInt(0))
                                {
                                    con.Send("say", "/godon " + p.username);
                                }
                    }
                }
                if (m.GetString(1) == ".help")//someone says help
                {
                 foreach (eeplayer p in playerList)
                    {
                              if (p.userID == m.GetInt(0))
                                {
                                    con.Send("say", "/pm " + p.username + "'.afk' '.unafk'");
                                }
                    }
                 }

Gooberg · 2015-07-19 02:45:58

I see what was wrong the whole time. I took a break for a while.

The bot starts the list for players who enter and leave, I start the bot after I enter, so commands do not work for me, therefore, I cannot test them.

Xfrogman43 · 2015-07-19 06:57:33

It should get players who are in the level already, unless you are doing it wrong.

John · 2015-07-19 09:14:18

>Ewoke: Don't spoon-feed code. How will he learn? I didn't learn to code from copy and pasting from a forum. Next time give him a task description. If you are going to spoon-feed, you should possibly take a screenshot so he has to type it manually. (Which would help him learn.)

On Topic:
You should use a Dictionary to store your players, I find it useful and convenient.

Xfrogman43, ewoke, Era

Gooberg · 2015-07-20 05:57:20

Thanks everyone :3 I got it working, though now I am trying to find out how to have a bot command work like this for secret, hidden, or just non-spammy commands.

/pm gooberg123 [command variable here]

I have been reading up on it and watching tutorials and I have been getting help, I am aware I should use the write function in EE Information created by capasha.

Though I am not sure how to get this working.

Thanks again! ^^

Xfrogman43 · 2015-07-20 07:19:01

Yes, you would use case "write":

Here's how you figure out what to do:

call string(0) first, and see what it produces, name that something easy for you to know. Do the same thing with the string(1). Go on from there.

Gooberg · 2015-07-20 15:07:51

Xfrogman43 wrote:

Yes, you would use case "write":
Here's how you figure out what to do:
call string(0) first, and see what it produces, name that something easy for you to know. Do the same thing with the string(1). Go on from there.

Thank you, I will start trying to get this working.

ewoke · 2015-07-20 23:45:40

Emalton wrote:

>Ewoke: Don't spoon-feed code. How will he learn? I didn't learn to code from copy and pasting from a forum. Next time give him a task description. If you are going to spoon-feed, you should possibly take a screenshot so he has to type it manually. (Which would help him learn.)
On Topic:
You should use a Dictionary to store your players, I find it useful and convenient.

some people learn by doing, others learn by reading on how to do it. ill use the screenshot method next time

Official Everybody Edits Forums

#1 2015-07-18 22:53:12

GooBot Help Thread

#2 2015-07-18 22:57:42

Re: GooBot Help Thread

#3 2015-07-18 22:59:43, last edited by Era (2015-07-18 23:04:43)

Re: GooBot Help Thread

#4 2015-07-18 23:25:23

Re: GooBot Help Thread

#5 2015-07-18 23:50:52, last edited by ewoke (2015-07-18 23:52:43)

Re: GooBot Help Thread

#6 2015-07-19 02:45:58

Re: GooBot Help Thread

#7 2015-07-19 06:57:33

Re: GooBot Help Thread

#8 2015-07-19 09:14:18

Re: GooBot Help Thread

#9 2015-07-20 05:57:20

Re: GooBot Help Thread

#10 2015-07-20 07:19:01

Re: GooBot Help Thread

#11 2015-07-20 15:07:51

Re: GooBot Help Thread

#12 2015-07-20 23:45:40

Re: GooBot Help Thread

Board footer