How to make a hero swapping spell between different players?

[Geshishouhu]

How to make this kind of spell? I have a problem with switching the player id of each player.
Thanks a lot.

 

[TriggerHappy]

How to make this kind of spell? I have a problem with switching the player id of each player.
Thanks a lot.

What do you mean switching the player id of each player?
This doesn't work?

• Actions
  • Unit - Change ownership of (Target unit of ability being cast) to (Owner of (Triggering unit)) and Change color
  • Unit - Change ownership of (Triggering unit) to (Owner of (Target unit of ability being cast)) and Change color

 

[Geshishouhu]

What do you mean switching the player id of each player?
This doesn't work?

• Actions
  • Unit - Change ownership of (Target unit of ability being cast) to (Owner of (Triggering unit)) and Change color
  • Unit - Change ownership of (Triggering unit) to (Owner of (Target unit of ability being cast)) and Change color

After the creation of each hero, I store it in a unit array. Moreover, all my spells are made based on that array, such as the damage of the spell. After the swapping, it seems that array is unchanged and all my triggered spells do not work.
I do not know how to fix this.....

 

[TriggerHappy]

After the creation of each hero, I store it in a unit array. Moreover, all my spells are made based on that array, such as the damage of the spell. After the swapping, it seems that array is unchanged and all my triggered spells do not work.
I do not know how to fix this.....

You can try something like this

• Actions
  • Set HeroArray[(Player Number of (Target unit of ability being cast))] = (Triggering unit)
  • Set HeroArray[(Player Number of (Triggering Unit))] = (Target unit of ability being cast)
  • Unit - Change ownership of (Target unit of ability being cast) to (Owner of (Triggering unit)) and Change color
  • Unit - Change ownership of (Triggering unit) to (Owner of (Target unit of ability being cast)) and Change color

You could also store (Triggering Unit) and (Target unit of ability being cast) in the beginning of the trigger to avoid so many function calls.

 

[Geshishouhu]

You can try something like this

• Actions
  • Set HeroArray[(Player Number of (Target unit of ability being cast))] = (Triggering unit)
  • Set HeroArray[(Player Number of (Triggering Unit))] = (Target unit of ability being cast)
  • Unit - Change ownership of (Target unit of ability being cast) to (Owner of (Triggering unit)) and Change color
  • Unit - Change ownership of (Triggering unit) to (Owner of (Target unit of ability being cast)) and Change color

You could also store (Triggering Unit) and (Target unit of ability being cast) in the beginning of the trigger to avoid so many function calls.

Thanks a lot.
Could you switch that trigger into context or Jass? I am using the following, which does not work:

 set udg_HeroOwner[GetUnitPointValue(udg_Swap_Hero[1])]=GetConvertedPlayerId(udg_Swap_Player[2])

I have been working on this all day -_-

Edit:
I think you are right, im working on it now...

 

[Vinz]

You use a point value as an index, and set it to Player Number of SwapPlayer[2]. What are you trying to do?

 

[pOke]

Can you post your trigger as to how you save the unit into an array?