Thinkphp 5.0 的代码我真是没法说了太渣了吧写框架的人不懂 OOP 思想?

/** * 验证单个字段规则 * @access protected * @param string $field 字段名 * @param mixed $value 字段值 * @param mixed $rules 验证规则 * @param array $data 数据 * @param string $title 字段描述 * @param array $msg 提示信息 * @return mixed */ protected function checkItem($field, $value, $rules, $data, $title = '', $msg = []) { if ($rules instanceof \Closure) { // 匿名函数验证支持传入当前字段和所有字段两个数据 $result = call_user_func_array($rules, [$value, $data]); } else { // 支持多规则验证 require|in:a,b,c|... 或者 ['require','in'=>'a,b,c',...] if (is_string($rules)) { $rules = explode('|', $rules); } $i = 0; foreach ($rules as $key => $rule) {

            if ($rule instanceof \Closure) {
                $result = call_user_func_array($rule, [$value, $data]);
            } else {
                // 验证
                if (is_numeric($key)) {
                    if (strpos($rule, ':')) {
                        list($type, $rule) = explode(':', $rule, 2);
                        if (isset($this->alias[$type])) {
                            // 判断别名
                            $type = $this->alias[$type];
                        }
                        $info = $type;
                    } elseif (method_exists($this, $rule)) {
                        $type = $rule;
                        $info = $rule;
                        $rule = '';
                    }else {
                        $type = 'is';
                        $info = $rule;
                    }
                } else {
                    $info = $type = $key;
                }

                // 如果不是 require 有数据才会行验证
                if (0 === strpos($info, 'require') || (!is_null($value) && '' !== $value)) {
                    // 验证类型
                    $callback = isset(self::$type[$type]) ? self::$type[$type] : [$this, $type];
                    // 验证数据
                    $result = call_user_func_array($callback, [$value, $rule, $data, $field]);
                } else {
                    $result = true;
                }
            }

            if (false === $result) {
                // 验证失败 返回错误信息
                if (isset($msg[$i])) {
                    $message = $msg[$i];
                    if (is_string($message) && strpos($message, '{%') === 0) {
                        $message = (substr($message, 2, -1));
                    }
                } else {
                    $message = $this->getRuleMsg($field, $title, $info, $rule);
                }
                return $message;
            } elseif (true !== $result) {
                // 返回自定义错误信息
                return $result;
            }
            $i++;
        }
    }
    return true !== $result ? $result : true;
}

我的天这是啥~

尤其是下面这几行代码我感觉要是在公司里写类似的会被别人打死的

if (is_numeric($key)) { if (strpos($rule, ':')) { list($type, $rule) = explode(':', $rule, 2); if (isset($this->alias[$type])) { // 判断别名 $type = $this->alias[$type]; } $info = $type; } elseif (method_exists($this, $rule)) { $type = $rule; $info = $rule; $rule = ''; }else { $type = 'is'; $info = $rule; } } else { $info = $type = $key; }

realpg

2016-12-18 11:57:03 +08:00

@whahuzhihao
@Syc

我能看懂也见过很多但是从来没考虑过为什么或者是谁教出来的之前只是归结成习惯问题了
反正我自己是不写这种的
我的开发环境对于允许出现判断的地方出现赋值运算符会直接报错认为是误写即使是“用技巧”特意这样写的也不让

反正我是难以阅读这种代码，我手底下也管理过大量写代码的人也做 code review ，也有不少从 C/JAVA 过来的，反正是没见过自己这么写的，自己用的代码我看不到，需要给别人看让别人审的代码，都是尽量照顾可读性

Jakesoft

2016-12-18 12:33:09 +08:00

@realpg 我的言论有点不友好，抱歉。

如果你是团队组织者，是可以决定该使用哪种写法，毕竟每个团队应该有统一规范。

PHP 中有使用 true === $result , 我同事也是这么写，我以前也是写 $result === true,后来发现换个位置其实没什么大不了。

之所以举 c++的例子是因为我看一些初级 c++教程都是这么写的，这点有装逼悬疑。

另外我目前也是写 php 。

---

顺手在 symfony 的源码里面搜了一下 `null !==` 有 100+的结果。 psr 规范没有规定应该使用哪种写法，所以一切看团队的建设咯。

realpg

2016-12-18 12:49:46 +08:00

@Jakesoft
我也道歉说话冲了点
我也不是没见过只是真没理解或者说没去探究为啥要这么写而且写 PHP PYTHON JAVA RUBY 的大量框架什么的都不会这么写

能运行那么代码本身就没错

还有就是标准 PSR 来说很多东西他规范的我们都故意不去按照这个去做其实规定只是为了有规定，比如连花括号都规定，对于很多大工程，需要上下回顾引用变量的变化的情况，我用 1080p 的显示器竖着放都嫌一页代码显示的行数不够照顾以及阅读原作者的意图，就是用自己的习惯当标准的生动代表。大厂在复杂算法下进行历史代码调错很多时候

基本上日常生产，公司的标准以方便别人阅读第一，开发速度第二，同一个文件里的变量命名法不同，其实并不会影响开发效率，也不会影响可读性，顶天是修改时候再打变量费劲

从日常快速阅读来说，本身人类的判断习惯就是一对多下变量先提。就像我生病了要请病假，给领导打电话，王总我要请病假，我生病了，生病了是状态的一种常量，很少有人说王总，生病了那个是我我要请病假。

在此例中，就是防止自己写错和让别人读的快 0.1s 哪个更优先的问题。作为公众代码，我个人倾向于语义优先。

您提到了 C++，我不开发 C++，略懂一点，上学也学过玩过，我估计是 C++的复杂工程逻辑判断让人欲仙欲死的原因导致的吧

Thinkphp 5.0 的代码我真是没法说了 太渣了吧 写框架的人不懂 OOP 思想?

Thinkphp 5.0 的代码我真是没法说了太渣了吧写框架的人不懂 OOP 思想?