`
hite
  • 浏览: 50628 次
  • 性别: Icon_minigender_1
  • 来自: 杭州
文章分类
社区版块
存档分类
最新评论

异常捕获的切面--java需要优雅吗?

阅读更多
我是je的新人,大学刚毕业,入司后就一直在看别人的代码。
今天看到前人的两行注释让我思考——
……也许我所述不是问题for YOU,但是我迷茫
    public synchronized void putInCache(String cacheName, String key, Object value){
        CacheExt cache= getCache(cacheName);
//        if (cache == null) {
//            throw new NeedsRefreshException("Cache is not exist");
//        }
        cache.putInCache(key, value);
    }

为什么要注释掉呢?可能是作者认为调用它的代码过滤了cache为空的可能,例如:
   
   public synchronized boolean hasCache(String cacheName) {
        return cacheMap.getKeys().contains(cacheName);
    } 
……
    if(hasCache(cacheName)) { putInCache(……);}
……

但下面:
    public synchronized Object getFromCache(String cacheName, String key){
        CacheExt cache = getCache(cacheName);
        if(cache == null)
            return null;
        try {
            return cache.getFromCache(key);
        } catch (NeedsRefreshException e) {
            cache.cancelUpdate(key);
            return null;
        }
    }

作者又在方法内部过滤这种异常,是作者的逻辑有问题吗?
当面对外部对本方法调用有多种可能性,而每种可能性的处理又个不相同时,就出现这样的问题:
本方法不能确定外部调用是否已经过滤了异常,而自己又很迫切的需要保证这一点时,你就得在自己内部过滤掉可能的异常。但是对于许多人来说:注意防止异常的发生是基本原则(至少老师是这么教我的——),所以对于一个充分解耦的程序来说多级的调用中,或者是分组开发里,出现这样的调用A-调用->B-调用->C-调用->D-调用->E-调用->F-调用->G的调用栈,G是最终的方法即本方法,内部有过滤,同样的A-调用->B有过滤,其他一样,既
if (cache == null) {
            throw new NeedsRefreshException("Cache is not exist");
        }

出现多次,判断N次(影响效率!?)。这样我看起来很不爽,这么麻烦!一点都不简约,不朴素。
当然,也可以不处理这样就简约了,朴素了-
待续……去看奥运会开幕式——

接续:
也就是说在类的外部和内部都不处理这种异常(有这样写代码的吗?),假设程序中不会出现异常;或者捕获这种异常的代价远远大于了重启的代价——

当然,这里依然还有一种方法,也就是整个系统的通用异常处理框架或机制。如此的话,在函数方法的内部和外部(不包含最外部)不用理睬过滤的事情了……

总结:当我们不确定异常发生的时机、异常处理的种类(也许有些异常出现后,可以使用默认参数代替,有些则报错崩溃)、异常处理代价时,就出现了异常处理的切面问题,也就是说在哪里会出现异常、哪里需要处理异常、如果处理异常的问题!
   我认为有4种模式:

  • 原始社会模式——个体与个体之间社交的匮乏,可以忽略任何一个人的错误(对应不处理方式)

  • 封建社会模式——个体在不经意见的错误会有官府等机制来纠正(对应方法外部过滤)

  • 资本主义模式——当每个个体的意识和法制强度达到一定程度时,个体对外界是干净的(对应内部过滤)

  • 社会主义模式——社会主义的每个公民都是优秀的,既是偶尔犯错,也会有社会主义的政府来处理(对应统一框架或机制处理)



如果是这样的话,我想Java代码在我们面前突然就变得优雅起来。
但在日益复杂的系统里,这可能实现吗?
分享到:
评论
11 楼 fjlyxx 2008-12-10  
异常的抛出其实和你业务框架有关系。没有一定的规则。如果你的异常没有必要要上级业务知道那你完全可以处理掉这个异常,但是如果你的异常会影响上级的处理那么你就得通知它。
10 楼 hite 2008-12-10  
不是说统一处理
fjlyxx 写道

huangking 写道我也支持bloodrate兄的观点...
DAO/Service/Controller都应该有各自不同的异常抛出..

能处理则处理,不能处理则抛出,这里抛出不一定直接throw 当前捕获的异常,可以抛出自定义的异常...当然顶层需要一个统一的处理,处理那些没有被处理掉的异常

不是说统一处理,而是你要让你的调用者知道你出异常了。你可以发出一个通告给你的调用者你没有按照预定的规则进行处理。 特别是在多线程中如果你不通告回掉业务的线程业务处理出问题了那么这条线程就可能永远挂起了。

是通告不是一定要统一处理。

通知机制,是不是说还要额外的代码来实现和管理呢?那通知的内容呢,包括类型等待吗,else还有?

我还有个问题:
当外侧调用者需要知道被调用的函数具体是什么异常,但是内部的异常是通过finally抛出的模糊异常,或者是某种类型IOexception的大异常,咋办?
9 楼 fjlyxx 2008-12-08  
huangking 写道
我也支持bloodrate兄的观点...
DAO/Service/Controller都应该有各自不同的异常抛出..

能处理则处理,不能处理则抛出,这里抛出不一定直接throw 当前捕获的异常,可以抛出自定义的异常...当然顶层需要一个统一的处理,处理那些没有被处理掉的异常


不是说统一处理,而是你要让你的调用者知道你出异常了。你可以发出一个通告给你的调用者你没有按照预定的规则进行处理。 特别是在多线程中如果你不通告回掉业务的线程业务处理出问题了那么这条线程就可能永远挂起了。

是通告不是一定要统一处理。
8 楼 huangking 2008-12-08  
我也支持bloodrate兄的观点...
DAO/Service/Controller都应该有各自不同的异常抛出..

能处理则处理,不能处理则抛出,这里抛出不一定直接throw 当前捕获的异常,可以抛出自定义的异常...当然顶层需要一个统一的处理,处理那些没有被处理掉的异常
7 楼 bloodrate 2008-12-08  
我认为异常不适于在一个统一的层面统一处理,由于系统是分层结构的,每一层都有各自的异常,异常属于这个层次的实现细节之一,一个层次不应该去处理其他层次的异常,比如你在写用户管理的Controller,调用UserServices,捕捉到NoUser异常,跳转到没有用户的错误页面,捕捉到ErrPassword跳转道密码错误页面,然而你发现还要处理FileNotFoundException,一下就蒙了,这个异常该怎么恢复?这说明在写UserServices没有把FileNotFoundException隐藏好...
6 楼 hite 2008-12-08  
请注意你上面的代码的方法定义
fjlyxx 写道

这不是优雅的问题,是你对程序结构理解的问题,一个没有好的异常体系的系统不是一个优秀的系统只是我这么觉得. 异常反复抓取是异常框架和程序员的问题.请注意你上面的代码的方法定义,方法并不会抛异常所以他多判断一次多抛出一次,这个就是开发人员自身的问题了.(对于可测异常)

我也觉得和系统的框架有关系,但是有些人这么写,有些人那么写,我难以确定那种更好些。
5 楼 fjlyxx 2008-12-02  
这不是优雅的问题,是你对程序结构理解的问题,一个没有好的异常体系的系统不是一个优秀的系统只是我这么觉得. 异常反复抓取是异常框架和程序员的问题.

请注意你上面的代码的方法定义,方法并不会抛异常所以他多判断一次多抛出一次,这个就是开发人员自身的问题了.(对于可测异常)

4 楼 jyasa 2008-09-28  

sorphi 写道

请看oscache的参考文档

3 楼 hite 2008-08-12  
leobluewing 写道
可能getCache()方法被优化过了

不会返回null也不一定。

sorphi 写道
请看oscache的参考文档


其实呢,我并不是说这个代码本身有什么含义,或者讨论它的健壮性。这段代码是个因子……
所以在我的题目里没有出现cache等具体字眼。
————见接前续————
2 楼 sorphi 2008-08-12  
请看oscache的参考文档
1 楼 leobluewing 2008-08-11  
可能getCache()方法被优化过了

不会返回null也不一定。

相关推荐

Global site tag (gtag.js) - Google Analytics