开发工具-PHP编码规范，这里该不该断行？

内容导读

互联网集市收集整理的这篇技术教程文章主要介绍了开发工具-PHP编码规范，这里该不该断行？，小编现在分享给大家，供广大互联网技能从业者学习和参考。文章包含1783字，纯文字阅读大概需要3分钟。

内容图文

phpStorm中自动的一个代码格式化规范。来自PSR01，“?”问号这里是否真的需要断行？
如何配置phpStorm去掉这个断行

//检测是否在SESSION 中
public function inSessionCheck()
{
    return ($this->UserId = session('UserId')) && self::isUserId($this->UserId) && ($this->UserName = session('UserName') && !empty($this->UserName))
        ? true : false;
}

回复内容：

phpStorm中自动的一个代码格式化规范。来自PSR01，“?”问号这里是否真的需要断行？
如何配置phpStorm去掉这个断行

//检测是否在SESSION 中
public function inSessionCheck()
{
    return ($this->UserId = session('UserId')) && self::isUserId($this->UserId) && ($this->UserName = session('UserName') && !empty($this->UserName))
        ? true : false;
}

看下面截图里面的（是一个下拉框）

这是代码过长不方便阅读的线 可以去掉

1. 配置的截图有人贴了我就不贴了

2. 关于PSR的相关规定在这里

一句话说就是每行的长度无论多长都不应该出错，但建议每行长度不超过80，检查器应当在长度超过120的时候报warning

1. XXX ? true : false实在啰嗦，!!XXX是转换为true/false的更短的写法，而题主的实例中表达式本身已经是and出来了，连!!都不需要，如果是我的话大概会写成

phpreturn $this->UserId = session('UserId'))
    && self::isUserId($this->UserId)
    && ($this->UserName = session('UserName')
    && !empty($this->UserName);

等等！

分完行我才发现，在一串and里面竟然藏了赋值，叫check的函数竟然有副作用，这都是坑队友的行为啊

phppublic function tryLoginBySession()
{
    $id = session('UserId');
    $name = session('UserName');

    if (!self::isUserId($id) || empty($name)) {
        return false;
    }

    $this->UserId = $id;
    $this->UserName = $name;

    return true;
}

a. 有副作用的方法就用动词词组，而非is check 之类的状态判断的方法名
b. 赋值就只赋值，判断就只判断，不要一边判断一遍赋值，比如说session里有ID但是self::isUserId失败的时候，这个方法返回false表示没登录，但 $this->UserId 竟然有值了，这是定时炸弹

最后才看到截图里后面那个方法也有? true : false，还是赶紧去掉吧

内容总结

以上是互联网集市为您收集整理的开发工具-PHP编码规范，这里该不该断行？全部内容，希望文章能够帮你解决开发工具-PHP编码规范，这里该不该断行？所遇到的程序开发问题。 如果觉得互联网集市技术教程内容还不错，欢迎将互联网集市网站推荐给程序员好友。

内容备注

版权声明：本文内容由互联网用户自发贡献，该文观点与技术仅代表作者本人。本站仅提供信息存储空间服务，不拥有所有权，不承担相关法律责任。如发现本站有涉嫌侵权/违法违规的内容， 请发送邮件至 gblab@vip.qq.com 举报，一经查实，本站将立刻删除。

内容手机端