Paginate images scanned from directory

Kleidi

Hello there,
i got this snippet for gettin all images from a directory but, i need to add pagination on it. A simple one, just to not mess up my design if will be found more then X images on the folder.
Or, if can add a limit for images. To display only XX images randomly, or newest.
The snippet is this:

$string =array();
$filePath='images/';  

$dir = opendir($filePath);
while ($file = readdir($dir)) { 
   if (preg_match('/.jpg/i', $file) || preg_match('/.gif/i', $file) || preg_match('/.png/i', $file)) { 
   $string[] = $file;
   }
}
while (sizeof($string) != 0){
  $img = array_pop($string);
  echo "<a class='lightbox-image' style='margin-bottom:5px;' href=".$filePath.$img." data-gal='prettyPhoto[gallery2]'>
		<img src=".$filePath.$img." width='150' height='100'>
		</a>
		";

}

Thank you in advance!

bradgrafelman

The topic of 'pagination' has come up quite often on the forums; have you tried searching first? Note that the fact that you're trying to paginate results of a directory scan rather than rows from a DB query is irrelevant; both can be considered as a large pile of data of which you want to display one "page" at a time.

Also, unrelated to your issue, note that this:

if (preg_match('/.jpg/i', $file) || preg_match('/.gif/i', $file) || preg_match('/.png/i', $file)) {

is very, very wrong for a few reasons:

Regular expressions aren't even being used, so using a preg_*() function is wasteful, misleading, and possibly prone to errors (which you've; see #2).
Note that the period is used as a wildcard, so this pattern:
```
/.jpg/i
```
will successfully match this string:
```
MyVirus_IsNotAJpg.exe
```
There's no need to do any string manipulation to find a file's extension; that's one of the uses of the built-in [man]pathinfo/man function.

Kleidi

bradgrafelman;10997290 wrote:
The topic of 'pagination' has come up quite often on the forums; have you tried searching first? Note that the fact that you're trying to paginate results of a directory scan rather than rows from a DB query is irrelevant; both can be considered as a large pile of data of which you want to display one "page" at a time.

Also, unrelated to your issue, note that this:
if (preg_match('/.jpg/i', $file) || preg_match('/.gif/i', $file) || preg_match('/.png/i', $file)) {
is very, very wrong for a few reasons:
Regular expressions aren't even being used, so using a preg_*() function is wasteful, misleading, and possibly prone to errors (which you've; see #2).
Note that the period is used as a wildcard, so this pattern:
/.jpg/i
will successfully match this string:
MyVirus_IsNotAJpg.exe
There's no need to do any string manipulation to find a file's extension; that's one of the uses of the built-in [man]pathinfo/man function.

Thank you for your reply.
Regarding pagination: The script of pagination that i have is exsclusivly for db rows. I have searched around for a pagination way for me 🙁
About problems with my script: Can you suggest me any other way, if you don't mind? ANd, please, remember that i'm new on php 😉

Thank you for your help!