Path: csiph.com!usenet.pasdenom.info!weretis.net!feeder4.news.weretis.net!feeds.phibee-telecom.net!newsfeed.xs4all.nl!newsfeed1a.news.xs4all.nl!xs4all!newsgate.cistron.nl!newsgate.news.xs4all.nl!post.news.xs4all.nl!not-for-mail
MIME-Version: 1.0
In-Reply-To: <mk8keahatht34gt0kq1ncmtaskdu2s3arc@4ax.com>
References: <6trfeate2ppvm1mcapgr0g4g2fd3vceab6@4ax.com> <mailman.18955.1424521415.18130.python-list@python.org> <mk8keahatht34gt0kq1ncmtaskdu2s3arc@4ax.com>
Date: Sun, 22 Feb 2015 13:15:09 -0600
Subject: Re: try pattern for database connection with the close method
From: Skip Montanaro <skip.montanaro@gmail.com>
To: Mario Figueiredo <marfig@gmail.com>
Content-Type: text/plain; charset=UTF-8
Cc: Python <python-list@python.org>
Precedence: list
Newsgroups: comp.lang.python
Message-ID: <mailman.19025.1424633976.18130.python-list@python.org>
Lines: 37
NNTP-Posting-Host: 2001:888:2000:d::a6
Xref: csiph.com comp.lang.python:86150

On Sun, Feb 22, 2015 at 12:41 PM, Mario Figueiredo <marfig@gmail.com> wrote:
> The sqlite context manager doesn't close a database connection on
> exit. It only ensures, commits and rollbacks are performed.

Sorry, I haven't paid careful attention to this thread, so perhaps
this has already been suggested, however... Can't you write your own
class which delegates to the necessary sqlite3 bits and has a context
manager with the desired behavior? Thinking out loud, you could define
a ConnectionMgr class which accepts a sqlite3 connection as a
parameter:

class ConnectionMgr(object):
  def __init__(self, conn):
    self.conn = conn

  def __enter__(self):
    ...

  def __exit__(self, type, value, exception):
    if self.conn is not None:
      ... close self.conn connection here ...
    self.conn = None

  def __getattr__(self, attr):
    return getattr(self.conn, attr)

then...

  try:
    with MyConnection(lite.connect('data.db')) as db:
      ...
  except lite.DatabaseError:
    ...

Might also have to __enter__ and __exit__ self.conn as appropriate.

Skip