[Trigger] Spawning same item!

[Dominic74]

Heey im trying to make a chest throw down a random item from a group table i make, but i drops the same item every time??
Any suggestions on what it is?

This is the Items that should be randomly dropped

Glove Items Table
    Events
        Map initialization
    Conditions
    Actions
        Set Glove_Items[0] = |cFFC000C0Nightslayer Gloves|r
        Set Glove_Items[1] = |cffc000c0Acherus Knight´s Gloves|r
        Set Glove_Items[2] = |cFFC000C0Lawbringer Gauntlets|r
        Set Glove_Items[3] = |cFFC000C0Gauntlets of Might|r
        Set Glove_Items[4] = |cFFC000C0Felheart Gloves|r
        Set Glove_Items[5] = |cFFC000C0Earthfury Gauntlets|r
        Set Glove_Items[6] = |cFFC000C0Cenarion Gloves|r
        Set Glove_Items[7] = |cFFBB00DDGloves of Prophecy|r
        Set Glove_Items[8] = |cFFBB00DDArcanist Gloves|r
        Set Glove_Items[9] = |cff3464e0Adventurers Gloves|r
        Set Glove_Items[10] = |cff3464e0Adventurers Gloves|r

This is the trigger to drop the random item in the selected table.

Open Chest
    Events
        Unit - A unit Is attacked
    Conditions
        (Unit-type of (Attacked unit)) Equal to Chest
    Actions
        If (All Conditions are True) then do (Then Actions) else do (Else Actions)
            If - Conditions
                ((Attacked unit) belongs to an enemy of (Owner of (Attacking unit))) Equal to True
            Then - Actions
                Animation - Play (Attacked unit)'s stand - 2 animation
                Unit - Change ownership of (Attacked unit) to Neutral Passive and Change color
                Unit - Remove Chest Mark Showing (+1) from (Attacked unit)
                Item - Create Glove_Items[(Random integer number between 0 and 10)] at ((Position of (Attacking unit)) offset by (100.00, 0.00))
                Animation - Reset (Attacked unit)'s animation
            Else - Actions

-Thanks in advance!

 

[Hotwer]

Use the

[trigger][/trigger]

bbtags instead of the code ones.

Set this random integer into a variable and make it show to all players to see if you're getting random numbers correctly

 

[Tasyen]

If you use the "test map"-Button from World Editor.
There is an option in World Editor - file - Preference - Map Testing - use fixed Random.
Make sure it is unchecked.

 

[MyPad]

There is an option in World Editor - file - Preference - Map Testing - use fixed Random.

I actually did not know that option exists...

Optimization

To do:
(Note: variables encased in <> are arbitrary)

Generate a global point variable:
<point_var>
Generate two global unit variables:
<unit_var1>
<unit_var2>

• Custom Script: local location loc2
• Set <unit_var1> = (Attacking Unit)
• Set <unit_var2> = (Triggering unit)
• If (All Conditions are True) then do (Then Actions) else do (Else Actions)
  • If - Conditions
    • ((Attacked unit) belongs to an enemy of (Owner of (Attacking unit))) Equal to True
  • Then - Actions
    • Set <point_var> = (Position of <unit_var1>)
    • Animation - Play <unit_var2>'s stand - 2 animation
    • Unit - Change ownership of <unit_var2> to Neutral Passive and Change color
    • Unit - Remove Chest Mark Showing (+1) from <unit_var2>
    • Custom Script: set loc2 = udg_<point_var>
    • Custom Script: set udg_<point_var> = Location(GetLocationX(loc2) + 100, GetLocationY(loc2))
    • Item - Create Glove_Items[(Random integer number between 0 and 10)] at <point_var>)
    • Animation - Queue <unit_var2>'s animation
    • Custom Script: call RemoveLocation(udg_<point_var>)
    • Custom Script: call RemoveLocation(loc2)
    • Custom Script: set loc2 = null
  • Else - Actions

 

[Dominic74]

I actually did not know that option exists...

Optimization

To do:
(Note: variables encased in <> are arbitrary)

Generate a global point variable:
<point_var>
Generate two global unit variables:
<unit_var1>
<unit_var2>

• Custom Script: local location loc2
• Set <unit_var1> = (Attacking Unit)
• Set <unit_var2> = (Triggering unit)
• If (All Conditions are True) then do (Then Actions) else do (Else Actions)
  • If - Conditions
    • ((Attacked unit) belongs to an enemy of (Owner of (Attacking unit))) Equal to True
  • Then - Actions
    • Set <point_var> = (Position of <unit_var1>)
    • Animation - Play <unit_var2>'s stand - 2 animation
    • Unit - Change ownership of <unit_var2> to Neutral Passive and Change color
    • Unit - Remove Chest Mark Showing (+1) from <unit_var2>
    • Custom Script: set loc2 = udg_<point_var>
    • Custom Script: set udg_<point_var> = Location(GetLocationX(loc2) + 100, GetLocationY(loc2))
    • Item - Create Glove_Items[(Random integer number between 0 and 10)] at <point_var>)
    • Animation - Queue <unit_var2>'s animation
    • Custom Script: call RemoveLocation(udg_<point_var>)
    • Custom Script: call RemoveLocation(loc2)
    • Custom Script: set loc2 = null
  • Else - Actions

Why is it better to put attacking and attacked unit in a variable?

 

[MyPad]

It saves function calls, which can be quite costly in comparison to reading a variable.