[Trigger] Chat Message + Selected Unit Event

[DroneFx]

Hi, I need help with a trigger. I've tried searching the forums to no avail as I don't know what the search term would be. What the trigger does is that when a unit is selected and the player types a message, the 'Actions' will trigger. But having 2 events won't work as only one is needed to trigger the Actions. How do you make it so that when you select a unit and type a message like '-sethealth 1000', it will set the selected unit's maximum health to 1000? Thanks.

 

[Just_Spectating]

I got this much for you, but i cant seem to find how to set the max hp to the value, however it can set the selected unit to any life below its max hp.

• Select
  • Events
    • Player - Player 1 (Red) types a chat message containing -sethealth as A substring
  • Conditions
  • Actions
    • For each (Integer A) from 1 to (Length of (Entered chat string)), do (Actions)
      • Loop - Actions
        • Set String = (Substring((Entered chat string), 11, (Integer((Entered chat string)))))
        • Set Integer = (Real(String))
    • Unit Group - Pick every unit in (Units currently selected by (Triggering player)) and do (Actions)
      • Loop - Actions
        • Unit - Set life of (Picked unit) to Integer

 

[Dr Super Good]

That trigger leaks a unit group.

 

[Rui]

Pst, DroneFx, haven't you introduced yourself to the Community yet? Check out our
Introduction Forum

 

[DroneFx]

@Just Spectating, thanks I will use that for the time being.

@Rui, I will get on to that.

 

[Bobo_The_Kodo]

• For each (Integer A) from 1 to (Length of (Entered chat string)), do (Actions)
• Loop - Actions
• Set String = (Substring((Entered chat string), 11, (Integer((Entered chat string)))))
• Set Integer = (Real(String))
• Unit Group - Pick every unit in (Units currently selected by (Triggering player)) and do (Actions)
• Loop - Actions
• Unit - Set life of (Picked unit) to Integer

You can just replace that with

• Unit Group - Pick every unit in (Units currently selected by (Triggering player)) and do (Actions)
• Loop - Actions
• Unit - Set life of (Picked unit) to Real( Substring((Entered chat string), 12, (Length of (Entered chat string))

 

[PurplePoot]

Set Integer = (Real(String))

I lol'd - your var naming could use some improvement

 

[DroneFx]

@Bobo_The_Kodo: Heh, that works too.

 

[Dr Super Good]

It still leaks though.

 

[edge[d1]]

and for multiple units, it repeats too much code.
but if he wants to avoid as much variables as he can and expects one unit to be selected, it's fine. just a bad coding practice...

 

[PurplePoot]

GUI is a bad coding practice, so that's a bit of a moot point.

 

[DroneFx]

Well since I don't seem to be able to set max hp, I will probaly assume it requires Jass, which I won't do in the near future. Thanks all for the help.

 

[PurplePoot]

Well since I don't seem to be able to set max hp, I will probaly assume it requires Jass, which I won't do in the near future. Thanks all for the help.

SetUnitMaxState by Blade.dk. It's Jass, but you can call it via a Custom Script action if you want to stick with GUI.