[Resolved] dynamic table background colors

[shanis42p]

I wrote a code that gathers user input on a variety of fields. One of the fields (called coating, from my database, drop down on the html form) has four possible options. The code accesses the field from the database and I would like it to assign a table background color based on a customer descision.

Heres the code:

<?php
//--------------------------------------------------------------
switch ($coating)
{

case "TiN":?>
		<th bgcolor="#FFCC33"><span class="style3" ><?php
		$coating = mysql_result($result,$i,"coating");
		print "$coating";?>
		</span></th><?php
		break;

case "TiCN":?>
		<th bgcolor="#999999" ><span class="style3" ><?php
		$coating = mysql_result($result,$i,"coating");
		print "$coating";?>
		</span></th><?php
		break;

case "TiALN":?>
		<th bgcolor="#660033"><span class="style3" ><?php
		$coating = mysql_result($result,$i,"coating");
		print "$coating";?>
		</span></th><?php
		break;

default:?>
		<th bgcolor="#000000"><span class="style3" ><?php
		$coating = mysql_result($result,$i,"coating");
		print "$coating";?>
		</span></th><?php
		break;	
}
?>

Here is the issue the code seems to set everything off by one row, the first row is always colorless, the color it should be is always one row below where it should be.

I think I may be coloring the cell after it puts a value in and moves to the next row.

Tell me what you think,
Shane

[jpmoriarty]

well just as a start... why don't you do:

<?php 
//-------------------------------------------------------------- 
switch ($coating) 
{ 
case "TiN":
        $colour = "FFCC33";
        break; 

case "TiCN": 
        $colour = "999999";
        break; 

case "TiALN":
        $colour = "660033";
        break; 

default: 
        $colour = "000000";
        break;     

}

echo "<th bgcolor=\"#$colour\"><span class=\"style3\" >\n"; 
$coating = mysql_result($result,$i,"coating"); 
echo  "$coating\n";
echo "</span></th>\n";
?> 

i think i'll need to see the code that surronds it though to see what's going on. you don't seem to have any <tr> defined, for example - is it elsewhere?

[shanis42p]

<?php 
//------------------------------------------------------------------------------------------
// INITIALIZING VARIABLES
	$hostname = "localhost";
	$username = "root";
	$password = "mastercut";
	$dbname = "mastercut";
	$usertable = "tblendmills";

//-------------------------------------------------------------------------------------------
// DEBUGGING TO SEE IF VARIABLE ARE PASSED, ..  VARIBLES POSTED FROM FORM
		print "You Entered into the form:";
		print "<BR><BR>";
		print "UNITS: $units";
		print "<BR>";
		print "TYPE: $type";
		print "<BR>";
		print "PART NUMBER: $partnum";
		print "<BR>";
		print "DIAMETER: $diameter";
		print "<BR>";
		print "CUTTING EDGE LENGTH $LOC";
		print "<BR>";
		print "SHANK $shank";
		print "<BR>";
		print "OAL $oal";
		print "<BR>";
		print "NO OF FLUTES: $flutes";
		print "<BR>";
		print "ENDTYPE $endtype";
		print "<BR>";
		print "COATING $coating";
		print "<BR><BR>";

//-----------------------------------------------------------------------------------------	
// CONNECTION TO DATABASE	
	mysql_connect($hostname,$username,$password) or die("Unable to connect to database");

mysql_select_db($dbname) or die("Unable to select database");


// REASSIGNING EMPTY VARIABLES TO WILDCARDS
//-------------------------------------------------------------------------------------------	
	if(empty($units)){
		$units = '%';
	}	
//-------------------------------------------------------------------------------------------	
	if(empty($partnum)){
		$partnum = '%';
	}
//-------------------------------------------------------------------------------------------	
	if(empty($diameter)){
		$diameter = '%';
	}
//-------------------------------------------------------------------------------------------
	if(empty($LOC)){
		$LOC = '%';
	}
//------------------------------------------------------------------------------------------	
	if(empty($shank)){
		$shank = '%';
	}
//-----------------------------------------------------------------------------------------
	if(empty($oal)){
		$oal = '%';
	}
//-----------------------------------------------------------------------------------------
// NO IF STATEMENT NEEDED FOR LAST TWO FORM VARIABLES THEY ARE ASSIGNED BY THE DROP DOWN
//-----------------------------------------------------------------------------------------


//QUERYING DATABASE FOR RECORDS
	$result = mysql_query("SELECT * FROM tblendmills WHERE 
		tblendmills.type LIKE '$type' AND
		tblendmills.units LIKE '$units' AND 
		tblendmills.partnum LIKE '$partnum' AND 
		tblendmills.diameter LIKE '$diameter' AND 
		tblendmills.LOC LIKE '$LOC' AND 
		tblendmills.shank LIKE '$shank' AND 
		tblendmills.oal LIKE '$oal' AND 
		tblendmills.flutes LIKE '$flutes' AND 
		tblendmills.coating LIKE '$coating' AND
		tblendmills.endtype LIKE '$endtype'");

$number = mysql_num_rows($result);

// SETTING UP TABLE HEADS FROM 
?>


<table width="757" border="1">
  <tr>
     <th scope="col">&nbsp;</th>
     <th scope="col">&nbsp;</th>
	<th scope="col">&nbsp;</th>
	<th scope="col">&nbsp;</th>
     <th colspan="4" scope="col" bgcolor="#0066FF"><span class="style3">Dimensions</span></th>
  	<th scope="col">&nbsp;</th>
	<th scope="col">&nbsp;</th>
  <tr>
  	<th scope="col" bgcolor="#0066FF"><span class="style3">Part Number </span></th>
     <th scope="col" bgcolor="#0066FF"><span class="style3">Endmill Type</span></th>
	<th scope="col" bgcolor="#0066FF"><span class="style3">Flutes</span></th>
	<th scope="col" bgcolor="#0066FF"><span class="style3">Endtype</span></th>
	<th scope="col" bgcolor="#0066FF"><span class="style3">Cutting Edge Diameter</span></th>
  	<th scope="col" bgcolor="#0066FF"><span class="style3">Cutting Edge Length </span></th>
     <th scope="col" bgcolor="#0066FF"><span class="style3">Shank Diameter </span></th>
	<th scope="col" bgcolor="#0066FF"><span class="style3">Overall Length </span></th>
     <th scope="col" bgcolor="#0066FF"><span class="style3">Coating</span></th>
	<th scope="col" bgcolor="#0066FF"><span class="style3">Price</span></th>

<?php	
	$i = 0;

if($number == 0){
	print "<center>No Parts Found</center>";
}
elseif ($number > 0){
	print "<center>There Were $number Endmills Found Matching Your Query: </center><BR>";

	while ($i < $number):
?>
			<tr>
    				<th><span class="style3">
					<?php
						$partnum = mysql_result($result,$i,"partnum");
						print "$partnum";
					?></span></th>
				<th><span class="style3">
					<?php
						$type = mysql_result($result,$i,"type");
						print "$type";
					?></span></th>
				<th><span class="style3">
					<?php
						$flutes = mysql_result($result,$i,"flutes");
						print "$flutes";
					?></span></th>	
				<th><span class="style3">
					<?php
						$endtype = mysql_result($result,$i,"endtype"); 
						print "$endtype";
					?></span></th>

			<th><span class="style3">
				<?php
					$diameter = mysql_result($result,$i,"Diameter");
					print "$diameter";
				?></span></th>
			<th><span class="style3">
				<?php
					$LOC = mysql_result($result,$i,"LOC");
					print "$LOC";
				?></span></th>
			<th><span class="style3">
				<?php
					$shank = mysql_result($result,$i,"Shank");
					print "$shank";
				?></span></th>
			<th><span class="style3">
				<?php
					$oal = mysql_result($result,$i,"oal");
					print "$oal";
				?></span></th>

			<?php
//------------------------------------------------------------------------------------
				switch ($coating) 
				{ 
     				case "TiN":?> 
        					<th bgcolor="#FFCC33"><span class="style3" ><?php 
        					$coating = mysql_result($result,$i,"coating"); 
        					print "$coating";?> 
        					</span></th><?php 
        					break; 

				case "TiCN":?> 
    					<th bgcolor="#999999" ><span class="style3" ><?php 
    					$coating = mysql_result($result,$i,"coating"); 
    					print "$coating";?> 
    					</span></th><?php 
    					break; 

				case "TiALN":?> 
    					<th bgcolor="#660033"><span class="style3" ><?php 
    					$coating = mysql_result($result,$i,"coating"); 
    					print "$coating";?> 
    					</span></th><?php 
    					break; 

				default:?> 
    					<th bgcolor="#000000"><span class="style3" ><?php 
    					$coating = mysql_result($result,$i,"coating"); 
    					print "$coating";?> 
    					</span></th><?php 
    					break;     
			}
			?>
			<th><span class="style3">
				<?php
					$list = mysql_result($result,$i,"list");
					print "$list";
				?></span></th>
		<?php
		$i++;
	endwhile;

}		

?>
</body>
</html>

Thats the whole Kit-And-Kaboodle

Thanks for the quick reply John,

Shane

[jpmoriarty]

ah. well there you go then.

I think the main problem is that you don't know your html. <th> is the table header code - it's for the cells that are at the top of the table. You can't have more than one table header.

for the majority of table's you want the following:

<table>
<tr>   <- table row start

<th>   <- table header start
head 1
</th>
<th>
head 2
</th>

</tr>

<tr>
<td>   <- table data start
data 1
</td>
<td>
data 2
</td>
</tr>
</table>

have a look at the w3c for the definitive lowdown.

Then, you need to learn how to use CSS properly because it will make things much, much easier (and it's also the way the web is heading, to get rid of sloppy, inefficient code). Head on over to the w3schools for all that jazz. You really don't need all those span's in there since you can define how tables look in css, nor should you have all the bgcolors there.

Not that I want to publicise another board, but there's a good CSS help page at webdeveloper.com - just tell em I sent 'ya!

[shanis42p]

I moved all of the code that prints the value of $coating outside the switch statement. But no luck, still puts the color one row after where it should be.

Thanks again,
Shane

One day thumb drives will rule our everday life.

[jpmoriarty]

okay can you post some of your generated html?

[shanis42p]

I see what you mean about useless code, I took all the spans out.

<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"
"http://www.w3.org/TR/html4/loose.dtd">
<html>
<head>
<title>Untitled Document</title>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<style type="text/css">
<!--
body {
	background-color: #000000;
	background-image: url(Images/MainFrameBG.gif);
}
.style1 {color: #FFFFFF}
body,td,th {
	color: #FFFFFF;
	font-family: Geneva, Arial, Helvetica, sans-serif;
	font-size: 12px;
	text-align: center;
}
-->
</style></head>

<body>
<div align="left"></div>

<?php 
//-----------------------------------------------------
// INITIALIZING VARIABLES
	$hostname = "localhost";
	$username = "root";
	$password = "mastercut";
	$dbname = "mastercut";
	$usertable = "tblendmills";

//-------------------------------------------------------
// DEBUGGING TO SEE IF VARIABLE ARE PASSED, ..  VARIBLES POSTED FROM FORM
	print "You Entered into the form:";
	print "<BR><BR>";
	print "UNITS: $units";
	print "<BR>";
	print "TYPE: $type";
	print "<BR>";
	print "PART NUMBER: $partnum";
	print "<BR>";
	print "DIAMETER: $diameter";
	print "<BR>";
	print "CUTTING EDGE LENGTH $LOC";
	print "<BR>";
	print "SHANK $shank";
	print "<BR>";
	print "OAL $oal";
	print "<BR>";
	print "NO OF FLUTES: $flutes";
	print "<BR>";
	print "ENDTYPE $endtype";
	print "<BR>";
	print "COATING $coating";
	print "<BR><BR>";

//---------------------------------------------------	
// CONNECTION TO DATABASE	
	mysql_connect($hostname,$username,$password) or die("Unable to connect to database");

mysql_select_db($dbname) or die("Unable to select database");


// REASSIGNING EMPTY VARIABLES TO WILDCARDS
//----------------------------------------------------	
	if(empty($units)){
		$units = '%';
	}	
//------------------------------------------	
	if(empty($partnum)){
		$partnum = '%';
	}
//-----------------------------------------------------	
	if(empty($diameter)){
		$diameter = '%';
	}
//------------------------------------------------
	if(empty($LOC)){
		$LOC = '%';
	}
//----------------------------------------------------	
	if(empty($shank)){
		$shank = '%';
	}
//-------------------------------------------------------
	if(empty($oal)){
		$oal = '%';
	}

//QUERYING DATABASE FOR RECORDS
	$result = mysql_query("SELECT * FROM tblendmills WHERE 
	tblendmills.type LIKE '$type' AND
	tblendmills.units LIKE '$units' AND 
	tblendmills.partnum LIKE '$partnum' AND 
	tblendmills.diameter LIKE '$diameter' AND 
	tblendmills.LOC LIKE '$LOC' AND 
	tblendmills.shank LIKE '$shank' AND 
	tblendmills.oal LIKE '$oal' AND 
	tblendmills.flutes LIKE '$flutes' AND 
	tblendmills.coating LIKE '$coating' AND
	tblendmills.endtype LIKE '$endtype'");

$number = mysql_num_rows($result);

// SETTING UP TABLE HEADS
?>

<table width="757" border="1">
  <tr scope="col">
     <th>&nbsp;</th>
     <th>&nbsp;</th>
     <th>&nbsp;</th>
     <th>&nbsp;</th>
     <th colspan="4" bgcolor="#0066FF">Dimensions</th>
     <th>&nbsp;</th>
     <th>&nbsp;</th></tr>
  <tr scope="col">
     <th bgcolor="#0066FF">Part Number</th>
     <th bgcolor="#0066FF">Endmill Type</th>
     <th bgcolor="#0066FF">Flutes</th>
     <th bgcolor="#0066FF">Endtype</th>
     <th bgcolor="#0066FF">Cutting Edge Diameter</th>
     <th bgcolor="#0066FF">Cutting Edge Length</th>
     <th bgcolor="#0066FF">Shank Diameter</th>
     <th bgcolor="#0066FF">Overall Length</th>
     <th bgcolor="#0066FF">Coating</th>
     <th bgcolor="#0066FF">Price</th></tr>


<?php	
	$i = 0;

if($number == 0){
	print "No Parts Found";
}
elseif ($number > 0){
	print "There Were $number Endmills Found Matching Your Query: <BR>";

	while ($i < $number):
?>
		<tr>
    		<td>
				<?php
					$partnum = mysql_result($result,$i,"partnum");
					print "$partnum";
				?></td>
			<td>
				<?php
					$type = mysql_result($result,$i,"type");
					print "$type";
				?></td>
			<td>
				<?php
					$flutes = mysql_result($result,$i,"flutes");
					print "$flutes";
				?></td>	
			<td>
				<?php
					$endtype = mysql_result($result,$i,"endtype"); 
					print "$endtype";
				?></td>
			<td>
				<?php
					$diameter = mysql_result($result,$i,"Diameter");
					print "$diameter";
				?></td>
			<td>
				<?php
					$LOC = mysql_result($result,$i,"LOC");
					print "$LOC";
				?></td>
			<td>
				<?php
					$shank = mysql_result($result,$i,"Shank");
					print "$shank";
				?></td>
			<td>
				<?php
					$oal = mysql_result($result,$i,"oal");
					print "$oal";
				?></td>
			<?php
				switch ($coating) 
				{ 
     				case "TiN":?> 
        				<td bgcolor="#FFCC33"><?php 
        				break; 

				case "TiCN":?> 
    				<td bgcolor="#999999"><?php 
    				break; 

				case "TiALN":?> 
    				<td bgcolor="#660033"><?php  
    				break; 

				default:?> 
    				<td bgcolor="#000000"><?php 
    				break;     
			}

			$coating = mysql_result($result,$i,"coating"); 
    		print "$coating";?> 
    		</td>

			<td>
				<?php
					$list = mysql_result($result,$i,"list");
					print "$list";
				?></td>
	</tr>
	<?php
	$i++;
endwhile;
}		

?>
</body>
</html>

My long a$$ code is makinkg this a long thread,

OK, you talked me into it, I removed all the spans from my table heads and table data. It defenitly makes it much easier to read.

Sorry for posting essential the same code two posts in a row.

Thanks,
Shane

[jpmoriarty]

i'm not entirely sure what the trouble is. I asked for you to post the HTML code, not the php. Run your script and post the generated html code, and say again what trouble you're having. when I copied and pasted bits of your code out i got a mainly blue table with 4 boxes toip right black and two top right black - but that's what you want isnt it?

[shanis42p]

Sorry John,

I thought you wanted to see the HTML that was above my php code. But anyways I tried to find something that did not return too many results (The Database has 7000+ records).

Here goes (this is the HTML when I hit view source while previewing)

DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN"
"http://www.w3.org/TR/html4/loose.dtd">
<html>
<head>
<title>Untitled Document</title>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<style type="text/css">
<!--
body {
background-color: #000000;
background-image: url(Images/MainFrameBG.gif);
}
.style1 {color: #FFFFFF}
body,td,th {
color: #FFFFFF;
font-family: Geneva, Arial, Helvetica, sans-serif;
font-size: 12px;
text-align: center;
}
-->
</style></head>

<body>
<div align="left"></div>

You Entered into the form:<BR><BR>UNITS: Metric<BR>TYPE: Medium Pitch Rougher<BR>PART NUMBER: <BR>DIAMETER: <BR>CUTTING EDGE LENGTH <BR>SHANK <BR>OAL <BR>NO OF FLUTES: 4<BR>ENDTYPE Square<BR>COATING %<BR><BR>
<table width="757" border="1">
<tr scope="col">
<th>&nbsp;</th>
<th>&nbsp;</th>
<th>&nbsp;</th>
<th>&nbsp;</th>
<th colspan="4" bgcolor="#0066FF">Dimensions</th>
<th>&nbsp;</th>
<th>&nbsp;</th></tr>
<tr scope="col">
<th bgcolor="#0066FF">Part Number</th>
<th bgcolor="#0066FF">Endmill Type</th>
<th bgcolor="#0066FF">Flutes</th>
<th bgcolor="#0066FF">Endtype</th>
<th bgcolor="#0066FF">Cutting Edge Diameter</th>
<th bgcolor="#0066FF">Cutting Edge Length</th>
<th bgcolor="#0066FF">Shank Diameter</th>
<th bgcolor="#0066FF">Overall Length</th>
<th bgcolor="#0066FF">Coating</th>
<th bgcolor="#0066FF">Price</th></tr>

<center>There Were 12 Endmills Found Matching Your Query: </center><BR>
<tr>
<td>86912</td>
<td>Medium Pitch Rougher</td>
<td>4</td>
<td>Square</td>
<td>12</td>
<td>25</td>
<td>12</td>
<td>75</td>
<td bgcolor="#000000">Uncoated</td>
<td>75.40</td>
</tr>

<tr>
<td>86913</td>
<td>Medium Pitch Rougher</td>
<td>4</td>
<td>Square</td>
<td>12</td>
<td>25</td>
<td>12</td>
<td>75</td>
<td bgcolor="#000000">TiN</td>
<td>83.00</td>
</tr>

<tr>
<td>86914</td>
<td>Medium Pitch Rougher</td>
<td>4</td>
<td>Square</td>
<td>12</td>
<td>25</td>
<td>12</td>
<td>75</td>
<td bgcolor="#FFCC33">TiCN</td>
<td>86.80</td>
</tr>

<tr>
<td>86915</td>
<td>Medium Pitch Rougher</td>
<td>4</td>
<td>Square</td>
<td>12</td>
<td>25</td>
<td>12</td>
<td>75</td>
<td bgcolor="#999999">TiALN</td>
<td>86.80</td>
</tr>

<tr>
<td>86916</td>
<td>Medium Pitch Rougher</td>
<td>4</td>
<td>Square</td>
<td>16</td>
<td>32</td>
<td>16</td>
<td>88</td>
<td bgcolor="#660033">Uncoated</td>
<td>129.00</td>
</tr>

<tr>
<td>86917</td>
<td>Medium Pitch Rougher</td>
<td>4</td>
<td>Square</td>
<td>16</td>
<td>32</td>
<td>16</td>
<td>88</td>
<td bgcolor="#000000">TiN</td>
<td>139.20</td>
</tr>

<tr>
<td>86918</td>
<td>Medium Pitch Rougher</td>
<td>4</td>
<td>Square</td>
<td>16</td>
<td>32</td>
<td>16</td>
<td>88</td>
<td bgcolor="#FFCC33">TiCN</td>
<td>145.10</td>
</tr>

<tr>
<td>86919</td>
<td>Medium Pitch Rougher</td>
<td>4</td>
<td>Square</td>
<td>16</td>
<td>32</td>
<td>16</td>
<td>88</td>
<td bgcolor="#999999">TiALN</td>
<td>145.10</td>
</tr>

<tr>
<td>86920</td>
<td>Medium Pitch Rougher</td>
<td>4</td>
<td>Square</td>
<td>20</td>
<td>38</td>
<td>20</td>
<td>100</td>
<td bgcolor="#660033">Uncoated</td>
<td>151.00</td>
</tr>

<tr>
<td>86921</td>
<td>Medium Pitch Rougher</td>
<td>4</td>
<td>Square</td>
<td>20</td>
<td>38</td>
<td>20</td>
<td>100</td>
<td bgcolor="#000000">TiN</td>
<td>163.00</td>
</tr>

<tr>
<td>86922</td>
<td>Medium Pitch Rougher</td>
<td>4</td>
<td>Square</td>
<td>20</td>
<td>38</td>
<td>20</td>
<td>100</td>
<td bgcolor="#FFCC33">TiCN</td>
<td>171.00</td>
</tr>

<tr>
<td>86923</td>
<td>Medium Pitch Rougher</td>
<td>4</td>
<td>Square</td>
<td>20</td>
<td>38</td>
<td>20</td>
<td>100</td>
<td bgcolor="#999999">TiALN</td>
<td>171.00</td>
</tr>

</body>
</html>

I did view source and touched it up a little bit to make it more readable on this post. I did not delete anything. OK here is the problem, take for example the first row that is a returned result.

<tr>
<td>86912</td>
<td>Medium Pitch Rougher</td>
<td>4</td>
<td>Square</td>
<td>12</td>
<td>25</td>
<td>12</td>
<td>75</td>
<td bgcolor="#000000">Uncoated</td>
<td>75.40</td>
</tr>

This row for example is correct the background color is #000000 which is correct for Uncoated.

The second Row:

<tr>
<td>86913</td>
<td>Medium Pitch Rougher</td>
<td>4</td>
<td>Square</td>
<td>12</td>
<td>25</td>
<td>12</td>
<td>75</td>
<td bgcolor="#000000">TiN </td>
<td>83.00</td>
</tr>

In this case the second row bgcolor="#000000" is incorrect for "TiN" TiN should be yellow in color not black.

TiN should be Yellow #FFCC33, TiCN should be gray #999999, TiALN should be purple (what ever that hex-code is).

My code is placing the color one data row lower than where assigned by my code.

The following row lists the yellow color, but it shound be gray because the data for that row indicates TiCN.
tr>[/COLOR]

Hope that will help some. I realized when you told me that you could not see the problem, you dont have my database so it would make it pretty hard to see anything. If there is any other information I can get fell free to let me know. I am pulling my hair out over this.

My finger is starting to hurt from all the scrolling.

Thank you, You are the best,
Shane

[jpmoriarty]

ahhh now i understand what you're doing!

Okay, I think it's (you'll kick yourself) fairly simple.

In the rest of your code you do your mysql_result etc and get the value from the database. Nice.

But then you just do a switch($coating) - you haven't got the new value from the db yet. You do that at the end, which means that when the code runs around again, it's got the $coating value from the previous row to work with and to run the switch on.

I would think that if you move your
$coating = mysql_result($result,$i,"coating");
to before the switch, not after it, then you'll be fine 🙂

oh also, you might find it easier to do mysql_fecth_array:

$row = mysql_fetch_array($result);
echo $row['partnum'];
echo $row['type'];

etc, rather than doing all your queries.

[shanis42p]

Thanks, that was it. Like you said I feel the need to give myself a wedgie for missing it.

Mad Props,

Much Appreachiated,
Shane