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

关于业务一直改导致的代码整洁度问题

  •  
  •   hHarvey · 55 天前 · 2136 次点击
    这是一个创建于 55 天前的主题,其中的信息可能已经有所发展或是发生改变。

    业务需求老是改,导致代码越改越乱,甚至改一个大的版本有些实体类都用不到了,这个时候你们会去整理代码嘛 比如删掉没用的方法?删掉没用的类?

    27 回复  |  直到 2019-09-27 13:47:44 +08:00
        1
    zjsxwc   55 天前
    删掉没用的呗。
    这个和产品经理有关,
    没有逻辑关联的为什么要在一起保留,
    就和楼下店铺一样,上上个月开服装店、上个月开火锅店、 这个月开理发店、下个月要开培训班一样
        2
    cnoder   55 天前
    要是下个版本又要加回来。。。
    可以删,但没必要,一定没用的就删了呗
        3
    Vegetable   55 天前
    我一般很少删的,主要是删了也没什么意义,但是被添加的新功能搞的很难看的代码,可能会尝试重构一下.
        4
    araraloren   55 天前
    所以 你们没有版本管理么。。。?
        5
    taogen   55 天前 via Android
    删。单独 commit 删除操作,便于以后查找。
        6
    luckyrayyy   55 天前
    会,相当程度的代码洁癖,甚至有点影响开发进度....
        7
    jadec0der   55 天前
    build 的时候集成一个静态检查工具,比如 find bug, 如果有没有用到的方法或者类就会 build 失败
        8
    czzhengkw   55 天前
    单元测试,覆盖率检测,覆盖不到的代码可以考虑删
        9
    ai277014717   55 天前
    产品在试错,如果转化率降低还可能下个版本还会加回来。
        10
    MrJeff   55 天前
    遇见没用的代码 我是一定要删掉的 反正有 git 做版本记录 代码洁癖
        11
    jaryur   55 天前
    两个字:重构
        12
    soulzz   55 天前
    删掉 包括注灰的部分
    有 git 后不需要留着
        13
    MuscleOf2016   55 天前   ♥ 1
    不会写着写着对这种代码就没感情了,已经是个被别人玩烂的东西了,能跑就行。
        14
    passerbytiny   55 天前
    如果你们有完成的评审或单元测试过程,那你们的 CI、CD、质量管理员会逼着你删掉过时代码。
        15
    ETO   55 天前
    会,自己写的代码没用的会删掉,精良精减。
    不过,同事的虽然看着很难受,还是不敢动。
        16
    tinycold   55 天前 via Android
    shi 山怎么来的,现在大家都知道了
        17
    l00t   55 天前
    不会次次删,但是隔段时间会删一次。变动太频繁的时候今天删明天加,就算有版本控制也很烦;隔段时间删一删,保持代码基本可控就行了。
        18
    maxxxxx   55 天前
    强业务代码乱就乱吧,无所谓。
        19
    Takamine   55 天前 via Android
    开闭原则就是被这么蹂躏的。_(:з」∠)_
        20
    loading   55 天前 via Android
    能跑就行,因为肯定会重构(个鬼)。
        21
    gaobing   55 天前
    代码分级。核心的功能和公用的功能保持条理和干净。易变的代码能用就行。
        22
    zhuzhibin   55 天前
    哎 前段时间做了一次重构 现在项目里面的 model 有很多比较复杂的业务逻辑,也是历史原因,刚开始的时候,orm 一直爽,现在有点难受了,违背了开闭原则,现在逐步往 server 层秦迁移,重构是一个漫长的过程。。。
        23
    weizhiyao008   55 天前
    不会,不删没事,删了出了 bug,这口锅有点大
        24
    Elethom   55 天前 via iPhone
    突然感觉我们团队好良心,三天上线一个功能完整的版本,然后用了两周重构优化补全测试,几十毫秒的 shortage 都专门发 issue。
        25
    feiffy   55 天前 via Android
    这个时候可能就需要应用设计模式的一些原则封装变化,把改动范围缩到最小,这样以后删起来更方便一些
        26
    davidyanxw   55 天前
    复用,抽出公用的逻辑、类库、工具库。
        27
    GreyYang   55 天前
    如果业务已经运行起来了, 重构还是要小心, 类似给正在飞行的飞机换点零件.
    关于   ·   FAQ   ·   API   ·   我们的愿景   ·   广告投放   ·   感谢   ·   实用小工具   ·   4331 人在线   最高记录 5043   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.3 · 43ms · UTC 06:25 · PVG 14:25 · LAX 22:25 · JFK 01:25
    ♥ Do have faith in what you're doing.