我该如何改进这个PHP代码？

Question

我有PHP代码如下，帮助我在脚本中得到照片的缩略图路径

这将需要提供的值这样从MySQL数据库“2/34/12/thepicture.jpg” 它将然后把它变成这个“2/34/12/thepicture_thumb1.jpg”

我相信有这样做的更好的表现方式，我愿意接受任何帮助，请

而且与50页用户的这将运行50次以获得50张不同的照片

// the photo has it is pulled from the DB, it has the folders and filename as 1 
$photo_url = '2/34/12/thepicture_thumb1.jpg'; 
//build the full photo filepath 
$file = $site_path. 'images/userphoto/' . $photo_url; 
// make sure file name is not empty and the file exist 
if ($photo_url != '' && file_exists($file)) { 
    //get file info 
    $fil_ext1 = pathinfo($file); 
    $fil_ext = $fil_ext1['extension']; 
    $fil_explode = '.' . $fil_ext; 
    $arr = explode($fil_explode, $photo_url); 
    // add "_thumb" or else "_thumb1" inbetween 
    // the file name and the file extension 2/45/12/photo.jpg becomes 2/45/12/photo_thumb1.jpg 
    $pic1 = $arr[0] . "_thumb" . $fil_explode; 
    //make sure the thumbnail image exist 
    if (file_exists("images/userphoto/" . $pic1)) { 
     //retunr the thumbnail image url 
     $img_name = $pic1; 
    } 
} 

1我很好奇的是它如何使用pathinfo（）来获取文件的扩展名，因为扩展名总是3位数，其他方法会让这个值的性能更好吗？

Answer 1

性能方面反馈意见，如果你调用内置的PHP函数，因为你正在运行在后台编译代码的性能非常出色。

当然，当你不需要时调用所有这些函数并不是一个好主意。在你的情况下，pathinfo函数返回你需要的各种路径。你叫的原始名称explode功能时，你可以建立文件名类似这样（注意，在“文件名”仅适用，因为PHP 5.2）：

$fInfo = pathinfo($file); 
$thumb_name = $fInfo['dirname'] . '/' . $fInfo['filename'] . '_thumb' . $fInfo['extension']; 

如果您还没有PHP 5千万。2，那么最简单的方法是忽略的功能和使用strrpos和substr：

// gets the position of the last dot 
$lastDot = strrpos($file, '.'); 
// first bit gets everything before the dot, 
// second gets everything from the dot onwards 
$thumbName = substr($file, 0, $lastDot) . '_thumb1' . substr($file, $lastDot); 

Answer 2

$img_name = preg_replace ('/^(.*)(\..*?)$/', '\1_thumb\2', $file);

编辑：设置高亮消失与\。

Answer 3

为什么你甚至关心这个功能的性能？假设你只调用它一次（比如生成“main”文件名时）并存储结果，与DB和文件系统访问相比，它的运行时间应该基本为零。如果您在每次访问时调用它以重新计算缩略图路径，那么这太浪费了，但它仍然不会对运行时产生重大影响。

现在，如果您希望它看起来更漂亮并且更易于维护，那么这是一个有价值的目标。

Answer 4

这段代码是否存在性能问题，还是只是过早优化？除非性能足够糟糕才成为可用性问题，并且分析器告诉您这个代码是应该承担责任的，否则此代码存在更多紧迫问题。

要回答这个问题：“我该如何改进此PHP代码？” 添加空格。

Answer 5

此代码的最好的优化就是提高它的可读性：

// make sure file name is not empty and the file exist 
if ($photo_url != '' && file_exists($file)) { 

    // Get information about the file path 
    $path_info = pathinfo($file); 

    // determine the thumbnail name 
    // add "_thumb" or else "_thumb1" inbetween 
    // the file name and the file extension 2/45/12/photo.jpg 
    // becomes 2/45/12/photo_thumb.jpg 
    $pic1 = "{$path_info['dirname']}/{$path_info['basename']}_thumb.{$fil_ext}"; 

    // if this calculated thumbnail file exists, use it in place of 
    // the image name 
    if (file_exists("images/userphoto/" . $pic1)) { 
     $img_name = $pic1; 
    } 
} 

我一直在使用换行符打散功能的部件，并用从pathinfo()返回的信息，以简化确定的过程缩略图名称。

更新从@DisgruntledGoat

Answer 6

解决这个问题的最简单方法是缩略图手之前的所有用户个人资料照片，并保持它周围，所以你不保持调整。