[RESOLVED] add image in table after 12 rows mysql

[planethax]

I would like to have

<tr><td colspan="12"><img src="chase.gif" alt="Race for the Chase" /></td></tr>

after 12 rows,

By reading I think I need to have something like this, but not sure how to implement it

$i = 1;  

$chase = '<tr><td colspan="12"><img src="chase.gif" alt="Race for the Chase" /></td></tr>'

foreach($standings as $rank=>$standing)
    if ($i%12 == 0) {
        echo $chase

my current code is as

		$standing_text = "";
foreach($standings as $rank=>$standing)
{
    $standing_text .= '<tr '.($standing['in_contention'] == 1 ? 'class="contendor"' : null).'>';
    $standing_text .= '<td>'.$rank.'</td>';
    foreach(array('rank_delta', 'driver', 'points', 'behind', 'starts', 'poles', 'wins', 'top5', 'top10', 'dnf', 'winnings') as $stat)
    {
        $standing_text .= '<td>'.$standing[$stat].'</td>';
    }
    $standing_text .= '</tr>';
}

$tpl -> AssignArray(array('standing' => $standing_text));

[johanafm]

$i = 0;
foreach($standings as $rank=>$standing)
{
	if (++$i % 12 == 0) {
		echo $chase 
	}
	else {
		// loop as it is now
	}
}

[planethax]

What have I done wrong here?

$standings = array();
while($row = mysql_fetch_assoc($result))
{
    $standings[$row['rank']] = $row;
}

	$standing_text = "";
$i = 0;  

$chase = '<tr><td colspan="12"><img src="chase-bar.gif" alt="Race for the Chase" /></td></tr>'

foreach($standings as $rank=>$standing)
{
    if (++$i % 12 == 0) {
        echo $chase
    }
    else {
       {
    $standing_text .= '<tr '.($standing['in_contention'] == 1 ? 'class="contendor"' : null).'>';
    $standing_text .= '<td>'.$rank.'</td>';
    foreach(array('rank_delta', 'driver', 'points', 'behind', 'starts', 'poles', 'wins', 'top5', 'top10', 'dnf', 'winnings') as $stat)
    {
        $standing_text .= '<td>'.$standing[$stat].'</td>';
    }
    $standing_text .= '</tr>';
}

} 

[johanafm]

Have a look at the reported error, and then indent your code properly to find what causes it.

[planethax]

johanafm;10926189 wrote:

Have a look at the reported error, and then indent your code properly to find what causes it.

There is no error, it just does not compile the page (blank section)

Being a newb, what doe you mean by indent properly? and what does that do?

Thanx for your reply.

[planethax]

Anyone help me here?
really trying to figure this out and see where I went wrong.

I am also thinking about removing the Class contender and just have every other row a diff class

eg row 1 class=d0 row 2 class=d1 row 3class=d0 row 4 class=d1 and so on.

[planethax]

The current code just has a blank section, how do I find out my error?

[johanafm]

Indenting is using tabs to properly structure your code for readability. Have a look at code formatting

Among other things, stay consistent

	/* usually you keep the opening bracket on the same line.
		but here you have it on both the opening line and the next one... */
    else {
       {
    $standing_text .= '<tr '.($standing['in_contention'] == 1 ? 'class="contendor"' : null).'>';
    $standing_text .= '<td>'.$rank.'</td>';
	// except sometimes it's on the next one.
    foreach(array('rank_delta', 'driver', 'points', 'behind', 'starts', 'poles', 'wins', 'top5', 'top10', 'dnf', 'winnings') as $stat)
    {
        $standing_text .= '<td>'.$standing[$stat].'</td>';
    }
    $standing_text .= '</tr>';

// If you code was properly indented, having two closing brackets in the same column position would be an indication that something isn't right.
}

}
?>

And there are lots of errors in the code. Turn on error reporting. Wether you echo to screen or just log them to file doesn't matter, as long as you get to see them one way or other.

You lack ; to end statements. You have an unmatched number of { and } which means you have ?> where php will expect }.

[planethax]

Ah, thank you.

How do I turn on error reporting or echo all errors?

Is this more on track?

	$standings = array();
	while($row = mysql_fetch_assoc($result))
	{
    	$standings[$row['rank']] = $row;
	}

	    $standing_text = "";
$i = 0;  
$chase = '<tr><td colspan="12"><img src="chase-bar.gif" alt="Race for the Chase" /></td></tr>';

foreach($standings as $rank=>$standing)
{
	if (++$i % 12 == 0) {
    	echo $chase;
						}
		else {
	$standing_text .= '<tr '.($standing['in_contention'] == 1 ? 'class="contendor"' : null).'>';
	$standing_text .= '<td>'.$rank.'</td>';
	foreach(array('rank_delta', 'driver', 'points', 'behind', 'starts', 'poles', 'wins', 'top5', 'top10', 'dnf', 'winnings') as 	$stat)
						{
    	$standing_text .= '<td>'.$standing[$stat].'</td>';
						}
	$standing_text .= '</tr>';
			}
}

I know its not correct, but closer I think, but now I get the image 6 times and to table and page al messed up.

[johanafm]

Have a look at http://se2.php.net/manual/en/errorfunc.configuration.php#ini.error-reporting.

Better but you're still not consistent in use of indentation. Compare the same code below for readability

/*	This is one way of doing it: opening bracket comes in the same column on the next line byt itself,
	and similar for the closing bracket. So, sticking with that for the entire script...*/
while($row = mysql_fetch_assoc($result))
{
	$standings[$row['rank']] = $row;
}

$standing_text = "";
$i = 0;  

$chase = '<tr><td colspan="12"><img src="chase-bar.gif" alt="Race for the Chase" /></td></tr>';

foreach($standings as $rank=>$standing)
{
	if (++$i % 12 == 0)
	{
		echo $chase;
	}
	else
	{
		$standing_text .= '<tr '.($standing['in_contention'] == 1 ? 'class="contendor"' : null).'>';
		$standing_text .= '<td>'.$rank.'</td>';

	foreach(array('rank_delta', 'driver', 'points', 'behind', 'starts', 'poles',
			'wins', 'top5', 'top10', 'dnf', 'winnings') as     $stat)
	{
		$standing_text .= '<td>'.$standing[$stat].'</td>';
	}
	$standing_text .= '</tr>';
}
}

Other than that, your html looks to be ok. However, you assign all table rows to $standing_text, while you echo the image.

$standing_text .= $chase;

[planethax]

Ok, I am starting to get it now, your code is must easier to decypher than mine and I could see how a missing or extra { or } would be seen easier!

Thank you.

Now the following code is almost there, however, it does NOT INSERT a row after 12 rows with the image but it OVERWRITES the twelfth row and EVERY twelfth row?

include_once('db_conn.php');
$query = "SELECT * FROM `nascar_standings` ORDER BY `rank` ASC";
$result = mysql_query($query) or die('MySQL Error: '.mysql_error());

$standings = array();
while($row = mysql_fetch_assoc($result))
{
    $standings[$row['rank']] = $row;
}

$standing_text = "";
$i = 0;  

$chase = '<tr><td colspan="12"><img src="chase-bar.gif" alt="Race for the Chase" /></td></tr>';

foreach($standings as $rank=>$standing)
{
    if (++$i % 12 == 0)
    {
        $standing_text .= $chase;
    }
    else
    {
        $standing_text .= '<tr '.($standing['in_contention'] == 1 ? 'class="contender"' : null).'>';
        $standing_text .= '<td>'.$rank.'</td>';

    foreach(array('rank_delta', 'driver', 'points', 'behind', 'starts', 'poles',
            'wins', 'top5', 'top10', 'dnf', 'winnings') as     $stat)
    {
        $standing_text .= '<td>'.$standing[$stat].'</td>';
    }
    $standing_text .= '</tr>';
}
}

$tpl -> AssignArray(array('standing' => $standing_text));

[johanafm]

Ah, you are correct and I didn't really pay attention to what you are trying to do.

foreach($standings as $rank=>$standing)
{
	// No ifcheck here. echo the actual rows each time
	$standing_text .= '<tr '.($standing['in_contention'] == 1 ? 'class="contender"' : null).'>';
	$standing_text .= '<td>'.$rank.'</td>';

foreach(array('rank_delta', 'driver', 'points', 'behind', 'starts', 'poles',
		'wins', 'top5', 'top10', 'dnf', 'winnings') as     $stat)
{
	$standing_text .= '<td>'.$standing[$stat].'</td>';
}
$standing_text .= '</tr>';

// Also add the image after each twelve rows.
if (++$i % 12 == 0)
{
	$standing_text .= $chase;
}
}

[planethax]

// Also add the image after each twelve rows.
    if (++$i % 12 == 0)
    {
        $standing_text .= $chase;
    }
}

how to change so its only after first 12 rows and not again?

thank you so much for your help and guiding me to learn.

[johanafm]

% is the modulo operator, which returns the "left over" (no idea what this is actually called in english) part in integer division.
1/12 = 0 rest 1
12/12 = 1 rest 0
13/12 = 1 rest 1

So instead of checking when division by 12 has a rest of 0, just if ($i == 12)

[planethax]

Thank you very much!

One more quick ?

(++$i == 12)
WHat is the ++ for?

[johanafm]

It's the increment operator, and exist in two versions: prefix and suffix.

$i = 1;
++$i;	// now $i is 2
$i++;	// and 3

$a = ++$i;	// $i is incremented, then assigned to $a. Both are 4
$b = $i++;	// $i is assigned to $b. then $i is incremented. $b is 4, $i is 5.