格式化if语句以提高可读性

本文介绍了格式化if语句以提高可读性的处理方法，对大家解决问题具有一定的参考价值，需要的朋友们下面随着小编来一起学习吧！ 问题描述

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 / $文件，$ thumbsize）; fwrite（$ log，date（Y-m-d）。@.date（H：i：s）。CREATED：$ thumbsdir / $ file \\\）;

解决方案

是一个图像的逻辑到自己的功能，这使得如果更具可读性，也允许您集中逻辑。

函数is_image（$ filename）{ $ image_extensions = array（'png'，'gif'，'jpg'）; $ b foreach（$ image_extensions as $ extension） if（strrpos（$ filename，。$ extension）！== FALSE） return true; 返回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 \\\）; }

What's the best way to format this for readability?

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"); }

解决方案

I'd extract the "is an image" logic into its own function, which makes the if more readable and also allows you to centralize the 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"); }