Compact IF Statement need help!

[Bin4ry]

mes "[Emperium Manager]";
mes "Global Prize Distribution:";
mes (($EmpBrk_PrzDG == 1)?"Enabled":"^FF0000Disabled^000000");
mes "Personal Prize Distribution:";
mes (($EmpBrk_PrzDP == 1)?"Enabled":"^FF0000Disabled^000000");
next;
switch(select("Toggle Global Prize Distribution:Toggle Personal Prize Distribution:Return")) {
case 1:
if ($EmpBrk_PrzDG == 1) set $EmpBrk_PrzDG, 0; goto L_TPD;
if ($EmpBrk_PrzDG == 0) set $EmpBrk_PrzDG, 1; goto L_TPD;

case 2:
if ($EmpBrk_PrzDP == 1) set $EmpBrk_PrzDP, 0; goto L_TPD;
if ($EmpBrk_PrzDP == 0) set $EmpBrk_PrzDP, 1; goto L_TPD;

case 3:
goto L_GM;
}
}

So here's the problem, once I enabled I can't disable it anymore. Not sure where I did wrong, please help!

[Euphy]

Could the global variable not be updating fast enough? That's all I can think of - you can test by switching to an NPC variable (.var) and if it works, that's the problem.

Edit: woops.

[Bin4ry]

 mes "[Emperium Manager]";
mes "Global Prize Distribution:";
mes ((.EmpBrk_PrzDG == 1)?"Enabled":"^FF0000Disabled^000000");
mes "Personal Prize Distribution:";
mes ((.EmpBrk_PrzDP == 1)?"Enabled":"^FF0000Disabled^000000");
next;
switch(select("Toggle Global Prize Distribution:Toggle Personal Prize Distribution:Return")) {
case 1:
if (.EmpBrk_PrzDG == 1) set .EmpBrk_PrzDG, 0; goto L_TPD;
if (.EmpBrk_PrzDG == 0) set .EmpBrk_PrzDG, 1; goto L_TPD;

case 2:
if (.EmpBrk_PrzDP == 1) set .EmpBrk_PrzDP, 0; goto L_TPD;
if (.EmpBrk_PrzDP == 0) set .EmpBrk_PrzDP, 1; goto L_TPD;

case 3:
goto L_GM;
}
}

Like this? Will show Disabled all time.

[KeyWorld]

It missing {} each time so it go directly on the goto.

Should be:

if ($EmpBrk_PrzDG == 1) { set $EmpBrk_PrzDG, 0; goto L_TPD; }
if ($EmpBrk_PrzDG == 0) { set $EmpBrk_PrzDG, 1; goto L_TPD; }

Or:

if ($EmpBrk_PrzDG == 1) set $EmpBrk_PrzDG, 0;
else set $EmpBrk_PrzDG, 1;
goto L_TPD;

Or even:

set $EmpBrk_PrzDG, !$EmpBrk_PrzDG; goto L_TPD;

[Bin4ry]

Hmm now it's working. Thanks both of you. and would you suggest using .var or $var in this case? Like an event NPC setting.

[Euphy]

Ah, missed that. Thanks, Keyworld! xD

would you suggest using .var or $var in this case? Like an event NPC setting.

NPC variables reset each time you reload scripts or the server restarts. If that doesn't matter, use NPC variables. If you need the values to be persistent, use global.

[Emistry]

no matter what variable you use...you still can access it....with this..

getvariableofnpc()

or other method..if there is any.....