LeanCloud 的 PHP SDK 中关于异常的处理是否不太合理?以及用户退出却不生效? - V2EX
V2EX = way to explore
V2EX 是一个关于分享和探索的地方
现在注册
已注册用户请  登录
工单节点使用指南
请用平和的语言准确描述你所遇到的问题
厂商的技术支持和你一样也是有喜怒哀乐的普通人类,尊重是相互的
如果是关于 V2EX 本身的问题反馈,请使用 反馈 节点
GDC
V2EX    全球工单系统

LeanCloud 的 PHP SDK 中关于异常的处理是否不太合理?以及用户退出却不生效?

  •  1
     
  •   GDC 这是一个创建于 2059 天前的主题,其中的信息可能已经有所发展或是发生改变。

    这两天准备接入 LeanCloud,使用官方的 PHP SDK 时发现了一个不太合理的地方。

    比如注册用户的时候,随便产生一个错误,假设用户名重复了,$user->signUp() 会抛出异常, 但 $th->getMessage() 得到的错误信息却不是纯粹的文字提示, 而是包含了 Request 的信息,例如这样:

    218 Invalid password, it must be a non-blank string.:{\"body\":{\"username\":\"spark\",\"password\":\"\"},\"method\":\"POST\",\"path\":\"\\/1.1\\/classes\\/_User\"}","code":218} 

    这样带来一个困扰就是,无法直接把错误信息反馈给前端、给用户看,只能通过 $th->getCode() 得到错误码之后自己去官方文档找对应的文字说明,再对应起来反馈回去。

    翻了 SDK 的代码后,最终在 BatchRequestError.php 文件中找到了原因:

    public function add($request, $response) { $error["code"] = isset($response["code"]) ? $response["code"] : 1; $error["error"] = "{$error['code']} {$response['error']}:" . json_encode($request); $this->errors[] = $error; return $this; } 

    可以看到在这里 $error["error"] 就已经被“合体”了。

    不知道有没有使用 LeanCloud 的同学,或者 LeanCloud 的人来说说,这个是 feature 还是 bug,目前已知错误码有 160 多个,难道要开发人员自己一个个做对应?


    另外还发现一个疑似 bug 的,User::logOut() 之后,下次请求使用相同的凭据 User::getCurrentUser() 还是能获取到当前用户,蛋疼。

    2 条回复    2020-02-25 16:43:59 +08:00
    FaceBug
        1
    FaceBug  
       2020-02-25 16:33:05 +08:00
    不能算 bug 吧,抛出错误的时候,把原始的请求也抛出是很常见的,不过一般我都是分两个字段的,msg 是 msg,request 是 request。

    就这个问题,你其实可以用那个冒号作为分隔符,前面的是你要展示给用户的信息。

    另外,就算单独给你 msg ( error )信息,我们通常也不会直接把这种英文的错误展示给终端用户吧(除非是内部用的系统,给程序员、运维看的),通常还是会通过 code 映射一个“人性化”的错误提示
    GDC
        2
    GDC  
    OP
       2020-02-25 16:43:59 +08:00
    @cepczkd 我认为分两个字段是比较合理的,另外在开发的网站本身就是英文的…
    关于     帮助文档     自助推广系统     博客     API     FAQ     Solana     5278 人在线   最高记录 6679       Select Language
    创意工作者们的社区
    World is powered by solitude
    VERSION: 3.9.8.5 40ms UTC 07:09 PVG 15:09 LAX 00:09 JFK 03:09
    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