Using the following code, I am trying to find and get a pointer for the target unit of a spell. Unfortunately, I find the code to be highly unstable in game: either the incorrect unit will receive the HP bonus (or none at all) or it will outright crash with the puppet->mainOrderId = 0 operation (it used to be commented out, in which case it never crashed, but no unit would receive the HP bonus unless I specifically targetted a preplaced Reaver).

I'm pretty sure the problem lies somewhere in the addresses and variable types. The method I am trying to use is to find the address of unit->orderTarget (the caster's target) and match it with puppet's address. I assume that these addresses would be the same. However, I am not certain. That is the theoretical basis of my current solution.

I am fully aware that I can just flag the target unit with a specific timer number by salvaging another value. However, if I wanted to use that solution, I would not have asked for help to either fix my current method's code or look for an entirely different solution.



EDIT:

Ugh, I could have just compared puppet's coordinates and match it with orderTarget's coordinates. Problem with this is, what if two units have the same coordinates? Sure, I can check their owners and unit IDs, but what if these are the same too?

However, if any fix can be applied to the code above, it would still be nice to know what went wrong. (Or if there is another completely different approach, that would be welcome too.)

Any suggestions must be able to find the index of the target unit in the unitTable. This, specifically, is what I am looking for, so that I can take the index and apply it to my own struct array.

A fix for the above code is still the most preferred.

Post has been edited 1 time(s), last time on Aug 9 2012, 11:00 am by Sand Wraith.

Aha, never mind, I am a genius after all.

As it turns out, it was working perfectly fine. Just because of other complications, however, it appeared like the problem was here.