Định dạng câu lệnh if để có thể đọc được

Question

Cách tốt nhất để định dạng tính năng này là dễ đọc?

if (strpos($file, '.jpg',1) && file_exists("$thumbsdir/$file") == false || strpos($file, '.gif',1) && file_exists("$thumbsdir/$file") == false || strpos($file, '.png',1) && file_exists("$thumbsdir/$file") == false) { 
    createThumb("$gallerydir/$file", "$thumbsdir/$file",$thumbsize); 
    fwrite($log,date("Y-m-d")." @ ".date("H:i:s")." CREATED: $thumbsdir/$file\n"); 
} 

Answer 1

Tôi muốn trích xuất logic "là hình ảnh" thành chức năng của riêng nó, làm cho số if dễ đọc hơn và cũng cho phép bạn tập trung vào logic.

function is_image($filename) { 
    $image_extensions = array('png', 'gif', 'jpg'); 

    foreach ($image_extensions as $extension) 
     if (strrpos($filename, ".$extension") !== FALSE) 
      return true; 

    return false; 
} 

if (is_image($file) && !file_exists("$thumbsdir/$file")) { 
    createThumb("$gallerydir/$file", "$thumbsdir/$file",$thumbsize); 
    fwrite($log,date("Y-m-d")." @ ".date("H:i:s")." CREATED: $thumbsdir/$file\n"); 
} 

Answer 2

if ((strpos($file, '.jpg',1) || 
    strpos($file, '.gif',1) || 
    strpos($file, '.png',1)) 
    && file_exists("$thumbsdir/$file") == false) 
{ 
    createThumb("$gallerydir/$file", "$thumbsdir/$file",$thumbsize); 
    fwrite($log,date("Y-m-d")." @ ".date("H:i:s")." CREATED: $thumbsdir/$file\n"); 
} 

Answer 3

tôi sẽ phá vỡ nó lên như thế này, đặt sang một bên những vấn đề dư thừa:

if (strpos($file, '.jpg',1) && file_exists("$thumbsdir/$file") == false 
|| strpos($file, '.gif',1) && file_exists("$thumbsdir/$file") == false 
|| strpos($file, '.png',1) && file_exists("$thumbsdir/$file") == false) { 
    createThumb("$gallerydir/$file", "$thumbsdir/$file",$thumbsize); 
    fwrite($log,date("Y-m-d")." @ ".date("H:i:s")." CREATED: $thumbsdir/$file\n"); 
} 

@Fire Lancer's câu trả lời giải quyết các dự phòng tốt.

Answer 4

Các file_exists kiểm tra dường như là không đổi cho mỗi loại tập tin, do đó, không so sánh chúng trừ khi việc kiểm tra file_exists đã được thông qua.

if (file_exists("$thumbsdir/$file") == false) 
{ 
    if(strpos($file, '.jpg',1) || 
    strpos($file, '.gif',1) || 
    strpos($file, '.png',1) 
    { 
    createThumb("$gallerydir/$file", "$thumbsdir/$file",$thumbsize); 
    fwrite($log,date("Y-m-d")." @ ".date("H:i:s")." CREATED: $thumbsdir/$file\n"); 
    } 
} 

Answer 5

function check_thumbnail($file) 
{ 
    return (strpos($file, '.jpg',1) && file_exists("$thumbsdir/$file") == false || 
      strpos($file, '.gif',1) && file_exists("$thumbsdir/$file") == false || 
      strpos($file, '.png',1) && file_exists("$thumbsdir/$file") == false); 
} 

if (check_thumbnail ($file)) { 
    createThumb("$gallerydir/$file", "$thumbsdir/$file",$thumbsize); 
    fwrite($log,date("Y-m-d")." @ ".date("H:i:s")." CREATED: $thumbsdir/$file\n"); 
} 

Sau khi giải nén logic để một chức năng riêng biệt, bạn có thể làm giảm sự trùng lặp:

function check_thumbnail($file) 
{ 
    return (strpos($file, '.jpg',1) || 
      strpos($file, '.gif',1) || 
      strpos($file, '.png',1)) && 
      (file_exists("$thumbsdir/$file") == false); 
} 

Answer 6

tôi sẽ tách các IFS là có một số mã lặp lại trong đó. Ngoài ra, tôi cố gắng thoát khỏi một thói quen càng sớm càng tốt:

if (!strpos($file, '.jpg',1) && !strpos($file, '.gif',1) && !strpos($file, '.png',1)) 
{ 
    return; 
} 

if(file_exists("$thumbsdir/$file")) 
{ 
    return; 
} 

createThumb("$gallerydir/$file", "$thumbsdir/$file",$thumbsize); 
fwrite($log,date("Y-m-d")." @ ".date("H:i:s")." CREATED: $thumbsdir/$file\n"); 

Answer 7

Tôi tìm những thứ sau để dễ đọc hơn bằng cách sử dụng getimagesize(). Tôi đang viết điều này ra khỏi đầu của tôi vì vậy nó có thể yêu cầu một số gỡ lỗi.

Mã dọc có thể đọc được nhiều hơn ngang, imho.

// Extract image info if possible 
    // Note: Error suppression is for missing file or non-image 
if (@$imageInfo = getimagesize("{$thumbsdir}/{$file}")) { 

    // Accept the following image types 
    $acceptTypes = array(
     IMAGETYPE_JPEG, 
     IMAGETYPE_GIF, 
     IMAGETYPE_PNG, 
    ); 

    // Proceed if image format is acceptable 
    if (in_array($imageInfo[2], $acceptTypes)) { 

     //createThumb(...); 
     //fwrite(...); 

    } 

} 

Hòa bình + hacking hạnh phúc.

Answer 8

Cũng có thể ném hai xu của tôi trong.

if(!file_exists($thumbsdir . '/' . $file) && preg_match('/\.(?:jpe?g|png|gif)$/', $file)) { 
    createThumb($gallerydir . '/' . $file, $thumbsdir . '/' . $file, $thumbsize); 
    fwrite($log, date('Y-m-d @ H:i:s') . ' CREATED: ' . $thumbsdir . '/' . $file . "\n"); 
}