Bug Value is always 0

This script used to work... and now suddenly it doesent work anymore...

Code (CSharp):

1. using System.Collections;
2. using System.Collections.Generic;
3. using UnityEngine;
4.  
5. public class HPregen : Ability
6. {
7.     public float healpercent;
8.     public float tickrate;
9.    
10.     public override void AbilityEffect(Unit unit)
11.     {
12.         if(timer >= tickrate)
13.         {
14.             timer = 0;
15.             int healamount = (int)((unit.baseHP / 100) * healpercent);
16.             Debug.Log(healamount);
17.             if(healamount <= 0)
18.             {
19.                 healamount = 1;
20.             }
21.             unit.currentHP += healamount;
22.             if(unit.currentHP > unit.baseHP)
23.             {
24.                 unit.currentHP = unit.baseHP;
25.             }
26.             unit.UpdateHealthbar();
27.         }
28.     }
29. }

The Debug.Log(healamount); is always 0

 

Before someone else pops in with a whole lot of tips on debugging, look at line 15 and decide "what would make that result in 0?" In general, when doing math on computers, especially with integers but even with floats, you want to do multiplies before divides.

 

I added 2 debugs ->
the math should work, wtf is happening
how could i change this calculation to make it work

Code (CSharp):

1. Debug.Log(unit.baseHP);
2. Debug.Log(healpercent);

(47 / 100) * 10 should be more than 0

 

(47/100)

is always going to be 0 if those are integers

That's what @halley is pointing out.

So for percentages, multiply first, THEN divide, then cast back to int

EDIT: in related news,

(100/47)

is always going to be 2, if those are integers

 

I just did a small test:

yeah mb, this is also 0

 

So try some variations.
Does moving healpercent to the front help?
Do removing some parentheses help?
Does multiplying by healpercent first help?

 

yep, ty alot guys, this now works and the result is 4 instead of 0

 

Defining 100 as an float 100f would also had solved above problem

 

Yep, same reason. Integer divide.