我該如何改進這個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

解決這個問題的最簡單方法是縮略圖手之前的所有用戶個人資料照片，並保持它周圍，所以你不保持調整。