Converting Process() To Events

Ed · 06-28-2009, 08:29 PM

So how do you need to do it? I have Eventmanager on my source but never used it

Do I change ALL of those

Code:

        if (npcDelay > 0) {
            npcDelay--;
        }
        if (teleDelay > 0) {
            teleDelay--;
        }
(..) etc

to this:

Code:

		public void statRestore(final Player p) {
        EventManager.getSingleton().addEvent(3000, new Event() { 
	    public void execute(EventContainer c) {        
		if (statRestoreDelay > 0) {
            statRestoreDelay--;
        } else {
		    c.stop();
		}
	   }
	});
    }

but just changing the bolded stuff to the previous codes? like this:

Code:

public void isDead(final Player p) {
        EventManager.getSingleton().addEvent(3000, new Event() { 
	    public void execute(EventContainer c) {        
        if (isDead) {
            deathDelay--;
            applyDead();
} else {
                        c.stop();
	   }
	});
    }
public void npc(final Player p) {
        EventManager.getSingleton().addEvent(3000, new Event() { 
	    public void execute(EventContainer c) {        
        if (npcDelay > 0) {
            npcDelay--;
} else {
                        c.stop();
	   }
	});
}

etc etc for all this if's? or what

**SilentZPrO** · 06-28-2009, 08:32 PM

Lol no! You change the void to what that process is rofl!

**Rowan** · 06-28-2009, 08:36 PM

kul, knew this, can you replace some of the code of your tut with this?:

public void statRestore(final Player p) {
EventManager.getSingleton().addEvent(2400, new Event() {
public void execute(EventContainer c) {
if (statRestoreDelay > 0)
statRestoreDelay--;
else
c.stop();
}
});
}

just conventions

Annexation · 06-28-2009, 08:44 PM

Originally Posted by Rowan

kul, knew this, can you replace some of the code of your tut with this?:

public void statRestore(final Player p) {
EventManager.getSingleton().addEvent(2400, new Event() {
public void execute(EventContainer c) {
if (statRestoreDelay > 0)
statRestoreDelay--;
else
c.stop();
}
});
}

just conventions

To bad you don't see spacing without tags, lol

and yea your teaching it wrong, listen to william

**Rowan** · 06-28-2009, 08:54 PM

just did conventions that i knew for that piece of code

XAFBX · 06-28-2009, 10:44 PM

Or, just make an event to call your process method process it every 500ms, then get rid of the process method being called through the Engine class and start the event on start up, oh my not multiple events pls.

**Soulevoker** · 06-29-2009, 02:04 AM

Originally Posted by William.D

You're still doing it wrong, mate. Let's say you got this in your process:

Code:

if (timer > 0) {
timer--;
} if (timer == 0) {
doJob();
}

Let's say you would set the timer to 6, which means 6x500MS that it would be executed after 3000 MS (3 seconds). Why does this suck? Because you are calling it 5 times to much, just execute an event after 3000 MS. Like this;

Code:

public void statRestore(final Player p) {
        EventManager.getSingleton().addEvent(3000, new Event() { 
        public void execute(EventContainer c) {        
        doJob();
                        c.stop();
       }
    });
    }

I always thought it was like this:

Code:

public void statRestore(final Player p) {
EventManager.getSingleton().addEvent( new Event() {
                    public void execute(EventContainer c) {
                       code here
                        c.stop();
                    }
                }, 3000);
}

This works just the same right?

**pHametic** · 06-29-2009, 02:07 AM

Well after you create the method for the new event where do you call it? or does it automatically get called?

**'Mystic Flow** · 06-29-2009, 03:55 AM

Originally Posted by Goodoo Dolls

I always thought it was like this:

Code:

public void statRestore(final Player p) {
EventManager.getSingleton().addEvent( new Event() {
                    public void execute(EventContainer c) {
                       code here
                        c.stop();
                    }
                }, 3000);
}

This works just the same right?

Goodoo replace your tick method in EventManager.java with this..

Code:

	public synchronized void addEvent(int tick, Event event) {
		events.add(new EventContainer(event,tick));
		notify();
	}

Thats how Simon is explaining it

**Soulevoker** · 06-29-2009, 04:44 AM

Originally Posted by Mystic Flow

Goodoo replace your tick method in EventManager.java with this..

Code:

    public synchronized void addEvent(int tick, Event event) {
        events.add(new EventContainer(event,tick));
        notify();
    }

Thats how Simon is explaining it

I still don't understand the difference other than changing the place where you place the tick time in the event void

Thread: Converting Process() To Events

Thread Tools

Users Browsing this Thread

User Tag List