这是一个创建于 2749 天前的主题，其中的信息可能已经有所发展或是发生改变。
两千行的配置文件，各种路径写死，迁移搞到我脑袋痛，头部还有一行抑制报错的代码，虽然他在入口文件以及写过一次，在这里又写了一次
一个大号 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