[Rudy_]

Hey everyone.
I need a bit of help, Here i'm detecting if team_hunter shoots team_prop's (Object) which is Prop[i]. But in-game if hunter shoots the object it dosen't kills prop :/ I have no idea what's wrong

pawn Код:

public OnPlayerWeaponShot(playerid, weaponid, hittype, hitid, Float:fX, Float:fY, Float:fZ)
{
    foreach(new ii: Player)
    if(GetPlayerTeam(ii) == team_hunter)
    {
        if(hittype == BULLET_HIT_TYPE_PLAYER_OBJECT)
        {
            foreach(new i:Player)
            {
                if(GetPlayerTeam(ii) != team_prop) continue;
                if(hitid == Prop[i])
                {
                    SetPlayerHealth(i, 0); 
                    SendClientMessage(i, -1, "Test");
                    break; 
                }
            }
        }
    }
    return 1;
}

I've also tried this.

pawn Код:

public OnPlayerWeaponShot(playerid, weaponid, hittype, hitid, Float:fX, Float:fY, Float:fZ)
{
    switch (hittype)
    {
        case BULLET_HIT_TYPE_PLAYER_OBJECT:
        {
            foreach(new i:Player)
            foreach(new ii: Player)
            {
                if(GetPlayerTeam(i) == team_hunter) continue;
                if(GetPlayerTeam(ii) == team_prop) continue;
                if(hitid == Prop[ii])
                {
                    SetPlayerHealth(ii, 0);
                    SendClientMessage(ii, -1, "You have been killed");
                    break;
                }
            }
        }
    }
    return 1;
}

[Stinged]

Try this.
(Also, are you sure Prop[playerid] is an object id?)

pawn Код:

public OnPlayerWeaponShot(playerid, weaponid, hittype, hitid, Float:fX, Float:fY, Float:fZ)
{
    if(GetPlayerTeam(playerid) == team_hunter) // No need for a loop, as playerid is in OnPlayerWeaponShot
    {
        if(hittype == BULLET_HIT_TYPE_PLAYER_OBJECT)
        {
            foreach(new i : Player)
            {
                if(hitid == Prop[i]) // Are you sure Prop[MAX_PLAYERS] (Prop[i]) is assigned to an object?
                {
                    SetPlayerHealth(i, 0);
                    SendClientMessage(i, -1, "Test");
                    break;
                }
            }
        }
    }
    return 1;
}

[ranme15]

BULLET_HIT_TYPE_PLAYER_OBJECT -> BULLET_HIT_TYPE_OBJECT

[Rudy_]

Yes, Prop[i] is attached to the player.
Under onplayerdeath i've added

pawn Код:

DestroyObject(Prop[i]);

it destroys the attached object

Plus: The script above didn't worked.. :/

Edit:

Quote:

Originally Posted by ranme15 BULLET_HIT_TYPE_PLAYER_OBJECT -> BULLET_HIT_TYPE_OBJECT

It's a player object

[Eth]

try this

pawn Код:

public OnPlayerWeaponShot(playerid, weaponid, hittype, hitid, Float:fX, Float:fY, Float:fZ)
{
    foreach(new ii: Player)
    if(GetPlayerTeam(ii) == team_hunter)
    {
        if(hittype == BULLET_HIT_TYPE_PLAYER_OBJECT)
        {
            foreach(new i:Player)
            {
                if(GetPlayerTeam(ii) != team_prop) continue;
                if(hitid == Prop[i])
                {
                     for(new i2=0; i<MAX_PLAYER_ATTACHED_OBJECTS; i2++)
                    {
                    if(IsPlayerAttachedObjectSlotUsed(playerid, i2)) RemovePlayerAttachedObject(playerid, i2);
                    }
                    SetPlayerHealth(i, 0); 
                    SendClientMessage(i, -1, "Test");
                    break; 
                }
            }
        }
    }
    return 1;
}

you have to un attach the object b4 destroying it

[IceCube!]

Rudy_

Remember you are duplicating the object to allow it to be seen outside of the players normal VW? If your still using the method we discussed. This would mean however that you would need to detect damage to a duplicated object (normal object) on the enemies screen.

[Rudy_]

Quote:

Originally Posted by IceCube! Rudy_ Remember you are duplicating the object to allow it to be seen outside of the players normal VW? If your still using the method we discussed. This would mean however that you would need to detect damage to a duplicated object (normal object) on the enemies screen.

Right, But atm i haven't added the duplicate, it's visible for everyone atm (for testing), so it should have worked :/

I tried this but didn't worked

pawn Код:

public OnPlayerWeaponShot(playerid, weaponid, hittype, hitid, Float:fX, Float:fY, Float:fZ)
{
    if(GetPlayerTeam(playerid) == team_hunter) // No need for a loop, as playerid is in OnPlayerWeaponShot
    {
        if(hittype == BULLET_HIT_TYPE_PLAYER_OBJECT)
        {
            foreach(new i : Player)
            {
                if(hitid == Prop[i]) // Are you sure Prop[MAX_PLAYERS] (Prop[i]) is assigned to an object?
                {
                    for(new i2=0; i<MAX_PLAYER_ATTACHED_OBJECTS; i2++)
                    {
                        DestroyObject(Prop[i2]);
                        SetPlayerHealth(i, 0);
                        SendClientMessage(i, -1, "Test");
                        break;
                    }
                }
            }
        }
    }
    return 1;
}

[IceCube!]

Quote:

Originally Posted by Stinged Try this. (Also, are you sure Prop[playerid] is an object id?) pawn Код: public OnPlayerWeaponShot(playerid, weaponid, hittype, hitid, Float:fX, Float:fY, Float:fZ) {     if(GetPlayerTeam(playerid) == team_hunter) // No need for a loop, as playerid is in OnPlayerWeaponShot     {         if(hittype == BULLET_HIT_TYPE_PLAYER_OBJECT)         {             foreach(new i : Player)             {                 if(hitid == Prop[i]) // Are you sure Prop[MAX_PLAYERS] (Prop[i]) is assigned to an object?                 {                     SetPlayerHealth(i, 0);                     SendClientMessage(i, -1, "Test");                     break;                 }             }         }     }     return 1; }

How didn't stinged code work? As it should kill the player as needed, but not destroy the object which is the only fault?

Does "Test" appear, or?

Stinged's code is also more efficent, have you tried debugging Stinged's code or even yours?

[Eth]

Quote:

Originally Posted by IceCube! How didn't stinged code work? As it should kill the player as needed, but not destroy the object which is the only fault? Does "Test" appear, or? Stinged's code is also more efficent, have you tried debugging Stinged's code or even yours?

^
this
also the question is, did he attach the object or not?

[Rudy_]

Quote:

Originally Posted by IceCube! How didn't stinged code work? As it should kill the player as needed, but not destroy the object which is the only fault? Does "Test" appear, or? Stinged's code is also more efficent, have you tried debugging Stinged's code or even yours?

No, it didn't appear the msg.. and didn't killed the player i shot with attached object.

Quote:

Originally Posted by Eth ^ this also the question is, did he attach the object or not?

Yes it was attached.