别人写的鸟程序展示一波(使用 thinkphp 框架)

2018-06-22 18:48:21 +08:00
 tangzhangming

两千行的配置文件,各种路径写死,迁移搞到我脑袋痛,头部还有一行抑制报错的代码,虽然他在入口文件以及写过一次,在这里又写了一次

一个大号 if

字体调最小,屏幕还是放不下他这个 if,他的目的是

if(!empty($html)){
    噼里啪啦几百行
}
return false;

这 TM 就不能

if( empty($html) ){
    return false;
}
下面再继续噼里啪啦

抹黑 thinkphp 和 thinkphp 爱好者?不,我没有

4109 次点击
所在节点    PHP
15 条回复
zhoushiya
2018-06-22 18:51:43 +08:00
已经不错了,还给你留了那么多中文注释。。。知足吧
Jakesoft
2018-06-22 19:48:37 +08:00
@zhoushiya 恕我直言,这的确是一坨翔
ifconfig
2018-06-22 20:09:19 +08:00
一看还觉得不错,仔细看原来有几十个 if
AJian
2018-06-22 20:37:47 +08:00
开发者自身水平的问题,就算换成别的语言也会写出类似的代码。
reself
2018-06-22 21:38:55 +08:00
屎山
heart4lor
2018-06-22 21:44:04 +08:00
空格缩进的点点看得脑阔疼
heart4lor
2018-06-22 21:45:17 +08:00
刚发现第二张图是 tab+space 混合缩进,强迫症要发作了
manhere
2018-06-22 21:45:55 +08:00
看格式,你说的千行配置应该是将全局 config 输出的结果数组放这了,应该是人为的为了增大维护难度。
torment5524
2018-06-22 21:50:04 +08:00
不会截图我就不发了,还是前阵子接手的平台,有个用户排行榜,2 个小时(有人去点才会执行判断是否经过 2 小时,热门时段排行榜不刷新)刷新一次,一次刷新要 5 分钟,我看下了,一个排行榜刷新要执行上千次 sql 查询(你没看错,他查询完一次统计数据,然后会 foreach 逐个用户再去查询用户信息),按照日和周和总计这几个维度,每个维度都这么单独来一遍,然后把所有的结果拼成一条字符串,再存到 redis 里面。就两张表,一张积分记录表,一张用户表,分别 1600w 条数据和 60 万条数据。
thinkcmf 的,也算是基于 thnkphp 的吧?。
现在需要把这块改成实时的,脑仁疼。
tomczhen
2018-06-22 21:56:13 +08:00
比我司的代码还是好一些的,至少只有一个配置文件起作用。
Lucups
2018-06-22 22:02:15 +08:00
维护这样的代码还是偷着乐吧

我之前维护的一个项目的前任的前任在代码里是直接下毒的。。。

tangzhangming
2018-06-23 09:19:38 +08:00
@tomczhen 你这话逗乐我了,我也遇到过那种配置不起作用的项目,另外还遇到过一个抑制报错,把错误都屏蔽,最后线上跑错误日志把硬盘写满了的
ben1024
2018-06-25 15:15:44 +08:00
跟框架无关,个人水平
nbboy
2018-07-20 09:46:21 +08:00
个人喜好
astome
2018-07-27 17:31:38 +08:00
写的不错了

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/465119

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX