跳到内容

中午好,记得休息片刻,补充能量。

  • 5 回复
  • 345 浏览

SonarQube 扫描越改越乱

参与讨论

搞了三天 SonarQube,修了一堆“bug”,结果 CI 还是红。有些规则就是教条,你不改它说你代码烂,改了反而逻辑炸了。这玩意到底有没有坑,还是我配置没弄对?

讨论区

按楼层回复,支持引用与表情;使用 @用户名 或 @昵称 可提醒对方;发言计入圈子贡献字数。

5s 条评论

  1. 运维祭天
    运维祭天 成长5,225

    别外传哈,我上次也被这玩意坑过。@路过点个赞 默认规则别全信,先对着日志筛一遍,有些就是为代码洁癖设计的。调warning比直接关靠谱,不然真漏了关键bug更蛋疼。

  2. 运维祭天
    运维祭天 成长5,225

    这玩意儿默认规则确实挺教条的,我项目里之前被强制要求所有魔法数字都得抽常量,结果改完一堆配置类反而更难读了。@路过点个赞 建议你先把那些和业务逻辑强相关的规则调成warning级别,别一刀切关掉,不然关键问题漏了就真炸了。

  3. 咖啡续命中
    咖啡续命中 萌芽1,007

    这玩意确实坑,有些规则就是教条。我上次也踩过,后来把不合理的规则直接关了,CI就过了。@路过点个赞 建议你先把默认规则砍一半再跑。

  4. 收藏=学会
    收藏=学会 成长4,514
    引用 咖啡续命中

    这玩意确实坑,有些规则就是教条。我上次也踩过,后来把不合理的规则直接关了,CI就过了。@路过点个赞 建议你先把默…

    @咖啡续命中 关规则确实能过,但有些规则其实是逻辑问题,你关了不排查后面还会炸。你当时关了哪些?有清单么

  5. 剪到凌晨四点
    剪到凌晨四点 成长3,175
    引用 咖啡续命中

    这玩意确实坑,有些规则就是教条。我上次也踩过,后来把不合理的规则直接关了,CI就过了。@路过点个赞 建议你先把默…

    @咖啡续命中 关规则确实能过,但有些规则关了排查不彻底,后面容易炸。我建议你@收藏=学会 先看日志再开白名单,别一刀切。

发表评论

登录后发表评论

登录即可参与楼层讨论,支持引用回复与 @ 提醒。

前往登录页

Welcome! This site is in Chinese. Tap EN in the top bar to read in English.