FUDforum: comp.lang.php » Checking if file is an image

Home » Imported messages » comp.lang.php » Checking if file is an image

Show: Today's Messages :: Polls :: Message Navigator

Checking if file is an image [message #170857]

Sat, 04 December 2010 01:37

jwcarlton
Messages: 76
Registered: December 2010

Karma:

Member

I have a section that shows an uploaded image. I was doing this:

list($width, $height, $type, $attr) = @getimagesize("/path/to/
$image");

if (!$width) {
$image = "noimage.gif";
$width = "75";
$height = "95";
}

I understand that error suppression has a bit of a performance hit,
though, so I'm curious if either of these would be better:

if (is_readable("/path/to/$image"))
list($width, $height, $type, $attr) = @getimagesize("/path/to/
$image");

if (getimagesize("/path/to/$image"))
list($width, $height, $type, $attr) = @getimagesize("/path/to/
$image");

Either would still be followed with "if (!$width)...". I know that
neither are fool-proof because they don't check that someone didn't
rename an .xls or .exe to .jpg, but I can probably control that better
on the upload side than checking here on every page load. Here, I'm
more concerned with speed, and not showing any error messages for
faulty images.

TIA,

Jason

Report message to a moderator

[Message index]

		Checking if file is an image By: jwcarlton on Sat, 04 December 2010 01:37
		Re: Checking if file is an image By: Magno on Sat, 04 December 2010 02:06
		Re: Checking if file is an image By: jwcarlton on Sat, 04 December 2010 02:38
		Re: Checking if file is an image By: Michael Fesser on Sat, 04 December 2010 11:39
		Re: Checking if file is an image By: Magno on Sat, 04 December 2010 15:10
		Re: Checking if file is an image By: Jerry Stuckle on Sat, 04 December 2010 03:34
		Re: Checking if file is an image By: Michael Fesser on Sat, 04 December 2010 11:39
		Re: Checking if file is an image By: The Natural Philosoph on Sat, 04 December 2010 19:19

Previous Topic:	Efficiency of a lot of variables
Next Topic:	web solutions for global presence

Goto Forum:

-=] Back to Top [=-

[ Syndicate this forum (XML) ] [

]

Current Time: Tue May 20 21:00:23 GMT 2025

Total time taken to generate the page: 0.03627 seconds