3

编辑

感谢您的所有回答，尤其是@Mailerdaimon，他注意到我没有在imagecopyresampled函数中使用计算值。

我不再得到黑色图像，但我仍然得到一些黑色部分，所以我认为我的比率公式应该更新：如果我上传横向图像，新图像的高度小于 170 像素，然后有一些黑色显示。

我怎样才能确保图像的高度一直走？

下面是一个简单的脚本，允许用户上传图片。上传完成后，图片将显示为 170px(h) x 150px(w) 的缩略图。

调整大小部分确实有效，因为输出图像是 170x150px 但如果我仍然得到一些黑色区域

if ($_SERVER["REQUEST_METHOD"] == "POST") 
{

$maxWidth  = 150;
$maxHeight = 170;

$name = $_FILES ['image'] ['name'];
$type = $_FILES ["image"] ["type"];
$size = $_FILES ["image"] ["size"];
$tmp_name = $_FILES ['image'] ['tmp_name'];
list($originalWidth, $originalHeight) = getimagesize($tmp_name);


 if ($originalWidth > $originalHeight) 
 {
   $thumbnail_height = floor(($originalHeight/$originalWidth)*$maxWidth);
   $thumbnail_width  = $maxWidth;
 } else {
   $thumbnail_width  = floor(($originalWidth/$originalHeight)*$maxHeight);
   $thumbnail_height = $maxHeight;
 }

 // Resample  
  $image_p = imagecreatetruecolor($maxWidth, $maxHeight);
  imagecreatefrompng($tmp_name);
  imagecopyresampled($image_p, 0, 0, 0, 0, $thumbnail_width, $thumbnail_height, 
  $originalWidth, $originalHeight);

//start upload process
$RandomNumber = uniqid();
$location = "uploads/$RandomNumber";
imagejpeg($image_p,  $location, 100);      

$sql=query("UPDATE users SET image = '".$location."' WHERE id = '$id'"); 

        } 
  }

知道我做错了什么吗？

4

3 回答 3

1
  1. 确保文件夹“上传”是可写的或由 Web 服务器拥有
  2. 尝试使用绝对路径：/var/www/mysite/uploads
  3. 尝试使用与随机 ID 不同的东西，例如行 ID 的哈希或 md5 或图像本身以避免重复。

您可以更改最后一部分：

...

$RandomNumber = md5($image_p); $location = "/var/www/mysite/uploads/$RandomNumber"; if( imagejpeg( $image_p, $location, 100 ) ){
query("UPDATE users SET image = '".$location."' WHERE id = '$id'"); }

...

于 2013-10-23T20:53:30.293 回答
1

我从未使用过 php，所以这可能不正确，但我看不到您在哪里使用计算的宽度和高度值！

if ($originalWidth > $maxWidth || $originalHeight > $maxHeight)
 {
      if ($originalWidth / $maxWidth > $originalHeight / $maxHeight) 
     {
       // width is the limiting factor
       $width = $maxWidth;
       $height = floor($width * $originalHeight / $originalWidth);

     } else { 
       // height is the limiting factor
       $height = $maxHeight;
       $width = floor($height * $originalWidth / $originalHeight);
     }

在这里你计算高度和宽度，然后它们不会出现在你的代码中的任何地方......

编辑： 您使用：

$image_p = imagecreatetruecolor($maxWidth, $maxHeight);
imagecreatefrompng($tmp_name);
imagecopyresampled($image_p, 0, 0, 0, 0, $thumbnail_width, $thumbnail_height,$originalWidth, $originalHeight);

但是文档说明imagecopyresmapled该功能采用以下

imagecopyresampled ($dst_image ,$src_image ,$dst_x ,$dst_y ,$src_x ,$src_y ,$dst_w , $dst_h ,$src_w ,$src_h )

虽然我认为你的代码应该是

imagecopyresampled($image_p,$tmp_name, 0, 0, 0, 0, $thumbnail_width, $thumbnail_height,$originalWidth, $originalHeight);

正如@Manolo Salsas（和其他几个人）已经提到的那样

EDIT2： 另一个问题在于如何计算高度和宽度值。如果$height被计算喜欢这样

$height = floor($width * $originalHeight / $originalWidth);

它可以小于$maxHeight，尽管在结果图像中创建了一个黑色（未填充）区域，因为您使用最大值创建了此图像

$image_p = imagecreatetruecolor($maxWidth, $maxHeight);

要解决此问题，请使用您计算的 $thumbnail_height 和 $thumbnail_width 值来创建图像

 $image_p = imagecreatetruecolor($thumbnail_width, $thumbnail_height);

或始终将图像大小调整为最大值

imagecopyresampled($image_p,$tmp_name, 0, 0, 0, 0, $maxWidth, $maxHeight,$originalWidth, $originalHeight);

这可能会扭曲图像。如果您不想扭曲图像，请考虑先调整大小，使尺寸适合缩略图，然后裁剪较长的一侧。

于 2013-10-28T08:37:06.967 回答
0

首先，

@getimagesize($_FILES['userfile']['tmp_name']))

此函数检查文件是否为真实图像（不仅仅是扩展名）。如果文件不是图像，它将返回 false（和警告，这就是 at 的原因）。

如果文件是真实图像，可以根据需要返回：

list($width, $height, $type, $attributes) = @getimagesize($_FILES['userfile']['tmp_name']))

或者只是尺寸：

$size = @getimagesize($_FILES['userfile']['tmp_name']))

所以如果你使用这个功能，记得加上at，并使用if条件来检查你是否接收到一个真实的图像。

另一方面，在确认您收到的是真实图像并且小于您想要的尺寸后，您可以执行以下操作：

if(move_uploaded_file($_FILES['userfile']['tmp_name'], $imagePath)

where$imagePath是您要保护图像的路径。

然后，你可以这样做：

$image = imagecreatefromjpeg($imagePath);
$canvas=imagecreatetruecolor($maxwidth,$maxheight);
imagecopyresampled($canvas, $image ,0,0,0,0,$maxwidth, $maxheight, $originalWidth, $originalHeight);

并且您将调整正确的图像大小。

于 2013-10-28T20:35:35.330 回答