help with function

[dadamssg]

i have a function that checks the extension to make sure its either a jpeg ,gif, or png. Its a bit messy and overkill i think. Can someone help me clean it up and more efficient? thanks ahead

if(!preg_match('/[.](jpg)|(gif)|(png)|(jpeg)|(GIF)|(PNG)|(JPG)|(JPEG)$/', $_FILES['fupload']['name']))  {

$_SESSION['mess'] = "Images must be JPG, GIF, or PNG.";

  extract($_POST);

   header("location: picupload.php?id={$_GET['id']}");

   exit();

}

 

[nrg_alpha]

Perhaps a pattern like this?

 

/\.(?:gif|jpe?g|png)$/i

 

The i modifier takes care of case insentivity, so gif or GIF / png or PNG will both work. as for jpg's I used jpe?g (since the e is optional, this will match jpg, JPG, jpeg, JPEG).