我有这样的代码:
try {
do_stuff();
return do_more_stuff();
} catch (UnsupportedEncodingException e) {
throw CustomException.programmer_error(e);
} catch (ProtocolException e) {
throw CustomException.programmer_error(e);
} catch (MalformedURLException e) {
throw CustomException.programmer_error(e);
} catch (SocketTimeoutException e) {
throw new CustomException(e);
} catch (IOException e) {
throw CustomException.unexpected_error(e);
}
我现在需要将所有这些catch块放在另一个类似的函数中。在这里避免重复的最好方法是什么?
请注意,两个try块中的代码并不十分相似。
另外,我真的不能把捕获物放在更高的位置。
注意,我更倾向于避免:
try {
do_stuff();
return do_more_stuff();
} catch (Exception e) {
handle_exception_via_rtti(e);
}
发布于 2010-02-16 20:15:27
就我个人而言,我会尝试让
do_stuff();
return do_more_stuff();
部分符合更通用的格式,以便应用策略(作为模式)。
然后,您可以重构所有调用这种块的地方,以便它们可以调用更通用的块(其中捕获只放置一次)。
发布于 2010-02-16 20:16:38
这取决于do_stuff
和do_more_stuff
抛出检查异常的原因。他们这样做是为了迫使用户处理异常吗?如果是,则您试图避免:
try {
do_stuff();
return do_more_stuff();
} catch (Exception e) {
handle_exception_via_rtti(e);
}
是一件好事。
如果您的代码只是捕获了Exception
,而do_stuff
和do_more_stuff
后来添加了额外的异常,那么您可能永远不会知道这种更改以及您的代码现在可能出错的事实。
因此,您可能必须处理throwing the kitchen sink并处理所有这些异常,除非这些方法可以切换为未检查的异常。
另一方面,如果这些方法抛出检查异常只是因为程序员懒于处理它们,只是想推卸责任,也许你从错误的角度看待这个问题。
发布于 2010-02-16 20:52:12
引入一个可以被子类化的泛型Action类怎么样:
public class Action {
public void runWithHandlers() throws Exception {
try {
run();
} catch (UnsupportedEncodingException e) {
throw CustomException.programmer_error(e);
} catch (ProtocolException e) {
throw CustomException.programmer_error(e);
} catch (MalformedURLException e) {
throw CustomException.programmer_error(e);
} catch (SocketTimeoutException e) {
throw new CustomException(e);
} catch (IOException e) {
throw CustomException.unexpected_error(e);
}
}
public void run() throws Exception {
// TODO subclasses of Action must implement this
}
}
然后在代码中的其他地方,实例化Action的一个子类并调用runWithHandlers():
new MyAction().runWithHandlers();
https://stackoverflow.com/questions/2272714
复制相似问题