PyQtで2つのラベルを一つにまとめたソースコードの添削をお願いいたします

Question

前提・実現したいこと

PyQt5で画像ラベルとテキストラベルを横に並べ、ペアにしてリストのように表示し、
ラベルペアに対して同じ動作をさせたいと思っております。

具体的には、マウスオーバーしたら色を変え、クリックしたらメッセージを表示する、です。

今回ソースコードのような形で実装したのですが、PyQtを触り始めたばかりで無駄が多いのではないかと思っております。
よりきれいなPyQt向きなコードを教えていただけないでしょうか。
よろしくお願いいたします。

該当のソースコード

from PyQt5.QtWidgets import *
from PyQt5.QtGui import *
from PyQt5.QtCore import *
import sys
 
class LabelTest(QWidget):
    def __init__(self):
        super().__init__()
        self.initUI()
 
    def initUI(self):
        # ラベル参照用辞書
        self.lbl_dict = dict()
 
        # 背景色
        tmpWidget =  QWidget()
        self.backGroundColor = tmpWidget.palette().color(QPalette.Window);
 
        # 設定
        icosize = 16
        width = 120
 
        for i in range(10):
            # 画像
            lblImg = QLabel(self)
            lblImg.setObjectName(str(i))
            lblImg.setText( "●" ) #lblImg.setPixmap( QPixmap("D:\\test.ico") ) #16x16 ico
            lblImg.installEventFilter(self)
            lblImg.move(0, i*icosize)
            lblImg.resize(icosize,icosize)
 
            # テキスト
            lblText = QLabel(self)
            lblText.setObjectName(str(i))
            lblText.setText("Text")
            lblText.installEventFilter(self)
            lblText.move(icosize, i*icosize)
            lblText.resize(width-icosize,icosize)
 
            # ラベルペアを参照用辞書に追加
            self.lbl_dict.setdefault(str(i), (lblImg,lblText))
 
        self.show()
 
    def eventFilter(self, object, event):
        # マウスオーバーしたラベルペアを黄色にする
        if event.type()== QEvent.Enter:
            for key, value in self.lbl_dict.items():
                if object.objectName() == key:
                    img, lbl = value
 
                    palette_yellow = QPalette()
                    palette_yellow.setColor(QPalette.Window, Qt.yellow)
 
                    img.setPalette(palette_yellow)
                    img.setAutoFillBackground(True)
                    lbl.setPalette(palette_yellow)
                    lbl.setAutoFillBackground(True)
 
            return True
 
        # マウスアウトしたラベルペアを背景色にする
        elif event.type()== QEvent.Leave:
            for key, value in self.lbl_dict.items():
                if object.objectName() == key:
                    img, lbl = value
 
                    palette_blue = QPalette()
                    palette_blue.setColor(QPalette.Window, self.backGroundColor )
 
                    img.setPalette(palette_blue)
                    img.setAutoFillBackground(True)
                    lbl.setPalette(palette_blue)
                    lbl.setAutoFillBackground(True)
 
            return True
 
        # クリックしたらメッセージを表示する
        elif event.type() == QEvent.MouseButtonPress:
            if event.button() == Qt.LeftButton:
                print(object.objectName(), "clicked!")
                return True
 
        return False
 
if __name__== '__main__':
    app= QApplication(sys.argv)
    ex= LabelTest()
    sys.exit(app.exec_())

Answer 1

• import * は使わない。(理由: PEP8 推奨スタイル)
  デバッグ時に名前空間を見る時、不要な未使用クラスを表示しない。
  個別に列挙が面倒な場合は、IDEやEditorに支援機能があります。
• QLabel で埋め込み画像を使う。簡易タグ表記
  setPixmap を使う場合、画像かテキストどちらか片方か表示できませんが、
  テキスト内で簡単なタグは使えます。
• スタイルシートを使う。背景色を変更していたイベント処理を省けます
  但し、マウスカーソルの変更などスタイルでは対応していない項目もあるので、
  全てスタイルシートで済むとは限りません。
• レイアウトマネージャを使う。→ポジションやサイズ計算
• initUI() -> _initUI() 外部から呼び出さないメソッド名の慣習。
  メソッド一覧等のドキュメントを生成する際に、プライベートメンバを表示しないように抑制できたりします。
  アンダーバーを２つにすると、外部から呼び出し際にエラーが出るようにもできます。コードの読みやすさと相談。
• ラベル参照用辞書の撤去。Qt のオブジェクト、生成時に親を指定する事で生存サイクルを管理できます。
  必要があれば、親オブジェクト(ウィジェット) から目的のラベルウィジェットを探索し参照することが可能です。
• 追記2: ラベルのクリックに対応。イベント通知には Qt ではシグナル・スロットの仕組みを使います。
  イベント自体の処理(マウスの左ボタン判別)と、イベントで起こしたい処理(objectNameを表示)の分離

from PyQt5.QtCore import Qt, pyqtSignal
from PyQt5.QtWidgets import (
    QApplication,
    QWidget,
    QLabel,
    QVBoxLayout,
)

CUSTOM_CSS = """
   QLabel:hover {
     background: yellow;
   }
"""

# ラベルにクリック対応
# 参考: https://wiki.qt.io/Clickable_QLabel
class ClickableLabel(QLabel):
    clicked = pyqtSignal()

    def mousePressEvent(self, event):
        if event.button() == Qt.LeftButton:
            self.clicked.emit()


class LabelTest(QWidget):
    def __init__(self, *args, **kw):
        super().__init__(*args, **kw)
        self._initUI()

    def _initUI(self):
        vbox = QVBoxLayout(self)
        img = '<img width=16 height=16 src="google-chrome-icon.png">'

        for num in range(10):
            name = f"Test {num:02}"
            label = ClickableLabel(self, objectName=name)
            label.setText(f"{img} {name}")
            label.clicked.connect(self.clicked)
            vbox.addWidget(label)

    def clicked(self):
        label = self.sender()
        name = label.objectName()
        print(f"{name} clicked")


def main():
    import sys

    app = QApplication(sys.argv)
    app.setStyleSheet(CUSTOM_CSS)
    win = LabelTest()
    win.show()
    sys.exit(app.exec_())


if __name__ == '__main__':
    main()

---

追記: スタイルシートと埋め込みタグにより不要になりますが、
質問のコードの改善点について。辞書内のオブジェクトを全て走査してますが、
辞書には番号をキーにして、画像・テキストのペアを格納すれば、for 文による走査を省けます。

Answer 2

無駄が多いのだろうとは思っていましたが、まさかここまでスッキリするなんて…
目から鱗とはまさにこのことです。

PythonとPyQtの魅力をより感じるようになりました。
素敵なコードを教えていただきまして、本当にありがとうございました。

頑張ってきれいに作り上げたいと思います。

追記 2022-02-13
シグナルとスロットの例までありがとうございます。
一つの関数の中で処理を分けると長くなってしまうことに少し悩んでいたので、
教えていただいたシグナル・スロットの仕組みで書き直したいと思います。