2013-05-28 161 views
0

我的download.php不起作用。下载完成后,打开文件的错误是“存档文件已损坏或损坏”。 我不知道错误的原因。所以请帮助我。该代码是一个硬代码。我通过给出图像的名称来测试此代码。它的工作原理是正确的,但是当我从数据库中检索图像并尝试运行此代码时,出现了问题。下载错误....存档文件损坏或损坏

 <?php 

echo "<br/>" , $product_id=$_REQUEST['product_id']; 
echo "<br/>" , $query= ("SELECT image_name FROM " . $db_prefix ."product WHERE image_id = $product_id"); 


$test = class_exists('ZipArchive'); 

///$url='upload_images/'; 
$url = $_SERVER['DOCUMENT_ROOT'].'photohive/upload_images/'; 

$zip = new ZipArchive; 
$zip->open($_SERVER['DOCUMENT_ROOT'].'photohive/downloads/file_' . time() . '.zip', ZipArchive::CREATE); 
$result = mysql_query($query); 
while($row= @mysql_fetch_assoc($result)) 
{ 
    //echo $url. $row['image_name']; 
    $file_name = $url.$row['image_name']; 
    if(file_exists($file_name)){ 
     //$zip->addFile($file_name); 
     //$zip->addFromString(basename($file_name), file_get_contents($file_name)); 

     $zip->addFile(realpath($file_name), $file_name); 
    } 
} 

$zip->close(); 


$encoding = 'binary'; 
$filename = $_SERVER['DOCUMENT_ROOT'].'photohive/downloads/file_' . time() . '.zip'; 
header('Pragma: public'); 
header('Expires: 0'); 
header('Content-Description: File Transfer'); 
header('Content-Disposition: filename="' . basename($filename). '"'); 
header('Content-Type: application/x-zip'); 
header('Content-Transfer-Encoding: ' . $encoding); 
header('Content-Length: ' . filesize($filename)); 

$file = readfile($filename); 

//print($file); 
exit; 




?> 
+0

? – Danack

+0

此外 - 你已经发布的代码显然不会工作,因为它在例如[echo“
”] – Danack

回答

0

在你的代码使用这个字符串来获得两倍的文件名:$_SERVER['DOCUMENT_ROOT'].'photohive/downloads/file_' . time() . '.zip';

在第二次,time()可能会返回不同的时间戳记,因为有些秒会通过,而你是压缩图片。

同样使用time()方法会在高流量站点上出现冲突,您可能会在一秒钟内获得两个或更多请求。

我建议你使用tempnam()函数来生成文件。请记住在下载完成后删除它。

下面是一个代码:

$filename = tempnam($_SERVER['DOCUMENT_ROOT'].'photohive/downloads', "zip"); 
... 
$zip->open($filename, ZipArchive::OVERWRITE); 

并删除这一行:你的第一个

$filename = $_SERVER['DOCUMENT_ROOT'].'photohive/downloads/file_' . time() . '.zip'; 

$encoding = 'binary';

+0

谢谢。它的工作现在。 – user1756650

+0

完美,请记得投票。并接受答案。 – Kostanos