导航菜单
首页 > 网络编程 > PHP编程 » 正文

如何避免PHP实例代码中的一些坏代码

后台-系统设置-扩展变量-手机广告位-内容正文顶部
本篇文章给大家分享了如何在PHP实例代码中发现坏代码以及如何修复的问题,有兴趣的朋友参考下。

做PHP开发已经有快一年的时间了,在这一年的时间中,学习了很多生产环境中的技巧,学习了很多东西,期间也阅读了一些优秀的源码和关于代码的书,对写代码这一块有了一定的思考,也看过很多别人写的好的代码和坏的代码,这里说说自己的感悟和改进吧。

本篇博客直说自己的感悟,在写代码时,我给自己立下的规则,这样可以让代码清晰可读并少走一些坑。这些简单的规则虽然没有设计模式看起来那么激动人心,但是,平常注意可以让代码看起来很清爽。

1. 不要在对象外使用未声明的变量

这个问题其实表述起来可能不容易理解。这个问题是因为PHP语言本身的特点决定的。由于PHP是一个弱类型的动态脚本语言,所以很多情况下,给了这个语言本省很宽松的条件让开发者去编写代码。但是往往这些便利也会变为坑,所以在使用一些动态语言很方便的写法的时候,尤其要注意。

下面我们先声明一个类,暂且叫这个类为用户类,这个User类的背景设定为,框架自带,不允许修改,并且隐藏在框架深处,不容易发现,实际案例可以参考laravel框架的Request类,代码如下:

class User {  public $username;  public $password;    public $otherInfo = [];      public function readUserInfo() {    return [      'username' => $this->username,      'password' => $this->password,    ];  }    public function addAtri($info) {    array_push($this->otherInfo, $info);  }}

这样的代码看似中规中矩,但是接下来,我们需要对这个类进行操作:

$user = new User();$user->userRealName = "hello world";

这样的代码在PHP中是完全可以运行的,并且不会报错,但是这样的代码会对之后的一些事情做为干扰。我们现在假定,上边的代码是在PHP web项目中是一个拦截器,或者叫做中间件也可以,然后我们在controller中会会使用到这个类的实例,并且使用到这个中间件中添加的这个变量,如下:

 class WebOperate {   public function doOprate(User $user) {     $user->userRealName = "hello world";     next($user);   } }

这里设定的场景是,WebOperate是一个中间件,所有的Controller都会走这个中间件后到达Controller,之后,在处理相应的Controller的功能,接下来,Controller会将中间件的实例注入进来,供控制器使用,而中间件开发人员不是很在意其的存在:

 class IndexController {   public function index(User $user) {     return $user->userRealName;   } }

而这样的代码是可以完美运行的,接下来,开发人员想要的实另一个User类,这个User类中添加一些其他功能,正如之前所说,这个类在框架深处并且很难找到,且不允许修改,因为其他功能使用了这个类,所以,我们只有继承并添加方法。根据开发经验,开发人员会认为User类中存在这个userRealName变量,所以就造成了这个写法:

首先是基于这个User衍生出来的Teacher类:

 class Teacher extends User {   public function sayHello() {     return "hello world";   } }

这样,我们的Teacher就可以sayhello了,但是,这个时候,在我们的Controller中还想知道老师的真实姓名,怎么办?根据经验,我们可以将注入的类换成Teacher并且返回真实姓名:

 class IndexController {   public function index(Teacher $user) {     return $user->userRealName;   } }

那么这下问题来了,其实User类中并没有这个类,所以这个变量根本没有数值,但是根据经验,是中间件已经赋值过一次了,所以我们应该可以直接使用,但是并没有这个数值,我们开始看源码发现,继承的User类中根本不存在这个变量,那么这个变量之前为什么可以使用呢,因为在中间件中,给User的实力付了值。

所以我们的不能这样直接使用未声明的变量,在一个类中。

我们应该这样写:

class WebOperate {  public function doOprate(User $user) {    $user->addAtri([      'userRealName' => 'hello world',    ]);    next($user);  }}

这样的中间件,在调用的时候继承类也可以使用同样的方法,很简单并且很不容易出现坏的味道。

2. 类or数组

其实这个问题同时也衍生出了另外的问题,就是函数返回值的问题。

收藏此文 赞一个 ( ) 打赏本站

如果本文对你有所帮助请打赏本站

  • 打赏方法如下:
  • 支付宝打赏
    支付宝扫描打赏
    微信打赏
    微信扫描打赏
后台-系统设置-扩展变量-手机广告位-内容正文底部

相关推荐:

留言与评论(共有 0 条评论)
   
验证码:
二维码