日期:2014-05-20  浏览次数:20899 次

新手~~关于异常的使用问题,谢谢!
今天写程序,用了一个简单工厂模式,工厂类有一个静态工厂方法,传入一个参数,根据参数生成不同的对象: 


代码
Java code
public static GateWayCaller getGateWayCaller(String serviceKey)   
{   
    if("alerting".equals(serviceKey))   
    {   
        return new AlertingCaller();   
    }   
    else if("report".equals(serviceKey))   
    {   
        return new ReportCaller();   
    }   
    //...   
    else  
    {   
        return null;   
    }   
}   



一同事硬说我return null不对,应该抛出一个checked异常,由客户端调用者去处理该异常。

我的想法很简单,既然这是一个生成对象的方法,如果调用者将参数传错导致要生成的对象不存在,就应该返回一个空对象,而不是抛出异常。不然的话所有客户端代码都要处理一个没用的异常!!

搞的好郁闷,最受不了的是老说我什么什么观点是错误的。俺也没有什么太多的编程经验,也不知道该怎么去反驳。我认为没有哪个观点是绝对错误的,只有比较之下哪种做法更好一点?听听大家的意见?


------解决方案--------------------
支持你。
这个时候不应该抛出异常。
------解决方案--------------------
你的同事说的只能说是一种解决方案,但并不是说你现在这个就不对。至于抛不抛异常,完全应该有应用的需要来决定。如果按你的写法,你的javadoc里应该注明,在什么情况下,此方法返回null值,这是个好习惯。

题外话:如果是sting值映射到一个实例的情况,建议用个map来做,不要用if,一是代码难看,二是维护成本增加。如果你的那几个Caller都是单例的,那map里直接放个实例就行;如果不能是单例的,那就放个Class对象,然后用调用其newInstance()方法;如果需要特殊的构造,可以写个接口来专门做构造。
------解决方案--------------------
你同事的观点是对的,应该抛出异常。

你返回null的话,看似简单,其实会让别人很头痛。这是因为:

1. 他一直要到用到这个返回对象的时候才知道他得到的是一个null(通过运行时的NullPointerException得知)。
2. 他只知道他得到了一个null,却不知道实际原因是什么,实际上的原因是他传入了不被接受的参数,但返回null并没有告诉他这一点。

所以,建议你的代码写成这样:

Java code

    if("alerting".equals(serviceKey))   
        return new AlertingCaller();  
    else if("report".equals(serviceKey))
        return new ReportCaller();
    //...   
    else  
        throw new RuntimeException("Invalid Service Key!");