[Bugsy]

Try this.

1. Minor update applied for get_user_health() efficiency

PHP Code:

#include <amxmodx>
#include <hamsandwich>
#include <fakemeta>

new const Version[] = "0.2";

#define MAX_PLAYERS 32
#define MAX_NAME_LENGTH 32

enum PlayerStats
{
    Shots,
    DamageGiven,
    DamageReceived
}

new g_pzStats[ MAX_PLAYERS + 1 ][ MAX_PLAYERS + 1 ][ PlayerStats ];
new g_pEnabled;

public plugin_init() 
{
    register_plugin( "Round Damage" , Version , "bugsy" );
    
    RegisterHam( Ham_TakeDamage , "player" , "HamTakeDamage" , true );
    register_logevent( "RoundEnd" , 2 , "1=Round_End" ); 
    
    g_pEnabled = register_cvar( "rd_enabled" , "1" );
}

public HamTakeDamage( victim , inflictor , attacker , Float:fDamage , bitDamage ) 
{     
    if ( get_pcvar_num( g_pEnabled ) )
    {
        new iActualDamage = pev( victim , pev_dmg_take ); 

        g_pzStats[ attacker ][ victim ][ Shots ]++;
        g_pzStats[ victim ][ attacker ][ DamageReceived ] += iActualDamage;
        g_pzStats[ attacker ][ victim ][ DamageGiven ] += iActualDamage;
    }
}

public RoundEnd()
{
    new iPlayers[ 32 ] , iNum , iAttackerPlayer , iVictimPlayer , iDamageGiven , iDamageTaken , iHealth[ MAX_PLAYERS + 1 ] , szName[ MAX_PLAYERS + 1 ][ 32 ];
    
    get_players( iPlayers , iNum );
    
    for ( new i = 0 ; i < iNum ; i++ )
    {
        iAttackerPlayer = iPlayers[ i ];
        
        for ( new p = 0 ; p < iNum ; p++ )
        {
            iVictimPlayer = iPlayers[ p ];
            
            if ( ( i != p ) && g_pzStats[ iAttackerPlayer ][ iVictimPlayer ][ DamageGiven ] || g_pzStats[ iAttackerPlayer ][ iVictimPlayer ][ DamageReceived ] )
            {
                iDamageGiven = min( g_pzStats[ iAttackerPlayer ][ iVictimPlayer ][ DamageGiven ] , 100 );
                iDamageTaken = min( g_pzStats[ iAttackerPlayer ][ iVictimPlayer ][ DamageReceived ] , 100 );
            
                if ( iDamageGiven || iDamageTaken )
                {
                    if ( !iHealth[ iVictimPlayer ] )
                        iHealth[ iVictimPlayer ] = is_user_alive( iVictimPlayer ) ? get_user_health( iVictimPlayer ) : -1;
                        
                    if ( szName[ iVictimPlayer ][ 0 ] == EOS )
                        get_user_name( iVictimPlayer , szName[ iVictimPlayer ] , charsmax( szName[] ) );
                        
                    client_print( iAttackerPlayer , print_chat , "(%d with %d) damage, (%d with %d) taken, %s (%dHP)" , iDamageGiven , g_pzStats[ iAttackerPlayer ][ iVictimPlayer ][ Shots ] , iDamageTaken , g_pzStats[ iVictimPlayer ][ iAttackerPlayer ][ Shots ] , szName[ iVictimPlayer ] , iHealth[ iVictimPlayer ] == -1 ? 0 : iHealth[ iVictimPlayer ] );
                }
            }
        }
    }

    ClearArray();
}

ClearArray()
{
    for ( new i = 1 ; i <= MAX_PLAYERS ; i++ )
    {
        for ( new p = 1 ; p <= MAX_PLAYERS ; p++ )
        {
            g_pzStats[ i ][ p ][ Shots ] = 0;
            g_pzStats[ i ][ p ][ DamageGiven ] = 0;
            g_pzStats[ i ][ p ][ DamageReceived ] = 0;
        }
    }
} 


[deprale]

Originally Posted by Bugsy Try this PHP Code: #include <amxmodx> #include <hamsandwich> #include <fakemeta> new const Version[] = "0.2"; #define MAX_PLAYERS 32 #define MAX_NAME_LENGTH 32 enum PlayerStats {     Shots,     DamageGiven,     DamageReceived } new g_pzStats[ MAX_PLAYERS + 1 ][ MAX_PLAYERS + 1 ][ PlayerStats ]; new g_pEnabled; public plugin_init()  {     register_plugin( "Round Damage" , Version , "bugsy" );          RegisterHam( Ham_TakeDamage , "player" , "HamTakeDamage" , true );     register_logevent( "RoundEnd" , 2 , "1=Round_End" );           g_pEnabled = register_cvar( "rd_enabled" , "1" ); } public HamTakeDamage( victim , inflictor , attacker , Float:fDamage , bitDamage )  {          if ( get_pcvar_num( g_pEnabled ) )     {         new iActualDamage = pev( victim , pev_dmg_take );                   g_pzStats[ attacker ][ victim ][ Shots ]++;         g_pzStats[ victim ][ attacker ][ DamageReceived ] += iActualDamage;         g_pzStats[ attacker ][ victim ][ DamageGiven ] += iActualDamage;     } } public RoundEnd() {     new iPlayers[ 32 ] , iNum , iAttackerPlayer , iVictimPlayer , iDamageGiven , iDamageTaken , iHealth[ MAX_PLAYERS + 1 ] , szName[ MAX_PLAYERS + 1 ][ 32 ];          get_players( iPlayers , iNum );          for ( new i = 0 ; i < iNum ; i++ )     {         iAttackerPlayer = iPlayers[ i ];                  for ( new p = 0 ; p < iNum ; p++ )         {             iVictimPlayer = iPlayers[ p ];                          if ( ( i != p ) && g_pzStats[ iAttackerPlayer ][ iVictimPlayer ][ DamageGiven ] || g_pzStats[ iAttackerPlayer ][ iVictimPlayer ][ DamageReceived ] )             {                 iDamageGiven = min( g_pzStats[ iAttackerPlayer ][ iVictimPlayer ][ DamageGiven ] , 100 );                 iDamageTaken = min( g_pzStats[ iAttackerPlayer ][ iVictimPlayer ][ DamageReceived ] , 100 );                              if ( iDamageGiven || iDamageTaken )                 {                     if ( !iHealth[ iVictimPlayer ] )                         iHealth[ iVictimPlayer ] = is_user_alive( iVictimPlayer ) ? get_user_health( iVictimPlayer ) : 0;                                              if ( szName[ iVictimPlayer ][ 0 ] == EOS )                         get_user_name( iVictimPlayer , szName[ iVictimPlayer ] , charsmax( szName[] ) );                                              client_print( iAttackerPlayer , print_chat , "(%d with %d) damage, (%d with %d) taken, %s (%dHP)" , iDamageGiven , g_pzStats[ iAttackerPlayer ][ iVictimPlayer ][ Shots ] , iDamageTaken , g_pzStats[ iVictimPlayer ][ iAttackerPlayer ][ Shots ] , szName[ iVictimPlayer ] , iHealth[ iVictimPlayer ] );                 }             }         }     }     ClearArray(); } ClearArray() {     for ( new i = 1 ; i <= MAX_PLAYERS ; i++ )     {         for ( new p = 1 ; p <= MAX_PLAYERS ; p++ )         {             g_pzStats[ i ][ p ][ Shots ] = 0;             g_pzStats[ i ][ p ][ DamageGiven ] = 0;             g_pzStats[ i ][ p ][ DamageReceived ] = 0;         }     } }

Rather interesting approach of getting health but problem is, players can still damage each other after round end(since it's called 5 seconds before a newround).
The way I do it is - I hook eventdeath and eventhealth, I use a bool for freezetime, set it to true on newround, and false on startround, in eventhealth I gather HP if it's not freezetime, and it works properly. This way I don't have to care if somebody changed mp_round_restart_delay (regamedll) or if they're using some mod that changes the amount of seconds it takes for new round to start.

PHP Code:

public EventHealth(client){ 
    if( !g_FreezeTime ) // Have to do this because EventHealth is called after you die (or maybe I'm stupid) and it sets your health to 100.
        g_RoundEndHp[client] = get_user_health(client);
} 


Also may I ask why you chose min instead of clamp?

[theuser]

Originally Posted by Bugsy Try this PHP Code: #include <amxmodx> #include <hamsandwich> #include <fakemeta> new const Version[] = "0.2"; #define MAX_PLAYERS 32 #define MAX_NAME_LENGTH 32 enum PlayerStats {     Shots,     DamageGiven,     DamageReceived } new g_pzStats[ MAX_PLAYERS + 1 ][ MAX_PLAYERS + 1 ][ PlayerStats ]; new g_pEnabled; public plugin_init()  {     register_plugin( "Round Damage" , Version , "bugsy" );          RegisterHam( Ham_TakeDamage , "player" , "HamTakeDamage" , true );     register_logevent( "RoundEnd" , 2 , "1=Round_End" );           g_pEnabled = register_cvar( "rd_enabled" , "1" ); } public HamTakeDamage( victim , inflictor , attacker , Float:fDamage , bitDamage )  {          if ( get_pcvar_num( g_pEnabled ) )     {         new iActualDamage = pev( victim , pev_dmg_take );                   g_pzStats[ attacker ][ victim ][ Shots ]++;         g_pzStats[ victim ][ attacker ][ DamageReceived ] += iActualDamage;         g_pzStats[ attacker ][ victim ][ DamageGiven ] += iActualDamage;     } } public RoundEnd() {     new iPlayers[ 32 ] , iNum , iAttackerPlayer , iVictimPlayer , iDamageGiven , iDamageTaken , iHealth[ MAX_PLAYERS + 1 ] , szName[ MAX_PLAYERS + 1 ][ 32 ];          get_players( iPlayers , iNum );          for ( new i = 0 ; i < iNum ; i++ )     {         iAttackerPlayer = iPlayers[ i ];                  for ( new p = 0 ; p < iNum ; p++ )         {             iVictimPlayer = iPlayers[ p ];                          if ( ( i != p ) && g_pzStats[ iAttackerPlayer ][ iVictimPlayer ][ DamageGiven ] || g_pzStats[ iAttackerPlayer ][ iVictimPlayer ][ DamageReceived ] )             {                 iDamageGiven = min( g_pzStats[ iAttackerPlayer ][ iVictimPlayer ][ DamageGiven ] , 100 );                 iDamageTaken = min( g_pzStats[ iAttackerPlayer ][ iVictimPlayer ][ DamageReceived ] , 100 );                              if ( iDamageGiven || iDamageTaken )                 {                     if ( !iHealth[ iVictimPlayer ] )                         iHealth[ iVictimPlayer ] = is_user_alive( iVictimPlayer ) ? get_user_health( iVictimPlayer ) : 0;                                              if ( szName[ iVictimPlayer ][ 0 ] == EOS )                         get_user_name( iVictimPlayer , szName[ iVictimPlayer ] , charsmax( szName[] ) );                                              client_print( iAttackerPlayer , print_chat , "(%d with %d) damage, (%d with %d) taken, %s (%dHP)" , iDamageGiven , g_pzStats[ iAttackerPlayer ][ iVictimPlayer ][ Shots ] , iDamageTaken , g_pzStats[ iVictimPlayer ][ iAttackerPlayer ][ Shots ] , szName[ iVictimPlayer ] , iHealth[ iVictimPlayer ] );                 }             }         }     }     ClearArray(); } ClearArray() {     for ( new i = 1 ; i <= MAX_PLAYERS ; i++ )     {         for ( new p = 1 ; p <= MAX_PLAYERS ; p++ )         {             g_pzStats[ i ][ p ][ Shots ] = 0;             g_pzStats[ i ][ p ][ DamageGiven ] = 0;             g_pzStats[ i ][ p ][ DamageReceived ] = 0;         }     } }

I haven't tested yet but I think there is a problem
In the function you specified, the minimum value is damaged and consider 100
for example :
The opponent's damage is 40
We shoot him a awp (we know that each awp shot to the body reduces 115 damage)
In this case, the plugin considers the minimum between 115 and 100, which becomes 100!
While he has to print the number 40 (because the opponent's damage was 40 and we took 40 damage from him)

Is my opinion really correct or did you anticipate this?

And what can be done for the comment that our friend said in the previous replay? (Roundend function that still has 5 seconds left when called)

[Natsheh]

Originally Posted by deprale Rather interesting approach of getting health but problem is, players can still damage each other after round end(since it's called 5 seconds before a newround). The way I do it is - I hook eventdeath and eventhealth, I use a bool for freezetime, set it to true on newround, and false on startround, in eventhealth I gather HP if it's not freezetime, and it works properly. This way I don't have to care if somebody changed mp_round_restart_delay (regamedll) or if they're using some mod that changes the amount of seconds it takes for new round to start. PHP Code: public EventHealth(client){      if( !g_FreezeTime ) // Have to do this because EventHealth is called after you die (or maybe I'm stupid) and it sets your health to 100.         g_RoundEndHp[client] = get_user_health(client); }  Also may I ask why you chose min instead of clamp?

Can you explain how this is related to the thread topic?

[Bugsy]

1. I didn't consider the scenario where someone could hurt someone before round start. I personally still consider it as damage issued in the round.
2. clamp() would only make sense if you want to enforce damage within a range (eg. 50-100). I do not need a lower boundary for damage since all damage counts, we just want to limit the max. Please provide your argument for using clamp().
3. My goal is to only call get_user_health() on alive players, and only call it once per alive player per round. I updated the code here to avoid redundant calls.
4. I added max() to limit the damage to 100 since you requested it here:

Yes, sometimes it is more than 100, which I want to be a maximum of 100. And that sometimes it is not displayed correctly For example, I do 100 damage and the plugin writes 80 thank you

[theuser]

Originally Posted by Bugsy 1. I didn't consider the scenario where someone could hurt someone before round start. I personally still consider it as damage issued in the round. 2. clamp() would only make sense if you want to enforce damage within a range (eg. 50-100). I do not need a lower boundary for damage since all damage counts, we just want to limit the max. Please provide your argument for using clamp(). 3. My goal is to only call get_user_health() on alive players, and only call it once per alive player per round. I updated the code here to avoid redundant calls. 4. I added max() to limit the damage to 100 since you requested it here:

The plugin still has a problem, for example, I give 100 damage to the opponent and the plugin writes 70 !!!
What is the reason for this problem?

[Bugsy]

If a victim has 10hp and I blast him in the head with an awp, do you want to see another 100 (or whatever) damage issued in, or only the 10 loss in hp? I think this is the issue .. I built this from a loss in hp perspective, assuming this is what you wanted.

Try this

[theuser]

Originally Posted by Bugsy If a victim has 10hp and I blast him in the head with an awp, do you want to see another 100 (or whatever) damage issued in, or only the 10 loss in hp? I think this is the issue .. I built this from a loss in hp perspective, assuming this is what you wanted. Try this

I want to see only 10

[Bugsy]

Doesn't that contradict this? https://forums.alliedmods.net/showpo...4&postcount=16

[theuser]

Originally Posted by Bugsy Doesn't that contradict this? https://forums.alliedmods.net/showpo...4&postcount=16

No, I meant that my opponent's damage is 100 and I do 100 damage to him, but the plugin mistypes 70 (I solved this problem myself)

And that if possible show the text of the message in next round freeze time

And show the damage we get from the enemy
For example, when a player's damage is 60 and I kill him, it is shown in the chat that I have taken 60 damage from him (Even if I had taken 400 damages from him)
(Note that the following may occur):
-The opponent's damage in the first round is 100
-I do 10 damage to him and his damage becomes 90
-Then my teammate deals 20 damage to him and his damage becomes 70
-In the end I kill him
Now in my chat it should say 80 (because I got 10 damage in the first round and 70 damage when I killed him.
Does the plugin calculate like this or it is possible?

And another problem
When someone drops a bomb and is damaged by a bomb, he sees himself as an attacker in the chat.