try-with-resources无法调用close()

时间:2015-03-26 10:12:31

标签: java try-with-resources

我正在使用方便的try-with-resources语句来关闭连接。这在大多数情况下都很有效,但只有一种完全简单的方法才能正常工作。即,在这里:

public boolean testConnection(SapConnection connection) {
  SapConnect connect = createConnection(connection);
  try ( SapApi sapApi = connect.connect() ) {
    return ( sapApi != null );
  } catch (JCoException e) {
    throw new UncheckedConnectionException("...", e);
  }
}

sapApi对象为非null,该方法返回true,但从不调用sapApi的close()方法。我现在使用了一个工作正常的finally块。但这非常令人费解。 Java字节代码还包含一个关闭调用。有没有人见过这种行为?

编辑以澄清情况:

这是SapApi,当然它实现了AutoCloseable。

class SapApi implements AutoCloseable {

  @Override
  public void close() throws JCoException {
    connection.close(); // this line is not hit when leaving testConnection(..)
  }
..
}

以下是与testConnection(..)在同一个类中的另一种方法。这里在返回之前调用SapApi.close()。

@Override
public List<Characteristic> selectCharacteristics(SapConnect aConnection,   InfoProvider aInfoProvider) {
  try (SapApi sapi = aConnection.connect()) {
    return sapi.getCharacteristics(aInfoProvider);
  } catch ( Exception e ) {
    throw new UncheckedConnectionException(e.getMessage(), e);
  }
}

修改 2: 这是SapConnect.connect():

SapApi connect() {
  try {
    ... // some setup of the connection
    return new SapApi(this); // single return statement
  } catch (Exception e) {
    throw new RuntimeException(e);
  }
}

SapApi没有子类。如上所述,close方法只有一个实现。特别是,没有空的close()。

3 个答案:

答案 0 :(得分:2)

为了调用close(),SapApi必须实现AutoCloseable接口,但由于我们讨论的是连接,因此SapApi实现抛出IOException的Closable接口更为合适。

阅读本文: http://tutorials.jenkov.com/java-exception-handling/try-with-resources.html

答案 1 :(得分:1)

我不知道你的意思

  

这在大多数情况下效果很好

1通常情况下SapApi在try-with-resources

中使用时会被关闭

2它通常适用于SapApi

以外的资源

我正在回答第2号的假设。

Try-with-resources仅适用于Java 7,用于实现AutoCloseable接口的资源。因此,我的第一个建议是让您检查SapConnectSapApi的API(无论它们是什么)以确定是否属于这种情况。

答案 2 :(得分:0)

一个猜测:也许connect会返回SapApi的子类或代理类。如果未进行任何更改,则close会被覆盖以执行任何操作,否则只会调用super.close()

我将它作为答案给出,因为即使没有调用任何方法,AutoCloseable也能正常工作。