Nestled loops to show pictures, i think?

inkfish

Hello,

I've got a problem. I have tabel in a database that has fields: id, gallery and path. now i want to show every post that has a specific gallery like this:

<table border="0" cellpadding="0" cellspacing="0">
	<tr><td height="20"></td></tr>
	<tr>
		<td width="20"></td>
		<td width="100" height="130" align="center" valign="middle" bgcolor="#F3F3F3"><a href="emmyivitt0.jpg"><img src="emmyivitt0_t.jpg" border="0"></a></td>
		<td width="20"></td>
		<td width="100" height="130"  align="center" valign="middle" bgcolor="#F3F3F3"><a href="emmyivitt1.jpg"><img src="emmyivitt1_t.jpg" border="0"></a></td>
		<td width="20"></td>
		<td width="100" height="130"  align="center" valign="middle" bgcolor="#F3F3F3"><a href="emmyivitt2.jpg"><img src="emmyivitt2_t.jpg" border="0"></a></td>
		<td width="20"></td>
		<td width="100" height="130"  align="center" valign="middle" bgcolor="#F3F3F3"><a href="emmyivitt3.jpg"><img src="emmyivitt3_t.jpg" border="0"></a></td>
		<td width="20"></td>
		<td width="100" height="130"  align="center" valign="middle" bgcolor="#F3F3F3"><a href="emmyivitt4.jpg"><img src="emmyivitt4_t.jpg" border="0"></a></td>
		<td width="20"></td>
	</tr>
	<tr><td height="20"></td></tr>
	<tr>
		<td width="20"></td>
		<td width="100" height="130" align="center" valign="middle" bgcolor="#F3F3F3"><a href="emmyivitt5.jpg"><img src="emmyivitt5_t.jpg" border="0"></a></td>
		<td width="20"></td>
		<td width="100" height="130"  align="center" valign="middle" bgcolor="#F3F3F3"><a href="emmyivitt6.jpg"><img src="emmyivitt6_t.jpg" border="0"></a></td>
		<td width="20"></td>
		<td width="100" height="130"  align="center" valign="middle" bgcolor="#F3F3F3"><a href="emmyivitt7.jpg"><img src="emmyivitt7_t.jpg" border="0"></a></td>
		<td width="20"></td>
		<td width="100" height="130"  align="center" valign="middle" bgcolor="#F3F3F3"><a href="emmyivitt8.jpg"><img src="emmyivitt8_t.jpg" border="0"></a></td>
		<td width="20"></td>
		<td width="100" height="130"  align="center" valign="middle" bgcolor="#F3F3F3"></td>
		<td width="20"></td>
	</tr>
</table>

There the pictures (the path in the tabel is a path to a picture) get put up in a nice way. But here i've not taken anything from the database. What i want is the script to first take five pictures and put them out and if its more then five pictures (in that gallery) then it starts a new <tr></tr> and puts them out and if there is only 8 pictures then the last two <td>'s will be left empty like my last one is in the example.

I know im maybe making it hard to understand but tell me what i should explain more and i'll do it.

Thanks!!

inkfish

I've tried to figure it out but i just don't get how to do it! Anybody? Please?

wilsonodk

Here is a idea of a direction to take with it...
It only uses one of your database columns,
so it will probably need tweaking.

// code removed

| Use corrected code below |

inkfish

Hello, wilsonodk!

Thank you for taking time doing this for me =) Have i got it right if i think that $variabel .= "Something"; Adds the "something" to the variabel? I've never used .= before.

One more thing. This part:

 
      $output .= ".jpg\"><img_src=\"$data['path']";
      $output .= '_t.jpg" border="0"></a></td>';

when you do that i don't need the thumb field in the tabel right?
the thumb field had the path to the thumbnail.

I think i understand the most of it,

Thank you VERY much! =)

inkfish

Back again, now with an error message. =/

I get this:

Parse error: parse error, unexpected T_ENCAPSED_AND_WHITESPACE, expecting T_STRING or T_VARIABLE or T_NUM_STRING in d:\webdesign\webroot\inkfishave\photoset.php on line 42

Line 42 is this:
$output .= "<a_href=\"$data['path']";

I tried to take away the _ from a_href but nothing happend, same error message.

Any ideas?

wilsonodk

inkfish,

You're correct with your guess about .= !!!
You can also use +=, -=, *=, /=, %= the same way.

// line 42 - lets try
$output .= '<a href="' . $data['path'];

Try this and let me know how it works.

inkfish

Hello again,

That line works now i think ;P BUT..
Now it complains about line 43:

Parse error: parse error, unexpected T_ENCAPSED_AND_WHITESPACE, expecting T_STRING or T_VARIABLE or T_NUM_STRING in d:\webdesign\webroot\inkfishave\photoset.php on line 43

Line 43: $output .= ".jpg\"><img_src=\"$data['path']";

i tried: $output .= ".jpg\"><img_src="' . $data['path']";
but that didn't work =/

wilsonodk

Hello Inkfish,

For reasons that I don't fully understand, when cutting and pasteing code some times an extra underscore is added.

I'll go and clean up my example using this stuff, but in the mean time...

// Line 43: try this
$output .= '.jpg"><img src="' . $data['path'];

Good Luck

inkfish

No error message but nothing happends when i clik on the link:

<a href="photoset.php?photoscat_id=<? print $row["id"] ?>" target="photoset"><? print $cat_name ?></a>

Photoset.php = the page where the code we have been working with is.

My code now looks like this. The connection to the database is done and i know that is working. Look on the SQL Query, i've changed it to how i want it; so the code we did only show the pictures in a speciall gallery.

<? 
if (isset($photoscat_id)) {

$photos_query = "SELECT * FROM photos WHERE gallery = '$photoscat_id'";
$photos_result = mysql_query($photos_query) or die (mysql_error());

// Now lets get down to work...
// define the blank row
$blankrow = '<tr><td_height="20"></td><td>&nbsp;</td></tr>';
// define the padding cells
$paddingcell = '<td width="20">&nbsp;</td>';

// first start the table
$output = '<table_border="0"_cellpadding="0"_cellspacing="0">';
$output .= $blankrow;



while ($data = mysql_fetch_array($photos_result)) {
	$output .= '<tr>';
	for ($i = 0; $i < 5; $i++) {
		$output .= $paddingcell;
		$output .= '<td width="100" height="130" align="center" valign="middle" bgcolor="#F3F3F3">';
		if ($data[$i]) {
			$output .= '<a href="' . $data['path'];
			$output .= '.jpg"><img src="' . $data['path'];
   		$output .= '_t.jpg" border="0"></a></td>';
		} 
		else {
  		$output .= '&nbsp;</td>';
		}
	}
	$output .= '</tr>';
	$output .= $blankrow;
}
$output = '</table>';
echo $output;
}
?>

EDIT:
I changed $output = '</table>'; to $output .= '</table>'; now something shows up. but it doesn't seam as the <tr> is working as it should. It also seems as it puts only 4 pictures side by side then a space. I'll attach a picture so u can see for ur self.

inkfish

Okey i've fixed some of it. The problem with the layout was that we had some _ on some places instead of just a blank space.

Now to the problems: Every picture is still shown 4 times, i've send a new PICTURE with this post as an attachment. Look on it and you'll see.

A second problem: the picture doesn't come up as u see and this is why, the path to the pictures look like this: /work/photos/1/emmyivitt0.jpg_t.jpg

SO we have to use the $data['thumb'] cause that field has the path to the thumbnail.

$data['path'] = emmyivitt0.jpg
$data['thumb'] = emmyivitt0_t.jpg

inkfish

i changed the outputs to this:

$output .= '<a href="' . $data['path'];
$output .= '"><img src="' . $data['thumb'];
$output .= '" border="0"></a></td>';

now the pictures shows but still 1 picture is showed 4 times then a new <tr> is created and the second picture is showed 4 times and another <tr> is created and here the 3rd picture is shown 4 times and so on... hmmmm...

wilsonodk

Hello Martin,

This turned out to be a little harder than I first thought.
But, here it is all the same, I've tried it out on my machine and got it working, but you might want to sweep through the code to make sure I got everything named back to what you are using and that no underscores creeped in where they don't belong.

I've added a new variable, called $cell, it is the <td> info for each image.

if (isset($photoscat_id)) {

$photos_query = "SELECT * FROM photos WHERE gallery = '$photoscat_id'";
$photos_result = mysql_query($photos_query) or die (mysql_error());

// Now lets get down to work...
// define the blank row
$blankrow = '<tr><td height="20" colspan="11">&nbsp;</td></tr>';
// define the padding cells
$paddingcell = '<td width="20">&nbsp;</td>';
// define an image cell
$cell = '<td width="100" height="130" align="center" valign="middle" bgcolor="#F3F3F3">';

// first start the table
$output = '<table border="0" cellpadding="0" cellspacing="0">'."\n";
$output .= $blankrow;

// The entry counter
$i = 0;

while ($data = mysql_fetch_array($photos_result)) {
	// Is this the fifth entry to a row? 
 // because we start at 0, 4 is the fifth entry
	if ($i == 4) {
		// It is:
  // Place the final entry, add an additional padding cell, 
  // add end the row, add blank row, reset the counter
		$output .= $paddingcell;
		$output .= $cell;

  $output .= '<a href="' . $data['path']; 
  $output .= '"><img src="' . $data['thumb']; 
  $output .= '" border="0"></a></td>'; 

	$output .= $paddingcell;
	$output .= '</tr>'."\n";
	$output .= $blankrow;
	$i = 0;
} else {
	// It is not: 
  // Check if first image of row, place image info, increment
		if ($i==0) $output .= '<tr>';

	$output .= $paddingcell;
	$output .= $cell;

  $output .= '<a href="' . $data['path']; 
  $output .= '"><img src="' . $data['thumb']; 
  $output .= '" border="0"></a></td>'; 

  // incrementing the counter
		$i += 1;

}

}

// Find out how many rows there were 
// and if they were divisible by 5
$numrows = mysql_num_rows($photos_result);

if (($numrows%5) != 0) {
	// we need more entry spots
	while ($i < 5) {
		if ($i == 4) {
			// Last spot
			$output .= $paddingcell;
			$output .= $cell;
			$output .= '&nbsp;</td>';
			$output .= $paddingcell;
			$output .= '</tr>'."\n";
			$output .= $blankrow;
			$i += 1;
		} else {
			// another blank 
			$output .= $paddingcell . $cell . '&nbsp;</td>';
			$i += 1;
		}
	}
} 

$output .= '</table>';
echo $output;
}

I tried to place the counting code inside of the while loop and that didn't work. Sorry, hope this helps you now.

wilson wise

inkfish

Woho! Works perfect! =)

wilsonodk, thank you very much for taking your time to do this =)

you don't have icq or msn? good to have contacts with good knowledge! =)

Thanks again.