python QtでのQPushButtonでのインスタンス生成で同じものが生まれてしまうことについて

Question

打ち消し線### 前提・実現したいこと

pythonでQPushButtonばらばらなインスタンスを2重ループで生成したい

発生している問題・エラーメッセージ

pythonでQPushButtonインスタンスを2重ループで生成する際に同じものが生成されてしまう

該当のソースコード

import sys
from PyQt5.QtWidgets import *
import datetime

class MyWindow(QMainWindow):
    
    def __init__(self):
        """ インスタンスが生成されたときに呼び出されるメソッド """
        super(MyWindow, self).__init__()
        self.initUI()
    
    def on_click1(self):
        print("clicked!")

    
    def initUI(self):
        """ UIの初期化 """
        self.resize(250, 150)           # 250x150ピクセルにリサイズ
        self.setWindowTitle('MyWindow') # タイトルを設定
        
        self.button=[[0]*8]*8
        hbox=[0]*8
        button=[[0]*8]*8
        print(button)
        for i in range(8):
            for j in range(8):
                self.button[i][j] = QPushButton('上')
                print(i,j,self.button[i][j])
        

        self.show()

def main():
    app = QApplication(sys.argv)
    w = MyWindow()
    app.exec_()

if __name__ == '__main__':
    main()

補足情報（FW/ツールのバージョンなど）

プリントした結果

0 0 <PyQt5.QtWidgets.QPushButton object at 0x7fd8edf8a0f0>
0 1 <PyQt5.QtWidgets.QPushButton object at 0x7fd8edf8a190>
0 2 <PyQt5.QtWidgets.QPushButton object at 0x7fd8edf8a230>
0 3 <PyQt5.QtWidgets.QPushButton object at 0x7fd8edf8a2d0>
1 0 <PyQt5.QtWidgets.QPushButton object at 0x7fd8edf8a370>
1 1 <PyQt5.QtWidgets.QPushButton object at 0x7fd8edf8a0f0>
1 2 <PyQt5.QtWidgets.QPushButton object at 0x7fd8edf8a190>
1 3 <PyQt5.QtWidgets.QPushButton object at 0x7fd8edf8a230>
2 0 <PyQt5.QtWidgets.QPushButton object at 0x7fd8edf8a2d0>

に同じインスタンスが含まれてしまいボタンの識別ができない
ばらばらなものにするにはどうしたらいいでしょうか？

Answer 1

問題点: 2次元配列の初期化

# NG
buttons = [[0]*8]*8

# OK
buttons = [[0]*8 for _ in range(8)]

上のコードで生成したリストは、リストの参照となっている為、
buttons[0] と buttons[1] は同じリストになってます。

[1][0] へボタンを代入した時に、[0][0] のボタンは破棄されます。
その為、次の[1][1] からは、その破棄されたボタンのアドレスを再利用する
といった具合になってます。

また、アドレスが重複してるように見えるのは、別の問題も絡んでいて
QPushButton に 親ウィジェットを指定していない為に起こっています。
レイアウトに入れるなり、インスタンス生成時に親を指定するとこの現象は回避できます。

解決策

• 2次元リストの初期化を修正する
• ボタンのインスタンス生成時に親ウィジェットを指定する

---

引用 FAQ 多次元のリストを作るにはどうしますか？

このようにして多次元の配列を作ろうとしてしまったことがあるでしょう:

>>>
>>> A = [[None] * 2] * 3

これを表示したときには問題なさそうに見えます:

>>> A
[[None, None], [None, None], [None, None]]

しかし値を代入すると、その値が複数の場所に現れてしまいます:

>>> A[0][0] = 5
>>> A
[[5, None], [5, None], [5, None]]

これは、* を使ったリストの複製がコピーを作らず、存在するオブジェクトへの参照を作るだけだからです。