Need help with PHP and math for a salarycap feature

Avenger

I believe this is the right area for this. I was told to come here from c/c++ forums. Said you guys would be better suited for this:

I run a madden league and we use a stat tool that takes stats and puts them into html pages. First question is this. We have this php file that takes rosters info and puts out a salary cap info page. That shows how many players the team has, the amount the team has spent, what is left to spend, and how much the person went over as well. But this isnt working correctly. The only things working are the amount of players, and amount spent. I believe the math is wrong, could someone please help me with this. I bought the the tool and have no idea how php works. Here is the file and what is says:

// CONNECT TO DATABASE
   $db = @mysql_connect($host,$user,$password) or die("<html><body bgcolor=$bgcolor>User Error!</body></html>");
   @mysql_select_db($database,$db) or die("<html><body bgcolor=$bgcolor>Database Error!</body></html>");

   $mysql_query = "SELECT max(season) as season FROM game_summary WHERE league_id=$league_id";
   $res=mysql_query($mysql_query,$db);
   $season = mysql_result($res,0,"season");

   $mysql_query = "SELECT salary_cap FROM league where id=1";
   $res=mysql_query($mysql_query,$db);
   $salary_cap = mysql_result($res,0,"salary_cap")/1000;

   // GET THE CURRENT WEEK FOR THE SPECIFIED LEAGUE
   $mysql_query = "SELECT nick_name as team,
                          sum(sal) as spent,
                          sum(1) as tot_players,
                          cap_penalty FROM team_players, team
                   WHERE team.team_abbr=team_players.team
                         and team.league_id=team_players.league_id
                         and team.league_id=1
                         and season=$season
                   GROUP BY team ORDER BY team";

   $res=mysql_query($mysql_query,$db);
   $num_rows = mysql_num_rows($res);

   $captmp = "\$76.7" . $salary_cap/1000 . "M";

   echo "<center><font size=4 color=000000 face=verdana>
         <b>Salary Cap Totals ($captmp / 53 Players)</b><br><br></font>
         <table cellpadding=1 cellspacing=1 border=0 bgcolor=000000>
         <tr><td><table bgcolor=FFFFF cellpadding=2 cellspacing=0 border=0F>";

   echo "<tr bgcolor=000000>
         <td width=75><font size=1 color=FFFFFF face=verdana>Team</td>
         <td width=50><font size=1 color=FFFFFF face=verdana>Players</td>
         <td align=right width=75><font size=1 color=FFFFFF face=verdana>Used</td>
         <td align=right width=75><font size=1 color=FFFFFF face=verdana></td>
         <td align=right width=75><font size=1 color=FFFFFF face=verdana>Available</td>
         </tr>";

   $rotate[0]="cecece";
   $rotate[1]="FFFFFF";

   for($count=0;$count<$num_rows;$count++){
       $team = mysql_result($res,$count,"team");

   $tot_players = mysql_result($res,$count,"tot_players");

   $spent = mysql_result($res,$count,"spent")/1000;
   $left = $salary_cap - $spent;

   if($spent<1000) $spent = "\$" . $spent . "K";
   else $spent = "\$" . $spent/1000 . "M";


   $cur_color=$rotate[$count%2];

   if($left<1000 & $left>1) $left = "\$" . $spent . "K";
   elseif($left<1000){
       $cur_color="red";
       if($left>-1000) $left = "\$" . $spent . "K";
       else $left = "\$" . $left/1000 . "M";
   }
   else $left = "\$" . $left/1000 . "M";


   echo "<tr bgcolor=$cur_color>
         <td><font size=1 face=verdana>$team</td>
         <td><font size=1 face=verdana>$tot_players</td>
         <td align=right><font size=1 face=verdana>$spent</font></td>
         <td align=right><font size=1 face=verdana>$penalty</font></td>
         <td align=right><font size=1 face=verdana>$left</font></td>
         </tr>";
   }

   echo "</table></td></tr></table><center><br>";

any help would be greatly appreciated! Thanks

if you want to see what I am talking about go here:

http://www.putm.wired-hub.net/league/gm/salarycap.php?league_id=1&season=2002

Also in some areas of this I get that error at the top of the pages. Something like unable to jump from 0. I bought the program that does this so I am no help here. Any help would be greatly appreciated. Thanks

rinjani

You paid for this?

The word penalty is missing in this line:

Unfortunately the variable $penalty has not been defined anywhere so the loop to produce it in your table is going to be blank.

The variable $left that produces the number in the available column is wrong. I'm assuming it should be something like $salary_cap - $spent but that is not what the code is creating. The /1000 is to get the numbers into the correct $xyzK or $xyzM.

The lines in the output are supposed to alternate color I think but that isn't happening because the number coming out for $left is incorrect.

The code looks like they never got around to finishing it (or they sent you the wrong file).

I would contact the person who wrote and point out that $penalty isn't being defined anywhere and that this is @#$%& shite.. [you paid them right - so you get to complain].

Sorry not much help

Rinjani

Avenger

I wish I could contact him lol he ran with around 1,000 dollars from a lot of people and never set them up like he said...he did give a tool that was about 98% completed. But I was just trying to get this part right...dang was hoping someone could fill in the holes...

Avenger

another question do you know why I get these errors:

Warning: Unable to jump to row 0 on MySQL result index 7 in /home/avenger4/public_html/league/stats/stats.php on line 145

Warning: Unable to jump to row 0 on MySQL result index 7 in /home/avenger4/public_html/league/stats/stats.php on line 146

I get those on this page:

http://www.putm.wired-hub.net/league/stats/index.php?league_id=1&season=2002&category=passing&player=Tim%20Couch

and I get this error on the salarycap tool

Warning: Unable to jump to row 0 on MySQL result index 3 in /home/avenger4/public_html/league/gm/salarycap.php on line 17

rinjani

Well, I for one don't know enough about what you need to see to give definitive answer but lets start here.

Assuming you have a proper text editor/dev environment and not MS word

comment out the offending area

/*
if($left<1000 & $left>1) $left = "\$" . $spent . "K";
elseif($left<1000){
$cur_color="red";
if($left>-1000) $left = "\$" . $spent . "K";
else $left = "\$" . $left/1000 . "M";
}
else $left = "\$" . $left/1000 . "M";

and try to build the correct output.

What is ths value for salary cap?

If the number 76.7 located around line 29 is correct then

$left = (76.7 - $spent);

If that gives the correct number then build the statement to get the format correct.

if ($left >= 1000) {
$left ="\$" . $left/1000 . "M";
}
and so on.

Good luck

Avenger

thank you for the quick reply and thank you for your help 🙂

Avenger

also I have no idea what I am doing so finishing this will be impossible lol...I tried and I havent been able to..Here is what I have so far:

<?php
    include("/home/avenger4/public_html/league/domain.php");
    include("/home/avenger4/public_html/cgi-bin/dbConfig.php");

$league_id=1;

// CONNECT TO DATABASE
$db = @mysql_connect($host,$user,$password) or die("<html><head><body bgcolor=$bgcolor alink=$bglinkcolor vlink=

$bglinkcolor link=$bglinkcolor>User Error!</body></html>");
    @mysql_select_db($database,$db) or die("<html><head><body bgcolor=$bgcolor alink=$bglinkcolor vlink=$bglinkcolor 

link=$bglinkcolor>Database Error!</body></html>");

$mysql_query = "SELECT max(season) as season FROM game_summary WHERE league_id=$league_id";
$res=mysql_query($mysql_query,$db);
$season = mysql_result($res,0,"season");

$mysql_query = "SELECT salary_cap FROM league where id=1";
$res=mysql_query($mysql_query,$db);
$salary_cap = mysql_result($res,0,"salary_cap")/1000;

// GET THE CURRENT WEEK FOR THE SPECIFIED LEAGUE
$mysql_query = "SELECT nick_name as team, sum(sal) as spent, sum(1) as tot_players, cap_penalty FROM team_players, 

team where team.team_abbr=team_players.team and team.league_id=team_players.league_id and team.league_id=1 and season

=$season GROUP BY team ORDER BY team";

$res=mysql_query($mysql_query,$db);
$num_rows = mysql_num_rows($res);

$captmp = "\$76." . $salary_cap/1000 . "M";

echo "<center><font size=4 color=000000 face=verdana><b>Salary Cap Totals ($captmp / 53 Players)</b><br><br></

font><table cellpadding=1 cellspacing=1 border=0 bgcolor=000000><tr><td><table bgcolor=FFFFF cellpadding=2 

cellspacing=0 border=0F>";
    echo "<tr bgcolor=000000><td width=75><font size=1 color=FFFFFF face=verdana>Team</td><td width=50><font size=1 

color=FFFFFF face=verdana>Players</td><td align=right width=75><font size=1 color=FFFFFF face=verdana>Used</td><td 

align=right width=75><font size=1 color=FFFFFF face=verdana></td><td align=right width=75><font size=1 color=FFFFFF 

face=verdana>Available</td></tr>";

$rotate[0]="cecece";
$rotate[1]="FFFFFF";

for($count=0;$count<$num_rows;$count++){
    $team = mysql_result($res,$count,"team");

    $tot_players = mysql_result($res,$count,"tot_players");

    $spent = mysql_result($res,$count,"spent")/1000;
    $left = (76. - $spent); 

    if($spent<1000) $spent = "\$" . $spent . "K";
    else $spent = "\$" . $spent/1000 . "M";

    if ($penalty==0) $penalty = "-";
    elseif($penalty<1000) $penalty = "\$" . $penalty . "K";
    else $penalty = "\$" . $penalty/1000 . "M";        
    $cur_color=$rotate[$count%2];

    if ($left >=1000) {$left ="\$" . $left/1000 . "M"; 
                $cur_color="red";
        if($left >=-1000) $left = "\$" . $spent . "K";
        else $left = "\$" . $left/-1000 . "M";
    }
    else $left = "\$" . $left/-1000 . "M";


    echo "<tr bgcolor=$cur_color><td><font size=1 face=verdana>$team</td><td><font size=1 face=

verdana>$tot_players</td><td align=right><font size=1 face=verdana>$spent</font></td><td align=right><font size=1 

face=verdana>$penalty</font></td><td align=right><font size=1 face=verdana>$left</font></td></tr>";
    }

echo "</table></td></tr></table><center><br>";
?>

I tried what you said and I just cant get it to work....lol I apologize for being incompetent 🙁

rinjani

OK,

Sorry I took just a quick look at this. The code you have at the bottom is now different from earlier with the -1000 in it.

It looks like $salary_cap is not coming up with a value so the approx. line 53

$left = $salary_cap - $spent;

is coming out as a negative number.

If 76.7 is the salary cap try replacing $salary_cap with 76.7. But I have a feeling that $salary_cap is different for each team and needs to come from the database. Right now it is coming out incorrectly.

if the SQL statement is approx correct at line 11 then try this (no certainty that it'll work).

$mysql_query = "SELECT salary_cap FROM league where id=1";
$res=mysql_query($mysql_query,$db);
// $salary_cap = mysql_result($res,0,"salary_cap")/1000;

while ($result = mysql_fetch_row($res)) {

$salary_cap = ($result[0]/1000);
}

Try cutting and pasting that whole piece. I have added // to the original $salary_cap line to comment it out.

you can then do

echo $salary_cap;

to see if there's a value for it.

Good luck.

I won't be able to help anymore today (not that I have helped much) sorry

rinjani

Avenger

Ok this is what your suggestion did: First it took taht error at the top of the page off. Which is nice....Thanks Second it is subtracting but I dont believe it is doing it right: Here is my cap numbers take a look at it:

49ers 55 $69M  $-68.924M 
Bears 54 $68.15M  $-68.074M 
Bengals 55 $64.35M  $-64.274M 
Bills 52 $64.2M  $-64.124M 
Broncos 55 $70.2M  $-70.124M 
Buccaneers 54 $71.7M  $-71.624M 
Cardinals 55 $64.75M  $-64.674M 
Chargers 53 $71.1M  $-71.024M 
Chiefs 55 $69.1M  $-69.024M 
Colts 54 $71.1M  $-71.024M 
Cowboys 52 $67M  $-66.924M 
Dolphins 53 $71.1M  $-71.024M 
Eagles 54 $68.35M  $-68.274M 
Falcons 54 $66.65M  $-66.574M 
Giants 50 $59.15M  $-59.074M 
Jaguars 55 $59.2M  $-59.124M 
Jets 52 $66.6M  $-66.524M 
Lions 54 $62.85M  $-62.774M 
Packers 54 $67.75M  $-67.674M 
Panthers 55 $62.35M  $-62.274M 
Patriots 55 $60M  $-59.924M 
Raiders 54 $71.3M  $-71.224M 
Rams 53 $66.4M  $-66.324M 
Ravens 44 $34.15M  $-34.074M 
Redskins 52 $66.25M  $-66.174M 
Saints 52 $57.9M  $-57.824M 
Seahawks 55 $63.35M  $-63.274M 
Steelers 53 $67.95M  $-67.874M 
Texans 52 $63.65M  $-63.574M 
Titans 55 $68.8M  $-68.724M 
Vikings 55 $52M  $-51.924M

Avenger

Used is on the left and available is on the right. But the math part it looks like is subtracting from what is used and showing it on the available part. So the available is 7,600,000 less then what is used. If that makes sense lol

On another subject. How did you get that error at the top to go away? I get that error on some of the names on the stats tool I use. Like this name: http://www.putm.wired-hub.net/league/stats/index.php?league_id=1&season=2002&category=passing&player=Tim%20Couch

Would you happen to know on that? Again I thank you very much 🙂 You have really went out of your way. Again thanks

Avenger

Oh and here is the code as it is now:

<?php
    include("/home/avenger4/public_html/league/domain.php");
    include("/home/avenger4/public_html/cgi-bin/dbConfig.php");

$league_id=1;

// CONNECT TO DATABASE
$db = @mysql_connect($host,$user,$password) or die("<html><head><body bgcolor=$bgcolor alink=$bglinkcolor vlink=$bglinkcolor link=$bglinkcolor>User Error!</body></html>");
@mysql_select_db($database,$db) or die("<html><head><body bgcolor=$bgcolor alink=$bglinkcolor vlink=$bglinkcolor link=$bglinkcolor>Database Error!</body></html>");

$mysql_query = "SELECT max(season) as season FROM game_summary WHERE league_id=$league_id";
$res=mysql_query($mysql_query,$db);
$season = mysql_result($res,0,"season");

$mysql_query = "SELECT salary_cap FROM league where id=1"; 
$res=mysql_query($mysql_query,$db); 
// $salary_cap = mysql_result($res,0,"salary_cap")/1000; 

while ($result = mysql_fetch_row($res)) { 

$salary_cap = ($result[0]/1000); 
} 


// GET THE CURRENT WEEK FOR THE SPECIFIED LEAGUE
$mysql_query = "SELECT nick_name as team, sum(sal) as spent, sum(1) as tot_players, cap_penalty FROM team_players, team where team.team_abbr=team_players.team and team.league_id=team_players.league_id and team.league_id=1 and season=$season GROUP BY team ORDER BY team";

$res=mysql_query($mysql_query,$db);
$num_rows = mysql_num_rows($res);

$captmp = "\$76." . $salary_cap/1000 . "M";

echo "<center><font size=4 color=000000 face=verdana><b>Salary Cap Totals ($captmp / 53 Players)</b><br><br></font><table cellpadding=1 cellspacing=1 border=0 bgcolor=000000><tr><td><table bgcolor=FFFFF cellpadding=2 cellspacing=0 border=0F>";
echo "<tr bgcolor=000000><td width=75><font size=1 color=FFFFFF face=verdana>Team</td><td width=50><font size=1 color=FFFFFF face=verdana>Players</td><td align=right width=75><font size=1 color=FFFFFF face=verdana>Used</td><td align=right width=75><font size=1 color=FFFFFF face=verdana></td><td align=right width=75><font size=1 color=FFFFFF face=verdana>Available</td></tr>";

$rotate[0]="cecece";
$rotate[1]="FFFFFF";

for($count=0;$count<$num_rows;$count++){
    $team = mysql_result($res,$count,"team");

    $tot_players = mysql_result($res,$count,"tot_players");

    $spent = mysql_result($res,$count,"spent")/1000;
    $left = (76. - $spent); 

    if($spent<1000) $spent = "\$" . $spent/1000 . "K";
    else $spent = "\$" . $spent/1000 . "M";

$cur_color=$rotate[$count%2];

    if ($left >=1000) {$left ="\$" . $left/1000 . "M"; 
                $cur_color="red";
        if($left >=1000) $left ="\$" . $left/1000 . "K";
        else $left = "\$" . $left/1000 . "M";
    }
    else $left ="\$" . $left/-1000 . "M";


    echo "<tr bgcolor=$cur_color><td><font size=1 face=verdana>$team</td><td><font size=1 face=verdana>$tot_players</td><td align=right><font size=1 face=verdana>$spent</font></td><td align=right><font size=1 face=verdana>$penalty</font></td><td align=right><font size=1 face=verdana>$left</font></td></tr>";
}

echo "</table></td></tr></table><center><br>";
?>

Avenger

just a friendly bump... 😃

Avenger

oh well....thanks guys for all your help...it at least works now..just wish I could get the math part right 🙂

rinjani

Check your message box, I just sent you a quick note

Rinjani

Avenger

Just wanted everyone to know that rinjani is the man. He took time out of his schedule to do this for me and got it working. Just wanted to say thanks to this community as a whole and Rinjani. Thanks 🙂