Zugg Software :: View topic

Posted: Wed Oct 08, 2003 7:21 pm

I am trying to use a the following trigger to capture my text, then create a record for each of my skills. The trigger is as follows.

#CLASS {Skills}
#VARIABLE EndLine {>}
#TRIGGER "SkillCatch" {show skills} {#SHOW hi}
##CONDITION {} {#IF (%line =~ "(*):%s(*).") {
#SHOW "%1"
#SHOW "%2"
#ADDKEY "%1" {CL="%2"|NL=0|Synched=true}
} {#IF (%line =~ "~>") {#SET 2}}} {looplines|param=30}CONDITION {>} {#SHOW Goodbye}
#CLASS 0

The italicized condition is the focus of my question, and the bolded portion is the code snippet I'm having problems with. The text it should match is as follows

MASTER Joined: 28 Nov 2000 Posts: 4817 Location: USA

It might be typos.
##CONDITION
{looplines|param=30}CONDITION

This seems a little simpler.
#CLASS {Skills}
#VARIABLE EndLine {>}
#TR SkillCatchOff {^~>} {#T- SkillCatchOff;#STATE Skillcatch 0;#SHOW Goodbye}
#TRIGGER "SkillCatch" {show skills} {#T+ SkillCatchOff;#SHOW hi}
#CONDITION {(*):%s(*).} {
#SHOW "%1"
#SHOW "%2"
#ADDKEY {%replace( "%1", " ", "_")} {CL="%2"|NL=0|Synched=true}
} {looplines|param=99}
#CLASS 0

NOTES:
1. * doesn't match #, so language skills will be in @catfolk instead of @language#catfolk, etc.
2. Since you do two #SHOWs after each match, you triple the number of lines. I increased the Loopline param to 99 to compensate.
3. Spaces aren't really desirable in variable names. I used %replace to get rid of them.

Posted: Thu Oct 09, 2003 6:48 am

The ## is a typo, and the SHOW %1 and SHOW %2 were just to make sure I wasn't going crazy because I didn't see those values.

I'm going to try what you said though.

Thanks for the help.

Posted: Thu Oct 09, 2003 7:26 am

Thanks that did the trick.

I'm re-learning zMud scripting and I've forgotten alot. :-D Dang Java!

Posted: Thu Oct 09, 2003 8:49 pm

I'm trying to take the script to the next level now. I want to check if the variables exist already. To do this I'm using an if statement, but it doesn't seem to be working. Any advice?

The show statement is just to keep track of what actually gets executed. The bolded if statement always evaluates to false. Is this the best way to check for the existence of a variable when you get the variable name from another variable?

Basically what I'm trying to say is

SubAdmin Joined: 18 Nov 2001 Posts: 5182

Try using %defined(@_temp). Currently an existing variable with null value would fail the test, why it always fails I can't tell. You might want to use a #SHOW, #ECHO, or even #INPUT to give you an idea why, most likely cause is that _temp does not have the right value.

Posted: Thu Oct 09, 2003 11:28 pm

That didn't work either. @_temp is defined. It's the name of a variable for example, if @temp = sword then the variable I want is @sword.

For some reason the expression evaluation @{@_temp}}) doesn't quite go the way it should, even with %define(@{@_temp})

I found a workaround though that seemed to do the trick.

MASTER Joined: 28 Nov 2000 Posts: 4817 Location: USA

#ADDKEY updates the existing variable if one by that name exists, and creates one otherwise. Since you use the same command in either case, do you really need to check whether one exists?

This snippet seems to work. I eliminated a couple spaces from ( @{@_temp} ) and added one between it and the {.

#IF (@{@_temp}) {
#SHOW variable exists
#IF (%2 = %db( @{@_temp}, CL)) {#SHOW variable should be update}
} {
#ADDKEY @_temp {Skill=%1|CL="%2"|NL=0|Synched=true}
}

If @_temp equals "sword" and @sword equals stonecutter
%defined( _temp) shows whether @_temp exists
%defined( @_temp) shows whether @sword exists
%defined( @{@_temp}) shows whether @stonecutter exists
An alias, macro, or path of that name (_temp, sword, stonecutter) would give the same result for each test.

It's usually best to keep related questions to a single thread.