北京快三预测号码推荐:如何避免PHP实例代码中的一些坏代码

北京快三走势图 www.mxld9.cn  更新时间:2018年07月08日 16:29:15   作者:风为南_   我要评论

本篇文章给大家分享了如何在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数组

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

首先,我明确表示,一个函数做多种类型的返回值是我个人感觉是不好的,在动态语言中虽然很常见,很多PHP的原生方法也有这样的,但是,在生产中使用这样的方式会造成函数返回的不确定性,我们需要作出很多判断来证明我们的结论,但是,如果返回值类型只有一种,我们就可以直接判断返回值就好了。

就像如下代码:

public function addNewUser() {
    $res = $this->addData();
    if ($res) {
      return true;
    } else {
      return [
        'error' => 1,
        'errormsg' => "没有添加成功"
      ];
    }
  }

这样的代码在作为调用者往往会多一次判断,如下:

public function index() {
    $res = $this->addNewUser();
    if (is_array($res) && isset($res['error'])) {
      return isset($res['errormsg']) ? $res['errormsg'] : "未知错误";
    }
    return "成功";
  }

这样的代码几乎每一次调用完成这个函数都会有这一套出现,不仅代码不美观,而且很臃肿。

这样的代码需要改善,首先限制住函数的返回值。比如,我们只让这个函数返回bool类型的数:

public function addNewUser() {
  $res = $this->addData();
  if ($res) {
    return true;
  } else {
    return false;
  }
}

但是,显然,很多时候,我们要的不是简单的真价值,所以,我们会选择返回更多信息,这个时候,我们可以有三种处理方式。

1)返回int类型的数,然后通过这个int类型的数去判断处理结果,我们可以添加上映射关系:

class Operate{
  public $operateRes = [
    0 => '成功',
    1 => '添加失败',
    2 => '未知错误',
  ];
  
  
  public function addNewUser() {
    $res = $this->addData();
    if ($res) {
      return 0;
    } else if ($res > 1) {
      return 1;
    }
    return 2;
  }
  
}

这样方法的调用者就可以很简单的使用方法并给出提示了:

$opera = new Operate();
$res = $opera->addNewUser();
return $opera->operateRes[$res];

给出统一的返回值类型的时候就完全不需要判断返回值类型而且可以设置一个规范返回提示。

2)我们也可以使用数组

3)数组给人不缺定性,因为很多时候,数组里可以认为的少写一些元素,如果少写了,程序直接报错,很不好。

所以第三种方式就是建议将固定格式的返回,写成一个类,做返回的时候,使用这个类:

class Operate{
  public function addNewUser() {
    $res = $this->addData();
    $result = new Result();
    if ($res) {
      $result->errno = 0;
      $result->errmsg = "成功";
    } else if ($res > 1) {
      $result->errno = 1;
      $result->errmsg = "失败";
    }
    $result->errno = 2;
    $result->errmsg = "未知错误";
    return $result;
  }
  
}

class Result {
  public $errno;
  public $errmsg;
}

这样的返回,保证了所有变量的存在,同样可以减少一次判断。

所以,综合以上,在我们返回结果的时候,尽量使用同种类型的变量,尽量减少使用数组返回。

  • PHP实现的构造sql语句类实例

    PHP实现的构造sql语句类实例

    这篇文章主要介绍了PHP实现的构造sql语句类,结合实例形式分析了PHP针对常用SQL语句的动态构造与生成技巧,需要的朋友可以参考下
    2016-02-02
  • php自定义加密与解密程序实例

    php自定义加密与解密程序实例

    这篇文章主要介绍了php自定义加密与解密程序,实例分析了自定义加密解密类文件及相关用法,具有一定参考借鉴价值,需要的朋友可以参考下
    2014-12-12
  • php将日期格式转换成xx天前的格式

    php将日期格式转换成xx天前的格式

    这篇文章主要介绍了php将日期格式转换成xx天前的格式,涉及php时间操作及正则匹配的技巧,非常具有实用价值,需要的朋友可以参考下
    2015-04-04
  • ThinkPHP跳转页success及error模板实例教程

    ThinkPHP跳转页success及error模板实例教程

    这篇文章主要介绍了ThinkPHP跳转页success及error模板实例教程,可以帮助初学者进一步熟悉ThinkPHP的页面跳转流程,需要的朋友可以参考下
    2014-07-07
  • PHP处理excel cvs表格的方法实例介绍

    PHP处理excel cvs表格的方法实例介绍

    PHP处理excel cvs表格想必有很多朋友对此还是很陌生的吧,下面以实例为大家介绍下,感兴趣的朋友可以参考下哈,希望对你有所帮助
    2013-05-05
  • php数组函数序列之in_array() - 查找数组中是否存在指定值

    php数组函数序列之in_array() - 查找数组中是否存在指定值

    in_array()定义和用法 in_array() 函数查找数组中是否存在指定值
    2011-11-11
  • win7安装php框架Yii的方法

    win7安装php框架Yii的方法

    这篇文章主要介绍了win7安装php框架Yii的方法,较为详细的分析讲述了Yii框架的下载、安装及相应问题的解决方法,需要的朋友可以参考下
    2016-01-01
  • 学习php设计模式 php实现原型模式(prototype)

    学习php设计模式 php实现原型模式(prototype)

    这篇文章主要介绍了php设计模式中的原型模式,使用php实现原型模式,感兴趣的小伙伴们可以参考一下
    2015-12-12
  • PHP中key和current,next的联合运用实例分析

    PHP中key和current,next的联合运用实例分析

    这篇文章主要介绍了PHP中key和current,next的联合运用,结合实例形式分析了key和current,next操作数组元素的相关技巧,需要的朋友可以参考下
    2016-03-03
  • 最新评论

    常用在线小工具

  • 中共中央直属机关党校 2019-04-21
  • 全国自由搏击锦标赛落下帷幕 2019-04-20
  • 近来人民网强国社区的评论少了很多,那是因为人们反映事实存在的问题,绝大多数都是说了等于白说,于是渐渐地失望了,渐渐地越来越多的人“算了,不说了!” 2019-04-19
  • 热巴窦骁携手演绎经典神话 2019-04-19
  • 神奇!沈阳现3D稻田画 2019-04-18
  • 中国和巴基斯坦将合拍电影《天路》 2019-04-18
  • 电视剧抱团出海 又有哪些作品走出了国门 2019-04-17
  • 谈判与加关税的不同在于,后者没有给小左唱赞歌的机会 2019-04-16
  • 全世界人民都要顺应人类社会发展规律,不断扩大社会财富公有制的范围,不断缩小社会财富私有制的范围,以便最终消灭社会财富私有制,建立共产主义社会财富公有制。 2019-04-15
  • 新时代·新征程十九大精神在基层---“各地聚焦”--湖南频道--人民网 2019-04-14
  • 首部由中国人翻译的120回《三国演义》问世 2019-04-14
  • 2022年冬奥会筹备进行时 2019-04-13
  • 牙膏的10个超实用功能 速收藏 2019-04-13
  • 新华保险荆州中支助力首届荆楚文化旅游节开幕 2019-04-12
  • 未来十天 山西以对流性天气为主 多阵性降水 2019-04-12
  • 765| 94| 196| 653| 634| 263| 2| 310| 416| 43|