我有一个关于异常处理和资源管理的问题,我想知道是否有人可以分享他们的意见.我需要执行一系列操作:阅读应用程序设置,设置环境,进行操作,然后最终清理.清理涉及拆除环境,但这只有在首先成功设置后才能发生.
这是我的第一种(也是la脚的)方法:
try {
readSettings();
setupEnvironment();
} catch (Exception ex) {
logStackTrace(ex);
displayError(ex);
closeCommThreads();
return;
}
try {
// do stuff
} catch (Exception ex) {
logStackTrace(ex);
displayError(ex);
} finally {
teardownEnvironment();
closeCommThreads();
}
这看起来有些丑陋,所以我决定寻找更好的解决方案.我做了一些背景阅读,并且有很多文章投票赞成更大的try / catch块,并最终使用(双关语)进行清理.所以这是我的第二次尝试:
try {
readSettings();
setupEnvironment();
// do stuff
} catch (Exception ex) {
logStackTrace(ex);
displayError(ex);
} finally {
teardownEnvironment();
closeCommThreads();
}
为了完成这项工作,我必须从teardownEnvironment()中删除顺序耦合,以便可以在setupEnvironment()之前或之后的任何时间调用它(对于编辑者:有什么改进的方法吗?).这是正确的方法吗?在设置之前,我确实感到有点不可思议.
编辑:
只是为了使其更加明确:我通过在teardownEnvironment内包括一个额外的检查来消除了顺序耦合-类似于if(!isSetup())return;.