[Dexon]

Hello, could anyone help in this?
I tried with the following:

public kd_check(id){
    ColorChat(id, "!tYour KD: !g%i", (get_user_frags(id)/get_user_deaths(id)))
}

Log:
Run time error 11: divide
[0] bounty.sma::kd_check (line 43)

[iceeedr]

Check that the frag is >0 before dividing.

[OciXCrom]

Deaths > 0*
Dividing zero is fine, dividing by zero is not.
Also, you need to use %f, not %i because the result will most likely be a floating point number, not an integer.

[fysiks]

Both get_user_frags() and get_user_deaths() return integers so the result will be an integer (see "integer division" e.g. 10/3 = 3, 5/10 = 0, etc.). To get a proper value for the KDR, you need to first convert them to floating point values with float() and then use %f in your format string.

Note: Do the 0 deaths check before converting to a float (i.e. store the deaths in a variable first so you don't need to call it twice).

[Dexon]

Thank you all for the answer, now I got this:

if(get_user_frags(id) > 0){
    ColorChat(id, "!tKD-d: !g%f", get_user_frags(id)/get_user_deaths(id))
}

But it shows the same error. :/

And im not sure with this, may you show me how you thought?

you need to first convert them to floating point values with float()

[+ARUKARI-]

1 / 1 = 1, 1 / 0 = ?

[OciXCrom]

new iDeaths = get_user_deaths(id)

if(iDeaths)
{
    ColorChat(id, "!tKD-d: !g%f", float(get_user_frags(id)) / float(get_user_deaths(id)))
}

[Dexon]

Thank you all, it's all fine.

But usually K/D counts even if user has 0 deaths.
So may anyone have an idea about:
if a user has 0 deaths, make the value to 1, but not the get_user_deaths, because it would just give the player +1 death in the scoreboard?

[OciXCrom]

if(iDeaths) print your regular message
else print 1

?

[Natsheh]

PHP Code:

public kd_check(id){
    ColorChat(id, "!tYour KD: !g%.2f", ( get_user_frags( id ) / float( max( get_user_deaths( id ), 1 ) ) ) )
} 


here you go.

also their method works fine the only problem it won't print in chat if the user has a 0 deaths.