Integer A Value

Felipe Gormadoc · May 4, 2013

i make one loop, 1 to 5, but when i create an action for display value of Integer A, it shows 6, WHY? :vw_wtf:

deathismyfriend · May 4, 2013

do u show it in the loop or after ? if its after it will display 6

and plz if its trigger related post the triggers

Felipe Gormadoc · May 4, 2013

I show in the Loop

look the trigger:

For each (Integer A) from 1 to 5, do (Actions)
- Loop - Actions
  - Set MEQS_Integer[0] = (Integer A)
  - Set MEQS_Integer[1] = (Load (19 + MEQS_Integer[0]) of MEQS_InfoQN from MEQS_Hashtable)
  - Custom script: set udg_MEQS_UnitType[1] = udg_MEQS_Integer[1]
  - If (All Conditions are True) then do (Then Actions) else do (Else Actions)
    - If - Conditions
      - MEQS_UnitType[1] Equal to MEQS_UnitType[0]
    - Then - Actions
      - Hashtable - Save ((Load (16 + MEQS_Integer[0]) of MEQS_QuestNumber from MEQS_Hashtable) + 1) as (16 + MEQS_Integer[0]) of MEQS_QuestNumber in MEQS_Hashtable
      - Trigger - Run sTRING <gen> (checking conditions)
      - Game - Display to (All players) the text: (String((Integer A)))
    - Else - Actions

deathismyfriend · May 4, 2013

u show it in the ITE. so the values have to be equal to show it. also u should not use integer A instead make ur own integer and use that. to display integer properly take it out of the ITE.

Felipe Gormadoc · May 4, 2013

what is it? ITE?

deathismyfriend · May 4, 2013

Felipe Gormadoc said:
what is it? ITE?

if then else

Felipe Gormadoc · May 4, 2013

but this has no logic, what problem be within of ITE?

I put out, more problem still remains the same:

If (All Conditions are True) then do (Then Actions) else do (Else Actions)
- If - Conditions
  - MEQS_UnitType[1] Equal to MEQS_UnitType[0]
- Then - Actions
  - Hashtable - Save ((Load (16 + MEQS_Integer[0]) of MEQS_QuestNumber from MEQS_Hashtable) + 1) as (16 + MEQS_Integer[0]) of MEQS_QuestNumber in MEQS_Hashtable
  - Trigger - Run sTRING <gen> (checking conditions)
- Else - Actions
Game - Display to (All players) the text: (String((Integer A)))

deathismyfriend · May 4, 2013

if its in the ITE then it only fires when the condition is true.

Felipe Gormadoc · May 4, 2013

but he should display the current number of Loop, but he displays a 6, even the loop having 5 as the maximum. I really do not know what it make :goblin_cry:

deathismyfriend · May 4, 2013

did u try moving the message and displaying the number out of the ITE block ? what does it display when u do that ?

Felipe Gormadoc · May 4, 2013

display 1,2,3,4,5 (OBS: I switched Integer A for Integer B)

I need more just a number exact, it would have to operate within THEN ACTIONS of ITE, but when he is there, it displays (1,2), right would only 1

For each (Integer B) from 1 to 5, do (Actions)
- Loop - Actions
  - Set MEQS_Integer[0] = (Integer B)
  - Set MEQS_Integer[1] = (Load (19 + MEQS_Integer[0]) of MEQS_InfoQN from MEQS_Hashtable)
  - Custom script: set udg_MEQS_UnitType[1] = udg_MEQS_Integer[1]
  - If (All Conditions are True) then do (Then Actions) else do (Else Actions)
    - If - Conditions
      - MEQS_UnitType[1] Equal to MEQS_UnitType[0]
    - Then - Actions
      - Hashtable - Save ((Load (16 + MEQS_Integer[0]) of MEQS_QuestNumber from MEQS_Hashtable) + 1) as (16 + MEQS_Integer[0]) of MEQS_QuestNumber in MEQS_Hashtable
      - Trigger - Run sTRING <gen> (checking conditions)
      - Game - Display to (All players) the text: (String((Integer B)))
    - Else - Actions

deathismyfriend · May 4, 2013

Don't use integer b either I said create ur own integer and use that. Plz elaborate more I have no idea what ur tlking about.

Felipe Gormadoc · May 4, 2013

Ok, exist:
5 counter
5 Strings
5 Unit Type

I need to know, which drive who was killed, to add +1 for your counter and display the string of this counter.

for example: I kill Unit Type 3, then +1 for Counter 3 and display String 3.

Look:

Untitled Trigger 001
- Events
  - Unit - A unit Dies
- Conditions
- Actions
  - Set MEQS_Unit[0] = (Killing unit)
  - Set MEQS_Player = (Owner of MEQS_Unit[0])
  - Set MEQS_UnitType[0] = (Unit-type of (Triggering unit))
  - For each (Integer MEQS_LOOP) from 1 to MEQS_Quest_Index, do (Actions)
    - Loop - Actions
      - Set MEQS_QuestNumber = ((Player number of MEQS_Player) x (1000 + MEQS_LOOP))
      - Set MEQS_InfoQN = MEQS_LOOP
      - If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        
        If - Conditions
        
        (Load 1 of MEQS_QuestNumber from MEQS_Hashtable) Equal to 1
        
        Then - Actions
        
        For each (Integer B) from 1 to 5, do (Actions)
        
        Loop - Actions
        
        Set MEQS_Integer[0] = (Integer B)
        
        Set MEQS_Integer[1] = (Load (19 + MEQS_Integer[0]) of MEQS_InfoQN from MEQS_Hashtable)
        
        Custom script: set udg_MEQS_UnitType[1] = udg_MEQS_Integer[1]
        
        If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        
        If - Conditions
        
        MEQS_UnitType[1] Equal to MEQS_UnitType[0]
        
        Then - Actions
        
        Hashtable - Save ((Load (16 + MEQS_Integer[0]) of MEQS_QuestNumber from MEQS_Hashtable) + 1) as (16 + MEQS_Integer[0]) of MEQS_QuestNumber in MEQS_Hashtable
        
        Trigger - Run sTRING <gen> (checking conditions)
        
        Game - Display to (All players) the text: (String((Integer B)))
        
        Else - Actions
        
        Else - Actions

defskull · May 4, 2013

So, Unit Type 1, Counter 1, Message 1, right ?

Why don't you do it like this;

Initialize
- Events
  - Map initialization
- Conditions
- Actions
  - -------- Index 1 --------
  - Set UnitType[1] = Footman
  - Set Message[1] = Footman has died.
  - -------- /////////////// --------
  - -------- .................. --------
  - -------- .................. --------
  - -------- Index n --------
  - Set UnitType[5] = Knight
  - Set Message[5] = Knight has died.
  - -------- /////////////// --------
  - -------- Total Index --------
  - Set TotalIndex = 5

Unit Dies
- Events
  - Unit - A unit Dies
- Conditions
- Actions
  - For each (Integer LoopingInteger) from 1 to TotalIndex, do (Actions)
    - Loop - Actions
      - If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        
        If - Conditions
        
        (Unit-type of (Triggering unit)) Equal to UnitType[LoopingInteger]
        
        Then - Actions
        
        Set Counter[LoopingInteger] = (Counter[LoopingInteger] + 1)
        
        Set LoopingInteger = TotalIndex
        
        Else - Actions

As said by others, don't use IntegerA/B, use your custom-variable, namely LoopingInteger - For Each Integer Variable

Maker · May 4, 2013

Does the sTRING trigger use integer A or integer B?

They are global variables so their values can be altered in other triggers.

deathismyfriend · May 4, 2013

defskull said:
So, Unit Type 1, Counter 1, Message 1, right ?

Why don't you do it like this;

Initialize

Events

Map initialization

Conditions

Actions

-------- Index 1 --------

Set UnitType[1] = Footman

Set Message[1] = Footman has died.

-------- /////////////// --------

-------- .................. --------

-------- .................. --------

-------- Index n --------

Set UnitType[5] = Knight

Set Message[5] = Knight has died.

-------- /////////////// --------

-------- Total Index --------

Set TotalIndex = 5

Unit Dies

Events

Unit - A unit Dies

Conditions

Actions

For each (Integer LoopingInteger) from 1 to TotalIndex, do (Actions)

Loop - Actions

If (All Conditions are True) then do (Then Actions) else do (Else Actions)

If - Conditions

(Unit-type of (Triggering unit)) Equal to UnitType[LoopingInteger]

Then - Actions

Set Counter[LoopingInteger] = (Counter[LoopingInteger] + 1)

Set LoopingInteger = TotalIndex

Else - Actions

As said by others, don't use IntegerA/B, use your custom-variable, namely LoopingInteger - For Each Integer Variable

if i understand what ur trying to do then do it this way. only thing to note is try not to use to many of the unit type of unit as they leak and cant be fixed.

defskull · May 5, 2013

as they leak

Explain.

deathismyfriend · May 5, 2013

defskull said:
Explain.

im guessing this is for me ?

try doing this in ur map

unit enters region event
condition
actions

for each integer from 1 to 100
. if unit type of trigger unit = no unit // make sure this is false to see the problem
. then
. else

then use a handle counter and watch it rapidly rise.

Dr Super Good · May 6, 2013

It cannot leaks because it maps to integer globals. These neither allocate or deallocate anything when used.

JASS:

integer bj_forLoopAIndex=0
integer bj_forLoopAIndexEnd=0
integer bj_forLoopBIndex=0
integer bj_forLoopBIndexEnd=0

deathismyfriend · May 6, 2013

Dr Super Good said:
It cannot leaks because it maps to integer globals. These neither allocate or deallocate anything when used.

JASS:

integer bj_forLoopAIndex=0 integer bj_forLoopAIndexEnd=0 integer bj_forLoopBIndex=0 integer bj_forLoopBIndexEnd=0

ik what they are converted into thats what stumped me.
nvm my bad i found a bug in the triggers. i thought it was the unit type id but it was the group trying to limit units by type.

Integer A Value

Similar threads