公司 Sonar 规则强制要求所有 select 语句添加 limit 字段 - V2EX
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
请不要在回答技术问题时复制粘贴 AI 生成的内容
Waverly
V2EX    程序员

公司 Sonar 规则强制要求所有 select 语句添加 limit 字段

  •  
  •   Waverly 2024-05-29 20:19:41 +08:00 2703 次点击
    这是一个创建于 529 天前的主题,其中的信息可能已经有所发展或是发生改变。

    公司 sonar 规则中强制 select 语句需要加 limit ;

    但有些情况下,已经使用了 PageMethod ,在运行时会根据 pageSize 和 pageNum 使用拦截器自动拼接 limit ,这些场景下就不能手动添加 limit ;

    ps:公司不能用 mybatis plus ,但能用 mybatis

    求一个方案,又能使用 PageMethod ,又能过 Sonar 扫描

    6 条回复    2024-05-30 15:40:07 +08:00
    Lucups
        1
    Lucups  
       2024-05-29 20:51:55 +08:00
    骚操作:为使用 PageMethod 的 SQL 手动添加 limit 123 ,然后写个优先级高于 PageMethod 的拦截器把 limit 123 过滤掉。
    9dP06m83vIV00l72
        2
    9dP06m83vIV00l72  
       2024-05-29 20:52:57 +08:00
    积极沟通,酌情添加;
    orioleq
        3
    orioleq  
       2024-05-30 01:30:54 +08:00 via iPhone
    1.把你觉得不合理的 sonar 规则去掉;
    2.有个注释叫 //NOSONAR 。
    yc8332
        4
    yc8332  
       2024-05-30 08:48:22 +08:00
    正常。以前的公司 mysql 不加 where limit 根本不让执行。。就怕有愣头青给你 select * from table , 大表数据库直接挂
    e3c78a97e0f8
        5
    e3c78a97e0f8  
       2024-05-30 12:24:28 +08:00
    limit 2147483647
    Waverly
        6
    Waverly  
    OP
       2024-05-30 15:40:07 +08:00
    @Lucups 是个方案,可以讨论一下可行性;
    @andytao 规则一刀切,没办法;
    @orioleq 规则没法改变,上面规定的;//NOSONAR 只是个临时方案,目前也有在用,但期望是找到一个通用的解决方案;
    @yc8332 这个也能理解,但是代码规范已经规定必须分页查询的情况下再使用这个规则多少有点不合理;
    关于     帮助文档     自助推广系统     博客     API     FAQ     Solana     4639 人在线   最高记录 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 32ms UTC 09:59 PVG 17:59 LAX 01:59 JFK 04:59
    Do have faith in what you're doing.
    ubao msn snddm index pchome yahoo rakuten mypaper meadowduck bidyahoo youbao zxmzxm asda bnvcg cvbfg dfscv mmhjk xxddc yybgb zznbn ccubao uaitu acv GXCV ET GDG YH FG BCVB FJFH CBRE CBC GDG ET54 WRWR RWER WREW WRWER RWER SDG EW SF DSFSF fbbs ubao fhd dfg ewr dg df ewwr ewwr et ruyut utut dfg fgd gdfgt etg dfgt dfgd ert4 gd fgg wr 235 wer3 we vsdf sdf gdf ert xcv sdf rwer hfd dfg cvb rwf afb dfh jgh bmn lgh rty gfds cxv xcv xcs vdas fdf fgd cv sdf tert sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf sdf shasha9178 shasha9178 shasha9178 shasha9178 shasha9178 liflif2 liflif2 liflif2 liflif2 liflif2 liblib3 liblib3 liblib3 liblib3 liblib3 zhazha444 zhazha444 zhazha444 zhazha444 zhazha444 dende5 dende denden denden2 denden21 fenfen9 fenf619 fen619 fenfe9 fe619 sdf sdf sdf sdf sdf zhazh90 zhazh0 zhaa50 zha90 zh590 zho zhoz zhozh zhozho zhozho2 lislis lls95 lili95 lils5 liss9 sdf0ty987 sdft876 sdft9876 sdf09876 sd0t9876 sdf0ty98 sdf0976 sdf0ty986 sdf0ty96 sdf0t76 sdf0876 df0ty98 sf0t876 sd0ty76 sdy76 sdf76 sdf0t76 sdf0ty9 sdf0ty98 sdf0ty987 sdf0ty98 sdf6676 sdf876 sd876 sd876 sdf6 sdf6 sdf9876 sdf0t sdf06 sdf0ty9776 sdf0ty9776 sdf0ty76 sdf8876 sdf0t sd6 sdf06 s688876 sd688 sdf86