pythonで配列にクラスを入れた際の挙動について

Question

#データベースからマスタの値を配列に読み込んでいるのですが、追加後配列の中身を参照すると最後のデータのみになってしまいます。件数と読み込みデータは正しいです。

#idと名前のみのクラス
class myClass(object):
    def __init__(self, i_id,i_name):
        self.id = i_id
        self.name = i_name
        
        
        


#配列を用意
aaa = []

cow_breeds = Data.objects.raw('SELECT * FROM cow_breeds_data order by id asc')
for p in cow_breeds:
	#idと名前を格納
    w_aaa = myClass
    w_aaa.id = p.id
    w_aaa.name = p.name
    logging.debug(str(w_aaa.id) + "," + w_aaa.name)
    #配列に追加
    aaa.append(w_aaa)

1,'aaa'
2,'bbb'
3,'ccc'

のデータを読み込んだ



#配列の中身が最後のデータ
for q in aaa:
    logging.debug(str(q.id) + "," + q.name)
    
    
しかし配列を見ると

3,'ccc'
3,'ccc'
3,'ccc'

になってしまいます

Answer 1

w_aaa = myClass この部分でクラス myClass 自体を参照しているのが原因です。
コード的には myClass のインスタンスを作ってリスト aaa に append する処理に見えます。

また、 w_aaa.id = p.id は有効ですが、 myClassのidとnameは
インスタンス生成時に渡す形で記述されているのでそれに沿った形で記述してみました。

from pprint import pp
import logging
logging.basicConfig(level=logging.DEBUG)

#idと名前のみのクラス
class myClass(object):
    def __init__(self, i_id,i_name):
        self.id = i_id
        self.name = i_name

# SQL実行結果の代わりのクラス
class Breeds():
    def __init__(self, id, name):
        self.id = id
        self.name = name

#配列を用意
aaa = []

# cow_breeds = Data.objects.raw('SELECT * FROM cow_breeds_data order by id asc')
cow_breeds = []
cow_breeds.append(Breeds(1, 'aaa'))
cow_breeds.append(Breeds(2, 'bbb'))
cow_breeds.append(Breeds(3, 'ccc'))

for p in cow_breeds:
    #idと名前を格納
    w_aaa = myClass(p.id, p.name)
    # w_aaa.id = p.id ↑で渡してあるので不要
    # w_aaa.name = p.name
    logging.debug(str(w_aaa.id) + "," + w_aaa.name)
    #配列に追加
    aaa.append(w_aaa)

# 配列の中身をダンプ
pp(aaa)

#配列の中身が最後のデータ
for q in aaa:
    logging.debug(str(q.id) + "," + q.name)

Comments

[sarasa_38765]

回答ありがとうございます。参考にします。

Answer 2

次のように修正すると期待通りに動作します:

for p in cow_breeds:
    #idと名前を格納
    # ↓ インスタンス生成していません、同じクラスを何度も直接使っています
    # w_aaa = myClass
    # w_aaa.id = p.id
    # w_aaa.name = p.name

　　# ↓インスタンス生成します
    w_aaa = myClass(p.id, p.name)
    logging.debug(str(w_aaa.id) + "," + w_aaa.name)
    #配列に追加
    aaa.append(w_aaa)

動作結果:

$ python test.py
1,aaa
2,bbb
3,ccc

原因

質問欄のコードは、クラスをインスタンス生成せずに使ってしまっています

    w_aaa = myClass

インスタンスとクラスは、よく「お菓子作り」の例で説明されます

インスタンスはクッキーで、クラスはクッキーを作るための金型です

お菓子作りにたとえると

金型????でクッキー????を大量生産して箱????につめようとしています

質問欄のコードは、金型????を直接箱????につめこんでいます
しかも、何度も同じ金型????を取り出しては、箱????につめこんでいます

箱????の中には金型????しか入っていません

そうではなく、
金型????からクッキー????をたくさん作って、箱????につめこむ必要がありますね

Comments

[sarasa_38765]

分かりやすい説明ありがとうございます。よく理解できました。

Answer 3

w_aaa = myClass
w_aaa.id = p.id
w_aaa.name = p.name

これは次の誤りでしょうね。

w_aaa = myClass(p.id, p.name)

Comments

[sarasa_38765]

正解です。ありがとうございます。