PyQt4におけるシグナル/スロットについて。無限ループしてしまう。

Question

毎度のことお世話になっております。
現在、python3.4言語でPyQt4を学んでいます。

本当ならPyQt4の書籍で学びたいのですが、
今現在、日本語によるものがないので、仕方なしに
Qt4の書籍を読み、PyQt4のドキュメントと照らし合わせながら学習を進めているという状況です。

ここで本題の質問なのですが、このコードを見てください。

class Fusen(QWidget):
    def __init__(self, mainwindow):
        super().__init__()
        self.createButton(mainwindow)
    def createButton(self, mainwindow):
        newFusenTB = QToolButton()
        newFusenTB.setText('+')
        newFusenTB.clicked.connect(lambda: mainwindow.createNewFusen()) # 問題のコード
    ####Widgetの設定は省略####

class MainWindow(QMainWindow):
    def __init__(self):
        super().__init__()
        self.createWindow()
    def self.createWindow(self):
        self.m_newFusenAct = QAction('&New Fusen', self)
        self.m_newFusenAct.triggered.connect(lambda: self.createNewFusen())
        self.menubar = self.menuBar()
        filemenu = self.menuBox.addMenu('&File')
        filemenu.addAction(self.m_newFusenAct)
     ###Windowの設定は省略###
    def createNewFusen(self):
        ptr = Fusen(self) #Fusenクラスのオブジェクト生成
        ptr.show()

MainWindow
|-------------------------|
|File|                    | # MenuBar
|-------------------------|
|                         |
|                         |
|-------------------------|

Fusen
|-------------------------|
| [+]                     | # [+] は ToolButton
|-------------------------|
|                         |
|                         |
|-------------------------|

このプログラムを起動させると、まずMainWindowが表示されて、そのあとメニューバーのFile->NewFusenをクリックします。

そうすると、別のウィンドウであるFusenが表示される。
そして、Fusenの[+]（ToolButton）をクリックしても、同じく新しいFusenが表示される。というプログラムになる予定です。

---

ですが、バグが発生しまして、
メニューのNewFusenのボタンをクリックして、Fusenクラスのオブジェクトが生成されるとき、
Fusenクラスのコンストラクタで、[+]ツールボタンが生成されると同時にconnectされているスロットが読まれて、また新たにFusenクラスのオブジェクトが生成される。(もしかしたら生成されるというのは勘違い？)

という、無限ループに陥ってしまうというわけです。

---

質問としては、
第一に無限ループを脱出する方法なのですが、
考えてみたのですが、どうも思い浮かびません。

解決策の予想ですが、どうもこの「シグナルとスロット」の関係にあるのではないかと思っているのですが、
どうなのでしょうか。

皆さんのお力を借りたく質問させてもらいましたが、
こちらの質問、説明に不備等がありましたら、
訂正いたしますので、ご意見をお聞かせください。

どうかよろしくお願いします。

Answer 1

クリック時にウィンドウを開くのであれば、QWidgetではなく、QDialogかQMainWindowを使いましょう。
Fusenクラス内から、スロット関数MainWindow::createNewFusen()を呼び出したい場合は以下のようなコードになりますが、素直にFusen::createNewFusen()を実装した方が良いと思います。

# -*- coding: utf-8 -*-
import sys
from PyQt4 import QtCore, QtGui


class Fusen(QtGui.QDialog):
    def __init__(self, parent=None):
        super(Fusen, self).__init__(parent)
        self.setupUi(self)
        self.button.clicked.connect(parent.createNewFusen)
        self.action_newFusen.triggered.connect(parent.createNewFusen)

    def setupUi(self, Fusen):
        self.hboxlayout = QtGui.QHBoxLayout(Fusen)
        self.button = QtGui.QPushButton("new Fusen")
        self.hboxlayout.addWidget(self.button)
        self.setLayout(self.hboxlayout)

        self.menubar = QtGui.QMenuBar(Fusen)
        self.action_newFusen = QtGui.QAction(Fusen)
        self.action_newFusen.setText('new Fusen')
        self.menubar.addAction(self.action_newFusen)
        self.hboxlayout.setMenuBar(self.menubar)


class MainWindow(QtGui.QMainWindow):
    def __init__(self, parent=None):
        super(MainWindow, self).__init__(parent)
        self.setupUi(self)
        self.button.clicked.connect(self.createNewFusen)
        self.action_newFusen.triggered.connect(self.createNewFusen)

    def setupUi(self, MainWindow):
        self.centralwidget = QtGui.QWidget(MainWindow)
        self.hboxlayout = QtGui.QHBoxLayout(self.centralwidget)
        self.button = QtGui.QPushButton("new Fusen")
        self.hboxlayout.addWidget(self.button)
        MainWindow.setCentralWidget(self.centralwidget)
        self.menubar = QtGui.QMenuBar(MainWindow)
        self.action_newFusen = QtGui.QAction(MainWindow)
        self.action_newFusen.setText('new Fusen')
        self.menubar.addAction(self.action_newFusen)
        MainWindow.setMenuBar(self.menubar)

    def createNewFusen(self):
        ptr = Fusen(self)
        ptr.show()

if __name__ == '__main__':
    app = QtGui.QApplication(sys.argv)
    main = MainWindow()
    main.show()
    app.exec_()

setupUi()はQt Designerを使うと書かずに済みますので、実際のコードは更に簡潔となります。

Comments

[mosquito]

返信が遅れまして、申し訳ありません。
WidgetではなくDialogかMainWindowですね！
そういうことなのですね。

それと、parentに指定していませんでした。見落としていました。
とてもわかりやすい解説をありがとうございます！

Answer 2

Qt はわかりますが、Python も PyQt も全くわからないので勘で回答します。

class Fusen(QWidget):
    def __init__(self, mainwindow):
        super().__init__()
        def self.createButton(mainwindow)

self.createButton(mainwindow) の前の def は不要ではないでしょうか。

        newFusenTB.clicked.connect(mainwindow.createNewFusen()) # 問題のコード

mainwindow.createNewFusen() ではなく、lambda: mainwindow.createNewFusen() ではないでしょうか。

Comments

[mosquito]

ご回答ありがとうございます。
そして、大変申し訳にくいのですが、

self.createButton(mainwindow)の　前の「def」は付け間違えです。
大変申し訳ありません。
と、同時に、
lambdaもつけ忘れでございます。

質問する側が、このようなことで、大変申し訳ありません。