两千行的配置文件，各种路径写死，迁移搞到我脑袋痛，头部还有一行抑制报错的代码，虽然他在入口文件以及写过一次，在这里又写了一次
一个大号 if
字体调最小，屏幕还是放不下他这个 if，他的目的是
if(!empty($html)){
噼里啪啦几百行
}
return false;
这 TM 就不能
if( empty($html) ){
return false;
}
下面再继续噼里啪啦
抹黑 thinkphp 和 thinkphp 爱好者？不，我没有
This topic created in 3058 days ago, the information mentioned may be changed or developed.
 |
1
已经不错了，还给你留了那么多中文注释。。。知足吧
|
1
 |
ifconfigJun 22, 2018
3
一看还觉得不错，仔细看原来有几十个 if
|
 |
AJianJun 22, 2018 via Android
4
开发者自身水平的问题，就算换成别的语言也会写出类似的代码。
|
 |
reselfJun 22, 2018 via Android
5
屎山
|
 |
heart4lorJun 22, 2018
6
空格缩进的点点看得脑阔疼
|
|
heart4lorJun 22, 2018
7
刚发现第二张图是 tab+space 混合缩进，强迫症要发作了
|
 |
manhereJun 22, 2018 via Android
8
看格式，你说的千行配置应该是将全局 config 输出的结果数组放这了，应该是人为的为了增大维护难度。
|
 |
torment5524Jun 22, 2018
9
不会截图我就不发了，还是前阵子接手的平台，有个用户排行榜，2 个小时（有人去点才会执行判断是否经过 2 小时，热门时段排行榜不刷新）刷新一次，一次刷新要 5 分钟，我看下了，一个排行榜刷新要执行上千次 sql 查询（你没看错，他查询完一次统计数据，然后会 foreach 逐个用户再去查询用户信息），按照日和周和总计这几个维度，每个维度都这么单独来一遍，然后把所有的结果拼成一条字符串，再存到 redis 里面。就两张表，一张积分记录表，一张用户表，分别 1600w 条数据和 60 万条数据。
thinkcmf 的，也算是基于 thnkphp 的吧？。 现在需要把这块改成实时的，脑仁疼。 |
 |
tomczhenJun 22, 2018 via Android
10
比我司的代码还是好一些的，至少只有一个配置文件起作用。
|
 |
LucupsJun 22, 2018
11
维护这样的代码还是偷着乐吧
我之前维护的一个项目的前任的前任在代码里是直接下毒的。。。  |
 |
12
@tomczhen 你这话逗乐我了，我也遇到过那种配置不起作用的项目，另外还遇到过一个抑制报错，把错误都屏蔽，最后线上跑错误日志把硬盘写满了的
|
 |
ben1024Jun 25, 2018
13
跟框架无关，个人水平
|
 |
nbboyJul 20, 2018
14
个人喜好
|
 |
astomeJul 27, 2018
15
写的不错了
|
Select Language