2025年2月24日 星期一 甲辰(龙)年 腊月廿四 设为首页 加入收藏
rss
您当前的位置:首页 > 计算机 > 编程开发 > PHP

提高php代码质量 36计(上)

时间:09-21来源:作者:点击数:49

1.不要使用相对路径

常常会看到: 

  • require_once('../../lib/some_class.php'); 

该方法有很多缺点:

它首先查找指定的php包含路径, 然后查找当前目录.

因此会检查过多路径.

如果该脚本被另一目录的脚本包含, 它的基本目录变成了另一脚本所在的目录.

另一问题, 当定时任务运行该脚本, 它的上级目录可能就不是工作目录了.

因此最佳选择是使用绝对路径:

  • define('ROOT' , '/var/www/project/');  
  • require_once(ROOT . '../../lib/some_class.php');  
  •  
  • //rest of the code 

我们定义了一个绝对路径, 值被写死了. 我们还可以改进它. 路径 /var/www/project 也可能会改变, 那么我们每次都要改变它吗? 不是的, 我们可以使用__FILE__常量, 如: 

  • //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 

现在, 无论你移到哪个目录, 如移到一个外网的服务器上, 代码无须更改便可正确运行.

2. 不要直接使用 require, include, include_once, required_once

可以在脚本头部引入多个文件, 像类库, 工具文件和助手函数等, 如: 

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

还可做得更多:

为同样文件查找多个目录

能很容易的改变放置类文件的目录, 无须在代码各处一一修改

可使用类似的函数加载文件, 如html内容.

3. 为应用保留调试代码

在开发环境中, 我们打印数据库查询语句, 转存有问题的变量值, 而一旦问题解决, 我们注释或删除它们. 然而更好的做法是保留调试代码.

在开发环境中, 你可以:

  • define('ENVIRONMENT' , 'development');  
  •  
  • if(! $db->query( $query )  
  • {  
  •     if(ENVIRONMENT == 'development')  
  •     {  
  •         echo "$query failed";  
  •     }  
  •     else 
  •     {  
  •         echo "Database error. Please contact administrator";  
  •     }  

在服务器中, 你可以:

  • define('ENVIRONMENT' , 'production');  
  •  
  • if(! $db->query( $query )  
  • {  
  •     if(ENVIRONMENT == 'development')  
  •     {  
  •         echo "$query failed";  
  •     }  
  •     else 
  •     {  
  •         echo "Database error. Please contact administrator";  
  •     }  

4. 使用可跨平台的函数执行命令

system, exec, passthru, shell_exec 这4个函数可用于执行系统命令. 每个的行为都有细微差别. 问题在于, 当在共享主机中, 某些函数可能被选择性的禁用. 大多数新手趋于每次首先检查哪个函数可用, 然而再使用它.

更好的方案是封成函数一个可跨平台的函数. 

  • /**  
  •     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'); 

上面的函数將运行shell命令, 只要有一个系统函数可用, 这保持了代码的一致性. 

5. 灵活编写函数

  • function add_to_cart($item_id , $qty)
  • {
  • $_SESSION['cart']['item_id'] = $qty;
  • }
  •  
  • add_to_cart( 'IPHONE3' , 2 );

使用上面的函数添加单个项目. 而当添加项列表的时候,你要创建另一个函数吗? 不用, 只要稍加留意不同类型的参数, 就会更灵活. 如:

  • 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) ); 

现在, 同个函数可以处理不同类型的输入参数了. 可以参照上面的例子重构你的多处代码, 使其更智能.

6. 有意忽略php关闭标签

我很想知道为什么这么多关于php建议的博客文章都没提到这点.

  • <?php  
  •  
  • echo "Hello";  
  •  
  • //Now dont close this tag 

这將节约你很多时间. 我们举个例子:

一个 super_class.php 文件

  • <?php  
  • class super_class  
  • {  
  •     function super_function()  
  •     {  
  •         //super code  
  •     }  
  • }  
  • ?>  
  • //super extra character after the closing tag 

index.php

  • require_once('super_class.php');  
  •  
  • //echo an image or pdf , or set the cookies or session data 

这样, 你將会得到一个 Headers already send error. 为什么? 因为 “super extra character” 已经被输出了. 现在你得开始调试啦. 这会花费大量时间寻找 super extra 的位置.

因此, 养成省略关闭符的习惯:

  • <?php  
  • class super_class  
  • {  
  •     function super_function()  
  •     {  
  •         //super code  
  •     }  
  • }  
  •  
  • //No closing tag 

这会更好. 

7. 在某地方收集所有输入, 一次输出给浏览器

这称为输出缓冲, 假如说你已在不同的函数输出内容:

  • function print_header()  
  • {  
  •     echo "<div id='header'>Site Log and Login links</div>";  
  • }  
  •  
  • function print_footer()  
  • {  
  •     echo "<div id='footer'>Site was made by me</div>";  
  • }  
  •  
  • print_header();  
  • for($i 0 ; $i 100$i++)  
  • {  
  •     echo "I is : $i <br />';  
  • }  
  • print_footer(); 

替代方案, 在某地方集中收集输出. 你可以存储在函数的局部变量中, 也可以使用ob_start和ob_end_clean. 如下:

  • function print_header()  
  • {  
  •     $o "<div id='header'>Site Log and Login links</div>";  
  •     return $o;  
  • }  
  •  
  • function print_footer()  
  • {  
  •     $o "<div id='footer'>Site was made by me</div>";  
  •     return $o;  
  • }  
  •  
  • echo print_header();  
  • for($i 0 ; $i 100$i++)  
  • {  
  •     echo "I is : $i <br />';  
  • }  
  • echo print_footer(); 

为什么需要输出缓冲:

>>可以在发送给浏览器前更改输出. 如 str_replaces 函数或可能是 preg_replaces 或添加些监控/调试的html内容.

>>输出给浏览器的同时又做php的处理很糟糕. 你应该看到过有些站点的侧边栏或中间出现错误信息. 知道为什么会发生吗? 因为处理和输出混合了.

8. 发送正确的mime类型头信息, 如果输出非html内容的话.

输出一些xml.

  • $xml = '<?xml version="1.0" encoding="utf-8" standalone="yes"?>';  
  • $xml = "<response>  
  •   <code>0</code>  
  • </response>";  
  •  
  • //Send xml data  
  • echo $xml; 

工作得不错. 但需要一些改进.

  • $xml = '<?xml version="1.0" encoding="utf-8" standalone="yes"?>';  
  • $xml = "<response>  
  •   <code>0</code>  
  • </response>";  
  •  
  • //Send xml data  
  • header("content-type: text/xml");  
  • echo $xml; 

注意header行. 该行告知浏览器发送的是xml类型的内容. 所以浏览器能正确的处理. 很多的javascript库也依赖头信息.

类似的有 javascript , css, jpg image, png image:

JavaScript

  • header("content-type: application/x-javascript");  
  • echo "var a = 10"

CSS

  • header("content-type: text/css");  
  • echo "#div id { background:#000; }"; 

9. 为mysql连接设置正确的字符编码

曾经遇到过在mysql表中设置了unicode/utf-8编码,  phpadmin也能正确显示, 但当你获取内容并在页面输出的时候,会出现乱码. 这里的问题出在mysql连接的字符编码.

  • //Attempt to connect to database  
  • $c = mysqli_connect($this->host , $this->username, $this->password);  
  •  
  • //Check connection validity  
  • if (!$c)&nbsp;  
  • {  
  •     die ("Could not connect to the database host: <br />". mysqli_connect_error());  
  • }  
  •  
  • //Set the character set of the connection  
  • if(!mysqli_set_charset ( $c 'UTF8' ))  
  • {  
  •     die('mysqli_set_charset() failed');  

一旦连接数据库, 最好设置连接的 characterset. 你的应用如果要支持多语言, 这么做是必须的.

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 else  
  • chmod("/somedir/somefile"0644);  
  •  
  • // Everything for owner, read and execute for others  
  • chmod("/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 time  
  • function delay()  
  • {  
  •     $sync_delay = get_option('sync_delay');  
  •  
  •     echo "<br />Delaying for $sync_delay seconds...";  
  •     sleep($sync_delay);  
  •     echo "Done <br />";  

用静态变量取代:

  • //Delay for some time  
  • function delay()  
  • {  
  •     static $sync_delay null;  
  •  
  •     if($sync_delay == null)  
  •     {  
  •     $sync_delay = get_option('sync_delay');  
  •     }  
  •  
  •     echo "<br />Delaying for $sync_delay seconds...";  
  •     sleep($sync_delay);  
  •     echo "Done <br />";  

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

  • $_SESSION['username'] = $username;  
  • $username = $_SESSION['username']; 

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

从现在开始, 使用应用相关的key和一个包装函数:

  • define('APP_ID' , 'abc_corp_ecommerce');  
  •  
  • //Function to get a session variable  
  • function session_get($key)  
  • {  
  •     $k = APP_ID . '.' . $key;  
  •  
  •     if(isset($_SESSION[$k]))  
  •     {  
  •         return $_SESSION[$k];  
  •     }  
  •  
  •     return false;  
  • }  
  •  
  • //Function set the session variable  
  • function 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内建有同名的函数, 这样可以避免冲突.

另一种看法是, 你可以在同个应用中为同个类维护多个版本, 而不导致冲突. 这是封装的基本好处, 无它.

方便获取更多学习、工作、生活信息请关注本站微信公众号城东书院 微信服务号城东书院 微信订阅号
推荐内容
相关内容
栏目更新
栏目热门
本栏推荐