Official Everybody Edits Forums

cercul1

Member

From: USA

Joined: 2015-02-18

Posts: 91

[Resolved] handling events without using invoke

Is there another way to do this code without using Invoke:

public void Onmessage(object s, PlayerIOClient.Message m)
        {
            Invoke(new EventHandler(delegate
            {
                switch (m.Type)
                {
                    #region Init
                    case "init":
                        {
                            con.Send("init2");
                            con.Send("say", "Hi!");
                        }
                        break;
                        #endregion
                }
            }));

        }

---

Gosha

Member

From: Russia

Joined: 2015-03-15

Posts: 6,211

Re: [Resolved] handling events without using invoke

why not just

public void Onmessage(object s, PlayerIOClient.Message m)
        {
              switch (m.Type)
                {
                    case "init":
                        {
                            cl.con.Send("init2");
                            cl.con.Send("say", "Hi!");
                        }
                        break;
                }

        }

cercul1

Member

From: USA

Joined: 2015-02-18

Posts: 91

Re: [Resolved] handling events without using invoke

This did not work, it won't even recognize any message coming in

---

MartenM

Member

From: The Netherlands

Joined: 2016-03-31

Posts: 978

Website

Re: [Resolved] handling events without using invoke

Did you do this?

con.OnMessage += new MessageReceivedEventHandler(messageHandler.handlemsg);

---

Ingame: marten22        My steam: MartenM

capasha

Member

Joined: 2015-02-21

Posts: 4,066

Re: [Resolved] handling events without using invoke

Is he trying to use client to send messages? Why is there cl.con.send?

cercul1

Member

From: USA

Joined: 2015-02-18

Posts: 91

Re: [Resolved] handling events without using invoke

Did you do this?

con.OnMessage += new MessageReceivedEventHandler(messageHandler.handlemsg);

yes i have

and

i've removed the cl, i had the event loader in a class before and had the class be a form but that didn't work, so i removed it sorry about that

---

Gosha

Member

From: Russia

Joined: 2015-03-15

Posts: 6,211

Re: [Resolved] handling events without using invoke

yea, that's weird.

First get a client

var Client = PlayerIO.QuickConnect.SimpleConnect({gameid}, {email}, {pass}, null);

then get a connection

var con = Client.Multiplayer.CreateJoinRoom({worldid},"Everybodyedits"+Client.BigDB.Load("config","config")["owner"], true, null, null)

Now set up message handler

you can do this in 2 ways.
1.

con.Send("init");
con.OnMessage += OnMessage;

public void OnMessage(object sender, Message m)
{
     if(m.Type == "init")
     {
         con.Send("init2");
     }
}

2.

con.Send("init");
con.OnMessage += (o,m) => 
{
      if(m.Type == "init")
     {
         con.Send("init2");
     }
}

To send messages use connection:

con.Send("init")

cercul1

Member

From: USA

Joined: 2015-02-18

Posts: 91

Re: [Resolved] handling events without using invoke

I'm running into a really confusing bug.  I'm not actually sure if it's connecting or not.  It returns the connected value as true, but he doesn't show up in the world until I refresh or open a new tab, my friend's list says he's still offline, any database i check says i'm offline.  I'm not really sure what's happening.

---

Gosha

Member

From: Russia

Joined: 2015-03-15

Posts: 6,211

Re: [Resolved] handling events without using invoke

Bots are offline becasue they don't connect to the lobby.

make sure you:
1) sent "init"
2) when recieved "init" you sent "init2"

or, even better, show your code. (make sure to hide email and password if it's hardcoded)

cercul1

Member

From: USA

Joined: 2015-02-18

Posts: 91

Re: [Resolved] handling events without using invoke

i ended up finding the mistake, i was sending "inut" instead of "init" lol i guess it's 5:15 AM and I still haven't gone to bed so that explains that, thank you so much for helping me with such a silly mistake.

---

LukeM

Member

From: England

Joined: 2016-06-03

Posts: 3,009

Website

Re: [Resolved] handling events without using invoke

Just to let you know, you may need to use invoke if you are using forms or timers or anything, as if you dont, you can run into threading problems. Using Invoke right at the start though is actually a clever way to fix this that I hadnt thought of before... :O

If you do want to use it though, I would advise doing this instead, as it looks better, and is probably a bit faster as you dont create events every time a message is recieved:

public void Onmessage(object s, PlayerIOClient.Message m)
{
    switch (m.Type)
    {
        case "init":
            {
                con.Send("init2");
                con.Send("say", "Hi!");
            }
            break;
    }
}

// Somewhere else
con.OnMessage += (s, m) => BeginInvoke((Action<object, Message>)OnMessage, s, m);

This way you are effectively changing a multithreaded program (one main thread and one thread for PlayerIO) into a single threaded program, so you should no longer get the usual problems associated with threading

hummerz5

Member

From: wait I'm not a secret mod huh

Joined: 2015-08-10

Posts: 5,853

Re: [Resolved] handling events without using invoke

This way you are effectively changing a multithreaded program (one main thread and one thread for PlayerIO) into a single threaded program, so you should no longer get the usual problems associated with threading

true, but then you get the usual problem with threading: putting your UI and work tasks on the same thread. While the way you suggest is easier and more manageable imo, I don't think it's the optimized course of action

---

thanks HG, HG, HG, HG! anatoly, zumza! zoey!

LukeM

Member

From: England

Joined: 2016-06-03

Posts: 3,009

Website

Re: [Resolved] handling events without using invoke

This way you are effectively changing a multithreaded program (one main thread and one thread for PlayerIO) into a single threaded program, so you should no longer get the usual problems associated with threading

true, but then you get the usual problem with threading: putting your UI and work tasks on the same thread. While the way you suggest is easier and more manageable imo, I don't think it's the optimized course of action

Well yes, but for most bots youre limited mostly by internet speed, not by time taken to actually do things. The only bot Ive made that was limited by processing time was Mars Mine, and it only happened when generating the world (a few seconds every hour or so), so didnt really matter

Edit: Just remembered there were also a few JavaScript things too, but... well... its JavaScript, so you cant really compare it to the speed of C# etc.