对于 PHP 老项目续命的思考 - V2EX
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
请不要在回答技术问题时复制粘贴 AI 生成的内容
zjsxwc
V2EX    程序员

对于 PHP 老项目续命的思考

  •  
  •   zjsxwc
    zjsxwc 2017-11-13 15:57:47 +08:00 3733 次点击
    这是一个创建于 2891 天前的主题,其中的信息可能已经有所发展或是发生改变。

    比如 ecshop ( https://github.com/shopex/ecshop), 这种 每个 php 文件直接对应一个 url 地址, 依赖完全靠魔法全局变量传递, 不使用面向对象方式组织代码

    这种老项目的特点是大部分时候都只是做 sql 手动拼接与简单 html 渲染; 代码里变量以 associative array 为主; 业务处理会混淆在 sql 与 php 代码之间; 规模一般不会太大, 一旦大了依赖复杂就玩崩, 最后难以维护, 所有续命必须要做的第一步就是把依赖关系理清, 然后才能使用我们熟悉的依赖注入容器等手段.

    如何续命:

    1. 原有的老代码保持原样, 不需要去改动

    2. 从数据库表生成领域实体面向对象的 PHP 文件(比如用 symfony 的 doctrine orm 相关命令工具)

    3. 业务分为老业务与新业务. 由于老业务代码不能改动, 我们需要通过写个代码转换器, 把老代码的每个 PHP 文件, 转换到新的面向对象的 PHP 类里面, 这样新老业务都是基于面向对象了. 这个代码转换器, 关键是要能够识别出老代码里使用的魔法全局变量, 把魔法全局变量写到生成类的构造函数里, 并替换老代码的魔法全局变量, 为类的属性, 以及把老代码里 function 改为类的 method, 实现把依赖都抽为类之间的依赖

    4. 之后新老业务就可以通过依赖注入容器来相互调用了.

    16 条回复    2017-11-14 12:39:03 +08:00
    sunsh2017
        1
    sunsh2017  
       2017-11-13 16:02:08 +08:00 via Android
    什么狗屁东西
    ORZRRR
        2
    ORZRRR  
       2017-11-13 16:28:04 +08:00
    @sunsh2017 楼主教你怎么续命呢
    sunding0308
        3
    sunding0308  
       2017-11-13 17:32:07 +08:00
    @sunsh2017 楼主教你怎么续命呢
    Moker
        4
    Moker  
       2017-11-13 17:34:23 +08:00
    现在就在维护类似的项目 太惨了
    chairuosen
        5
    chairuosen  
       2017-11-13 17:38:39 +08:00
    要插一个第 0 步,写测试
    picone
        6
    picone  
       2017-11-13 17:41:51 +08:00
    改不动的,ecshop 2.7 一堆旧的写法,用 php5.6 跑一堆 notice。
    除此之外还有页面渲染的链接等等,太多要改了,没必要续命。
    sytnishizuiai
        7
    sytnishizuiai  
       2017-11-13 17:59:25 +08:00
    牛,当初下个官网的 ecshop,全是 notice,改了我好久才能用,而且后续还会是不是出现
    mlkr
        8
    mlkr  
       2017-11-13 18:19:50 +08:00
    还不如重新来过
    pqee
        9
    pqee  
       2017-11-13 18:45:55 +08:00
    5. 离职

    我说真的。
    tongz
        10
    tongz  
       2017-11-13 19:29:04 +08:00 via Android
    续命?魔法?
    akira
        11
    akira  
       2017-11-13 20:18:52 +08:00
    基本上整体重做会更快
    askfilm
        12
    askfilm  
       2017-11-13 20:38:35 +08:00
    赞一个,虽然说的太笼统
    litter123
        13
    litter123  
       2017-11-13 20:54:30 +08:00
    @tongz 不不不,是膜法
    lsido
        14
    lsido  
       2017-11-14 02:06:20 +08:00 via iPhone
    掐死重生吧
    qianmeng
        15
    qianmeng  
       2017-11-14 07:33:24 +08:00 via Android
    想办法一点一点改造然后替换吧,否则很痛苦的
    onion83
        16
    onion83  
       2017-11-14 12:39:03 +08:00
    了解业务流程,搞清楚数据库结构和存储逻辑,兼容 URL,慢慢做切换流量。

    老代码通常有历史包袱,跟着旧的思路很容易陷进去。
    关于     帮助文档     自助推广系统     博客     API     FAQ     Solana     1281 人在线   最高记录 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 26ms UTC 23:44 PVG 07:44 LAX 16:44 JFK 19:44
    Do have faith in what you're doing.
    ubao snddm index pchome yahoo rakuten mypaper meadowduck bidyahoo youbao zxmzxm asda bnvcg cvbfg dfscv mmhjk xxddc yybgb zznbn ccubao uaitu acv GXCV ET GDG YH FG BCVB FJFH CBRE CBC GDG ET54 WRWR RWER WREW WRWER RWER SDG EW SF DSFSF fbbs ubao fhd dfg ewr dg df ewwr ewwr et ruyut utut dfg fgd gdfgt etg dfgt dfgd ert4 gd fgg wr 235 wer3 we vsdf sdf gdf ert xcv sdf rwer hfd dfg cvb rwf afb dfh jgh bmn lgh rty gfds cxv xcv xcs vdas fdf fgd cv sdf tert sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf shasha9178 shasha9178 shasha9178 shasha9178 shasha9178 liflif2 liflif2 liflif2 liflif2 liflif2 liblib3 liblib3 liblib3 liblib3 liblib3 zhazha444 zhazha444 zhazha444 zhazha444 zhazha444 dende5 dende denden denden2 denden21 fenfen9 fenf619 fen619 fenfe9 fe619 sdf sdf sdf sdf sdf zhazh90 zhazh0 zhaa50 zha90 zh590 zho zhoz zhozh zhozho zhozho2 lislis lls95 lili95 lils5 liss9 sdf0ty987 sdft876 sdft9876 sdf09876 sd0t9876 sdf0ty98 sdf0976 sdf0ty986 sdf0ty96 sdf0t76 sdf0876 df0ty98 sf0t876 sd0ty76 sdy76 sdf76 sdf0t76 sdf0ty9 sdf0ty98 sdf0ty987 sdf0ty98 sdf6676 sdf876 sd876 sd876 sdf6 sdf6 sdf9876 sdf0t sdf06 sdf0ty9776 sdf0ty9776 sdf0ty76 sdf8876 sdf0t sd6 sdf06 s688876 sd688 sdf86