数码控科技猎奇Iphone动漫星座游戏电竞lolcosplay王者荣耀攻略allcnewsBLOGNEWSBLOGASKBLOGBLOGZSK全部技术问答问答技术问答it问答代码软件新闻开发博客电脑/网络手机/数码笔记本电脑互联网操作系统软件硬件编程开发360产品资源分享电脑知识文档中心IT全部全部分类全部分类技术牛文全部分类教程最新网页制作cms教程平面设计媒体动画操作系统网站运营网络安全服务器教程数据库工具网络安全软件教学vbscript正则表达式javascript批处理更多»编程更新教程更新游戏更新allitnewsJava新闻网络医疗信息化安全创业站长电商科技访谈域名会议专栏创业动态融资创投创业学院 / 产品经理创业公司人物访谈营销开发数据库服务器系统虚拟化云计算嵌入式移动开发作业作业1常见软件all电脑网络手机数码生活游戏体育运动明星影音休闲爱好文化艺术社会民生教育科学医疗健康金融管理情感社交地区其他电脑互联网软件硬件编程开发360相关产品手机平板其他电子产品摄影器材360硬件通讯智能设备购物时尚生活常识美容塑身服装服饰出行旅游交通汽车购房置业家居装修美食烹饪单机电脑游戏网页游戏电视游戏桌游棋牌游戏手机游戏小游戏掌机游戏客户端游戏集体游戏其他游戏体育赛事篮球足球其他运动球类运动赛车健身运动运动用品影视娱乐人物音乐动漫摄影摄像收藏宠物幽默搞笑起名花鸟鱼虫茶艺彩票星座占卜书画美术舞蹈小说图书器乐声乐小品相声戏剧戏曲手工艺品历史话题时事政治就业职场军事国防节日风俗法律法规宗教礼仪礼节自然灾害360维权社会人物升学入学人文社科外语资格考试公务员留学出国家庭教育学习方法语文物理生物工程学农业数学化学健康知识心理健康孕育早教内科外科妇产科儿科皮肤科五官科男科整形中医药品传染科其他疾病医院两性肿瘤科创业投资企业管理财务税务银行股票金融理财基金债券保险贸易商务文书国民经济爱情婚姻家庭烦恼北京上海重庆天津黑龙江吉林辽宁河北内蒙古山西陕西宁夏甘肃青海新疆西藏四川贵州云南河南湖北湖南山东江苏浙江安徽江西福建广东广西海南香港澳门台湾海外地区

如何避免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 获取视频时长的实例代码
  • php多维数组指定多字段排序的示例代码
  • php多进程实现代码
  • php 计算两个特别大的整数实例代码
  • php生成(支持多模板)二维码海报代码
  • php实现huffman编码/解码的示例代码
  • php实现数据库的增删查改功能及完整代码
  • php无限级评论嵌套实现代码
  • php比较两个字符串长度的方法
  • php基于session实现数据库交互的类实例
  • php中的观察者模式
  • php5.3安装zend guard loader图文教程
  • php cache类代码(php数据缓存类)
  • 对php新手的一些建议(php学习经验总结)
  • php检测接口traversable用法详解
  • php实现根据图片色界在不同位置加水印的方法
  • php自定义时间转换函数示例
  • php小技巧之过滤ascii控制字符
  • 免责声明 - 关于我们 - 联系我们 - 广告联系 - 友情链接 - 帮助中心 - 频道导航
    Copyright © 2017 www.zgxue.com All Rights Reserved