[boynedmaster]

I have this snippet of code.

if(headshotProtect[victimId]){
		if(damagecustom == TF_CUSTOM_HEADSHOT){
			damage = floatModulus(damage, 90.0);
			PrintToServer("%f", damage);
			changed = true;
		}
	}
	
	return changed ? Plugin_Changed : Plugin_Continue;

I have a bunch of other if statements that change the damage, and they all work just fine.
floatModulus is this function:

public float floatModulus(float a, float b){
	while(a > b)
		a -= b;
	
	return a;
}

How can I fix this? I've tried putting + 0.0, - 0.0, and * 1.0 at the end of the float, but none work.
The PrintToServer function tells me the damage I want it to be, but it doesn't actually change the damage.

[Powerlord]

Just to make sure, this function looks something like this, right?

PHP Code:

public Action OnTakeDamage(int victim, int &attacker, int &inflictor, float &damage, int &damagetype, int &weapon,
        float damageForce[3], float damagePosition[3], int damagecustom)
{
    bool changed;

    if(headshotProtect[victimId]){
        if(damagecustom == TF_CUSTOM_HEADSHOT){
            damage = floatModulus(damage, 90.0);
            PrintToServer("%f", damage);
            changed = true;
        }
    }
    
    return changed ? Plugin_Changed : Plugin_Continue;
} 


With the hook being created like this?

PHP Code:

public void OnClientPutInServer(int client)
{
    SDKHook(client, SDKHook_OnTakeDamage, OnTakeDamage);
} 


[boynedmaster]

Originally Posted by Powerlord Just to make sure, this function looks something like this, right? PHP Code: public Action OnTakeDamage(int victim, int &attacker, int &inflictor, float &damage, int &damagetype, int &weapon,         float damageForce[3], float damagePosition[3], int damagecustom) {     bool changed;     if(headshotProtect[victimId]){         if(damagecustom == TF_CUSTOM_HEADSHOT){             damage = floatModulus(damage, 90.0);             PrintToServer("%f", damage);             changed = true;         }     }          return changed ? Plugin_Changed : Plugin_Continue; }  With the hook being created like this? PHP Code: public void OnClientPutInServer(int client) {     SDKHook(client, SDKHook_OnTakeDamage, OnTakeDamage); }

Yep, except it's:

PHP Code:

bool changed = false; 


[boynedmaster]

Also victimId is this:

PHP Code:

int victimId = GetEventInt(event, "userid");
int attackerId = GetEventInt(event, "attacker"); 


But like I said, other if statements that change the damage work just fine with these.

EDIT:
I copied and pasted from the wrong function, lol.

PHP Code:

int attackerId = GetClientUserId(attacker);
int victimId = GetClientUserId(victim); 


[pcmaster]

If you are using victimid like that, you need to convert it to a client index first (otherwise your array access won't work properly).

[boynedmaster]

Originally Posted by pcmaster If you are using victimid like that, you need to convert it to a client index first (otherwise your array access won't work properly).

What? It's working perfectly fine. Everything works except the damage doesn't change. I get the PrintToServer that's said right after damage is set, and other if statements using victimId work 100% as intended.

[boynedmaster]

Originally Posted by pcmaster If you are using victimid like that, you need to convert it to a client index first (otherwise your array access won't work properly).

Oh, I copied and pasted from the wrong code. I edited the post.

[WildCard65]

If your using OnTakeDamage from SDKHooks, you don't need to do GetClientOfUserId as SDKHooks OnTakeDamage passes a client index.

[boynedmaster]

Originally Posted by WildCard65 If your using OnTakeDamage from SDKHooks, you don't need to do GetClientOfUserId as SDKHooks OnTakeDamage passes a client index.

Oh

[boynedmaster]

Originally Posted by WildCard65 If your using OnTakeDamage from SDKHooks, you don't need to do GetClientOfUserId as SDKHooks OnTakeDamage passes a client index.

Tested this. It's wrong.