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

如何快速提升已有屎山稳定性?业务逻辑非常复杂,重构不现实

  •  
  •   edis0n0 · 87 天前 · 3072 次点击
    这是一个创建于 87 天前的主题,其中的信息可能已经有所发展或是发生改变。
    公司一套十几年前买的外包写的外贸商城系统,3 个程序员修修补补用了十几年,什么日志都没有,变量名乱起,函数名和作用不一致,平均每月炸两次,一大半能重启解决(莫名其妙数据库连不上,日志里没有任何错误,重启服务没用、nginx 莫名其妙写了 1.6TB 稀疏文件把固态硬盘写满了,重启就会消失 这类完全不知道怎么排查的故障),关键还经常半夜炸,然后时差原因半夜还是客户最多的时候。

    不想润,因为系统没炸的时候非常闲,基本都在摸鱼,每周只要工作半小时那种
    37 条回复    2023-03-05 12:34:33 +08:00
    imdong
        1
    imdong  
       87 天前 via iPhone
    定时器,比如定时重启,
    edis0n0
        2
    edis0n0  
    OP
       87 天前
    @imdong #1 目前就是这样干的,检查到掉线就重启,但还有 40%左右重启不能解决
    edis0n0
        3
    edis0n0  
    OP
       87 天前
    单函数名和作用不一致这一个就被坑过好几次,例如 /t/899919 在发邮件通知的 PHP 文件里扣手续费导致账对不上,一晚上找不到问题出在哪
    edis0n0
        4
    edis0n0  
    OP
       87 天前
    部分模块是 ASP.NET 的,部分模块是 PHP 的系统,还有部分新的模块是 ASP.NET Core 的
    opengps
        5
    opengps  
       87 天前
    重启能解决,大概率是资源没及时释放,内存泄漏句柄泄漏等问题
    v2defe
        6
    v2defe  
       87 天前 via Android   ❤️ 4
    系统稳定了你不就不稳定了🙂
    edis0n0
        7
    edis0n0  
    OP
       87 天前
    @v2defe #6 炸得这么频繁我也不稳定🙂
    aureole999
        8
    aureole999  
       87 天前   ❤️ 1
    快速的办法:把所有不稳定的功能全删了就 ok 了。
    开个玩笑。既然都叫屎山了怎么会那么容易就让你改好,这东西只能一点一点改。
    比如先加上日志,这不算重构吧应该。然后改一些方法内局部变量名,再到改全局变量名,改方法名,加注释。改完名加完注释应该就通读了代码并有一定的理解了,加测试用例,自动回归测试。然后改小的模块,最后改大模块。别想一口吃个胖子,反正除了炸的时候都很清闲,那就慢慢改呗。
    fisherwei
        9
    fisherwei  
       87 天前
    首先,nginx 的那个问题应该好解决吧,你们这个系统是有大量上传文件的场景吗?
    edis0n0
        10
    edis0n0  
    OP
       87 天前
    @fisherwei #9 都是一些图片上传的场景,仓库出库图片、客户返图一类,十几年数据量 300GB ,nginx 请求 body 最大是 2MB 。
    night98
        11
    night98  
       87 天前
    简单点,上监控,各种监控搞起来,有了监控就能发现大部分主要问题,然后解决掉。

    次一些的,把核心链路拆出来重写,其他非核心链路的上监控慢慢调
    Chad0000
        12
    Chad0000  
       86 天前 via iPhone
    先上日志系统
    然后业务代码迁移到.net standard 项目以做好迁移到 core 的准备
    netnr
        13
    netnr  
       86 天前 via Android   ❤️ 2
    长时间稳定运行,会增加你 OUT 的风险
    azusematsuri
        14
    azusematsuri  
       86 天前 via Android
    按功能后端,一部分一部分重构到 go
    searene
        15
    searene  
       86 天前
    有可能是内存泄漏,建议加监控、日志
    Chad0000
        16
    Chad0000  
       86 天前 via iPhone
    @azusematsuri
    迁移到 go ?写惯了 C#就很难(不想)迁移到其他语言了。
    litchinn
        17
    litchinn  
       86 天前
    “系统没炸的时候非常闲”,万一搞好了,它不炸了,领导觉得不需要你了咋办 doge
    forgottencoast
        18
    forgottencoast  
       86 天前
    平时很闲,就平时慢慢改啊。。。。
    pengtdyd
        19
    pengtdyd  
       86 天前
    《计算机科学领域的任何问题都可以通过增加一个间接的中间层来解决》
    yjd
        20
    yjd  
       86 天前
    连半小时都省了,是不是离离职不远了^_^
    lingex
        21
    lingex  
       86 天前
    看着有点兴奋哈,每解决一个都是很有成就感的。

    先加日志吧,出问题的操作多加点,如有可能每个跳转都加上。
    jaggle
        22
    jaggle  
       86 天前 via iPhone
    好奇问一下楼主你们的日订单数量多少多少,能重启而没用户不抱怨吗
    centralpark
        23
    centralpark  
       86 天前
    加日志,加单元测试,加监控。切割好了才好改。当然,就像楼上说的,这一切都不要让你领导知道,不然你的地位危矣
    newskillsget
        24
    newskillsget  
       86 天前
    @v2defe 哈哈哈哈哈你是懂稳定的
    vacker
        25
    vacker  
       86 天前 via iPhone
    你是老板的话你可以直接重构,否则 保持现状修修补补就好
    charlie21
        26
    charlie21  
       86 天前 via Android
    jQuery 时代的吗?
    falcon05
        27
    falcon05  
       86 天前
    我现在也有一套这种的,非常恐怖,nginx 的配置太蛋疼了,换成 openresy ,准备学点 lua ,相当加了一层。
    enchilada2020
        28
    enchilada2020  
       86 天前 via Android
    @pengtdyd 求出处
    liangxin1998
        29
    liangxin1998  
       86 天前
    git 有一项规则,就是重构
    fds
        30
    fds  
       86 天前
    先给主要逻辑加上些测试?
    数据库连接断开可以加些 log 吧,还有压测或者手动 kill 连接试试。
    pengtdyd
        31
    pengtdyd  
       86 天前
    @centralpark 《“Any problem in computer science can be solved by another layer of indirection.”》
    billytom
        32
    billytom  
       86 天前
    @edis0n0 首先,你这是内存无法回收造成的 boom ,那就要定时重启,而不是出问题连不上再重启。把故障率控制到从 40% -> 20% 就够了,不要去碰程序主体(防止手贱问题越来越大),之后好好公司呆着就行。

    最后,偶尔半夜 boom 是好事,要让老板知道你在辛勤工作,这工资给得值
    zhengfan2016
        33
    zhengfan2016  
       86 天前
    楼上说的很对,维持现状修修补补就好了,

    重构好了,没有功劳,一旦经济下行,可能第一个毕业的是你,反正炒了系统也很稳定不会 boom

    重构不好,锅全是你的,该卷铺盖被炒的,还是你,搞不好让公司蒙受损失,毕业以后还得吃官司😂
    mafanding
        34
    mafanding  
       86 天前 via iPhone
    弄一个定时重启+一个磁盘使用量预警 足够了 总要有点工作量 不然你干嘛等老板来开你么
    em0miao0
        35
    em0miao0  
       86 天前
    @netnr 很有道理,我就是被这么干掉的,系统弄得太好,一度没有啥新需求,导致系统长达 4 ,5 个月没啥问题不用重启升级啥的然后被干掉了;所以。。OP 还是慢慢改吧。
    MrHyde
        36
    MrHyde  
       85 天前
    @billytom 32 楼有道理,你就主动重启,在白天,人少的时候
    esile
        37
    esile  
       85 天前
    如果你修好了就证明你随时可以被优化掉。
    关于   ·   帮助文档   ·   博客   ·   nftychat   ·   API   ·   FAQ   ·   我们的愿景   ·   实用小工具   ·   1169 人在线   最高记录 5634   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 45ms · UTC 17:49 · PVG 01:49 · LAX 10:49 · JFK 13:49
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.