Home/ Questions/Q 6726319
In Process

The Archive Base Latest Questions

Editorial Team
  • 0
Asked: 2026-05-26T09:54:48+00:00

I have written a simple image gallery scrip witch checks a folder and displays

  • 0

I have written a simple image gallery scrip witch checks a folder and displays all the images in the folder i have code to remove the . and .. but its not working and I cant see why any help would be great as im new to the dir functions in PHP. I have lightbox tags surrounding the link hence the extra echo’s The code is below:

<?
// USER OPTIONS DEDFINED HERE

$dir = "img/";      //folder with images
$height ="196px";   //image height on page its displayed in full in lightbox
$width ="320px";    //image width on page its displayed in full in lightbox
?>

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Strict//EN"     "http://www.w3.org/TR/xhtml1/DTD/xhtml1-strict.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head> 
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1" /> 
<title>Gallery</title> 
<link href="css/css.css" rel="stylesheet" type="text/css" />
<script type="text/javascript" src="js/prototype.js"></script>
<script type="text/javascript" src="js/scriptaculous.js?load=effects,builder"></script>
<script type="text/javascript" src="js/lightbox.js"></script>
</head>

<?

//open directory
if ($opendir = opendir($dir));

{

//read dir
while (FALSE !== ($file = readdir($opendir)))
{   
    if ($file!="."&&$file!="..")
        echo "<div class='imgwraper'>";
        echo"<a href='$dir$file' rel='lightbox' title='$filename'>";
        echo "<img src='$dir$file' alt='' width='$width' height='$height'/>";
        echo"</a>";
        echo "<div class='name_box'>";
        echo current(explode('.', $file));
        echo "</div>";
        echo "</div>";

}
closedir($opendir); 
}

?>

Thanks in Advance

Lewis

Share

Leave an answer

You must login to add an answer.

Need An Account,

1 Answer

  1. Editorial Team

    You have no braces after if ($file!="."&&$file!=".."), the if statement is only applicable for the first line of echo "<div class='imgwraper'>";.

    Wrap everything in braces like so:

    while (FALSE !== ($file = readdir($opendir)))
{   
    if ($file!="."&&$file!="..") {
        echo "<div class='imgwraper'>";
        echo"<a href='$dir$file' rel='lightbox' title='$filename'>";
        echo "<img src='$dir$file' alt='' width='$width' height='$height'/>";
        echo"</a>";
        echo "<div class='name_box'>";
        echo current(explode('.', $file));
        echo "</div>";
        echo "</div>";
    }
}

    By the way, you could consider using glob to simplify your code a bit more.

    • 0