V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
tangzhangming
V2EX  ›  PHP

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

  •  
  •   tangzhangming · 2018-06-22 18:48:21 +08:00 · 4064 次点击
    这是一个创建于 2105 天前的主题,其中的信息可能已经有所发展或是发生改变。

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

    avatar 一个大号 if

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

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

    这 TM 就不能

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

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

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

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

    tangzhangming
        12
    tangzhangming  
    OP
       2018-06-23 09:19:38 +08:00
    @tomczhen 你这话逗乐我了,我也遇到过那种配置不起作用的项目,另外还遇到过一个抑制报错,把错误都屏蔽,最后线上跑错误日志把硬盘写满了的
    ben1024
        13
    ben1024  
       2018-06-25 15:15:44 +08:00
    跟框架无关,个人水平
    nbboy
        14
    nbboy  
       2018-07-20 09:46:21 +08:00
    个人喜好
    astome
        15
    astome  
       2018-07-27 17:31:38 +08:00
    写的不错了
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   我们的愿景   ·   实用小工具   ·   3937 人在线   最高记录 6543   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 33ms · UTC 10:25 · PVG 18:25 · LAX 03:25 · JFK 06:25
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.