Different cell background color for different year

[chrisdee]

Hi,

Any idea how I can print out different background for my table below depending on wich year my mp3's are from ?.
For example I would like one background color for all cells with mp3 files from 2015, another color for all cells with mp3 files from 2014, and so on.

<style type="text/css"> a:link { text-decoration:  none; } table { border-collapse: collapse; } td { border: 1px dotted; } </style>
<?php
// Variables
$dir    = glob("*.mp3");
$sort    = $_GET['sort'];

// Functions for sorting each table column title
function sort_name($a,$b) {    return $a > $b;                            }
function sort_size($a,$b) {    return filesize($a) < filesize($b);        }
function sort_time($a,$b) {    return filemtime($b) - filemtime($a);    }

// Checks wich table column title is selected
if      ($sort=='name') {       usort($dir, "sort_name");  }
elseif  ($sort=='size') {       usort($dir, "sort_size");  }
else                    {       usort($dir, "sort_time");  }

// Table
echo "<table>";
echo "<tr>";
$titles = array('nr','name','size','date');
foreach($titles as $title) {
    if        ($title==$sort)    { echo "<td bgcolor=lightgreen>&nbsp;<b><a href=index.php?sort=$title>".ucfirst($title)."</a></b>&nbsp;</td>"; }
    elseif    ($title=='nr')    { echo "<td bgcolor=lightblue>&nbsp;<a href=./>".ucfirst($title)."</a>&nbsp;</td>"; }
    elseif    ($title=='time'){ echo "<td bgcolor=lightblue>&nbsp;".ucfirst($title)."&nbsp;</td>"; }
    else                    { echo "<td bgcolor=lightblue>&nbsp;<a href=index.php?sort=$title>".ucfirst($title)."</a>&nbsp;</td>"; }
}
echo "</tr>";

// Table content
for($i=0; $i<count($dir); $i++) {
    echo "<tr>";
        echo "<td>&nbsp;" . $i . "&nbsp;</td>";
        echo "<td>&nbsp;<a href=" . rawurldecode($dir[$i]) . ">" . $dir[$i] . "</a>&nbsp;</td>";
        echo "<td>&nbsp;" . round(filesize($dir[$i])/(1024)/(1024),1) . " MB&nbsp;</td>";
        echo "<td>&nbsp;" . date("d.m.Y", filemtime($dir[$i])) . "<sub> - " .date("H:i:s", filemtime($dir[$i])) . "</sub>&nbsp;</td>";
        echo "</tr>";
}

echo "</table><br>";
?>

[Bonesnap]

Just a couple things looking over your code:

1. In your sorting functions, what happens when the values are equal?
2. I'm pretty sure the HTML attribute "bgcolor" is deprecated. You're better off doing your colouring via CSS.
3. Move your call to count() outside your loop and instead provide the result to the for loop. Otherwise the function has to execute every time the loop runs.
4. Where are you getting the year from the .mp3 file? 😕

[cretaceous]

can't you do:

if(date("Y", filemtime($dir[$i])) ==2014)echo "<td style='background-color:#fc0'>&nbsp;" . date("d.m.Y", filemtime($dir[$i])) . "<sub> - " .date("H:i:s", filemtime($dir[$i])) . "</sub>&nbsp;</td>"; 

of course using appropriate switch or if/else structure and with class based css

as Bonesnap says bgcolor went out last century

[chrisdee]

Thank you to both.

Below is the working script. I'm thinking there must be a shorter and maby better way, but it works.

<style type="text/css"> a:link { text-decoration:  none; } table { border-collapse: collapse; } td { border: 1px dotted; } </style>
<?php
// Variables
$dir	= glob("*.mp3");
$sort	= $_GET['sort'];

// Functions for sorting each table column title
function sort_name($a,$b) {	return $a > $b;							}
function sort_size($a,$b) {	return filesize($a) < filesize($b);		}
function sort_time($a,$b) {	return filemtime($b) - filemtime($a);	}

// Checks wich table column title is selected
if      ($sort=='name') {       usort($dir, "sort_name");  }
elseif  ($sort=='size') {       usort($dir, "sort_size");  }
else                    {       usort($dir, "sort_time");  }

// Table
echo "<table>";
echo "<tr>";
$titles = array('nr','name','size','date');
foreach($titles as $title) {
	if		($title==$sort)	{ echo "<td bgcolor=lightgreen>&nbsp;<b><a href=index.php?sort=$title>".ucfirst($title)."</a></b>&nbsp;</td>"; }
	elseif	($title=='nr')	{ echo "<td bgcolor=lightblue>&nbsp;<a href=./>".ucfirst($title)."</a>&nbsp;</td>"; }
	elseif	($title=='time'){ echo "<td bgcolor=lightblue>&nbsp;".ucfirst($title)."&nbsp;</td>"; }
	else					{ echo "<td bgcolor=lightblue>&nbsp;<a href=index.php?sort=$title>".ucfirst($title)."</a>&nbsp;</td>"; }
}
echo "</tr>";

// Table content
for($i=0; $i<count($dir); $i++) {
	if(date("Y", filemtime($dir[$i])) == 2015) {
		echo "<tr>";
	        echo "<td style='background-color:#fc0'>&nbsp;" . $i . "&nbsp;</td>";
    	    echo "<td style='background-color:#fc0'>&nbsp;<a href=" . rawurldecode($dir[$i]) . ">" . $dir[$i] . "</a>&nbsp;</td>";
        	echo "<td style='background-color:#fc0'>&nbsp;" . round(filesize($dir[$i])/(1024)/(1024),1) . " MB&nbsp;</td>";
        	echo "<td style='background-color:#fc0'>&nbsp;" . date("d.m.Y", filemtime($dir[$i])) . "<sub> - " .date("H:i:s", filemtime($dir[$i])) . "</sub>&nbsp;</td>";
        echo "</tr>";
	}
	elseif(date("Y", filemtime($dir[$i])) == 2014) {
		echo "<tr>";
	        echo "<td style='background-color:#fc9'>&nbsp;" . $i . "&nbsp;</td>";
    	    echo "<td style='background-color:#fc9'>&nbsp;<a href=" . rawurldecode($dir[$i]) . ">" . $dir[$i] . "</a>&nbsp;</td>";
        	echo "<td style='background-color:#fc9'>&nbsp;" . round(filesize($dir[$i])/(1024)/(1024),1) . " MB&nbsp;</td>";
        	echo "<td style='background-color:#fc9'>&nbsp;" . date("d.m.Y", filemtime($dir[$i])) . "<sub> - " .date("H:i:s", filemtime($dir[$i])) . "</sub>&nbsp;</td>";
        echo "</tr>";
	}
	elseif(date("Y", filemtime($dir[$i])) == 2013) {
		echo "<tr>";
	        echo "<td style='background-color:#fcf'>&nbsp;" . $i . "&nbsp;</td>";
    	    echo "<td style='background-color:#fcf'>&nbsp;<a href=" . rawurldecode($dir[$i]) . ">" . $dir[$i] . "</a>&nbsp;</td>";
        	echo "<td style='background-color:#fcf'>&nbsp;" . round(filesize($dir[$i])/(1024)/(1024),1) . " MB&nbsp;</td>";
        	echo "<td style='background-color:#fcf'>&nbsp;" . date("d.m.Y", filemtime($dir[$i])) . "<sub> - " .date("H:i:s", filemtime($dir[$i])) . "</sub>&nbsp;</td>";
        echo "</tr>";
	}
		elseif(date("Y", filemtime($dir[$i])) == 2010) {
		echo "<tr>";
	        echo "<td style='background-color:#def'>&nbsp;" . $i . "&nbsp;</td>";
    	    echo "<td style='background-color:#def'>&nbsp;<a href=" . rawurldecode($dir[$i]) . ">" . $dir[$i] . "</a>&nbsp;</td>";
        	echo "<td style='background-color:#def'>&nbsp;" . round(filesize($dir[$i])/(1024)/(1024),1) . " MB&nbsp;</td>";
        	echo "<td style='background-color:#def'>&nbsp;" . date("d.m.Y", filemtime($dir[$i])) . "<sub> - " .date("H:i:s", filemtime($dir[$i])) . "</sub>&nbsp;</td>";
        echo "</tr>";
	}
			elseif(date("Y", filemtime($dir[$i])) == 2008) {
		echo "<tr>";
	        echo "<td style='background-color:#add'>&nbsp;" . $i . "&nbsp;</td>";
    	    echo "<td style='background-color:#add'>&nbsp;<a href=" . rawurldecode($dir[$i]) . ">" . $dir[$i] . "</a>&nbsp;</td>";
        	echo "<td style='background-color:#add'>&nbsp;" . round(filesize($dir[$i])/(1024)/(1024),1) . " MB&nbsp;</td>";
        	echo "<td style='background-color:#add'>&nbsp;" . date("d.m.Y", filemtime($dir[$i])) . "<sub> - " .date("H:i:s", filemtime($dir[$i])) . "</sub>&nbsp;</td>";
        echo "</tr>";
	}
	else {
	echo "<tr>";
        echo "<td>&nbsp;" . $i . "&nbsp;</td>";
        echo "<td>&nbsp;<a href=" . rawurldecode($dir[$i]) . ">" . $dir[$i] . "</a>&nbsp;</td>";
        echo "<td>&nbsp;" . round(filesize($dir[$i])/(1024)/(1024),1) . " MB&nbsp;</td>";
        echo "<td>&nbsp;" . date("d.m.Y", filemtime($dir[$i])) . "<sub> - " .date("H:i:s", filemtime($dir[$i])) . "</sub>&nbsp;</td>";
        echo "</tr>";
}}

echo "</table><br>";
?>

[Bonesnap]

You could move the output into a function that takes the year as an argument so you're not duplicating so much code:

function print_row($year)
{
    $class = '';

switch($year)
{
    case 2015:
        $class = 'colour1';
        break;

    case 2014:
        $class = 'colour2';
        break;

    case 2013:
        $class = 'colour3';
        break;

    case 2010:
        $class = 'colour4';
        break;

    default:
        $class = 'defaultcolor';
        break;
}

echo "<tr class='".$class."'>";
    echo "<td>&nbsp;" . $i . "&nbsp;</td>";
    echo "<td>&nbsp;<a href=" . rawurldecode($dir[$i]) . ">" . $dir[$i] . "</a>&nbsp;</td>";
    echo "<td>&nbsp;" . round(filesize($dir[$i])/(1024)/(1024),1) . " MB&nbsp;</td>";
    echo "<td>&nbsp;" . date("d.m.Y", filemtime($dir[$i])) . "<sub> - " .date("H:i:s", filemtime($dir[$i])) . "</sub>&nbsp;</td>";
echo "</tr>";
}

Then you can call it:

for($i=0; $i<count($dir); $i++)
{
    print_row(date("Y", filemtime($dir[$i])));
}

This follows the DRY principle, and as you can see makes your code a lot easier to read and maintain.

[chrisdee]

Thanks. But I'm not getting it to work properly.
It doesn't print out in the table and all it prints out all the files as MB 01.01.1970 - 01:00:00

<style type="text/css"> a:link { text-decoration:  none; } table { border-collapse: collapse; } td { border: 1px dotted; } </style>
<?php
// Variables
$dir    = glob("*.mp3");
$sort   = $_GET['sort'];

// Functions for sorting each table column title
function sort_name($a,$b) {     return $a > $b;                         }
function sort_size($a,$b) {     return filesize($a) < filesize($b);     }
function sort_time($a,$b) {     return filemtime($b) - filemtime($a);   }

// Checks wich table column title is selected
if      ($sort=='name') {       usort($dir, "sort_name");  }
elseif  ($sort=='size') {       usort($dir, "sort_size");  }
else                    {       usort($dir, "sort_time");  }

// Table
echo "<table>";
echo "<tr>";
$titles = array('nr','name','size','date');
foreach($titles as $title) {
        if      ($title==$sort) { echo "<td bgcolor=lightgreen>&nbsp;<b><a href=".$_SERVER['PHP_SELF']."?sort=$title>".ucfirst($title)."</a></b>&nbsp;</td>"; }
        elseif  ($title=='nr')  { echo "<td bgcolor=lightblue>&nbsp;<a href=./>".ucfirst($title)."</a>&nbsp;</td>"; }
        elseif  ($title=='time'){ echo "<td bgcolor=lightblue>&nbsp;".ucfirst($title)."&nbsp;</td>"; }
        else                    { echo "<td bgcolor=lightblue>&nbsp;<a href=".$_SERVER['PHP_SELF']."?sort=$title>".ucfirst($title)."</a>&nbsp;</td>"; }
}
echo "</tr>";

// Table content
function print_row($year) {
        $class = '';
        switch($year) {
                case 2015:
                        $class = 'orange';
                        break;
                case 2014:
                        $class = 'green';
                        break;
                case 2013:
                        $class = 'blue';
                        break;
                case 2010:
                        $class = 'red';
                        break;
                default:
                        $class = 'yellow';
                        break;
}

echo "<tr class='".$class."'>";
        echo "<td>&nbsp;" . $i . "&nbsp;</td>";
        echo "<td>&nbsp;<a href=" . rawurldecode($dir[$i]) . ">" . $dir[$i] . "</a>&nbsp;</td>";
        echo "<td>&nbsp;" . round(filesize($dir[$i])/(1024)/(1024),1) . " MB&nbsp;</td>";
        echo "<td>&nbsp;" . date("d.m.Y", filemtime($dir[$i])) . "<sub> - " .date("H:i:s", filemtime($dir[$i])) . "</sub>&nbsp;</td>";
echo "</tr>";
}

echo "</table><br>";

for($i=0; $i<count($dir); $i++) {
        print_row(date("Y", filemtime($dir[$i])));
}

?>

[cretaceous]

The date of 01.01.1970 indicates that it is picking up '0' when it gets the date value

This bit

for($i=0; $i<count($dir); $i++) { 
...
}

is the part producing the $i value so you need to output the table rows inside that loop so that $i has a value and it increments each time the loop runs. Where you have it $i does not exist.

[chrisdee]

I'v tried to move the for loop bit all around the place but I can't get it to work.
If I place it outside the function you created it echo'es 01.01.1970.
If I place it inside the function nothing is echo'ed.

[cretaceous]

You need to grasp the logic of what is happening 😉
It has to be inside your (not my) loop, as the loop defines $i - that is just a counter so it says 'get me item 0' on first run, on second run 'get me item 1' and so on.
$dir contains the file names of the files in your directory

What happens if you do this:

for($i=0; $i<count($dir); $i++) {
        echo "<tr class='".$class."'>";
        echo "<td>&nbsp;" . $i . "&nbsp;</td>";
        echo "<td>&nbsp;<a href=" . rawurldecode($dir[$i]) . ">" . $dir[$i] . "</a>&nbsp;</td>";
        echo "<td>&nbsp;" . round(filesize($dir[$i])/(1024)/(1024),1) . " MB&nbsp;</td>";
        echo "<td>&nbsp;" . date("d.m.Y", filemtime($dir[$i])) . "<sub> - " .date("H:i:s", filemtime($dir[$i])) . "</sub>&nbsp;</td>";
        echo "</tr>";

//and to debug:
echo "\n<tr><td colspan='4'>item: ",$i," filename: ",$dir[$i],"</td></tr>";
}

// add this after you do the glob on the files to see if $dir contains anything
echo 'file count: ',count ($dir);