C++新手在练手时写的,但也是产品刚搭架子的阶段。这个函数、这个函数的调用者都是这位写的,现在看到的是重构过了的
他还搞了一堆随便返回C风格的返回值0、-1、-2、-3、-4、-5...的函数,
我说不要这么搞,错误码起码要用enum class,他说这是局部错误码,殊不知“局部错误码”在调用者很多、调用层级多了之后就不是“局部”的,而是灾难。还不如"都只返回true/false + log error reason string",因为错误码最终也是为了定位错误原因。
【 在 finlab 的大作中提到: 】
: 这个返回的字符串其实包含了跟调用者约定的协议。要改,只能跟调用者一起改。
: 这个函数的问题主要是把真正的返回值跟错误信息编码在一起了。
: 用个tuple,或者写个专门包装返回信息的类
: ...................
--
修改:z16166 FROM 111.199.147.*
FROM 111.199.147.*