if over a number function

msnhockey

hi all.

I am new here to the forums and am seeking alittle help

I have a stats website and am trying to show a code if over a certain # and if over that number say ____ is over budget. if below that number leave at the total it is at

here is my code so far

$salary_total1 = mysql_result($result_stats,0,"salary");

$salary_total = number_format($salary_total1, 0,"",","); 

$cap_total1 = "25200000";

$cap_total = number_format($cap_total1, 0,"",","); 


if ($salary_total < $cap_total) {
$salary_total = "$salary_total"; 
} else {
$salary_total = "$salary_total is over budget";

}

any help is appriated

thanks

mike

Piranha

And the question is? Please state your question in a clear way, we have no way of knowing what problem you have unless you explain it.

I can't see any problem, have you tried to echo the values to see that it is correct?

msnhockey

here is my example

salary_cap = 25,200,000

if salary total is under salary_cap it will display the salary_total

if salary_total is over salary_cap I would like it to flag it somehow and show

salary_total is over budget

I have echoed it and stuff, but if over budget it just shows the salary_total and not with the is over budget

thanks

mike

Weedpacket

You want to compare your numbers before you format them; because after you format them they're not numbers any more, they're strings, and so compare differently (they've got commas in them, for a start). In other words, you want to compare (an unformatted) $salary_total with $cap_total1 (and you don't need $cap_total at all unless and until you want to display it).

In fact, $cap_total1 is a string anyway because of the quotes around it. Remove those to make it a number.

msnhockey

I am trying to use cap_total to display the cap on the page.

for a better example of what I am doing you can goto
http://www.snapshot-hockey.net/ufhl/?opt=viewteam&id=0003

thanks,

mike

msnhockey

i tried your example of removing the quotes and still the same. If i echo it I get the salary_total is over budget weather or not it's under or over the cap amount. if i remove the echo I get nothing at all just the salary_total

mike

Piranha

Please try the following:

$salary_total1 = 26200000;
$cap_total1 = 25200000;

if ($salary_total1 <= $cap_total1) {   // It should be <= to get it within budget even with it's exact the amount
echo "within budget";
} else {
echo "over budget";
}

If this works as you expect (and it probably will) then you know that the if - else is correct and that the error is somewhere else.

msnhockey

ok.. that worked great.

the only thing now is that is displays at the top of the page and not in the spot where i need it. I need it next to the salary in the table below

here is the address of the example
http://www.snapshot-hockey.net/ufhl/?opt=viewteam&id=0003

thanks

mike

msnhockey

never mind I have finished what I needed. I took out the echo and added $over and it works great.

I would like to thank you for the help

here the code if you want to look at what I did

if ($salary_total1 <= $cap_total1) { 
$over =  " "; 
} else { 
$over =  "IS OVER BUDGET"; 
}

thanks

mike

Weedpacket

msnhockey wrote:
i tried your example of removing the quotes and still the same.

Next time, try reading the rest of the post as well; not just the last line.