使用for循环为QScrollbars的valueChanged信号分配方法似乎不太合适

Question

我正在尝试使用for循环为QScrollbars的valueChanged信号分配方法，以使代码更干净。然而，这似乎并不正确。我做错什么了

import sys
from PyQt5.QtWidgets import QScrollBar, QDialog, QVBoxLayout, QApplication


class MainWindow(QDialog):
    def __init__(self):
        super().__init__()
        self.layout = QVBoxLayout(self)
        self.scrollbar1 = QScrollBar(self)
        self.scrollbar2 = QScrollBar(self)
        self.scrollbars = [self.scrollbar1, self.scrollbar2]
        self.names = ['scrollbar 1', 'scrollbar 2']
        self.layout.addWidget(self.scrollbar1)
        self.layout.addWidget(self.scrollbar2)
        for scrollbar, name in zip(self.scrollbars, self.names):
            print(scrollbar, name)
            scrollbar.valueChanged.connect(lambda: self.test(name))

        # self.scrollbar1.valueChanged.connect(
        #     lambda: self.test(self.names[0])
        # )
        # self.scrollbar2.valueChanged.connect(
        #     lambda: self.test(self.names[1])
        # )

        self.show()

    def test(self, scrollbar):
        print(scrollbar)


if __name__ == '__main__':
    app = QApplication(sys.argv)
    GUI = MainWindow()
    sys.exit(app.exec_())

“手动”分配方法按预期工作，即传递不同的名称。但是，当使用for循环时，对于两个滚动条，值更改时会打印相同的名称

编辑：这是我的快照滑块方法

旧的：

def snap_slider(scrollbar):
    x = np.modf(scrollbar.value() / scrollbar.singleStep())
    if x[0] < 0.5:
        scrollbar.setSliderPosition(
            int(x[1] * scrollbar.singleStep()))
    else:
        scrollbar.setSliderPosition(
            int((x[1]+1) * scrollbar.singleStep()))

新增：

def snap_slider(self):
    x = np.modf(self.sender().value() / self.sender().singleStep())
    if x[0] < 0.5:
        self.sender().setSliderPosition(
            int(x[1] * self.sender().singleStep()))
    else:
        self.sender().setSliderPosition(
            int((x[1] + 1) * self.sender().singleStep()))

Answer 1

这里有几点，因为您正在努力使代码更干净：

• 您应该更喜欢使用sender()方法而不是lambda函数
• 最好将UI设置隔离在一个单独的函数中，您可以在其他地方调用该函数
• 关于show()方法也一样：避免将它放在__init__方法中，因为您可能需要实例化一个MainWindow而不显示它（例如：出于测试目的）

这会产生如下结果：

import sys
from PyQt5.QtWidgets import QScrollBar, QDialog, QVBoxLayout, QApplication

class MainWindow(QDialog):
    def __init__(self):
        super().__init__()
        self.createWidgets()

    def createWidgets(self):
        self.layout = QVBoxLayout(self)

        self.scrollbar1 = QScrollBar(self)
        self.scrollbar2 = QScrollBar(self)

        for widget in [self.scrollbar1, self.scrollbar2]:
            widget.valueChanged.connect(self.test)
            self.layout.addWidget(widget)

    def test(self, event):
        print(self.sender())


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