PyQt5 按钮未连接 [英] PyQt5 buttons not connecting

问题描述

我正在尝试使用 PyQT5 构建一个简单的 GUI，其中有 3 个按钮用于打开文件浏览器，还有一个用于对所选文件进行处理，但是我无法让按钮连接到执行此操作所需的功能.

I am trying to build a simple GUI using PyQT5, with 3 buttons to open file browsers and one more to run processing with the selected files, but I can't get my buttons to connect to the functions needed to carry this out.

在Ctrl 类中，_connect_signals 函数似乎没有调用_input_select.谁能帮我找出原因?

In the Ctrl class, the _connect_signals function doesn't seem to be calling _input_select. Can anyone help me figure out why?

import sys

# Import QApplication and the required widgets from PyQt5.QtWidgets
from PyQt5.QtWidgets import QApplication
from PyQt5.QtWidgets import QMainWindow
from PyQt5.QtWidgets import QPushButton
from PyQt5.QtWidgets import QVBoxLayout
from PyQt5.QtWidgets import QWidget
from PyQt5.QtWidgets import QFileDialog


# Create a subclass of QMainWindow to setup the calculator's GUI
class UI(QMainWindow):
    """App's View (GUI)."""

    def __init__(self):
        """View initializer."""
        super().__init__()
        # Set some main window's properties
        self.setFixedSize(300, 150)
        # Set the central widget and the general layout
        self.generalLayout = QVBoxLayout()
        self._centralWidget = QWidget(self)
        self.setCentralWidget(self._centralWidget)
        self._centralWidget.setLayout(self.generalLayout)
        # Create the buttons
        self._create_buttons()

    def _create_buttons(self):
        """Create the buttons."""
        self.buttons = {}
        buttons_layout = QVBoxLayout()
        # Button text | position on the QVBoxLayout
        buttons = {
            "Select input file...": 0,
            "Select config file...": 1,
            "Select output file...": 2,
            "Run": 3,
        }
        # Create the buttons and add them to the grid layout
        for btnText, pos in buttons.items():
            self.buttons[btnText] = QPushButton(btnText)
            buttons_layout.addWidget(self.buttons[btnText], pos)
        # Add buttons_layout to the general layout
        self.generalLayout.addLayout(buttons_layout)


# Create a Controller class to connect the GUI and the model
class Ctrl:
    """App's Controller."""

    def __init__(self, setup, view):
        """Controller initializer."""
        self._view = view
        self._setup = setup
        # Connect signals and slots
        self._connect_signals()

    def _input_select(self):    # Not being called
        print("input selection")

        options = QFileDialog.Options()
        file_select, _ = QFileDialog.getOpenFileNames(
            self,
            'Select Input File...',
            '',
            'CSV Files (*.csv);;All Files (*)',
            options=options
        )
        if file_select:
            self._setup["input"] = file_select

    def _connect_signals(self):
        """Connect signals and slots."""
        self._view.buttons["Select input file..."].clicked.connect(self._input_select)  # Not working!


# Client code
def main():
    """Main function."""
    # Create an instance of `QApplication`
    app = QApplication(sys.argv)
    # Show the app's GUI
    view = UI()
    view.show()
    setup = {}
    # Create instance of the controller
    Ctrl(setup=setup, view=view)
    # Execute app's main loop
    sys.exit(app.exec_())


if __name__ == "__main__":
    main()

如果有帮助，我开始屠宰 这个示例代码来自真正的 Python 教程，但必须在此过程中破坏它.

In case it helps, I started out by butchering this example code from a Real Python tutorial, but must have broken it along the way.

推荐答案

问题是您没有保留对正在创建的 Ctrl() 实例的任何持久引用.这会导致 Python 在实例创建后立即对其进行垃圾回收.

The problem is that you are not keeping any persistent reference to the Ctrl() instance you are creating. This results in python garbage collecting it as soon as the instance is created.

要解决这个问题，只需将其分配给一个变量:

To solve the issue, just assign it to a variable:

def main():
    """Main function."""
    # Create an instance of `QApplication`
    app = QApplication(sys.argv)
    # Show the app's GUI
    view = UI()
    view.show()
    setup = {}
    # Create instance of the controller
    ctrl = Ctrl(setup=setup, view=view)
    # Execute app's main loop
    sys.exit(app.exec_())

一些注意事项:

• 虽然将逻辑与界面分离通常是一种很好的做法，但这是一个需要谨慎使用的概念，因为有时它只会使事情变得比应有的复杂.大多数时候(尤其是简单的程序)，它只会产生更大的代码库而没有带来任何实际好处:更难阅读和调试，而且您可能最终会不断地从逻辑部分和用户界面部分切换代码；
• 您的代码显示了该概念的缺点之一:当您创建文件对话框时，您使用的是 self，但在这种情况下，它指的是 Ctrl实例，而参数应该是 UI 实例而不是(这将导致崩溃，因为 Qt 将获得意外的参数类型)；您可以使用 self._view 代替，但是，如上所述，在这种情况下，整个分离只会使事情变得不必要地复杂；
• 将字符串用于引用内部对象的字典键很少是一个好主意(尤其是像您那样使用长描述性字符串时)；
• 当从一个模块导入多个元素时，通常最好将它们分组而不是使用单行导入:它使代码更整洁，更易于阅读和检查:from PyQt5.QtWidgets import (QApplication, QMainWindow, QPushButton, QVBoxLayout, QWidget, QFileDialog)

• while separating logic from interface is usually good practice, it's a concept that needs to be used with care, as sometimes it only makes things more complex than they should be. Most of the times (especially with simple programs), it only makes a bigger codebase without giving any actual benefit: it's harder to read and to debug, and you'll probably end up continuously switching from the logic parts and the ui parts of your code;
• your code shows one of the drawback of that concept: when you create the file dialog, you're using self, but in that case it refers to the Ctrl instance, while the argument should be the UI instance instead (which will result in a crash, as Qt will get an unexpected argument type); you can use self._view instead, but, as said, the whole separation in this case just makes things unnecessarily complex;
• using strings for dictionary keys that refer to internal objects is rarely a good idea (especially when using long descriptive strings like you did);
• when importing more than one element from a module, it's usually better to group them instead of using single line imports: it makes the code tidier and easier to read and inspect: from PyQt5.QtWidgets import (QApplication, QMainWindow, QPushButton, QVBoxLayout, QWidget, QFileDialog)

这篇关于PyQt5 按钮未连接的文章就介绍到这了，希望我们推荐的答案对大家有所帮助，也希望大家多多支持IT屋！