一个异常应该在一个log文件里面出现一次且只有一次

系统报了一堆异常，吓人一跳，看了看，发现有一个异常在同一个文件中，打了多次的情况。
比如A->B，B->C，C系统有一个异常，打到了error.log中，然后又向上抛，B捕获了这个异常，打到了error.log中，又向上抛，C系统捕获了这个异常，打到了error.log中。

感觉这种记录异常的方式是不合理的。

当时这么做的思路是所有的异常打在一个文件里面，方便排查故障。

但是这种一个root cause的异常在同一个文件打多次应该是不必要的。

传统的做法是一个异常只打一次log，但是这个只是一个理想情况，我的想法是，一个系统对于同一个root cause的异常只应该记录一次。

以上面的例子为例，C把异常打到c.error.log中，抛出去，B把异常打到b.error.log中，抛出去，A把异常打到a.error.log中。同时，所有的异常应该保留异常堆栈，这样，不论是在A还是B还是C都可以看到这个异常的真正的root cause。

评论

25 楼 zhang_xzhi_xjtu 2010-11-28  

gdpglc 写道

zhang_xzhi_xjtu 写道

false,null是返回值，是正常流程的一部分，不是异常流。

这算总原则吧，我不反对。
不过，我想的说的是：事情要具体情况具体分析，不可教条。某些事情，算什么是分不清楚的，比如：hibernate的load和get方法，功能一样，但找不到对象时，前者会抛异常，后者返回null。

软件有规则的一面，也有复杂多变的一面，很多简单的原则在软件开发上，都是有适用范围的。在应用软件开发的指导性原则时，一定要了解它的前因后果，防止在错误的上下文下应用某个设计原则，那只能是事倍功半。

同意你说的事情要具体情况具体分析，不可教条。

hibernate的load和get方法，功能一样，但找不到对象时，前者会抛异常，后者返回null。

这个设计不觉的很奇怪吗？

如果你不假思索的说这个好，并且以这个为参照物的话，是不是教条（迷信框架）了。

24 楼 gdpglc 2010-11-28  

zhang_xzhi_xjtu 写道

false,null是返回值，是正常流程的一部分，不是异常流。

这算总原则吧，我不反对。
不过，我想的说的是：事情要具体情况具体分析，不可教条。某些事情，算什么是分不清楚的，比如：hibernate的load和get方法，功能一样，但找不到对象时，前者会抛异常，后者返回null。

软件有规则的一面，也有复杂多变的一面，很多简单的原则在软件开发上，都是有适用范围的。在应用软件开发的指导性原则时，一定要了解它的前因后果，防止在错误的上下文下应用某个设计原则，那只能是事倍功半。

23 楼 zhang_xzhi_xjtu 2010-11-28  

false,null是返回值，是正常流程的一部分，不是异常流。

22 楼 gdpglc 2010-11-28  

zhang_xzhi_xjtu 写道

1，2 我认为以false和null这个处理方式是个设计的败笔。忽略的话，很多人catch住一样忽略，你是防不住的。

败在何处？

21 楼 zhang_xzhi_xjtu 2010-11-28  

gdpglc 写道

linliangyi2007 写道

我觉得这个问题的关键是，系统在哪个层面上拦截/处理异常？
从设计上说，层层往上抛异常，层层记录，却不处理，这样的设计有缺陷的。

底层的异常应该对上层屏蔽，外部API调用可能得到一个false或者null的结果，但不是异常本身。

1.请问底层的民常上层屏蔽的目的是什么？

2.以false和null表达异常，我觉得不合适。调用者很可能会忽略对异常的处理。

3.打印日志，我建议，catch异常的就要进行记录。在多人合作的情况下，这样的规则最容易统一，必免出现异常出现而不记录的情况。多看点日志对个程序员来说，也不算啥。

1，2 我认为以false和null这个处理方式是个设计的败笔。忽略的话，很多人catch住一样忽略，你是防不住的。
3 人生要有追求啊，为什么要多看日志。

20 楼 gdpglc 2010-11-28  

linliangyi2007 写道

我觉得这个问题的关键是，系统在哪个层面上拦截/处理异常？
从设计上说，层层往上抛异常，层层记录，却不处理，这样的设计有缺陷的。

底层的异常应该对上层屏蔽，外部API调用可能得到一个false或者null的结果，但不是异常本身。

1.请问底层的异常上层屏蔽的目的是什么？

2.以false和null表达异常，我觉得不合适。调用者很可能会忽略对异常的处理。

3.打印日志，我建议，catch异常的就要进行记录。在多人合作的情况下，这样的规则最容易统一，必免出现异常出现而不记录的情况。多看点日志对个程序员来说，也不算啥。

19 楼 zhang_xzhi_xjtu 2010-11-28  

抛出异常的爱 写道

zhang_xzhi_xjtu 写道

en 现在超时很多

用日志分析工具吧....
sh写一个

系统有专门的监控系统，我也有写awk脚本。
但是问题是这样打log，我觉得不合理。

18 楼 抛出异常的爱 2010-11-17  

zhang_xzhi_xjtu 写道

en 现在超时很多

用日志分析工具吧....
sh写一个

17 楼 agapple 2010-11-17  

这个还是异常的设计问题

需要规范好业务异常和系统异常，各司其职，在合适的地方捕获处理

16 楼 zhang_xzhi_xjtu 2010-11-16  

en 现在超时很多

15 楼 抛出异常的爱 2010-11-16  

zhang_xzhi_xjtu 写道

打在哪里呢，我是接受打多次的，但是不接受一个log文件里面打多次。
如果打两次还是在同一个文件里面，那么也是不好的。

你家代码中异常很多么?

14 楼 zhang_xzhi_xjtu 2010-11-16  

打在哪里呢，我是接受打多次的，但是不接受一个log文件里面打多次。
如果打两次还是在同一个文件里面，那么也是不好的。

13 楼 抛出异常的爱 2010-11-16  

zhang_xzhi_xjtu 写道

抛出异常的爱 写道

runtimeexception
只打二次.

when and where?

错误发生的地方 , 需要回滚的地方
其它的地方不用捕获也不用打日志.

12 楼 zhang_xzhi_xjtu 2010-11-15  

抛出异常的爱 写道

runtimeexception
只打二次.

when and where?

11 楼 zhang_xzhi_xjtu 2010-11-15  

buyajun 写道

楼主 你真要是 “C把异常打到c.error.log中，抛出去，B把异常打到b.error.log中，抛出去，A把异常打到a.error.log中”

调试程序的时候 会累死
得看 N个 log文件

no，对于c系统来说，只看c.error.log就好了。所有的堆栈信息都有的啊。

10 楼 zhang_xzhi_xjtu 2010-11-15  

snovian 写道

如果声明抛出异常，就不应该记录也不应该捕获

框架本身也要记录异常，来观测框架的使用情况。

9 楼 zhang_xzhi_xjtu 2010-11-15  

linliangyi2007 写道

我觉得这个问题的关键是，系统在哪个层面上拦截/处理异常？
从设计上说，层层往上抛异常，层层记录，却不处理，这样的设计有缺陷的。

底层的异常应该对上层屏蔽，外部API调用可能得到一个false或者null的结果，但不是异常本身。

这个情况太理想，而且也是可以商量的，异常吞没返回一个null或者false，未必是一个好的实践。

8 楼 zhang_xzhi_xjtu 2010-11-15  

xixix2004 写道

这个不是什么技术问题吧。

1，异常从C抛到B，再从B抛到A，A的异常堆栈信息能保存相应的信息。
2，C，B不能处理异常为什么要捕获。

很简单的方法，B，C直接抛出，只在A捕获，打印异常信息即可。

捕获异常的奥义：谁有能力处理谁捕获。

你懂的。

有些第3方库，就是捕获，打log，再抛出。这个我控制不了的。

7 楼 buyajun 2010-11-15  

楼主 你真要是 “C把异常打到c.error.log中，抛出去，B把异常打到b.error.log中，抛出去，A把异常打到a.error.log中”

调试程序的时候 会累死
得看 N个 log文件

6 楼 wwj85523 2010-11-15  

我觉得应该处理这个错误,让它下次不出错,
而不是处理这个log,让它出现而不显示,真正狼来了就倒霉了,呵呵