Использование цикла for для присвоения методов сигналам valueChanged QScrollbars, похоже, не работает

Question

Я пытаюсь использовать цикл for для назначения методов сигналам valueChanged QScrollbars, чтобы сделать мой код чище.Тем не менее, это, кажется, не работает правильно.Что я делаю не так?

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 для обеих полос прокрутки при изменении значения печатается одно и то же имя.

РЕДАКТИРОВАТЬ: вот мой метод snap_slider

old:

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 функции
• Рекомендуется изолировать настройку пользовательского интерфейса в отдельной функции, которую вы могли бы вызвать в другом месте
• То же самое в методе 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_())