egghelp.org community

[darton]

Hello!
Maybe it is just a stupid mistake, but I have made a script with global variables which don't work. Here is the script:

[nml375]

Well, what confuses me, if why you've put those two string equal-checks outside the proc...
_________________
NML_375, idling at #eggdrop@IrcNET

[darton]

because I have many procs in one tcl-file and I must use these variables in almost every proc.

[nml375]

What I mean is, do you update them at any time?
Would'nt it be easier to just make $nick global, and run the test inside the proc (won't increase load that much, probably won't even notice it)
_________________
NML_375, idling at #eggdrop@IrcNET

[darton]

Yes I update it at any time with "global bla blabla". But does that increase load? I think there is no difference if I write "if {$bla || $blabla}" or "if {[string equal -nocase $blabla $nick]}"

[nml375]

This is what I'd suggest... Assuming from your first post that you wish to check wether the bot is afk or not, and reply "<yournick> is AFK" if so...

[krimson]

@darton: the reason why the code in your first post won't work is because (probably) you didn't set $nick before setting $bla and $blabla. this is the reason why you should use both vars inside your procs

[nml375]

As I remember, $nick is a reserved variable, "always" set to whatever nick your bot currently uses..
Hav'nt dug into the source in a while, but I'd guess it's not set during startup..
I also get this distict feeling darton expects bla and blabla to be updated as soon as nick is changed... which is most definetly not the case, all "global" does is to link the local (function-instance) variable to the global one (ie making bla and blabla readable from within the function even tho they exist outside).

It feels like he actually wants to do some variable tracing (man n trace), although coding event-driven tcl isn't that pretty in eggies as I recall...
_________________
NML_375, idling at #eggdrop@IrcNET

[KrzychuG]

Can't you convert those global variables to procedures? I don't think you can use variable like you defined.

[krimson]

if you used an argument when declaring the procs, you must use it when calling them too

[KrzychuG]

[krimson]

that's no prob :P just wanted to make sure the info is recieved the right way ;)

[nml375]

Guess that's why one should never code when dead-tired (or quite intoxicated)
(learned that the hard way myself...)

Tho while we're at it, might aswell throw in something like this

[krimson]

here are some tips:

1. don't use global nick, you'd be better off using an argument for proc botaway instead

2. you don't have to use ${nick} there, you use that when you have any non-space char right after the var, and don't want it to get mixed up with something else

[nml375]

1. Depends on the purpose of the proc.. in this case, botaway would indicate it's this bot, hence requiring a nick to be supplied externally would be confusing. If the objective were to check a random nick, you'd be correct tho.
Considdering the ambiguos nature of darton's original code, I'd guess both are equally correct given it's interpretation of that code...

2. True. Just keep a habit of enclosing variable names with {} whenever it is adjacent to any non-whitespace character (Rather do it excessively, rather than forgetting it when really needed)
_________________
NML_375, idling at #eggdrop@IrcNET