1

在这个示例代码中，我想使用function db_properties函数上的变量connect_and_query。为了实现这一点，我选择了return. 因此，使用该策略，代码可以完美运行。但是，在这个例子中，db.properties 文件只有 4 个变量。也就是说，如果属性文件有 20 多个变量，我应该继续使用return吗？还是有一种最优雅/更清洁/正确的方法来做到这一点？

import psycopg2
import sys
from ConfigParser import SafeConfigParser

class Main:

    def db_properties(self):
        cfgFile='c:\test\db.properties'
        parser = SafeConfigParser()
        parser.read(cfgFile)
        dbHost = parser.get('database','db_host')
        dbName = parser.get('database','db_name')
        dbUser = parser.get('database','db_login')
        dbPass = parser.get('database','db_pass')
        return dbHost,dbName,dbUser,dbPass

    def connect_and_query(self):
        try:
            con = None

            dbHost=self.db_properties()[0]
            dbName=self.db_properties()[1]
            dbUser=self.db_properties()[2]
            dbPass=self.db_properties()[3]

            con = None
            qry=("select star from galaxy")
            con = psycopg2.connect(host=dbHost,database=dbName, user=dbUser,
                                   password=dbPass)
            cur = con.cursor()
            cur.execute(qry)
            data = cur.fetchall()
            for result in data:
                qryResult   = result[0]
                print "the test result is : " +qryResult
        except psycopg2.DatabaseError, e:
                print 'Error %s' % e
                sys.exit(1)
        finally:
            if con:
                con.close()

operation=Main()
operation.connect_and_query()

我正在使用 python 2.7 问候

4

4 回答 4

5

如果有很多变量，或者如果您想轻松更改正在读取的变量，请返回字典。

def db_properties(self, *variables):
    cfgFile='c:\test\db.properties'
    parser = SafeConfigParser()
    parser.read(cfgFile)
    return {
        variable: parser.get('database', variable) for variable in variables
    }

def connect_and_query(self):
    try:
        con = None
        config = self.db_properties(
            'db_host',
            'db_name',
            'db_login',
            'db_pass',
        )
        #or you can use:
        #   variables = ['db_host','db_name','db_login','db_pass','db_whatever','db_whatever2',...]
        #   config = self.db_properties(*variables)
        #now you can use any variable like: config['db_host']
        # ---rest of the function here---

编辑：我重构了代码，以便您可以指定要在调用函数本身中加载的变量。

于 2012-10-04T15:26:12.540 回答
4

您当然不想调用db_properties()4 次；只需调用一次并存储结果。

返回字典而不是元组也几乎肯定更好，因为调用者需要知道方法按顺序返回的内容，而不是仅仅通过名称访问值。随着传递的值数量的增加，这变得更加难以维护。

例如：

class Main:
    def db_properties(self):
        cfgFile='c:\test\db.properties'
        parser = SafeConfigParser()
        parser.read(cfgFile)
        configDict= dict()
        configDict['dbHost'] = parser.get('database','db_host')
        configDict['dbName'] = parser.get('database','db_name')
        configDict['dbUser'] = parser.get('database','db_login')
        configDict['dbPass'] = parser.get('database','db_pass')
        return configDict

    def connect_and_query(self):
        try:
            con = None
            conf = self.db_properties()

            con = None
            qry=("select star from galaxy")
            con = psycopg2.connect(host=conf['dbHost'],database=conf['dbName'],
                                   user=conf['dbUser'],
                                   password=conf['dbPass'])
于 2012-10-04T15:24:58.857 回答
0

注意：未经测试

您可以更改您的 db_properties 以返回一个字典：

from functools import partial

# call as db_properties('db_host', 'db_name'...)
def db_properties(self, *args):
    parser = SafeConfigParser()
    parser.read('config file')
    getter = partial(parser.get, 'database')
    return dict(zip(args, map(getter, args)))

但除此之外，最好将解析器保留为实例的属性，并提供一种方便的方法......

类任何（对象）：def init（self，*args，**kwargs）：#blah blah blah cfgFile='c:\test\db.properties' self._parser = SafeConfigParser() self._parser.read(cfgFile) @property def db_config(self, key): return self._parser.get('database', key)

然后使用con = psycopg2.connect(host=self.db_config('db_host')...)

于 2012-10-04T15:56:17.577 回答
0

我建议返回一个命名元组：

from collections import namedtuple

# in db_properties()
return namedtuple("dbconfig", "host name user password")(
             parser.get('database','db_host'),
             parser.get('database','db_name'),
             parser.get('database','db_login'),
             parser.get('database','db_pass'),
       )

现在您有了一个可以通过索引或属性访问的对象。

config = self.db_properties()
print config[0]     # db_host
print config.host   # same
于 2012-10-04T16:43:33.617 回答