This topic created in 4358 days ago, the information mentioned may be changed or developed.
最近在重构别人的代码,以前的代码类方法里面都这种:
public function method() {
if(查询出错)
return -100;
else if(查询结果为空)
return 0;
else
return 1;
}
//调用的时候判断数字。。。
我想得是,查询出错直接throw exception , 为空就是返回false, 有值就返回true.
不知道这样合理不,或者说还有更优雅的实现?
10 replies • 2014-12-03 10:24:44 +08:00
 |
|
java的话我倾向于这样,因为还是会有些时候用到查询的返回数据的 ``` public Object query() throws Exception{ if(查询出错){ throw new Exception(); } if(查询结果为空){ return null; } else{ return 查询到的数据; } }
``` 返回true,false一般用来check是否存在查询数据的,会再wrap一个函数 ``` public Boolean check() throws Exception{ return query() != null; } ```
|
 |
|
我觉得得看你的方法名字。
|
 |
|
要判断,这个错误是否会影响整体运行的结果,如果不影响,就包装一下好了,只有在可控范围内无法处理,而且严重影响其他代码运行的错误才需要throws exception。经常会有错误发生的话,C类的可以传递一个error对象进去当作输出参数,**error,调用返回后检查error是不是null。java的话就不行了,你能处理的,就尽量在函数中处理吧,要throws exception,就必须用try的方式调用,很麻烦,否则编译器报错。
|
 |
|
有值返回true的话值放在哪里……
一般而言我会用exception/array 这样,无论是否有结果都返回array,外面关心空结果的话用empty判断就好
如果是只查一条记录的,看这条记录是否“应该存在”,如果“应该存在”,那么出错和空抛不同exception,如果“不存在正常”的话,用exception/null/$data 这样
数据库倒了这么重要的事情当然要抛exception让外面处理
一般而言,除非方法本身就是判断句isXXX hasXXX,我会尽量不返回true/false
优雅不敢谈,但求不坑
|
 |
|
返回三种值的情况难道不该用三元吗 ...
|
 |
|
return result == null ? 0 : result;
|
 |
|
可以参考wp的错误信息返回设计。返回一个对象,用类似 is_error($obj) 来判断是否错误。
|
 |
|
@ mkeith 你看你看这种情况不就是为我这种三元狂魔量身定做的吗!
|
 |
|
查询出错应该抛出异常,没数据直接返回空数组,有数据就返回数据的数组
|