跳到内容

夜深了,注意休息,愿你今夜好梦。

  • 11 回复
  • 365 浏览

SonarQube 扫描越改越乱

参与讨论

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

讨论区

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

11s 条评论

  1. 运维祭天
    运维祭天 绽放14.1k

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

  2. 不测不信邪
    不测不信邪 绽放15.7k
    引用 运维祭天

    别外传哈,我上次也被这玩意坑过。@路过点个赞 默认规则别全信,先对着日志筛一遍,有些就是为代码洁癖设计的。调wa…

    @运维祭天 调warning是比直接关靠谱点。但我还是觉得这玩意儿就是给代码洁癖用的,业务跑得快比代码干净重要多了。有人遇到过改完规则反而多出bug的没?

  3. 摸鱼不划水
    摸鱼不划水 成长5,220
    引用 不测不信邪

    笑死,SonarQube 这玩意就是给代码洁癖准备的,业务逻辑改成一坨屎还过不了CI。@不测不信邪 我这边改完规则直接多出两个空指针,还不如不改。你后来咋处理的?

  4. 法务别找我
    法务别找我 绽放14.4k
    引用 运维祭天

    别外传哈,我上次也被这玩意坑过。@路过点个赞 默认规则别全信,先对着日志筛一遍,有些就是为代码洁癖设计的。调wa…

    @运维祭天 调warning确实比全关强,但有些规则开了反而改出一堆新bug。这玩意是不是得把团队代码风格统一了再跑?

  5. 运维祭天
    运维祭天 绽放14.1k

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

  6. 咖啡续命中
    咖啡续命中 成长10.6k

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

  7. 收藏=学会
    收藏=学会 成长11.6k
    引用 咖啡续命中

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

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

  8. 前端背锅侠
    前端背锅侠 成长6,961
    引用 收藏=学会

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

    @收藏=学会 我这边SonarQube 9.9,默认规则关了三分之一才过CI。有清单,回头发你。但有些规则其实是代码逻辑问题,关了后面还是得修,样本量够吗?

  9. 剪到凌晨四点
    剪到凌晨四点 成长4,580
    引用 咖啡续命中

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

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

  10. 平替猎人
    平替猎人 成长10.5k
    引用 咖啡续命中

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

    @咖啡续命中 关规则是治标不治本。我这项目之前被规则逼着把所有int改成final static常量,结果业务逻辑全变成配置地狱,改完反而多了一堆bug。你当时关的规则有具体清单吗?我想看看是不是同款。

  11. 深夜改BUG
    深夜改BUG 绽放14.4k
    引用 平替猎人

    SonarQube 有些规则就是给代码洁癖用的,业务逻辑一复杂反而被它带沟里。@平替猎人 你那个int改final static我见过,改完配置类直接变天书。调warning比直接关好点,但别指望它能查真bug。

发表评论

登录后发表评论

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

前往登录页

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