Groups > comp.lang.java.programmer > #37149

Re: Not closing a SQL(ite3) connection

Path	csiph.com!eternal-september.org!feeder.eternal-september.org!news.eternal-september.org!.POSTED!not-for-mail
From	Cecil Westerhof <Cecil@decebal.nl>
Newsgroups	comp.lang.java.programmer
Subject	Re: Not closing a SQL(ite3) connection
Date	Sat, 12 Aug 2017 16:44:53 +0200
Organization	Decebal Computing
Lines	55
Message-ID	<87y3qoyhei.fsf@munus.decebal.nl> (permalink)
References	<87a839um2q.fsf@munus.decebal.nl> <omg0la$1qe3$1@gioia.aioe.org> <874lte1pdj.fsf@munus.decebal.nl> <omli1h$jt7$1@gioia.aioe.org> <slrnootj2q.o2q.avl@logic.at> <87bmnlyrx7.fsf@munus.decebal.nl> <omn1cd$mor$1@gioia.aioe.org>
Mime-Version	1.0
Content-Type	text/plain; charset=utf-8
Content-Transfer-Encoding	8bit
Injection-Info	mx02.eternal-september.org; posting-host="a1c8c080b2cb6129343ef73ef98e693f"; logging-data="6221"; mail-complaints-to="abuse@eternal-september.org"; posting-account="U2FsdGVkX1/yeS+nJCOTWYuWaMZ0IcZJE2Eff5XyWm4="
User-Agent	Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux)
Cancel-Lock	sha1:vRHXHUTs8zXGirYthqOApTa7ryA= sha1:2FI8DRpXS/Yhlv/hiN+i6WvDwA8=
X-Homepage	http://www.decebal.nl/
Xref	csiph.com comp.lang.java.programmer:37149

Show key headers only | View raw

Arne Vajhøj <arne@vajhoej.dk> writes:

> On 8/12/2017 6:57 AM, Cecil Westerhof wrote:
>> This was only for testing purposes. My real code has become:
>>      @Override
>>      public void close() throws SQLException {
>>          boolean closedResultSet = false;
>>
>>          try {
>>              Statement stmt = resultSet.getStatement();
>>
>>              resultSet.close();
>>              closedResultSet = true;
>>              stmt.close();
>>          } catch (SQLException e) {
>>              if (closedResultSet) {
>>                  SQLException extException = new SQLException("Could not close Statement after closing ResultSet.");
>>
>>                  extException.addSuppressed(e);
>>                  e = extException;
>>              }
>>              throw e;
>>          }
>>      }
>>
>> The String will be done with a variable of-course. Beside that: is the
>> code OK, or can it be improved?
>
> I see little value in this over the much more simple:
>
>      @Override
>      public void close() throws SQLException {
>          Statement stmt = resultSet.getStatement();
>          resultSet.close();
>          stmt.close();
>      }
>
> :-)
>
> The original SQLException should contain sufficient information. I
> don't see "Could not close Statement after closing ResultSet." adding
> value.

Maybe I am gold-plating. I want to make sure that it is clear that it
is my function and not a user function where it goes wrong, but
of-course:
    nl.decebal.sql.SQLResultSet.close
makes that clear.

I probably should remove it.

-- 
Cecil Westerhof
Senior Software Engineer
LinkedIn: http://www.linkedin.com/in/cecilwesterhof

Thread

Not closing a SQL(ite3) connection Cecil Westerhof <Cecil@decebal.nl> - 2017-08-09 11:28 +0200
  Re: Not closing a SQL(ite3) connection Arne Vajhøj <arne@vajhoej.dk> - 2017-08-09 18:00 -0400
    Re: Not closing a SQL(ite3) connection Cecil Westerhof <Cecil@decebal.nl> - 2017-08-11 10:26 +0200
      Re: Not closing a SQL(ite3) connection Arne Vajhøj <arne@vajhoej.dk> - 2017-08-11 20:27 -0400
        Re: Not closing a SQL(ite3) connection Cecil Westerhof <Cecil@decebal.nl> - 2017-08-12 10:00 +0200
          Re: Not closing a SQL(ite3) connection Andreas Leitgeb <avl@logic.at> - 2017-08-12 08:51 +0000
            Re: Not closing a SQL(ite3) connection Cecil Westerhof <Cecil@decebal.nl> - 2017-08-12 11:05 +0200
          Re: Not closing a SQL(ite3) connection Arne Vajhøj <arne@vajhoej.dk> - 2017-08-12 09:52 -0400
            Re: Not closing a SQL(ite3) connection Cecil Westerhof <Cecil@decebal.nl> - 2017-08-12 16:26 +0200
              Re: Not closing a SQL(ite3) connection Arne Vajhøj <arne@vajhoej.dk> - 2017-08-12 21:43 -0400
                Re: Not closing a SQL(ite3) connection Cecil Westerhof <Cecil@decebal.nl> - 2017-08-15 08:19 +0200
          Re: Not closing a SQL(ite3) connection lewbloch@gmail.com - 2017-08-12 10:14 -0700
        Re: Not closing a SQL(ite3) connection Andreas Leitgeb <avl@logic.at> - 2017-08-12 09:33 +0000
          Re: Not closing a SQL(ite3) connection Cecil Westerhof <Cecil@decebal.nl> - 2017-08-12 12:57 +0200
            Re: Not closing a SQL(ite3) connection Andreas Leitgeb <avl@logic.at> - 2017-08-12 13:27 +0000
              Re: Not closing a SQL(ite3) connection Cecil Westerhof <Cecil@decebal.nl> - 2017-08-12 16:37 +0200
            Re: Not closing a SQL(ite3) connection Arne Vajhøj <arne@vajhoej.dk> - 2017-08-12 09:55 -0400
              Re: Not closing a SQL(ite3) connection Cecil Westerhof <Cecil@decebal.nl> - 2017-08-12 16:44 +0200
                Re: Not closing a SQL(ite3) connection Arne Vajhøj <arne@vajhoej.dk> - 2017-08-12 21:45 -0400
            Re: Not closing a SQL(ite3) connection Martin Gregorie <martin@address-in-sig.invalid> - 2017-08-12 21:04 +0000
              Re: Not closing a SQL(ite3) connection Cecil Westerhof <Cecil@decebal.nl> - 2017-08-13 00:30 +0200
                Re: Not closing a SQL(ite3) connection Martin Gregorie <martin@address-in-sig.invalid> - 2017-08-13 11:34 +0000
                Re: Not closing a SQL(ite3) connection Cecil Westerhof <Cecil@decebal.nl> - 2017-08-13 15:27 +0200
          Re: Not closing a SQL(ite3) connection Andreas Leitgeb <avl@logic.at> - 2017-08-12 13:06 +0000

csiph-web