このPHPコードを改善するにはどうすればよいですか？

Question

私はそれは意志 それは、MySQLからのDB「2/12分の34/thepicture.jpg」をこのように指定された値になります私は、スクリプト

で写真のサムネイル画像のパスを得るのを助ける、それ以下のPHPコードを持っていますその後、私は、私は任意の助けて下さい50とページ上も

に開いていますこれを行うのより良い性能の方法があると確信しています。この「2/12分の34/thepicture_thumb1.jpg」

にそれを回しますユーザーはこれを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; 
    } 
} 

私は、ファイル拡張子を取得するためにpathinfo（）をどのように使用しているのか不思議です。この拡張子は常に3桁であるため、この値をより良いパフォーマンスにする方法はありますか？

Answer 1

性能面からのフィードバックを組み込むことを更新しました

あなたは舞台裏でコンパイルされたコードを実行しているため、パフォーマンスが優れて機能します。

もちろん、これらの機能をすべて呼び出す必要はありませんが、良い考えではありません。あなたの場合、pathinfo関数は必要なさまざまなパスを返します。あなたがPHP 5をお持ちでない場合は

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

：あなたはこの（ノート、「ファイル名」はPHP 5.2以降のみ利用可能です）のようなファイル名を構築することができますときには、元の名前にexplode関数を呼び出します。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);

編集：BBコードは\で姿を消しました。

Answer 3

なぜこの機能のパフォーマンスについて懸念していますか？ （「メイン」ファイル名が生成されたときに）一度だけコールし、結果を格納すると仮定すると、ランタイムは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()から返された情報を使用していましたサムネイル名あなたは組み込みのPHP呼び出している場合、@DisgruntledGoat

Answer 6

この問題を解決する最も簡単な方法は、手の前に、すべてのユーザープロファイルの写真をサムネイルとあなたがリサイズ保持しませんので、周りにそれを維持することです。