V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
V2EX 提问指南
1oNflow
V2EX  ›  问与答

hotfix 时如果知道问题的位置,可以直接把这部分代码注释掉吗?

  •  
  •   1oNflow · 2020-03-22 17:18:29 +08:00 · 863 次点击
    这是一个创建于 1504 天前的主题,其中的信息可能已经有所发展或是发生改变。

    hotfix 或者出 bug 的的标准操作是怎样的?

    需要 revert 最近的提交,还是直接做出新的修改,比如直接修复,或者先注释掉新的提交(不影响核心功能),后面慢慢改?

    5 条回复    2020-03-22 22:15:58 +08:00
    123444a
        1
    123444a  
       2020-03-22 18:22:07 +08:00 via Android
    注释代码能解决问题?你的意思是取消某个功能吧?一般没有人会注释代码,都是用 flag 来控制功能的启动,就是 --help 出来那些 flag
    yhxx
        2
    yhxx  
       2020-03-22 19:22:10 +08:00
    你说的是线上故障?
    标准流程应该是先回滚,然后看情况再好好修
    1oNflow
        3
    1oNflow  
    OP
       2020-03-22 21:22:51 +08:00 via iPhone
    @yhxx
    感谢回复,目前的情况是个 web 应用的前端部分( react ),有个 component 获取数据后显示的逻辑出错了,因为能定位到它,所以想直接注释掉这个 component 用户就看不见了,可以后续慢慢修。

    回滚应该更好,但是需要回滚的部分不是最新 commit 的,中间也有别人开发的部分上线,害怕回滚了某个 commit 会影响别人的代码…

    @123444a
    目前用的是 nodejs,前端也并没有设定特定功能的 flag,这种做法在 java 后端用的比较多?
    123444a
        4
    123444a  
       2020-03-22 21:25:37 +08:00 via Android
    JS 开了混淆没有,开了就注释然后加个 ToDo,没开就删代码
    1oNflow
        5
    1oNflow  
    OP
       2020-03-22 22:15:58 +08:00
    @123444a 没,就写在 jsx 里的,小公司的项目没事,主要担心大公司流程是不是不能随便删(注释)代码就来问一下
    关于   ·   帮助文档   ·   博客   ·   API   ·   FAQ   ·   我们的愿景   ·   实用小工具   ·   2457 人在线   最高记录 6543   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 26ms · UTC 09:24 · PVG 17:24 · LAX 02:24 · JFK 05:24
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.