[Trigger] Tips Text Help

indomitable1319 · Apr 29, 2010

In certain games I have played, they have -tipson and -tipsoff commands. But how do they make the text 'loop' over and over again?

Tips
Events
- Player - Player 1 (Red) types -tips as an exact match
Conditions
Actions
- What trigger?

MortAr · Apr 29, 2010

Attachment shows how to make it.

On initialiazation tips are ON by default.
Type -tipsoff to turn them off or -tipson to turn them on.

indomitable1319 · Apr 29, 2010

You're the dude that was cursing on HW Chat...
Anyway, testing it now. Thanks.

Agh... Too much for me. Nevermind, I'll just use something that I planned if I couldn't do it.

Bribe · Apr 29, 2010

setup
- Events
  - Map Initialization
- Conditions
- Actions
  - Set Tip[0] = "Don't be a fool, stay in school."
  - Set Tip[1] = "Don't do drugs, idiot."

Looping Trig
- Events
  - Elapsed Time - Every 15.00 seconds of game time
- Conditions
  - ShowTips Equal To True
- Actions
  - Game - Text Message to All Players: Tip[Math - Random Integer Number between 0 and 1]

Off/on trig
- Events
  - -------- Put what event you want to disable/enable it --------
- Conditions
- Actions
  - Set ShowTips = False/True

tml616 · Apr 29, 2010

you could slightly improve Bribe's system by turning off the periodic trigger when you toggle off, and turn on the trigger when toggle on. But this could make it spam if you spam the toggle.

Bribe · Apr 29, 2010

A time difference of five nanoseconds every 15 seconds means nothing to a GUI user.

MortAr · Apr 29, 2010

You can improve it, using my map, and combine the on/off triggers into a single one, with a substring and just checking which command is typed.

indomitable1319 · Apr 30, 2010

Hmm.. Okay.

Cokemonkey11 · May 1, 2010

You can improve it, by writing it in vJass

JASS:

scope text initializer i
    globals
        private constant real TIMEOUT=15
        private constant integer HIGHBOUND=2
        private string array message
        private boolean bool=false
    endglobals
    
    private function p takes nothing returns nothing
        if bool then
            call DisplayTextToPlayer(GetLocalPlayer(),0,0,message[GetRandomInt(0,HIGHBOUND)])
        else
            call PauseTimer(time)
        endif
    endfunction
    
    private function c takes nothing returns boolean
        if bool then
            set bool=false
            call DisplayTextToPlayer(GetLocalPlayer(),0,0,"Tips turned off")
        else
            set bool=true
            call TimerStart(time,TIMEOUT,true,function p)
            call DisplayTextToPlayer(GetLocalPlayer(),0,0,"Tips turned on")
        endif
        return false
    endfunction
    
    private function i takes nothing returns nothing
        local trigger t=CreateTrigger()
        set message[0]="first message"
        set message[1]="another message"
        set message[2]="yet a third message."
        call TriggerRegisterPlayerChatEvent(t,Player(0),"-tips",true)
        call TriggerAddCondition(t,Condition(function c))
    endfunction
endscope

Bribe · May 1, 2010

You can improve your JASS, by using actions instead of conditions, like this:

JASS:

    private function c takes nothing returns nothing
        if bool then
            set bool = false
            call DisplayTextToPlayer(GetLocalPlayer(),0,0,"Tips turned off")
        else
            set bool = true
            call TimerStart(time,TIMEOUT,true,function p)
            call DisplayTextToPlayer(GetLocalPlayer(),0,0,"Tips turned on")
        endif
    endfunction
    
    private function i takes nothing returns nothing
        local trigger t = CreateTrigger()
        call TriggerRegisterPlayerChatEvent(t,Player(0),"-tips",true)
        call TriggerAddAction(t,function c)
        set message[0] = "first message"
        set message[1] = "another message"
        set message[2] = "yet a third message."
    endfunction

Cokemonkey11 · May 1, 2010

Actually no, that's terrible and not an improvement at all.

Edit: let me explain why since I'm sure you're not the only person that doesn't know.

Conditions are handled just like actions in this sense, except they have a few distinct advantages and disadvantages from actions.

First, Conditions CANNOT have TSA within them--it breaks the thread.
Second, Conditions can be destroyed within a dynamic trigger while actions cannot (the result is a non-leaking dynamic trigger if you choose to create one)
Third, because the majority of triggers have both a condition and an action, it's wiser to put the action within the condition than the condition within the action for speed purposes.

Bribe · May 1, 2010

Actually, yes, that's a great idea and a great improvement over your method.

Actions don't need two handles to be assigned to them. And if you're using both conditions and actions in the same trigger, you've used 3 handles instead of just 1. Conditions take 2? Yes:

JASS:

call TriggerAddCondition(t,Condition(function SomeCondition))
                          //^--- Here it generates a handle.
native TriggerAddCondition takes trigger whichTrigger, boolexpr condition returns triggercondition
                                                    //here it generates another handle --^

Also, unless you are somehow able to track both generated handles created by making a triggercondition, you've just leaked more than the triggeraction would have.

TriggerSleepAction - how about you just avoid using it altogether. That's absolutely no reason to needlessly allocate extra handles just because of useless TSA.

Using Condition as an action allows you to quickly pass parameters to that action. Passing parameters is faster than declaring/nulling local handles:

JASS:

function Action takes unit TrigUnit,unit Target,unit dummy returns nothing
 
if (condition) then
    call Action(GetTriggerUnit(),GetSpellTargetUnit(),null)
endif

Cokemonkey11 · May 2, 2010

What you say makes sense, but I think you're misunderstanding what I meant with TSA. I agree it's useless.

I have a question, if TriggerAddCondition() makes 2 handles, then why shouldn't you use TriggerAddAction() the same way I did, ignoring conditions entirely?

Edit: to clarify, I put the actions within my condition function, but if action functions are better, why shouldn't people put conditions within an actions function?

PurplePoot · May 2, 2010

Think of a condition like call foo() while an action is like call ExecuteFunc("foo").

Not a perfect analogy but it should hint at why conditions are generally better if all you care about is efficiency.

Bribe · May 2, 2010

All I changed is made function c return nothing instead of a boolean and swap TriggerAddCondition for TriggerAddAction.

What I wrote should, in terms of syntax, look like this:

JASS:

function Action takes unit TrigUnit,unit Target,unit dummy returns nothing
    //actions
endfunction
 
function Conditions takes nothing returns nothing
    if (GetSpellAbilityId() == 'A000') then
        call Action(GetTriggerUnit(),GetSpellTargetUnit(),null)
    endif
endfunction
 
function InitTrig takes nothing returns nothing
    local trigger t = CreateTrigger()
    call TriggerRegisterAnyUnitEventBJ(t,EVENT_PLAYER_UNIT_SPELL_EFFECT)
    call TriggerAddAction(t,function Conditions)
endfunction

Bribe · May 2, 2010

PurplePoot said:
Think of a condition like call foo() while an action is like call ExecuteFunc("foo").

Not a perfect analogy but it should hint at why conditions are generally better if all you care about is efficiency.

I don't see how creating an entire boolean expression is an efficient thing to do when 99% of triggers would do better to replace conditions for actions. Unless native event-triggered triggers execute conditions faster than they execute actions, the best plan is to allocate less RAM by generating Actions instead.

PurplePoot · May 2, 2010

Because an "entire boolean expression" is essentially a function pointer, and conditions remain in the same thread as their parent while actions create a new one.

The time difference is minimal, but creating a thread costs time while using an existing boolexpr costs less.

Bribe · May 2, 2010

When the speed difference comes to a fraction of a nanosecond, allocating a smaller amount RAM is more important.

PurplePoot · May 2, 2010

When the RAM difference is a few bytes, your code looking good is more important than either.

Also, believe it or not, all these little optimizations add up. While they don't matter for anything static, when you are using dynamic triggers as function pointers (and stuff like that) in complicated systems it can matter to actually use conditions, hence why .evaluate() is used much more often than .execute() in vJass.

Bribe · May 2, 2010

I don't have use for either one because I don't care for those calls, but what you say makes sense if that's the way you want to play those triggers. In most cases I've seen on this forum, that's not the best way to go. As for the first note you made, the code looking good? What do you mean?

PurplePoot · May 2, 2010

I mean I would rather have both an action and a condition for standard, non-speed-dependent triggers, because it flows nicer and is easier to organize, especially if you have a lot of locals.

Bribe · May 2, 2010

Then it's a matter of preference for you as it is for me. I like doing a "call actions" from within my condition trigger so that I can just pass some parameters right there because parameters don't need null. I'd rather add a little horizontal space than make a lot of vertical space, plus parameters are faster than declaring/nulling locals in scripting time.

indomitable1319 · May 2, 2010

JASS... To bad I don't know anything about it. Well, thanks for the help (or at least tried to) but I didn't use anything you guys gave me... :sad:

[Trigger] Tips Text Help

Attachments

Similar threads