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

Re: Checking if file is an image [message #170861 is a reply to message #170859]

Sat, 04 December 2010 02:38

jwcarlton
Messages: 76
Registered: December 2010

Karma:

Member

On Dec 3, 9:06 pm, Magno <marbar...@gmail.com> wrote:
> On 12/03/2010 10:37 PM, jwcarlton wrote:
>
>
>
>> 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
>
> ¿Why don’t you just better use ob_start(); and buffer any possible error
> output?
> If no error output, then render the image normally.
> If there is error, let yourself know by any way and then exit();
>
> To check if there is any error, I think that a single.-
> if (strlen($ob = ob_get_clean())) would do.

That's smart, thanks.

J

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: Sat Nov 30 07:16:42 GMT 2024

Total time taken to generate the page: 0.04731 seconds