有人认真看过 slim 的代码质量吗？

[whatisnew]

我就不多说了，能写出这种代码的人，还有人推崇。。。

https://github.com/slimphp/Slim/blob/2.x/Slim/Log.php#L308

if (!isset(self::$levels[$level])) {
    throw new \InvalidArgumentException('Invalid log level supplied to function');
} else if ($this->enabled && $this->writer && $level <= $this->level) {
....

throw new \InvalidArgumentException

-> then ->

else if ?

然后，该作者的 slim3 居然还是高度遵循 fig PSR ，我去。。。

[justanding]

不可饶恕，居然有一个 else if

[mahone3297]

@justanding 请教， else if 的问题是？

[manhere]

记得这样可以少写一对大括号，可以用的吧？

[bramblex]

烂得在能够接受的程度内就懒得造轮子

[notgod]

按我的理解
推崇的本身 并不是代码写的好不好 而是这种造物的精神 以及造出来以后开源给大家使用
避免重复造轮子的问题

在这个基础上 没人天生什么都会 一次性就能写好
都是慢慢去完善 根据个人喜好 一种逻辑 10 个人可能有 100 种写法
没有必要强加自己的逻辑 认为自己一定是正确的打开方式

仅仅是个人喜好而已
比如我 就喜欢做简单处理
强迫症 别人的代码 缩进混淆了我都要修正 不然不舒服

[zsx]

@mahone3297 这里 throw 出来后后续不会执行， else 是多余的，直接 if 就好了。

[yzyzsun]

看标题还以为是说 Ruby 的 Slim ……

[anewg]

@zsx throw 要上面的 if 成立才会进入啊， if 不成立就判断接下来的 else if ，跟直接 if 有什么关系？

[BOYPT]

看来我跟楼主水平差距太大，我硬是没看出问题在哪。
不过 3.x 的 slim 已经不用他自己的 Log 模块了，扔了很多包袱给其他包。

[raincious]

@anewg

他的意思是可以直接

if (!isset(self::$levels[$level])) {
throw new \InvalidArgumentException('Invalid log level supplied to function');
}

if (.....

不过其实没什么，不影响稳定性，后面重构的时候修改下就行了，并不需要惊讶。

[zsx]

@anewg 我解释错了……上面 if 成立就不会到 elseif 而且还 throw 了；如果把 elseif 改成 if 也不会有问题，上面 if 不成立就会到 elseif ，和直接 if 也没区别。

[Slienc7]

@zsx
明显 else if 更优雅。

[audi]

http://silex.sensiolabs.org 欢迎你

[lyf362345]

不知道这个是 slim2 还是 slim3, 我看过一部分 slim2 代码, 槽点更多.

[justanding]

@mahone3297 PSR2 里面是“应该使用关键词 elseif 代替所有 else if ，以使得所有的控制关键字都像是单独的一个词。”

[falcon05]

意思是 else if 多加了一个空格，所以问题很大？

[shiny]

正是提 pull request 好机会呀

[orvice]

@lyf362345 2.x

[mahone3297]

@zsx 怎么会多余。。。又不一定进 if 流程
看了你后面的解释，还是没明白。。。

@justanding 原来如此，有可能是你说的这个。

[zsx]

@mahone3297 所以我貌似直接忽略了 PSR （然后 get 错点了（