实践中, 函数/方法的参数的有效性、合法性校验应该由函数本身负责还是函数的调用者负责? 对应的异常处理呢? - V2EX
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
Sign Up Now
For Existing Member  Sign In
请不要在回答技术问题时复制粘贴 AI 生成的内容
dioxide

实践中, 函数/方法的参数的有效性、合法性校验应该由函数本身负责还是函数的调用者负责? 对应的异常处理呢?

  •  
  •   dioxide Nov 25, 2019 1993 views
    This topic created in 2347 days ago, the information mentioned may be changed or developed.

    在有严格接口格式声明的情况下.

    A 方法调用 B 方法,A 有责任对调用 B 时的传参和相应的异常做处理么? 还是说都应该有 B 负责. 这样更内聚,对 A 来说也符合最少知识原则.

    有什么说法、理论论述么? 或者实践经验.

    6 replies    2019-11-25 18:47:11 +08:00
    zhiguang
        1
    zhiguang  
       Nov 25, 2019   1
    理论上讲应该是 B 负责,实际上从头到尾就你一个人写,写哪都行
    passerbytiny
        2
    passerbytiny  
       Nov 25, 2019   1
    永远是被调用端。

    但是。

    在你的例子中,是 外部 调用 A,A 调用 B,此时对于外部来说,A 和 B 合成一体作为被调用方,A 和 B 至少要有一个做校验。至于是 A 做还是 B 做还是两个都做,取决于开发规范。

    如果不好理解的话,你这样看:假如 A 是最初的调用方,那么 A 就没有参数,压根没法做参数校验。
    eason1874
        3
    eason1874  
       Nov 25, 2019   1
    要看是内部还是外部吧,AB 都是自己组那就 A 负责,A 是别组 B 是自己那就 B 负责。
    zjp
        4
    zjp  
       Nov 25, 2019 via Android   1
    dioxide
        5
    dioxide  
    OP
       Nov 25, 2019
    有这么个事实可以确定: 若每个 B 的调用者,都预先做参数校验,那代码中无疑会存在冗余(违反 DRY 原则).
    GM
        6
    GM  
       Nov 25, 2019   1
    一般来说是函数本身负责。
    特殊情况是,该函数调用频率非常频繁,对性能要求非常高,这时候可以把职责外推给调用者(文档里写清楚)。
    About     Help     Advertise     Blog     API     FAQ     Solana     3182 Online   Highest 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 42ms UTC 13:39 PVG 21:39 LAX 06:39 JFK 09:39
    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