2010-03-14 262 views
1

我使用以下函数重命名缩略图。
例如,如果我将名为“image.png”的文件上载到上传文件夹,并且该文件夹中已经有一个名为“image.png”的文件,则新文件会自动更名为“image-copy-1巴纽”。如果还有一个名为“image-copy-1.png”的文件,它将被重命名为“image-copy-2.png”等。
以下函数返回新文件名。至少这就是它应该做的......
尽管如此,重命名并不能正常工作。有时会产生奇怪的结果,如:(我总是上传一个名为“1.png”的文件)
1-copy-1.png
1-copy-2.png
1-copy-2-copy-1 .png
1-copy-2-copy-3.png重命名具有相同名称的文件

我希望你明白我的问题,尽管我的描述有些复杂......你能告诉我这里出了什么问题吗? (奖金问题:是正则表达式做这种东西的正确工具?)

<?php 
function renameDuplicates($path, $file) 
{ 
    $fileName = pathinfo($path . $file, PATHINFO_FILENAME); 
    $fileExtension = "." . pathinfo($path . $file, PATHINFO_EXTENSION); 

    if(file_exists($path . $file)) 
    { 
     $fileCopy = $fileName . "-copy-1"; 

     if(file_exists($path . $fileCopy . $fileExtension)) 
     {   
      if ($contains = preg_match_all ("/.*?(copy)(-)(\\d+)/is", $fileCopy, $matches)) 
      { 
       $copyIndex = $matches[3][0];    
       $fileName = substr($fileCopy, 0, -(strlen("-copy-" . $copyIndex))) . "-copy-" . ($copyIndex + 1); 
      }      
     } 

     else 
     { 
      $fileName .= "-copy-1"; 
     } 
    } 

    $returnValue = $fileName . $fileExtension; 
    return $returnValue; 
}?> 

回答

2

简单,没有正则表达式;

function renameDuplicates($path, $file) 
{ 
    $fileName = pathinfo($path . $file, PATHINFO_FILENAME); 
    $fileExtension = "." . pathinfo($path . $file, PATHINFO_EXTENSION); 

    $returnValue = $fileName . $fileExtension; 

    $copy = 1; 
    while(file_exists($path . $returnValue)) 
    { 
     $returnValue = $fileName . '-copy-'. $copy . $fileExtension; 
     $copy++; 
    } 
    return $returnValue; 
} 

性能问题快速基准测试(32000在文件夹中的MAX文件):

$start = microtime(1); 
$c=0; 
while($c<32000) 
    if(file_exists(__FILE__)) 
     $c++; 
echo microtime(1) - $start; /* 0.44202709197998 */ 

超过半秒为最坏的情况那么少。并为100份 - 0.0013940334320068秒。至于正则表达式:

$start = microtime(1); 
$contains = preg_match_all ("/.*?(copy)(-)(\\d+)/is", __FILE__, $matches); 
echo microtime(1) - $start; /* 0.010906934738159 */ 

所以如果你打算有一个以上的文件,正则表达式〜800份将更快(某些微秒,但速度更快):)

+0

非常感谢您的回答(以及回答奖励问题)! – snorpey 2010-03-15 09:57:09

0

当其重命名像1-copy-2-copy-1.png的文件是什么,你要上传的文件的名称?

因为:
如果文件有此名1-copy-2.png这是很正常的1-copy-2-copy-1.png为重命名的文件...

否则就有问题:)

+0

我总是上传一个名为“1.png”的文件。 – snorpey 2010-03-14 23:02:13

0

它看起来像你的代码只检查名为1-copy-1.png的文件副本是否存在?当它发现它重命名为1-copy-2.png

然而,它不检查文件1-copy-2.png是否存在并且不增加它。

但在您的解决方案中,脚本需要检查文件夹中的每个文件。所以如果有很多文件。它会很慢。