V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
hzw94
V2EX  ›  编程

对于这种操作,各位有什么看法?

  •  
  •   hzw94 · 2020-09-07 17:11:00 +08:00 · 2533 次点击
    这是一个创建于 702 天前的主题,其中的信息可能已经有所发展或是发生改变。

    让我的头疼一会儿~

    12 条回复    2020-09-08 21:15:47 +08:00
    goodboy95
        1
    goodboy95  
       2020-09-07 17:17:30 +08:00
    是有点头疼,以前我也这么干过,后来发现不太好调试就分行了。
    mercurylanded
        2
    mercurylanded  
       2020-09-07 17:27:21 +08:00
    不怎么看,又不是我负责的代码
    xuanbg
        3
    xuanbg  
       2020-09-07 17:31:50 +08:00
    把异常吃掉!哈哈哈,这个操作可以有。

    如果 level 是个可有可无的值,且这个参数值完全不可控的话,这样其实挺合适的。如果不是的话……造孽啊
    chendy
        4
    chendy  
       2020-09-07 18:03:48 +08:00
    判断 null 是不可能判断的,一辈子都不会的
    只能靠吃异常,才勉强会写点 crud
    kop1989
        5
    kop1989  
       2020-09-07 18:06:02 +08:00
    如果 level 有默认值,且契合业务的话,应该也是合理的吧:
    如果传过来的值不合法,就忽略,采用默认值。
    kop1989
        6
    kop1989  
       2020-09-07 18:07:36 +08:00
    所以 C#有 TryParse(string,out int) return bool 这种语法糖。
    kop1989
        7
    kop1989  
       2020-09-07 18:08:36 +08:00
    @kop1989 #5 哦,抱歉,他只捕捉了空指针,这个我服……
    xuzheliang
        8
    xuzheliang  
       2020-09-07 18:27:07 +08:00
    这就是我会写的代码...不过这样写的问题在哪?一眼不就知道是什么意思了吗?(调试不方便除外)
    ThomasZ
        9
    ThomasZ  
       2020-09-07 20:22:03 +08:00 via Android
    @xuzheliang 问题他扑获的空指针异常啊。。。。 那个 level=l 那就傻了
    yeqizhang
        10
    yeqizhang  
       2020-09-07 20:40:31 +08:00 via Android
    借楼,改成三目运算符判断空怎样?
    hzw94
        11
    hzw94  
    OP
       2020-09-08 10:45:21 +08:00
    ---

    图上的`try...catch`的根本原因是为了防止前端未提交`level`参数,导致后端无法获取对象,进而在执行`trim()`语法时抛出`null 异常`。

    问题出在前端而不是后端,要保证前端参数正确,前端的锅不要丢给后端。

    开发者意识到了问题,处理方法却是错误的。

    ---

    我最想讨论的是,`try...catch`的正确用法是什么?

    我懂的也不多,但至少有一点要遵循,任何情况下`catch`中要将异常打印到日志里面,方便开发者调试,提醒接盘侠有异常。

    如果没有日志,在数百数千行代码定位一个隐形 bug,要命的!
    xuzheliang
        12
    xuzheliang  
       2020-09-08 21:15:47 +08:00
    @ThomasZ 没想到问题在这...受教了
    关于   ·   帮助文档   ·   API   ·   FAQ   ·   我们的愿景   ·   广告投放   ·   感谢   ·   实用小工具   ·   4200 人在线   最高记录 5497   ·     Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 · 26ms · UTC 05:52 · PVG 13:52 · LAX 22:52 · JFK 01:52
    Developed with CodeLauncher
    ♥ Do have faith in what you're doing.