Whats wrong with this script? Rewritten...

[ESCForums_com]

I've narrowed it down to this part of the script which is overloading the server, It works fine, so I don't know whats wrong with it!
$url.pic is the url of the folder with pictures
$pic is the pictures
$heightwidth gets inserted as the height/width of picture
$max = 400

This little thing will overload my server before a picture is even selected, I think its because its looking for the $pic var and their is none...could that be it?

$imagehw = getimagesize($urlpic .$pic); 
    $imagewidth = $imagehw[0]; 
    $imageheight = $imagehw[1]; 
	if (($imageheight > $max) || ($imagewidth > $max)) 
  {   

    if ($imageheight > $imagewidth) 
    { 
      $sizefactor = (double) ($max / $imageheight);
    } 
    else 
    {
      $sizefactor = (double) ($max / $imagewidth) ;
    }
  }    

  $newwidth = (int) ($imagewidth * $sizefactor);
  $newheight = (int) ($imageheight * $sizefactor); 
  if (($newheight == "0") || ($newwidth == "0")) {
  $heightwidth = "";
  }else{
  $heightwidth = "height=".$newheight." width=".$newwidth;
  }

Thanks, maybe now someone can help me!

-Phil

[michael]

That should not overload the server.

Echo the value of pic and path, and echo out the $imagehw using a foreach, and decide if those values are correct.

What version of GD, PHP, Apache?

[dawson1323]

I don't think it would make a difference but try to change:

$imagehw = getimagesize($urlpic .$pic);

TO:

$imagehw = getimagesize($urlpic$pic);

or:

$tempvar = $urlpic$pic;

$imagehw = getimagesize($$tempvar); (yes TWO $'s)

-D

[ESCForums_com]

I spoke to soon, after days of thinking, I've figured it out...

This is what overloaded the server:

getimagesize($urlpic .$pic);

It was looking to get the image size from a folder, since $pic would equal nothing, so I just tried it right now, by adding a simple "if ($pic == "") {} else {" to the script...now it works perfect and doesn't overload the server!

If i make no sense, horray for me!