Path: csiph.com!v102.xanadu-bbs.net!xanadu-bbs.net!feeder.erje.net!eu.feeder.erje.net!newsfeed.xs4all.nl!newsfeed1.news.xs4all.nl!xs4all!post.news.xs4all.nl!not-for-mail
To: python-list@python.org
From: Peter Otten <__peter__@web.de>
Subject: Re: Dictionary from sqlite3.Row and PyCharm unresolved reference
Date: Fri, 13 Feb 2015 12:33:22 +0100
Organization: None
References: <1a194e0a113d8d215970f15d910@nntp.aioe.org>
Mime-Version: 1.0
Content-Type: text/plain; charset="ISO-8859-1"
Content-Transfer-Encoding: 7Bit
User-Agent: KNode/4.13.3
Precedence: list
Newsgroups: comp.lang.python
Message-ID: <mailman.18727.1423827222.18130.python-list@python.org>
Lines: 43
NNTP-Posting-Host: 2001:888:2000:d::a6
Xref: csiph.com comp.lang.python:85639

Mario Figueiredo wrote:

> Currently i'm using the following code to transform a row fetched from an
> sqlite database into a dictionary property:

class Unknown:

>     def __init__(self, id_):
>         self.id = id_
>         self.data = None
>         ...
>         conn = sqlite3.connect('data')
>         conn.row_factory = sqlite3.Row
>         row = conn.execute(query, {'id': str(id_)}).fetchone()
>         conn.close()
> 
>         if row:
>             self.data = dict(zip(row.keys(), tuple(row)))
> 
> I have two questions:
> 
>   1. Is this an acceptable idiom for the construction of self.data
>   dictionary,
>       or do I have a better and more readable option?

If sqlite3.Row is not dict-like enough to use it directly

self.data = row

you can either convert the Row object with

self.data = dict(row)

or use the recipe for a row factory provided in the documentation at 

https://docs.python.org/dev/library/sqlite3.html#sqlite3.Connection.row_factory

that constructs a dict directly.

And now an unsolicited remark: if you have more than one instance of Unknown 
you might read the data outside the initialiser or at least keep the 
connection open and pass a connection or cursor object to the initialiser.