素材牛VIP会员
thinkphp中$this->reset && session($key, null); 这种写法是什么意思?
 隔***陈  分类:ThinkPHP  人气:801  回帖:5  发布于6年前 收藏
public function check($code, $id = '') {
        $key = $this->authcode($this->seKey).$id;
        // 验证码不能为空
        $secode = session($key);
        if(empty($code) || empty($secode)) {
            return false;
        }
        // session 过期
        if(NOW_TIME - $secode['verify_time'] > $this->expire) {
            session($key, null);
            return false;
        }

        if($this->authcode(strtoupper($code)) == $secode['verify_code']) {
            $this->reset && session($key, null);//这个地方为什么用&&,而不是三元运算符?
            return true;
        }

        return false;
    }

我的理解是应该写成这样:

$this->reset ? session($key, null):1;
 标签:thinkphpphp

讨论这个帖子(5)垃圾回帖将一律封号处理……

Lv1 新人
凌***志 移动开发工程师 6年前#1

根据&&的执行规则,如果左边返回true,则执行右边操作,否则不执行右边的操作。这样写既不会是二楼说的不规范,也不会是一楼说的用if代码太多。

Lv1 新人
非***扰 交互设计师 6年前#2

赞同 @zj飞得更高 同学的答案。

不过实际上,推荐用andor操作符,他们的优先级更低,更适合进行语句的链接。

更主要的是可读性不低啊,and是“就做...”,or是“否则做...”

Lv3 码奴
陈***子 PHP开发工程师 6年前#3

其实就是一个蹩脚的写法,虽然执行结果和三元运算符一样,没一点好处
可读性太差。

不赞同楼下的写法,既然有三元运算符,为什么还要用if?
一行代码可以搞定的问题,为什么要用3行?

Lv4 码徒
非***星 其它 6年前#4

你可以看做三元运算法理解,但是即使你看成三元的,也是一种不规范的写法。

其合理的写法应该是

if ($this->reset) {
    session($key, null);
}
Lv4 码徒
这***3 站长 6年前#5

一般称作短路运算
用于替代简化内容体只有一句的if语句.

不认为这是一种不规范的写法,存在即合理,这个运算符的用法就是这样的,只不过大多数用在条件判断里而已

记得在哪里看到过这种写法的效率是高于if的,没实测过,虽然这种性能效率可忽略
有些时候if嵌套的很多,但里面的内容却只有一句,用这个或者三目运算可读性反而会好点

 文明上网,理性发言!   😉 阿里云幸运券,戳我领取