在Eclipse
我收到了一条我不明白的警告Resource leak: 'ps' is not closed at this location
。
在我的Java
代码中,我将“ps”声明为预备语句,并且我多次使用(并关闭)它。然后我有以下顺序:
try {
if(condition) {
ps = c.prepareStatement("UPDATE 1 ...");
} else {
ps = c.prepareStatement("UPDATE 2 ...");
}
ps.executeUpdate();
} catch (SQLException e) {
// exception handling
} finally {
if (null != ps)
try {
ps.close();
} catch (SQLException e) {
// exception handling
};
}
“资源泄漏” - 警告来自其他部分的“更新” - 声明。
如果我在启动try块之前设置ps = null
,则没有警告。
如果第二个UPDATE-Statement被注释掉,则不会显示警告。
这是理解还是java / eclipse问题?
答案 0 :(得分:11)
如果您有此警告,则表示您使用的是Java 7.在这种情况下,您不应自行关闭实现AutoClosable
的资源。您应该在try
statementcommented:
// decide which update statement you need:
// (your if should be here)
String update = ....;
try (
ps = c.prepareStatement(update);
) {
// use prepared statement here.
} catch (SQLException) {
// log your exception
throw new RuntimeException(e);
}
// no finally block is needed. The resource will be closed automatically.
我确实不知道为什么存在if/else
语句导致警告出现或消失。但是java 7建议使用我上面描述的自动关闭资源的方法,所以试试这个。
答案 1 :(得分:3)
我认为,你正在使用的检查器存在问题。
将代码分成initialization
和use
块。此外,将异常抛出初始化块(或提前返回)。这样,在use
阻止
// initialization
// Note that ps is declared final.
// I think it will help to silence your checker
final PreparedStatement ps;
try {
if( bedingungen ... ) {
ps = c.prepareStatement("UPDATE 1 ...");
} else {
ps = c.prepareStatement("UPDATE 2 ...");
}
}
catch (SQLException e) {
log.error("Problem creating prepared statement, e );
throw e;
}
// use
try {
ps.executeUpdate();
} catch (SQLException e) {
log.error("Problem decrementing palets on " + srcElement.getName() +
": " + e.getMessage());
}
finally {
try {
ps.close();
} catch (SQLException e) {
log.warn("Error closing PreparedStatement: " + e.getMessage());
};
}
答案 2 :(得分:-5)
将变量名称从c更改为mC。我认为使用c作为变量名时这是一个奇怪的故障。谢谢查理