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