Whats wrong?

[medion555]

• city
  • Events
    • Time - Every 20.00 seconds of game time
  • Conditions
  • Actions
    • Set Taxincome = Small Town Hall
    • Unit Group - Pick every unit in (Units of type Small Town Hall) and do (Actions)
      • Loop - Actions
        • Unit Group - Pick every unit in (Units within 512.00 of (Position of (Picked unit))) and 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 Farm
              • Then - Actions
                • Player - Add 1 to (Owner of (Triggering unit)) Current gold
              • Else - Actions

It should pick all units of unit type farm in a 512 range of the unit type small town hall

It doesnt work

 

[ACdestinydream]

what did u set the taxincome for?
edit: oh yah the farm should be the one triggering the income if u set it that way... its wrong lol
do as wad did the post below me says. hes correct (i think)

 

[ED_Reborn]

I see the problem.

The reason for it not working is that in your if-then-else condition you use unit- type of triggering unit. There is no triggering unit.

I would change the second unit group to units within range matching condition and then use the (unit-type of (matching unit) = Farm) condition.

 

[medion555]

Negative i got this now

• city
  • Events
    • Time - Every 1.00 seconds of game time
  • Conditions
  • Actions
    • Unit Group - Pick every unit in (Units of type Small Town Hall) and do (Actions)
      • Loop - Actions
        • Unit Group - Pick every unit in (Units within 512.00 of (Position of (Picked unit)) matching ((Unit-type of (Picked unit)) Equal to Farm)) and do (Actions)
          • Loop - Actions
            • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
              • If - Conditions
                • (Unit-type of (Picked unit)) Equal to Farm
              • Then - Actions
                • Player - Add 1 to (Owner of (Picked unit)) Current gold
              • Else - Actions

 

[M0RT]

You can't put two "Pick units in group" inside each other, that doesn't work.
Also, the condition (Unit-type of (Picked unit)) equal to farm in IF/then/else is unnecessary as you already test it in the Unit group parameters. Also, you leak unit group.

 

[ED_Reborn]

Remove the whole if-then-else, including the condition. I think the problem is that you are referring to the unit as "picked unit", since there are two pick unit loops.

Edit: 2 minutes... I guess M0RT is right, you can't put two unit group loops within each other.

 

[ED_Reborn]

Here is a (or rather: three) triggers you can use instead. I haven't tested but they seem right. It should be noted that they leak unit groups.

Triggers

This initializes the hashtable

• Untitled Trigger 002
  • Events
    • Map initialization
  • Conditions
  • Actions
    • Hashtable - Create a hashtable
    • Set hashVar = (Last created hashtable)

This one saves the number of farms within range of all town halls to the hashtable.

• Untitled Trigger 001
  • Events
    • Time - Every 1.00 seconds of game time
  • Conditions
  • Actions
    • Unit Group - Pick every unit in (Units in (Playable map area) matching ((Unit-type of (Matching unit)) Equal to Town Hall)) and do (Actions)
      • Loop - Actions
        • Hashtable - Save (Number of units in (Units within 512.00 of (Position of (Picked unit)) matching ((Unit-type of (Matching unit)) Equal to Farm))) as 0 of (Key (Picked unit)) in hashVar

This one gives the player gold for every farm.

• Untitled Trigger 003
  • Events
    • Time - Every 20.00 seconds of game time
  • Conditions
  • Actions
    • Unit Group - Pick every unit in (Units in (Playable map area) matching ((Unit-type of (Matching unit)) Equal to Town Hall)) and do (Actions)
      • Loop - Actions
        • Player - Add (Load 0 of (Key (Picked unit)) from hashVar) to (Owner of (Picked unit)) Current gold

 

[Maker]

Here's the thing in one trigger, and this does not leak and it's MPI.

Trigger

• Untitled Trigger 075
  • Events
    • Time - Every 20.00 seconds of game time
  • Conditions
  • Actions
    • For each (Integer A) from 1 to 12, do (Actions)
      • Loop - Actions
        • Set Temp_Group_1 = (Units owned by (Player((Integer A))) of type Town Hall)
        • Unit Group - Pick every unit in Temp_Group_1 and do (Actions)
          • Loop - Actions
            • Set Temp_Loc_1 = (Position of (Picked unit))
            • Set Temp_Group_2 = (Units within 512.00 of Temp_Loc_1 matching (((Owner of (Matching unit)) Equal to (Player((Integer A)))) and ((((Matching unit) is alive) Equal to True) and ((Unit-type of (Matching unit)) Equal to Farm))))
            • Player - Add (Number of units in Temp_Group_2) to (Player((Integer A))) Current gold
            • Custom script: call RemoveLocation(udg_Temp_Loc_1)
            • Custom script: call DestroyGroup(udg_Temp_Group_2)
        • Custom script: call DestroyGroup(udg_Temp_Group_1)

 

[medion555]

• Set Temp_Group_2 = (Units within 512.00 of Temp_Loc_1 matching ((Owner of (Matching unit)) Equal to (Player((Integer A)))))

Trying now

The set tempgroup is a bit hard i only got to this

 

[Maker]

There's AND almost at the bottom of the list, use that to add more conditions.

 

[medion555]

thx i will try k i couldnt find the and .... are u using WEU or some ?

 

[Maker]

Select "units in range matching condition".
Click the "condition".
You get a list of comparisons.
At the bottom of the list, there's AND and OR. Use the AND.

 

[medion555]

Ok i want to be able to give 5 gold for every unit of unit type farm in unit_group_2 now it gives only the farms x 1

 

[Maker]

Use "arithmetic" and then <number of units in group> * 5.

 

[FearFigmen]

Unit Group - Pick every unit in (Units within 512.00 of (Position of (Picked unit)) matching ((Unit-type of (Picked unit)) Equal to Farm)) and do (Actions)

matching ((Unit-type of (Picked unit)) Equal to Farm)) <---- Change this to

matching ((Unit-type of (MATCHING unit)) Equal to Farm)) and it should work

 

[Azeroc]

Now I really don't understand why you guys need whole hashtable system for this, just put first unit group's picked unit in variable and set (triggering unit) to (picked unit), like this.

• city
• Events
  • Time - Every 20.00 seconds of game time
• Conditions
• Actions
• Set Taxincome = Small Town Hall
  • Unit Group - Pick every unit in (Units of type Small Town Hall) and do (Actions)
    • Loop - Actions
      • Set TempUnit = (Picked Unit)
  • Unit Group - Pick every unit in (Units within 512.00 of (Position of (TempUnit))) and do (Actions)
    • Loop - Actions
      • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        • If - Conditions
          • (Unit-type of (Picked Unit)) Equal to Farm
        • Then - Actions
          • Player - Add 1 to (Owner of (Picked Unit)) Current gold
        • Else - Actions

Also put Unit Groups in variables and remove em at the end, else you might experience fps drop or lags later.

 

[Dr Super Good]

Instead of doing unescescary enuming, just enum them once into a group and then for through that whole group all the time.

Basically On unit death you remove the farm from the farm group if the type is farm (of dieing unit). If the type is a town, then you enum all farms in range of it and then check for each one of them if there is a living town in range of them (if not remove them from the farm group).
On unit construction completion, if the type is a farm, you check if it is in range of a town, and if so add it to the farm group. If the type is a town you check all farms in the area are in the farm group, and if not add them to it.
On build start you check if the farm is in range of a town, if not you immediatly abort its construction and give an error message (why on earth should you be able to build usless farms?).

Ontop of the group of farms (used for checking which farms are counted), you have a counter which is the farm number (how many farms are working at any time).
Finally, every X seconds you simply add the farm number worth of gold to the player. Yes really it is that simple after the system above is added.

As you can see, my recommended solution has high processing load only when new buildings are made or destroyed. The only possible time it has a chance of lagging is when a town is destroyed near a lot of farms but this is not likely. However at any time it will be a hell of a lot more efficent than the perodic enum solution you are trying above by incomparable ammounts. My solution should also be highly scaleable unlike yours meaning the load at any time for any opperation of the system remains approximatly constant weather one or hundreds of units are in the system. By a few hundred your system should start to cause visible FPS reduction if not have the chance to crash due to the opt limit.

 

[medion555]

My map will hold a max of 50 of them

so maker his system works just fine

Thx anyway