Help with opendir and reading files and printing HTML

ukndoit

I did this in Perl and am trying to use it in PHP, by just changing the syntax, I am getting this error now:
Parse error: parse error, unexpected T_IF in /home/mitchell/sitepages/portfolios.php on line 17
Line 17 is this one:

if ($handle = opendir($photo_directory)) {

I don't know PHP, only just playing with it, can you please look at it and tell me what I did wrong?

Here is the 'PHP' code(transformed from Perl syntax to PHP as best as I could figure):

	$_picture_thumbnail_size = "45";
	$_counter = 0;
	$_a_counter = 0;
	$_table_counter = 0;
	$_table_to_print = "
		<tr valign=\"middle\">";
	$photo_directory = "/home/user/www/images/$gallery"
	if ($handle = opendir($photo_directory)) { 
		while (false !== ($file = readdir($handle))) {
			if ($file != "." && $file != "..") {
				if (!preg_match(".(jpg|gif|jpeg|png)", $file)) {
					next;
				}
				if ($_counter <= $_starting_value) {
					next;
				}
				if ($_def_display && ($_a_counter >= $_def_display)) {
					next;
				}
				$_counter++;
				if ($_table_counter && $_table_counter > 2) {
					$_table_counter = 0;
					$_table_to_print .= <<<EOF
			</tr>
			<tr valign="middle">
				<td bgcolor="#990000" style="color: #FFFFFF">
					$_counter
				</td>
				<td>
					<script language="JavaScript">
						document.write('<a href="javascript:;" onclick="window.open(\\'ShowPic.cgi?img=$_what_gallery_2_open/$DirFile\\',\\'POPUP_IMAGE\\',\\'toolbar=no,location=no,menu=no,resizable=no,status=no,scrollbars=yes,top=50,left=100,height=350,width=350\\'); return false;" target="_blank"><img width="$_picture_thumbnail_size" alt="Portfolio Image Number: $_counter" src="$DirFile" border="0"></a>');
					</script>
					<noscript><a href="ShowPic.cgi?img=$_what_gallery_2_open/$DirFile" target="_blank"><img width="$_picture_thumbnail_size" alt="Portfolio Image Number: $_counter" src="$DirFile" border="0"></a></noscript>
				</td>
EOF:
					$_table_counter++;
				} else {
					$_table_to_print .= <<<EOF
				<td bgcolor="#990000" style="color: #FFFFFF">
					$_counter
				</td>
				<td>
					<script language="JavaScript">
						document.write('<a href="javascript:;" onclick="window.open(\\'ShowPic.cgi?img=$_what_gallery_2_open/$DirFile\\',\\'POPUP_IMAGE\\',\\'toolbar=no,location=no,menu=no,resizable=no,status=no,scrollbars=yes,top=50,left=100,height=350,width=350\\'); return false;" target="_blank"><img width="$_picture_thumbnail_size" alt="Portfolio Image Number: $_counter" src="$DirFile" border="0"></a>');
					</script>
					<noscript><a href="ShowPic.cgi?img=$_what_gallery_2_open/$DirFile" target="_blank"><img width="$_picture_thumbnail_size" alt="Portfolio Image Number: $_counter" src="$DirFile" border="0"></a></noscript>
				</td>
EOF;
						$_table_counter++;
				}
				$_a_counter++;
			}
		}
		closedir($handle);
	}

$_FILE_NAME = <<<EOF
<table border="0" bordercolor="#990000" width="75%" align="center" cellpadding="2" cellspacing="2">
	$_table_to_print
	</tr>
	$_links
</table>		
EOF;
	echo $_FILE_NAME;

Thank you for any assistance you can be.
Richard

LordShryku

It's the line above that one missing a ;

ukndoit

Sweet Thanks!

One more Problem thus far:

if (!preg_match(".jpg|.gif|.jpeg|.png", $file)) {

Is that comparable to this Perl syntax:

if($file =~ /\.(jpg|gif|jpeg|png)$/i) {

the $ at the end of the line makes sure that is found at the END of the string. How can I do this in PHP?

Thank you again!
Richard

Weedpacket

The syntax is the same as in Perl (that's why they're called Perl-compatible regular expressions):

if (!preg_match("/\.(jpg|gif|jpeg|png)$/i", $file)) {

And you want [man]continue[/man], not next.

ukndoit

Excellent! Now I am just having a problem. Here is the PHP code, as of right now, I'll tell you after the code what it is doing and what the output is...

		$_counter = 0;
		$_a_counter = 0;
		$_table_counter = 0;
		$_tablecounterandall = 0;
		$_table_to_print = <<<EOF
		<tr valign="middle">
EOF;
		$photo_directory = "/home/user/public_html/images/" . $gallery;
		if ($handle = opendir($photo_directory)) {
			while (false !== ($file = readdir($handle))) {
				if ($file != "." && $file != "..") {
					if (!preg_match("/\.(jpg|gif|jpeg|png)$/i", $file)) {
						continue;
					}
					$_counter++;
					if ($_table_counter > 2) {
						$_table_counter = 0;
						$_tables_and_me = $_tables_and_me . "1+";
						$_table_to_print = $_table_to_print . <<<EOF
			</tr>
			<tr valign="middle">
				<td>
					<script language="JavaScript">
						document.write('<a href="javascript:;" onclick="window.open(\\'ShowPic.cgi?img=gallery/$file\\',\\'POPUP_IMAGE\\',\\'toolbar=no,location=no,menu=no,resizable=no,status=no,scrollbars=yes,top=50,left=100,height=350,width=350\\'); return false;"><img width="$_picture_thumbnail_size" alt="Portfolio Image Number: $_counter" src="$__gallery_url/$gallery/$file" border="0"></a>');
					</script>
					<noscript><a href="ShowPic.cgi?img=$gallery/$file" target="_blank"><img width="$_picture_thumbnail_size" alt="Portfolio Image Number: $_counter" src="$__gallery_url/$gallery/$file" border="0"></a></noscript>
				</td>
EOF:
						$_table_counter++;
						$_tablecounterandall = $_tablecounterandall+$_table_counter;
					} else {
						$_tables_and_me = $_tables_and_me . "1+";
						$_table_to_print = $_table_to_print . <<<EOF
				<td bgcolor="#990000" style="color: #FFFFFF">
					$_counter
				</td>
				<td>
					<script language="JavaScript">
						document.write('<a href="javascript:;" onclick="window.open(\\'ShowPic.cgi?img=gallery/$file\\',\\'POPUP_IMAGE\\',\\'toolbar=no,location=no,menu=no,resizable=no,status=no,scrollbars=yes,top=50,left=100,height=350,width=350\\'); return false;"><img width="$_picture_thumbnail_size" alt="Portfolio Image Number: $_counter" src="$__gallery_url/$gallery/$file" border="0"></a>');
					</script>
					<noscript><a href="ShowPic.cgi?img=gallery/$file" target="_blank"><img width="$_picture_thumbnail_size" alt="Portfolio Image Number: $_counter" src="$__gallery_url/$gallery/$file" border="0"></a></noscript>
				</td>
EOF;
						$_table_counter++;
						$_tablecounterandall = $_tablecounterandall+$_table_counter;
					}
					$_a_counter++;
				}
			}
			closedir($handle);
		}
		if($_table_to_print) {
			$_page_content .= <<<EOF
			<center>I found $_counter Pictures in this Gallery, click on one of them to view it full size...$_tables_and_me$_tablecounterandall</center>
	<table border="0" bordercolor="#990000" width="75%" align="center" cellpadding="2" cellspacing="2">
		$_table_to_print
		</tr>
	</table>
EOF;
		}

Ok, $_counter is working I know that because the output shows this:

I found 5 Pictures in this Gallery, click on one of them to view it full size...0

I put $_tablecounterandall in there just as a test to see if it is running all the way through but it is NOT. It appears to not be running past this:

	if ($_table_counter > 2) {

Right above that is where the $_counter get's incremented. So I know it's getting to there. This if statement that it appears to be failing at, is not even doing the "else" statement.

Can you see a reason why it would not be working?

Thank you again, very much!
Richard