Cheat Engine

[bknight2602]

[quote="AylinCE"]

[AylinCE]

The error persists because even if you define the sequence, there are two critical issues that commonly trigger a 'nil value' in HoMM V scripting when using triggers.

1. The Argument Pass Issue When you use Trigger(PLAYER_ADD_HERO_TRIGGER, PLAYER_1, "AddSkills");, the game engine automatically passes the name of the hero being hired as an argument to that function.
If your function is defined without a parameter, it can't handle the incoming data.

2. Case Sensitivity & Scope Lua is strictly case-sensitive.
If there is even a single letter difference between the trigger string "AddSkills" and the function AddSkills(), it will return nil.

Recommended Fix
You should restructure your added functions like this (place them above the triggers):

[bknight2602]

[AylinCE]

The issue you're describing with GetPlayerNames is a classic 'race condition' in game scripting.
It’s not necessarily a quirk of how the code is interpreted, but rather when it is interpreted.

Here is why it happens intermittently:

Initialization Timing:
When a map first loads, the script starts executing immediately.
Sometimes, the script reaches the GetPlayerNames line before the game engine has fully initialized the player slots or retrieved the network data (especially in multiplayer or maps with complex AI setups).
If the engine isn't 'ready,' it returns a nil or an unexpected number of arguments.

Memory State: When you restart the map, the game engine is already 'warm' and the data is often already cached in memory, which is why a restart usually fixes it.

How to make it more stable:

Instead of calling it globally at the very top of the script, try wrapping it in a small delay or calling it inside a specific initialization function.

For example:

[bknight2602]

I will try moving the GetPlayerHeroes() further down and implementing your "delay"
There is a issue that I think I see, when a hero is hired GetPlayerHeroes will add the new hero to the table and length will change also, but if GetPlayerHeroes isn't rerun it will be wrong, correct?

[AylinCE]

You hit the nail on the head! You are absolutely correct:
GetPlayerHeroes() returns a static table (a snapshot) of the heroes present at the exact moment the function is called.

If a player hires a new hero 5 minutes later, your playernames or playerheroes table will not automatically update. It doesn't have a "live link" to the game engine's memory; it’s just a copy of what existed at Time X.

To keep your script accurate, you have two main options:

1. The "Polling" Method (Simple but slightly heavier)
Instead of running the check once at the start, you run your logic inside a recurring trigger (e.g., every day or every few turns).