[GoldNux]

[CSTRIKE] Non-player entity 0 out of range [AMXX] Run time error 10: native error (native "cs_set_weapon_ammo") [AMXX] [0] gn_retake.sma::playerSpawned (line 127)

public playerSpawned(id)
{
    if (cs_get_user_team(id) == CS_TEAM_T)
    {
        cs_set_weapon_ammo(find_ent_by_owner(-1,"weapon_ak47",id),30)
        cs_set_weapon_ammo(find_ent_by_owner(-1,"weapon_glock18",id),20)
        cs_set_weapon_ammo(find_ent_by_owner(-1,"weapon_usp",id),12)
    }
}

This function does what I want it to do but I get this error and I would like to know why.
I'm not sure if the weapon entity index is set incorrectly or if the issue is that noone has a specific weapon.

Thanks!

[Bugsy]

http://www.amxmodx.org/api/engine/find_ent_by_owner

[CSTRIKE] Non-player entity 0 out of range
[AMXX] Run time error 10: native error (native "cs_set_weapon_ammo")
[AMXX] [0] gn_retake.sma::playerSpawned (line 127)

Return
Entity index if an entity was found, 0 otherwise 

[GoldNux]

Originally Posted by Bugsy http://www.amxmodx.org/api/engine/find_ent_by_owner [CSTRIKE] Non-player entity 0 out of range [AMXX] Run time error 10: native error (native "cs_set_weapon_ammo") [AMXX] [0] gn_retake.sma::playerSpawned (line 127) Return Entity index if an entity was found, 0 otherwise

Ahh I see, so this can be ignored. Thank you sir.

[HamletEagle]

Originally Posted by GoldNux Ahh I see, so this can be ignored. Thank you sir.

You need to check if this entities exist before doing stuff with them, so no, don't ignore the error, fix the code.

[CrazY.]

Honestly I have not seen anything informing to ignore this error on the site that Bugsy given to you...

[GoldNux]

Originally Posted by CrazY. Honestly I have not seen anything informing to ignore this error on the site that Bugsy given to you...

Well, I am far from a programmer and just starting out.
My plugin works without any issues, and I simply thought that it would return 0 if it didnt find any weapon entity.
There was not much further explanation, I still don't know how to properly check if an entity exists..

Something like this?

PHP Code:

            new ak47Ent;
            new glock18Ent;
            new uspEnt;
            ak47Ent = find_ent_by_owner(-1, "weapon_ak47", id);
            glock18Ent = find_ent_by_owner(-1, "weapon_glock18", id);
            uspEnt = find_ent_by_owner(-1, "weapon_usp", id);
            if(pev_valid(ak47Ent))
            {
                cs_set_weapon_ammo(find_ent_by_owner(-1,"weapon_ak47",id),30)
            }
            if(pev_valid(glock18Ent))
            {
                cs_set_weapon_ammo(find_ent_by_owner(-1,"weapon_glock18",id),20)
            }
            if(pev_valid(uspEnt))
            {
                cs_set_weapon_ammo(find_ent_by_owner(-1,"weapon_usp",id),12)
            } 


[CrazY.]

That's right.

[GoldNux]

Originally Posted by CrazY. That's right.

Thanks!

[Bugsy]

You should not call find_ent_by_owner() redundantly like that. You are calling it once and storing the value in a variable, so you should reuse that in cs_set_weapon_ammo().

PHP Code:

    new ak47Ent = find_ent_by_owner(-1, "weapon_ak47", id);
    new glock18Ent = find_ent_by_owner(-1, "weapon_glock18", id);
    new uspEnt = find_ent_by_owner(-1, "weapon_usp", id);
    
    if(pev_valid(ak47Ent))
    {
        cs_set_weapon_ammo(ak47Ent,30)
    }
    if(pev_valid(glock18Ent))
    {
        cs_set_weapon_ammo(glock18Ent,20)
    }
    if(pev_valid(uspEnt))
    {
        cs_set_weapon_ammo(uspEnt,12)
    }