1

我有一个类正在接受一个 Id 并尝试更新变量 current_account 但是当我打印出 current_account 的详细信息时它没有更新。

有人对此有任何想法吗?python的新手,所以可能会做一些我看不到的愚蠢的事情。

class UserData:
    def __init__(self, db_conn=None):
        if None == db_conn:
            raise Exception("DB Connection Required.")

        self.db = db_conn
        self.set_my_account()
        self.set_accounts()
        self.set_current_account()

    def set_current_account(self, account_id=None):
        print account_id
        if None == account_id:
            self.current_account = self.my_account
        else:
            if len(self.accounts) > 0:
                for account in self.accounts:
                    if account['_id'] == account_id:
                        self.current_account = account
                        print self.current_account['_id']
            else:
                raise Exception("No accounts available.")

假设set_my_account()得到一个账户数据字典和一个账户数据set_accounts()字典列表。

因此,当我执行以下操作时:

user_data = UserData(db_conn=db_conn)
user_data.set_current_account(account_id=account_id)

Wheredb_conn是一个有效的数据库连接并且account_id是一个有效的帐户 ID。

我从以上两行中得到以下内容。

None
518a310356c02c0756764b4e
512754cfc1f3d16c25c350b7

所以None值来自类的声明,然后接下来的两个来自对set_current_account(). 第一个id值是我要设置的值。第二个id值是已经从类__init__()方法中设置的值。

4

2 回答 2

3

有很多冗余和非 Pythonic 结构。我清理了代码以帮助我了解您要做什么。

class UserData(object):
    def __init__(self, db_conn):
        self.db = db_conn
        self.set_my_account()
        self.set_accounts()
        self.set_current_account()

    def set_current_account(self, account_id=None):
        print account_id
        if account_id is None:
            self.current_account = self.my_account
        else:
            if not self.accounts:
                raise Exception("No accounts available.")

            for account in self.accounts:
                if account['_id'] == account_id:
                   self.current_account = account
                   print self.current_account['_id']

user_data = UserData(db_conn)
user_data.set_current_account(account_id)

(db_conn=None)当没有显式参数的调用无效时,您使用了默认参数。是的,你现在可以打电话,__init__(None)但你也可以打电话__init__('Nalum');你不能保护一切。

通过移动“无帐户”例外,块快速失败,您可以节省一级缩进。

调用 UserData(db_conn=db_conn) 是有效的,但不必要地重复。

不幸的是,我仍然无法弄清楚您要完成什么,这可能是最大的缺陷。变量名对于帮助读者(可能是未来的你)理解代码非常重要。current_account, my_account,account_idcurrent_account['_id']如此模糊的意图,你应该真正考虑更多不同的,信息丰富的名字。

于 2013-05-09T14:33:48.657 回答
1

弄清楚它是什么。

数据正在代码库中的其他位置进行更改。它现在按预期工作。

感谢大家指出我做错的以 Python 为中心的事情,很高兴得到它。

于 2013-05-10T10:45:01.810 回答