Re: Using fopen or SQL to check image

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 25 June 2010 19:35, Ashley Sheridan <ash@xxxxxxxxxxxxxxxxxxxx> wrote:
> On Fri, 2010-06-25 at 19:31 +0200, Karl Cifius wrote:
>
>> Hi,
>>
>> I'm making a Facebook application that can generate images to user's
>> albums. To publish a story a thumbnail of this image is stored on my
>> server. Since this server currently is very limited I want to be able
>> to clean these thumbnails pretty often.
>>
>> To not get broken links in older facebook stories the address to the
>> thumbnail is a php script that checks if the thumbnail is available
>> and returns it, or otherwise returns a default thumbnail.
>>
>> I have solved this using the following code:
>>
>> $tImage = $_GET['i'];
>> $tURL   = "upload/$tImage.jpg";
>> if(!($fp=fopen($tURL,"rb"))){
>>    header("Location: thumb.jpg");
>> }else{
>>    header("Location: upload/$tImage.jpg");
>>    fclose($fp);
>> }
>>
>> My question is if it would be better to have a mysql database with
>> information about the thumbnail and check if the image is there,
>> instead of checking if the image file can be loaded? What is the most
>> optimized approach if I start to gain traffic?
>>
>>
>> Thanks,
>>
>> /Karl
>>
>
>
> I think checking for the existence of a file is probably going to be the
> quicker approach. Unless you have a server with loads of RAM and your DB
> is very small, it's unlikely your DB will exist entirely in memory, so
> you will at some point have to access the files that the DB uses, even
> though this is done by the server automatically.
>
> On another note, I would try to sanitise that $_GET variable a bit, as
> it could lead to issues down the line later. Maybe limit the string to
> patterns you expect for an image URL.
>
> Thanks,
> Ash
> http://www.ashleysheridan.co.uk
>

Might be quicker to do with a .htaccess file - you can avoid loading php at all.

Regards
Peter


-- 
<hype>
WWW: http://plphp.dk / http://plind.dk
LinkedIn: http://www.linkedin.com/in/plind
BeWelcome/Couchsurfing: Fake51
Twitter: http://twitter.com/kafe15
</hype>

-- 
PHP General Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php




[Index of Archives]     [PHP Home]     [Apache Users]     [PHP on Windows]     [Kernel Newbies]     [PHP Install]     [PHP Classes]     [Pear]     [Postgresql]     [Postgresql PHP]     [PHP on Windows]     [PHP Database Programming]     [PHP SOAP]

  Powered by Linux