PHP コードの品質を向上させる 36 のヒント

WBOY
リリース: 2016-06-20 12:33:49
オリジナル
1417 人が閲覧しました

1. 相対パスを使用しないでください。

次のようなことがよくあります。

require_once('../../lib/some_class.php');
ログイン後にコピー

この方法には多くの欠点があります。

最初に指定された php インクルード パスを検索し、次に現在のディレクトリを検索します。

スクリプトが別のディレクトリにあるスクリプトによってインクルードされている場合、チェックされるパスが多すぎます。

もう 1 つの問題は、スケジュールされたタスクがスクリプトを実行するとき、その親ディレクトリが作業ディレクトリではない可能性があるということです。選択肢は [絶対パスを使用する] です:

絶対パスを定義しましたが、値はハードコードされています。パス /var/www/project も変更される可能性があるため、変更することもできます。毎回変更する必要がありますか? いいえ、次のような __FILE__ 定数を使用できます。

define('ROOT' , '/var/www/project/');require_once(ROOT . '../../lib/some_class.php');//rest of the code
ログイン後にコピー
これで、外部サーバーへの移動など、どのディレクトリに移動しても、コードは変更する必要はありません。

2. require、include、include_once、required_once を直接使用しないでください。
//suppose your script is /var/www/project/index.php//Then __FILE__ will always have that full path.define('ROOT' , pathinfo(__FILE__, PATHINFO_DIRNAME));require_once(ROOT . '../../lib/some_class.php');//rest of the code
ログイン後にコピー

クラスなどのスクリプトの先頭に複数のファイルを導入できます。ライブラリ、ツール ファイル、ヘルパー関数など:

この使用法は、より柔軟に記述する必要があります。たとえば、

require_once('lib/Database.php');require_once('lib/Mail.php');require_once('helpers/utitlity_functions.php');
ログイン後にコピー
何か違いはありますか? このコードは、より信頼性の高い可読性です。

この関数は、必要に応じて、次のように拡張できます。
function load_class($class_name){    //path to the class file    $path = ROOT . '/lib/' . $class_name . '.php');    require_once( $path );}load_class('Database');load_class('Mail');
ログイン後にコピー

さらに次のこともできます。

同じファイルの複数のディレクトリを検索
function load_class($class_name){    //path to the class file    $path = ROOT . '/lib/' . $class_name . '.php');    if(file_exists($path))    {        require_once( $path );    }}
ログイン後にコピー

コードを 1 つずつ変更することなく、クラス ファイルを配置するディレクトリを簡単に変更できます。

同様の関数を使用して、 HTML コンテンツなどのファイルをロードします。

3 . アプリケーションのコードのデバッグを継続します

開発環境では、データベース クエリを出力し、問題のある変数値をダンプし、問題が解決されたら、コメントするか削除しますが、より良い方法は、コードのデバッグを続けることです。

開発環境では、次のことができます。

サーバーでは、次のことができます。

define('ENVIRONMENT' , 'development');if(! $db->query( $query ){    if(ENVIRONMENT == 'development')    {        echo "$query failed";    }    else    {        echo "Database error. Please contact administrator";    }}
ログイン後にコピー
4. クロスプラットフォーム関数の実行を使用する コマンド

system、exec、passthru、shell_exec これらの 4 つの関数は、それぞれの動作が若干異なります。共有ホスティングでは、一部の機能が選択的に無効になる可能性があるため、ほとんどの初心者は、使用する前に毎回どの機能が利用可能であるかを確認する傾向があります。
define('ENVIRONMENT' , 'production');if(! $db->query( $query ){    if(ENVIRONMENT == 'development')    {        echo "$query failed";    }    else    {        echo "Database error. Please contact administrator";    }}
ログイン後にコピー

より良い解決策は、その機能をクロスプラットフォーム機能にカプセル化することです。

上記の関数は、コードの一貫性を維持するシステム関数が存在する限り、シェル コマンドを実行します。

5.
/**	Method to execute a command in the terminal	Uses :	1. system	2. passthru	3. exec	4. shell_exec*/function terminal($command){	//system	if(function_exists('system'))	{		ob_start();		system($command , $return_var);		$output = ob_get_contents();		ob_end_clean();	}	//passthru	else if(function_exists('passthru'))	{		ob_start();		passthru($command , $return_var);		$output = ob_get_contents();		ob_end_clean();	}	//exec	else if(function_exists('exec'))	{		exec($command , $output , $return_var);		$output = implode("n" , $output);	}	//shell_exec	else if(function_exists('shell_exec'))	{		$output = shell_exec($command) ;	}	else	{		$output = 'Command execution not possible on this system';		$return_var = 1;	}	return array('output' => $output , 'status' => $return_var);}terminal('ls');
ログイン後にコピー

上記の関数を使用して 1 つの項目を追加します。また、項目をリストに追加するときは、別の関数を作成する必要がありますか? いいえ、さまざまな種類のパラメーターに注意を払う必要があります。例:

これで、同じ関数をさまざまなタイプの入力パラメータを処理できるようになり、コードの多くをリファクタリングしてよりスマートにすることができます。 🎜>
function add_to_cart($item_id , $qty){    $_SESSION['cart']['item_id'] = $qty;}add_to_cart( 'IPHONE3' , 2 );
ログイン後にコピー
6. php の終了タグを意図的に無視します

その理由を知りたいのですが、php のアドバイスに関する多くのブログ投稿でこれについて言及されていません。
function add_to_cart($item_id , $qty){    if(!is_array($item_id))    {        $_SESSION['cart']['item_id'] = $qty;    }    else    {        foreach($item_id as $i_id => $qty)        {            $_SESSION['cart']['i_id'] = $qty;        }    }}add_to_cart( 'IPHONE3' , 2 );add_to_cart( array('IPHONE3' => 2 , 'IPAD' => 5) );
ログイン後にコピー

これで救われます。例を見てみましょう: super_class.php ファイル

Index.php

<?phpecho "Hello";//Now dont close this tag
ログイン後にコピー

この場合、ヘッダーは既に送信済みというエラーが発生します。なぜかというと、「スーパー エクストラ文字」がすでに出力されているため、スーパー エクストラ文字の位置を見つけるのに時間がかかるからです。 >

//super extra character after the closing tag
ログイン後にコピー
これはより良いでしょう。

7. すべての入力をどこかに集めて、一度にブラウザに出力します
require_once('super_class.php');//echo an image or pdf , or set the cookies or session data
ログイン後にコピー

これは出力バッファリングと呼ばれます。さまざまな関数:

<!--?php class super_class{    function super_function()    {        //super code    }}//No closing tag</pre-->
ログイン後にコピー

または、出力を関数のローカル変数に保存することもできます。次のように、ob_start と ob_end_clean を使用することもできます。 🎜>出力バッファリングが必要な理由:>>str_replaces 関数や preg_replaces などの出力を変更したり、HTML コンテンツの監視/デバッグを追加したりできます。>>出力するのは非常に悪いです。ブラウザに送信し、同時に PHP で処理すると、処理と出力が混在するため、サイドバーまたは途中でエラー メッセージが表示されるのを見たことがありますか? >8. 非 HTML コンテンツを出力する場合は、正しい MIME タイプのヘッダーを送信します。

は正常に動作します。 🎜>ヘッダー行に注意してください。この行は、送信されたコンテンツが XML タイプであることをブラウザーに伝えるため、多くの JavaScript ライブラリーもヘッダー情報に依存します。 image:JavaScript

function print_header(){    echo "Site Log and Login links";}function print_footer(){    echo "Site was made by me";}print_header();for($i = 0 ; $i ';}print_footer();
ログイン後にコピー
CSS

function print_header(){    $o = "Site Log and Login links";    return $o;}function print_footer(){    $o = "Site was made by me";    return $o;}echo print_header();for($i = 0 ; $i ';}echo print_footer();
ログイン後にコピー
9. mysql 接続に正しい文字エンコーディングを設定します

これまでに発生した Unicode/utf-8 エンコーディングは、 mysql テーブルを作成し、phpadmin では正しく表示できますが、コンテンツを取得してページに出力すると、文字化けが発生します。ここで問題となるのは、mysql 接続の文字エンコーディングです。データベースに接続したら、アプリケーションが複数の言語をサポートする必要がある場合は、接続の文字セットを設定するのが最適です。

10. 使用 htmlentities 设置正确的编码选项

php5.4前, 字符的默认编码是ISO-8859-1, 不能直接输出如À â等.

$value = htmlentities($this->value , ENT_QUOTES , CHARSET);
ログイン後にコピー

php5.4以后, 默认编码为UTF-8, 这將解决很多问题. 但如果你的应用是多语言的, 仍然要留意编码问题,.

11. 不要在应用中使用gzip压缩输出, 让apache处理

考虑过使用 ob_gzhandler 吗? 不要那样做. 毫无意义. php只应用来编写应用. 不应操心服务器和浏览器的数据传输优化问题.使用apache的mod_gzip/mod_deflate 模块压缩内容.

12. 使用json_encode输出动态javascript内容

时常会用php输出动态javascript内容:

$images = array( 'myself.png' , 'friends.png' , 'colleagues.png');$js_code = '';foreach($images as $image){$js_code .= "'$image' ,";}$js_code = 'var images = [' . $js_code . ']; ';echo $js_code;//Output is var images = ['myself.png' ,'friends.png' ,'colleagues.png' ,];
ログイン後にコピー

更聪明的做法, 使用 json_encode:

$images = array( 'myself.png' , 'friends.png' , 'colleagues.png');$js_code = 'var images = ' . json_encode($images);echo $js_code;//Output is : var images = ["myself.png","friends.png","colleagues.png"]
ログイン後にコピー

优雅乎?

13. 写文件前, 检查目录写权限

写或保存文件前, 确保目录是可写的, 假如不可写, 输出错误信息. 这会节约你很多调试时间. linux系统中, 需要处理权限, 目录权限不当会导致很多很多的问题, 文件也有可能无法读取等等.确保你的应用足够智能, 输出某些重要信息.

$contents = "All the content";$file_path = "/var/www/project/content.txt";file_put_contents($file_path , $contents);
ログイン後にコピー

这大体上正确. 但有些间接的问题. file_put_contents 可能会由于几个原因失败:>>父目录不存在>>目录存在, 但不可写>>文件被写锁住?所以写文件前做明确的检查更好.

$contents = "All the content";$dir = '/var/www/project';$file_path = $dir . "/content.txt";if(is_writable($dir)){    file_put_contents($file_path , $contents);}else{    die("Directory $dir is not writable, or does not exist. Please check");}
ログイン後にコピー

这么做后, 你会得到一个文件在何处写及为什么失败的明确信息.

14. 更改应用创建的文件权限

在linux环境中, 权限问题可能会浪费你很多时间. 从今往后, 无论何时, 当你创建一些文件后, 确保使用chmod设置正确权限. 否则的话, 可能文件先是由”php”用户创建, 但你用其它的用户登录工作, 系统將会拒绝访问或打开文件, 你不得不奋力获取root权限, 更改文件的权限等等.

// Read and write for owner, read for everybody elsechmod("/somedir/somefile", 0644);// Everything for owner, read and execute for otherschmod("/somedir/somefile", 0755);
ログイン後にコピー

15. 不要依赖submit按钮值来检查表单提交行为

if($_POST['submit'] == 'Save'){    //Save the things}
ログイン後にコピー

上面大多数情况正确, 除了应用是多语言的. ‘Save’ 可能代表其它含义. 你怎么区分它们呢. 因此, 不要依赖于submit按钮的值.

if( $_SERVER['REQUEST_METHOD'] == 'POST' and isset($_POST['submit']) ){    //Save the things}
ログイン後にコピー

现在你从submit按钮值中解脱出来了.

16. 为函数内总具有相同值的变量定义成静态变量

//Delay for some timefunction delay(){    $sync_delay = get_option('sync_delay');    echo "Delaying for $sync_delay seconds...";    sleep($sync_delay);    echo "Done ";}
ログイン後にコピー

用静态变量取代:

//Delay for some timefunction delay(){    static $sync_delay = null;    if($sync_delay == null)    {	$sync_delay = get_option('sync_delay');    }    echo "Delaying for $sync_delay seconds...";    sleep($sync_delay);    echo "Done ";}
ログイン後にコピー

17. 不要直接使用 $_SESSION 变量

某些简单例子:

$_SESSION['username'] = $username;$username = $_SESSION['username'];
ログイン後にコピー

这会导致某些问题. 如果在同个域名中运行了多个应用, session 变量可能会冲突. 两个不同的应用可能使用同一个session key. 例如, 一个前端门户, 和一个后台管理系统使用同一域名.从现在开始, 使用应用相关的key和一个包装函数:

define('APP_ID' , 'abc_corp_ecommerce');//Function to get a session variablefunction session_get($key){    $k = APP_ID . '.' . $key;    if(isset($_SESSION[$k]))    {        return $_SESSION[$k];    }    return false;}//Function set the session variablefunction session_set($key , $value){    $k = APP_ID . '.' . $key;    $_SESSION[$k] = $value;    return true;}
ログイン後にコピー

18. 將工具函数封装到类中

假如你在某文件中定义了很多工具函数:

function utility_a(){    //This function does a utility thing like string processing}function utility_b(){    //This function does nother utility thing like database processing}function utility_c(){    //This function is ...}
ログイン後にコピー

这些函数的使用分散到应用各处. 你可能想將他们封装到某个类中:

class Utility{    public static function utility_a()    {    }    public static function utility_b()    {    }    public static function utility_c()    {    }}//and call them as $a = Utility::utility_a();$b = Utility::utility_b();
ログイン後にコピー

显而易见的好处是, 如果php内建有同名的函数, 这样可以避免冲突.另一种看法是, 你可以在同个应用中为同个类维护多个版本, 而不导致冲突. 这是封装的基本好处, 无它.

19. Bunch of silly tips

>>使用echo取代print>>使用str_replace取代preg_replace, 除非你绝对需要>>不要使用 short tag>>简单字符串用单引号取代双引号>>head重定向后记得使用exit>>不要在循环中调用函数>>isset比strlen快>>始中如一的格式化代码>>不要删除循环或者if-else的括号不要这样写代码:

if($a == true) $a_count++;
ログイン後にコピー

这绝对WASTE.写成:

if($a == true){    $a_count++;}
ログイン後にコピー

不要尝试省略一些语法来缩短代码. 而是让你的逻辑简短.>>使用有高亮语法显示的文本编辑器. 高亮语法能让你减少错误.

20. 使用array_map快速处理数组

比如说你想 trim 数组中的所有元素. 新手可能会:

foreach($arr as $c => $v){	$arr[$c] = trim($v);}
ログイン後にコピー

但使用 array_map 更简单:

$arr = array_map('trim' , $arr);
ログイン後にコピー

这会为$arr数组的每个元素都申请调用trim. 另一个类似的函数是 array_walk. 请查阅文档学习更多技巧.

21. 使用 php filter 验证数据

你肯定曾使用过正则表达式验证 email , ip地址等. 是的,每个人都这么使用. 现在, 我们想做不同的尝试, 称为filter.php的filter扩展提供了简单的方式验证和检查输入.

22. 强制类型检查

$amount = intval( $_GET['amount'] );$rate = (int) $_GET['rate'];
ログイン後にコピー

这是个好习惯.

23. 如果需要,使用profiler如xdebug

如果你使用php开发大型的应用, php承担了很多运算量, 速度会是一个很重要的指标. 使用profile帮助优化代码. 可使用xdebug和webgrid.

24. 小心处理大数组

对于大的数组和字符串, 必须小心处理. 常见错误是发生数组拷贝导致内存溢出,抛出Fatal Error of Memory size 信息:

$db_records_in_array_format; //This is a big array holding 1000 rows from a table each having 20 columns , every row is atleast 100 bytes , so total 1000 * 20 * 100 = 2MB$cc = $db_records_in_array_format; //2MB moresome_function($cc); //Another 2MB ?
ログイン後にコピー

当导入或导出csv文件时, 常常会这么做.不要认为上面的代码会经常因内存限制导致脚本崩溃. 对于小的变量是没问题的, 但处理大数组的时候就必须避免.确保通过引用传递, 或存储在类变量中:

$a = get_large_array();pass_to_function(&$a);
ログイン後にコピー

这么做后, 向函数传递变量引用(而不是拷贝数组). 查看文档.

class A{    function first()    {        $this->a = get_large_array();        $this->pass_to_function();    }    function pass_to_function()    {        //process $this->a    }}
ログイン後にコピー

尽快的 unset 它们, 让内存得以释放,减轻脚本负担.

25. 由始至终使用单一数据库连接

确保你的脚本由始至终都使用单一的数据库连接. 在开始处正确的打开连接, 使用它直到结束, 最后关闭它. 不要像下面这样在函数中打开连接:

function add_to_cart(){    $db = new Database();    $db->query("INSERT INTO cart .....");}function empty_cart(){    $db = new Database();    $db->query("DELETE FROM cart .....");}
ログイン後にコピー

使用多个连接是个糟糕的, 它们会拖慢应用, 因为创建连接需要时间和占用内存.特定情况使用单例模式, 如数据库连接.

26. 避免直接写SQL, 抽象之

不厌其烦的写了太多如下的语句:

$query = "INSERT INTO users(name , email , address , phone) VALUES('$name' , '$email' , '$address' , '$phone')";$db->query($query); //call to mysqli_query()
ログイン後にコピー

这不是个建壮的方案. 它有些缺点:>>每次都手动转义值>>验证查询是否正确>>查询的错误会花很长时间识别(除非每次都用if-else检查)>>很难维护复杂的查询因此使用函数封装:

function insert_record($table_name , $data){    foreach($data as $key => $value)    {	//mysqli_real_escape_string        $data[$key] = $db->mres($value);    }    $fields = implode(',' , array_keys($data));    $values = "'" . implode("','" , array_values($data)) . "'";    //Final query    $query = "INSERT INTO {$table}($fields) VALUES($values)";    return $db->query($query);}$data = array('name' => $name , 'email' => $email  , 'address' => $address , 'phone' => $phone);insert_record('users' , $data);
ログイン後にコピー

看到了吗? 这样会更易读和扩展. record_data 函数小心的处理了转义.最大的优点是数据被预处理为一个数组, 任何语法错误都会被捕获.该函数应该定义在某个database类中, 你可以像 $db->insert_record这样调用.查看本文, 看看怎样让你处理数据库更容易.类似的也可以编写update,select,delete方法. 试试吧.

27. 將数据库生成的内容缓存到静态文件中

如果所有的内容都是从数据库获取的, 它们应该被缓存. 一旦生成了, 就將它们保存在临时文件中. 下次请求该页面时, 可直接从缓存中取, 不用再查数据库.好处:>>节约php处理页面的时间, 执行更快>>更少的数据库查询意味着更少的mysql连接开销

28. 在数据库中保存session

基于文件的session策略会有很多限制. 使用基于文件的session不能扩展到集群中, 因为session保存在单个服务器中. 但数据库可被多个服务器访问, 这样就可以解决问题.在数据库中保存session数据, 还有更多好处:>>处理username重复登录问题. 同个username不能在两个地方同时登录.>>能更准备的查询在线用户状态.

29. 避免使用全局变量

>>使用 defines/constants>>使用函数获取值>>使用类并通过$this访问

30. 在head中使用base标签

没听说过? 请看下面:

<img alt="" src="happy.jpg" />
ログイン後にコピー

base 标签非常有用. 假设你的应用分成几个子目录, 它们都要包括相同的导航菜单.

www.domain.com/store/home.php

www.domain.com/store/products/ipad.php

在首页中, 可以写:

<a href="home.php">Home</a><a href="products/ipad.php">Ipad</a>
ログイン後にコピー

但在你的ipad.php不得不写成:

<span style="color:#333333;font-family:''Helvetica, Arial, sans-serif'';"><a href="../home.php">Home</a><a href="ipad.php">Ipad</a></span>
ログイン後にコピー

因为目录不一样. 有这么多不同版本的导航菜单要维护, 很糟糕啊.

因此, 请使用base标签.

<a href="home.php">Home</a><a href="products/ipad.php">Ipad</a>
ログイン後にコピー

现在, 这段代码放在应用的各个目录文件中行为都一致.

31. 永远不要將 error_reporting 设为 0

关闭不相的错误报告. E_FATAL 错误是很重要的.

ini_set('display_errors', 1);error_reporting(~E_WARNING & ~E_NOTICE & ~E_STRICT);
ログイン後にコピー

32. 注意平台体系结构

integer在32位和64位体系结构中长度是不同的. 因此某些函数如 strtotime 的行为会不同.

在64位的机器中, 你会看到如下的输出.

$ php -aInteractive shellphp > echo strtotime("0000-00-00 00:00:00");-62170005200php > echo strtotime('1000-01-30');-30607739600php > echo strtotime('2100-01-30');4104930600
ログイン後にコピー

但在32位机器中, 它们將是bool(false).

33. 不要过分依赖 set_time_limit

如果你想限制最小时间, 可以使用下面的脚本:

<span style="color:#333333;font-family:''Helvetica, Arial, sans-serif'';">set_time_limit(30);//Rest of the code</span>
ログイン後にコピー

高枕无忧吗?  注意任何外部的执行, 如系统调用,socket操作, 数据库操作等, 就不在set_time_limits的控制之下.

 因此, 就算数据库花费了很多时间查询, 脚本也不会停止执行. 视情况而定.

34. 使用扩展库

一些例子:

>>mPDF — 能通过html生成pdf文档

>>PHPExcel — 读写excel

>>PhpMailer — 轻松处理发送包含附近的邮件

>>pChart — 使用php生成报表

使用开源库完成复杂任务, 如生成pdf, ms-excel文件, 报表等.

35. MVC フレームワークを使用する

codeigniter のような MVC フレームワークを使用する時が来ました。MVC フレームワークは、php コードを HTML から分離するだけです。

>>php コードと html コードを明確に区別できるため、チームでの共同作業に適しており、デザイナーとプログラマーが同時に作業できます。

>>オブジェクト指向に設計された機能により、メンテナンスが容易になります。 > 多くの提案、トリック、ハックがフレームワークに実装されています

36. 常に phpbench を参照してください

phpbench は、いくつかの小さな構文変更を示すベンチマーク結果を提供します。大きな違いをもたらす

PHP サイトのコメントを確認し、IRC で質問し、時々オープンソース コードを読み、Linux を使用して開発します。

ソース:php.cn
このウェブサイトの声明
この記事の内容はネチズンが自主的に寄稿したものであり、著作権は原著者に帰属します。このサイトは、それに相当する法的責任を負いません。盗作または侵害の疑いのあるコンテンツを見つけた場合は、admin@php.cn までご連絡ください。
人気のチュートリアル
詳細>
最新のダウンロード
詳細>
ウェブエフェクト
公式サイト
サイト素材
フロントエンドテンプレート