This is the method I have on my Timer class
public int getPerHour(int value) {
return (int) (value * 3600000d / (System.currentTimeMillis() - start));
}
About the same as you have, except for the fact I multiply first instead of diving so there are no rounding errors