有人认真看过 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 ，我去。。。

[hylent]

psr 是一个问题
另外 elseif 本身不需要 直接 if 不就可以了么
反正我个人的习惯 很少用 elseif
if 要么抛出异常 要么 return 跳出函数 主体流程都是保持在一个缩进里的 这样看起来更舒服
对于实在要用 elseif 的 我宁可转为 switch

[alexapollo]

这个槽点明显不正确吧？

if -> else if 这样的逻辑可以做著名的的火车选路，可以做更好的跳转的。运行起来会比多个 if 更快。

[arakashic]

除非 code style 上有要求，要不然这个 case 里面
if () {
throw
} else if {
}
和
if () {
throw
}
if {
}
有啥区别？不能光凭这一点就吐槽“能写出这种代码的人，还有人推崇。。。”吧。

[xiamx]

没啥不好，这种写法完整的描述了一个 state transition

[qhgongzi]

确实是不用 else if ，增加了理解难度，可以在后面直接 if
but....这么点小瑕疵，就上升到"能写出这种代码的人，还有人推崇。。。", 楼主酸味真浓。
要不楼主亮亮你的代码，看看有多少值得推崇的好代码

[whatisnew]

@qhgongzi 这里只是举个典型的例子，你可以看看其他的代码，比如 Slim.php 之类的，随手翻一下

[Slienc7]

@whatisnew 能贴一下自己的代码让我们学习一下吗？

[hanzhao]

我也习惯这样写 有问题吗？

[whatisnew]

你这是逗我玩么
https://github.com/slimphp/Slim/blob/2.x/Slim/Slim.php#L116
https://github.com/slimphp/Slim/blob/2.x/Slim/Slim.php#L984


are you fucking kidding me
https://github.com/slimphp/Slim/blob/2.x/Slim/Middleware/PrettyExceptions.php#L92
https://github.com/slimphp/Slim/blob/2.x/Slim/Middleware/SessionCookie.php#L94
https://github.com/slimphp/Slim/blob/2.x/Slim/Environment.php#L169
https://github.com/slimphp/Slim/blob/2.x/Slim/Environment.php#L176
https://github.com/slimphp/Slim/blob/2.x/Slim/Route.php#L165
https://github.com/slimphp/Slim/blob/2.x/Slim/Router.php#L245

这 tm 整个框架设计的就有问题，从 slim.php 到 route 到 middleware 到 log 到 能跑起来真是走运

[whatisnew]

关于 http://php.net/manual/en/class.iteratoraggregate.php 和 https://github.com/slimphp/Slim/blob/2.x/Slim/Helper/Set.php#L77

关于 session cookie http://php.net/manual/zh/function.session-set-save-handler.php
关于 log exception http://php.net/manual/zh/class.logicexception.php
关于 log write https://github.com/yiisoft/yii2/blob/master/framework/log/Logger.php

关于如何设计一个 PHP 框架 http://symfony.com/doc/current/create_framework/index.html

我敢说作者是个野鸡

[ioven]

有其他小巧好用的逛街推荐不？

[jeffw]

帖一篇王垠博客的文章，直接拉到页面最后面开始看，第 2 条。
http://www.yinwang.org/blog-cn/2013/03/15/language-design-mistake1

然后再读读他的这篇文章，我觉得他说的是很有道理的。
http://www.yinwang.org/blog-cn/2015/11/21/programming-philosophy