[EasyUO] Find the mistake

[Luna]

Deleted. Some people here are really not worth it. Sorry to all the nice and good people of Excelsior

[Stephen]

Why not just set a pause for whatever the timer you set it for? Then it will say it. pause that time. then repeat.


So like

Code: Select all

Set %waittime 3600s

while #TRUE
{
    event macro 1 0 [c PS's, Bee-Friendly Plants, House Add-Ons, Thorns, Deco, Golden Skulls, Parasitic Plants And More @Luna's!
    wait %waittime
  }

I mean way easier imho. or even just

Code: Select all

while #TRUE
{
    event macro 1 0 [c PS's, Bee-Friendly Plants, House Add-Ons, Thorns, Deco, Golden Skulls, Parasitic Plants And More @Luna's!
    wait 3600s
  }

Just seems easier since it will wait the timer either way.



Also your

Code: Select all

wait 20 ; wait 1 second

20 milliseconds is all that is waiting there. 1000 would be 1 second or set it to just 1s for 1 second.

[Luna]

Also your

Code: Select all

wait 20 ; wait 1 second

20 milliseconds is all that is waiting there. 1000 would be 1 second or set it to just 1s for 1 second.

EasyUO is 1/20th second, please refer to the documentation for more details

[Stephen]

You are right. I was wrong on it. I am used to using Python for scripting UO scripts....Therefore I overlooked it. Doesn't change the fact you keep adding unnecessary code that causes your problems.

Id fix your problems but.....I am sure you figured it out already

[Luna]

Of course, you can see the new script here when I posted it to allow people to follow the new April 2018 rules for advertisements.

[Yoda]

....wanders by notices conversation....

why are we talking about automated ad scripts at all? they are lame, make the chat lame, and are a lame thing to code, use, or otherwise.. having said that, your code is entirely too complicated for what it does.. it is like a rube goldberg machine..

https://www.youtube.com/watch?v=B17OvPYM040

easyuo is terrible at being consistent at tick speed, meaning its timers are unreliable trust me on this I was coding an internal display system and was trying to time the vertical retrace, which was impossible because easy uo is a scripting language not a functional programming language.

I wish there was a market channel for this sort of stuff but honestly.. I can see your problems in that script there is more than one, and no I won`t help you spot the error because much like an autohunter the real fix for this script is obvious...

Delete it...

[Stephen]

Delete! Lol

[Stephen]

Of course, you can see the new script here when I posted it to allow people to follow the new April 2018 rules for advertisements.

The April 2018 Rules for advertisement the new part was not giving unsolicited pms. The 60 minutes 12 lines ha been the rule for years now. Just an FYI.

[Luna]

Deleted