LazyPHP icon indicating copy to clipboard operation
LazyPHP copied to clipboard

一些改进

Open koboshi opened this issue 11 years ago • 5 comments
trafficstars

include_once改为require 避免重复包含和嵌套引起bug 错误信息显示与否现在可通过常量DEBUG控制

koboshi avatar Jan 03 '14 08:01 koboshi

value || action; 这种写法是不推荐的。应该让自己的写法更加清晰。 http://programmers.stackexchange.com/questions/109138/is-it-bad-practice-to-use-short-circuit-evaluation-instead-of-an-if-clause

error_reporting 应该一直是 E_ALL^E_NOTICE,不应该可以通过配置关闭。 http://www.v2ex.com/t/67309

picasso250 avatar Jan 04 '14 02:01 picasso250

include_once 如果该文件已经包含,则不会再次包含。

lichunqiang avatar Jan 04 '14 14:01 lichunqiang

入口文件用once(或者过于依赖once)并不是一个好主意

koboshi avatar Jan 04 '14 14:01 koboshi

@lichunqiang 我同意 @koboshi 关于 include_once 的意见。 http://www.laruence.com/2012/09/12/2765.html

picasso250 avatar Jan 05 '14 01:01 picasso250

@picasso250 @koboshi 至少lp里使用了once,如果是自己写东西,当然不使用once,但是作为框架级别的,不能保证每个使用者都会注意到文件的引用关系。

lichunqiang avatar Jan 06 '14 02:01 lichunqiang