insure a variable has a .gif or .jpg extension?

[_0_05_]

Hi, my script opens a directory, and loops through each file, and i want to insure that each file has a .jpg .gif or .jpeg file extension. I realize that there are also files called . and ..

[_0_05_]

here's my code

if ($handle = opendir('images')) {

while (false !== ($file = readdir($handle))) { 
    $file2 = strtolower($file);
	$x=0;
	if ( ereg( "\\.jpg |\\.gif |\\.jpeg", $file2, $array)){
		print "blah";
		print "$array[$x]<br>";
		$x=$x+1;
	}
	//print $file2."<br>";
}


closedir($handle); 
}

[_0_05_]

(double post)

[_0_05_]

WOW! 4 replies to my own post!
i thought that some of you might want to see how i did it

$across = 4;
if ($handle = opendir('test-images/')) {

$x=0;
    while (false !== ($file = readdir($handle))) { 
        $file2 = strtolower($file);

	if ((preg_match ("/.gif/i", "$file2")) or (preg_match("/.jpg/i", "$file2")) or (preg_match("/.jpeg/i", "$file2"))) {
		$x++;		
		if ($x == $across){
			print "</tr>\n";
			print "<tr>\n";
			$x=0;
		}

		print "\n<td width=\"25%\" align=\"center\">\n";
		print "<a target=\"_blank\" href=\"test-images/$file2\">\n";
		print "<img border=\"0\" src=\"test-images/$file2\" width=\"122\" height=\"124\"></a></td>\n";					

	}
}


closedir($handle); 
}

[Weedpacket]

Well, if it works - but why haul preg_match in multiple times if you don't actually use any regular expressions? You don't even use case-insensitive matching, to be honest, because you strtolower() the filenames anyway.

I.e., why not just use one that reads
/.(gif|jpe?g)$/
Where the \ makes ure you match a . and not just any old character, and the $ makes sure you really are looking at the file extension and not something in the middle of the filename.

And "$variable" is just redundant extra work; just use $variable.