[Resolved] Adding pages from opendir

[fr8]

Im looking to advance my gallery a little. Im getting picture sets that are way over 200 pictures and i would like to divide them up into pages.

But im not sure how to do this from opedir.

My gallery from phpbuilder

This is what I have when I pull from DB and make the pages.

$pid = preg_replace('/[^0-9]/i', '',$_GET['pid']);
$page = preg_replace('/[^0-9]/i', '',$_GET['page']);

// limit value for number of records to be shown per page 
// query database to find total number of records to display 
$limit = 12;  

$query_count = 'select * from js_gallery WHERE pid='.$pid.'';  

$result_count = mysql_query($query_count);  

$totalrows = mysql_num_rows($result_count);  

if(empty($page))  

$page = 1;  

$limitvalue = $page * $limit - ($limit);  

$query = 'select * from js_gallery WHERE pid='.$pid.' LIMIT '.$limitvalue.', 

'.$limit.'';  

$result = mysql_query($query) or die('Error: ' . mysql_error());  

$count_result = mysql_num_rows($result);  

// Display links at the top to indicate current page and number of pages displayed 
$numofpages = ceil($totalrows / $limit); 
echo'<font class="boxcontent"><br><p><b>'.$totalrows.'</b> Picture(s)</p></font>'; 
$from=$limit*$page-$limit+1; 
$to=$from + $count_result-1; 

// start 123 next>> table 
echo '<table align="center" width="90%" border="0" cellspacing="0" cellpadding="0" 

class="boxcontent"> 
            <tr><td width="120" align="left">';

if($numofpages>1){
	echo 'Showing: '.$from.' - '.$to.'</td><td align="right" 

valign="middle"><b>Go to page:</b>&nbsp;';}  

// display previous link if page is not 1 
if($page != 1){  

$pageprev = $page - 1;  

echo('<a href="'.$PHP_SELF.'?pid='.$pid.'&page='.$pageprev.'">Prev</a>&nbsp;');  

}  

// display page nos if not 1 
for($i = 1; $i <= $numofpages; $i++){  

if($numofpages>1){ 
if($i == $page)  

echo('&nbsp;'.$i.'&nbsp;');  

else 
echo('&nbsp;<a href="'.$PHP_SELF.'?pid='.$pid.'&page='.$i.'">'.$i.'</a>&nbsp;');  

}} 

// display next page link if there is more than one 
if(($totalrows - ($limit * $page)) > 0){  

$pagenext = $page + 1;  

echo('<a href="'.$PHP_SELF.'?pid='.$pid.'&page='.$pagenext.'">Next</a>');  

} 
// end 123 next>> table 
echo"</td></tr></table><br>";

Anyone have some ideas?

[bradgrafelman]

Try using [man]scandir/man to load all the image filenames into an array, then establish a limit of # of pics per page.

This is the code that I made... I don't know how efficient it is, but it seems to work as far as I know.

if(!function_exists('scandir')) { 
	function scandir($dir) {
		$dh  = opendir($dir);
		while (false !== ($filename = readdir($dh))) {
			$files[] = $filename;
		}
	return $files;
	}
}


$page = @$_GET['page'] ? $_GET['page'] : 0;

$fileArray = scandir('images/');

$limit = 2; // 20 pics per page.
for($i = ($page*$limit), $stop = (($page*$limit)+$limit); $i < $stop && $i < count($fileArray); $i++) {
    if($fileArray[$i] == '.' || $fileArray[$i] == '..') {
        $stop++;
        continue;
    } else {
        echo 'Filename: ' . $fileArray[$i] . '<br>';
        // do whatever you want with the filename here...
    }
}

Page 0 and Page 1 are the same thing, so start with Page 1, then Page 2, etc.

EDIT: scandir() is a command introduced in PHP 5. If you're using PHP 4, don't worry! I included PHP's code to create this function if it doesn't exist.

[fr8]

very nice. thanks for the fast reply. ill test that out when i get some down time.

[fr8]

It worked to an extent.

Now im trying to figure out to replace this line of code from the 123page numbers from above in my first post.

$numofpages = ceil($totalrows / $limit);

any ideas?

[bradgrafelman]

Try this:

$numofpages = ceil((count($fileArray)-2) / $limit);

[man]count/man will tell you how many items are in the array $fileArray. I subtracted 2 to get rid of the '.' (current directory) and '..' (parent directory) entries that are stored using readdir().

[fr8]

well its moving along.

EDIT: Fixed the last page problem.

now how do i make it start with page one and still have it work.

New Version of the code.

$limit = 5;

if(!function_exists('scandir')) {
    function scandir($dir) {
        global $files;
        $dh  = opendir($dir);
        while (false !== ($filename = readdir($dh))) {
            $files[] = $filename;
        }
    return $files;
    }
}

$page = @$_GET['page'] ? $_GET['page'] : 0;

$fileArray = scandir($dir);

// 20 pics per page.
for($i = ($page*$limit), $stop = (($page*$limit)+$limit); $i < $stop && $i < count($fileArray); $i++) {
    if($fileArray[$i] == '.' || $fileArray[$i] == '..') {
        $stop++;
        continue;
    } else {
        echo '<a href="'.$dir.$fileArray[$i].'"><img src="' .$thumb.$fileArray[$i].'" border="0"></a><br>';
        // do whatever you want with the filename here...
    }
}

$numofpages = ceil((count($fileArray)-2) / $limit);

$ddir = opendir($dir);
while ($file = readdir($ddir)) {
  if (!is_dir($file))
    $totalfiles++;
}
// display previous link if page is not 1
if($page != 0){
$pageprev = $page - 1;
echo('<a href="'.$PHP_SELF.'?pid='.$pid.'&page='.$pageprev.'">Prev</a>&nbsp;');
}

// display page nos if not 1
for($i = 0; $i <= $numofpages; $i++){
        if($numofpages>1){
                if($i == $page){
                        echo('&nbsp;'.$i.'&nbsp;');   }
        else
                echo('&nbsp;<a href="'.$PHP_SELF.'?pid='.$pid.'&page='.$i.'">'.$i.'</a>&nbsp;');
        }
}

// display next page link if there is more than one
if(($totalfiles - ($limit * $page)) > 0){
$pagenext = $page + 1;
echo('<a href="'.$PHP_SELF.'?pid='.$pid.'&page='.$pagenext.'">Next</a>');
}

[bradgrafelman]

$page = @$_GET['page'] ? $_GET['page'] : 0; 

This line uses the ternary operator. It's like a condensed IF-ELSE statement.

"If $GET['page'] is set, $page = $GET['page'], else $page = 0"

Simply change the '0' at the end to a '1'.

[fr8]

Originally posted by bradgrafelman

$page = @$_GET['page'] ? $_GET['page'] : 0; 

This line uses the ternary operator. It's like a condensed IF-ELSE statement.

"If $GET['page'] is set, $page = $GET['page'], else $page = 0"

Simply change the '0' at the end to a '1'. [/B]

that does fix things. But it loses the "real" first page. it starts with the second page. from some reason page zero is page one. and to see that page i must leave that set to zero.

[bradgrafelman]

Weird, I could have swore it worked as '1' for me. Alrighty.. try this:

$limit = 5; 

function scandir($dir) { 
    global $files; 
    $dh  = opendir($dir); 
    while (false !== ($filename = readdir($dh))) { 
        if(!is_dir($filename)) $files[] = $filename; 
    } 
return $files; 
}

$page = @$_GET['page'] ? $_GET['page'] : 1; 

$fileArray = scandir($dir); 

// 20 pics per page. 
for($i = (($page-1)*$limit), $stop = ((($page-1)*$limit)+$limit); $i < $stop && $i < count($fileArray); $i++) { 
        echo '<a href="'.$dir.$fileArray[$i].'"><img src="' .$thumb.$fileArray[$i].'" border="0"></a><br>'; 
        // do whatever you want with the filename here... 
} 

$numofpages = ceil(count($fileArray) / $limit); 

$totalfiles = count($fileArray);

// display previous link if page is not 1 
if($page != 1){ 
$pageprev = $page - 1; 
echo('<a href="'.$PHP_SELF.'?pid='.$pid.'&page='.$pageprev.'">Prev</a>&nbsp;'); 
} 

// display page nos if not 1 
for($i = 1; $i <= $numofpages; $i++){ 
        if($numofpages>1){ 
                if($i == $page){ 
                        echo('&nbsp;'.$i.'&nbsp;');   } 
        else 
                echo('&nbsp;<a href="'.$PHP_SELF.'?pid='.$pid.'&page='.$i.'">'.$i.'</a>&nbsp;'); 
        } 
} 

// display next page link if there is more than one 
if(($totalfiles - ($limit * $page)) > 1){ 
$pagenext = $page + 1; 
echo('<a href="'.$PHP_SELF.'?pid='.$pid.'&page='.$pagenext.'">Next</a>'); 
}

Also, note that I changed this:

$ddir = opendir($dir); 
while ($file = readdir($ddir)) { 
  if (!is_dir($file)) 
    $totalfiles++; 
} 

to this:

$totalfiles = count($fileArray);

Just one less call to the directory, and since we already have an array of filenames, its just plain easier.

NOTE: This script is no longer compatible with PHP 5. Going to PHP 5 with this script unedited would have previously thrown off things a bit. A couple of modifications would be needed to use this script in PHP 5. If you would like me to show you those modifications, let me know in your next post.

[fr8]

works.

I should of picked up on count()'ing that array. very nice addition.

as for the PHP5. Wouldnt if just change the function to check for the scandir()? or is it a little more elaborate than that?

also thank you for the help.

[bradgrafelman]

Well, I included the is_dir() in my scandir() command to leave off the "." and "..". PHP5's function doesn't do this, and you'd have to go back to the "if($filename != ".")" etc.

I suppose if I wasn't being lazy I'd make this script PHP 4 & PHP 5 compliant. If you want such a version, let me know. Otherwise, don't forget to mark this thread resolved using the link at the bottom of the page.

[fr8]

Here is my shot at it. Not tested.

if(!function_exists('scandir')) { 
    function scandir($dir) { 
        global $files; 
        $dh  = opendir($dir); 
        while (false !== ($filename = readdir($dh))) { 
		if($filename == '.' || $filename == '..') {
			continue;
		}else{
			$files[] = $filename; 
		}
        } 
    return $files; 
    } 
}

This right or is there a better way?

[bradgrafelman]

Sorry - didn't clarify. The function I wrote for PHP 4 is just fine; no need to modify it. It already gets rid of any directories in the array. To make it PHP 5 compatible, all you do is use function_exists() to know whether to define it or not, like this:

if(!function_exists('scandir')) {  

    function scandir($dir) {  

        global $files;  

        $dh  = opendir($dir);  

        while (false !== ($filename = readdir($dh))) {  

            if(!is_dir($filename)) $files[] = $filename;  

        }  

    return $files;  

    } 
}

Since you can't modify a function already defined (like in PHP 5 in this case), you need to modify the data when you process it. Here is where you would check for "." or ".." in our code:

for($i = (($page-1)*$limit), $stop = ((($page-1)*$limit)+$limit); $i < $stop && $i < count($fileArray); $i++) {  

        echo '<a href="'.$dir.$fileArray[$i].'"><img src="' .$thumb.$fileArray[$i].'" border="0"></a><br>';  

        // do whatever you want with the filename here... 
}  

An example solution would be to check if $fileArray[$i] is '.' or '..' or not. If it is, add 1 back to the "stop" or limit variable (so that we aren't shorted an image or two on the output), and continue on to the next iteration. Something like this:

for($i = (($page-1)*$limit), $stop = ((($page-1)*$limit)+$limit); $i < $stop && $i < count($fileArray); $i++) {
		if($fileArray[$i] == '.' || $fileArray[$i] == '..') {
			$stop++;
			continue;
		}
        echo '<a href="'.$dir.$fileArray[$i].'"><img src="' .$thumb.$fileArray[$i].'" border="0"></a><br>';  

        // do whatever you want with the filename here... 
}  

I believe that would do the trick for PHP 4+ compatibility.

[fr8]

very cool. thank you for the time you put in helping. I'll be sure to mark this thread resolved. 🙂