Почему не привязываются функции на кнопки на создаваемом виджете из моего класса?

Question

[TemaKam]

при нажатии кнопки вызываю функцию класса (из другого модуля), которая возвращает форму (с кнопками и т.д.)
все действия, которые прописаны в __init__ выполняются, а когда попытался в этом классе привязать функцию к кнопке - то на создаваемой форме эта функция уже не работает
почему так?

если кратко код, то примерно так:
модуль 1.py

from 2.py import Task
...
self.tasks.addTab(Task(параметры __init__).new_task(), self.task_name.toPlainText())

модуль 2.py:

class Task:
    def __init__(self, параметры):
        super().__init__()
        self.tab_2 = QtWidgets.QTabWidget()
        self.tab_2.setObjectName("tab_2")
        # кнопка для примера
        self.start_task = QtWidgets.QPushButton(self.tab_2)
        self.start_task.setGeometry(QtCore.QRect(300, 510, 171, 23))
        font = QtGui.QFont()
        font.setPointSize(9)
        self.start_task.setFont(font)
        self.start_task.setObjectName("start_task")

        self.start_task.clicked.connect(self.test)

    def new_task(self):
        return self.tab_2

    def test(self):
        print('111')

вот и получается, что форма с кнопкой создаётся (ну и другие объекты там еще), а действие по нажатию кнопки не происходит

Answer 1

[bbkmzzzz]

class Task:
    def __init__(self, параметры):
        super().__init__()

От чего наследован класс? Как запускается приложение?
clicked сигнал испускается при нажатии на кнопку. Сигналы могут испускать и принимать только объекты-наследники QObject и только в основном цикле событий. Task таковым не является.

Либо дополняем код, либо идем читать про Qt.

Comments

[TemaKam]

ни от чего, просто в другом файле объявил его отдельно
запускается из первого модуля:

class ExampleApp(QtWidgets.QMainWindow, test_ui.Ui_MainWindow):
    def __init__(self):
        super().__init__()
        self.setupUi(self)
...
def main():
    app = QtWidgets.QApplication(sys.argv)
    window = ExampleApp()
    window.show()
    app.exec_() 


if __name__ == '__main__': 
    main()

нужно и во втором модуле сделать class Task(QtWidgets.QMainWindow) или как?

[bbkmzzzz]

В процессе выполнения нет "файлов" и "модулей", условно говоря. Все, что импортируется - добавляется в то пространство имен куда импортируется, как будто оно там всегда и было.

class ExampleApp(QtWidgets.QMainWindow, test_ui.Ui_MainWindow):

Не надо наследоваться от двух классов, если точно не знаете зачем это требуется. Отделите мух от котлет, логику от визуала.

from test_ui import Ui_MainWindow as Main_Ui
class ExampleApp(QtWidgets.QMainWindow):
    def __init__(self):
        super().__init__()
        self.ui = Main_Ui() # все визуальные элементы теперь будут тут
        self.ui.setupUi(self)

Сигналы могут испускать и принимать только объекты-наследники QObject и только в основном цикле событий. Task таковым не является.

Да, Taks должен быть наследником QObject.

Task(QtWidgets.QMainWindow) или как?

QMainWindow - главное окно, обычно оно одно. Можете наследовать практически от любого класса Qt, за небольшим исключением.

[TemaKam]

bbkmzzzz, это делал просто из примера в интернете, так и осталось
я попробовал и Task(QtWidgets.QMainWindow) и Task(QtWidgets.QTabWidget), все равно не работает функция по кнопке

[bbkmzzzz]

Да как так-то? Должно же работать!!11 Надо еще как-нибудь попробовать. Гадать на коде из интернета, почему не работает другой код - это круто, конечно. Либо код, который работает не так, как от него хочется, либо будут общие фразы о том, как должно быть организовано все, что бы работало. Воспроизводимый и запускающийся код, который я могу запустить на своей машине.

[TemaKam]

bbkmzzzz, ну вы же сказали, что класс должен быть наследником класса Qt
в исходном классе ExampleApp работают же все функции на кнопках, а тут нет даже с наследованием
нужно еще что-то предоставить?

[bbkmzzzz]

, а тут нет даже с наследованием нужно еще что-то предоставить?

тут это где? там? там нет наследования. Или есть. но не там?

Скинуть тот код, именно тот, в котором непонятки. Не на котором он основан, а тот с которым проблемы.

[TemaKam]

bbkmzzzz,

неактуально

бред какой-то, сейчас вырезал все чтобы самое главное оставить, теперь другая ошибка

вот код 1.py

spoiler

import sys
from PyQt5 import QtWidgets, QtGui, QtCore, Qt
from PyQt5.QtGui import QIcon
import ui_1
from task import Task


class ExampleApp(QtWidgets.QMainWindow, ui_1.Ui_MainWindow):
    def __init__(self):
        super().__init__()
        self.setupUi(self)
        self.add_task.clicked.connect(self.add_task_f)

    # добавление задачи
    def add_task_f(self):
        self.suppliers_and_add_tasks.addTab(Task().new_task(), self.task_name.toPlainText())


def main():
    app = QtWidgets.QApplication(sys.argv)
    window = ExampleApp()
    window.show()
    app.exec_()


if __name__ == '__main__':
    main()

ui_1.py

spoiler

from PyQt5 import QtCore, QtGui, QtWidgets


class Ui_MainWindow(object):
    def setupUi(self, MainWindow):
        MainWindow.setObjectName("MainWindow")
        MainWindow.resize(1479, 781)
        sizePolicy = QtWidgets.QSizePolicy(QtWidgets.QSizePolicy.Fixed, QtWidgets.QSizePolicy.Fixed)
        sizePolicy.setHorizontalStretch(0)
        sizePolicy.setVerticalStretch(0)
        sizePolicy.setHeightForWidth(MainWindow.sizePolicy().hasHeightForWidth())
        MainWindow.setSizePolicy(sizePolicy)
        MainWindow.setMinimumSize(QtCore.QSize(1479, 781))
        MainWindow.setMaximumSize(QtCore.QSize(1479, 781))
        font = QtGui.QFont()
        font.setPointSize(9)
        MainWindow.setFont(font)
        self.centralwidget = QtWidgets.QWidget(MainWindow)
        sizePolicy = QtWidgets.QSizePolicy(QtWidgets.QSizePolicy.Fixed, QtWidgets.QSizePolicy.Fixed)
        sizePolicy.setHorizontalStretch(0)
        sizePolicy.setVerticalStretch(0)
        sizePolicy.setHeightForWidth(self.centralwidget.sizePolicy().hasHeightForWidth())
        self.centralwidget.setSizePolicy(sizePolicy)
        self.centralwidget.setMinimumSize(QtCore.QSize(1479, 781))
        self.centralwidget.setMaximumSize(QtCore.QSize(1479, 781))
        self.centralwidget.setObjectName("centralwidget")
        self.all_tabs = QtWidgets.QTabWidget(self.centralwidget)
        self.all_tabs.setGeometry(QtCore.QRect(0, 0, 1479, 781))
        sizePolicy = QtWidgets.QSizePolicy(QtWidgets.QSizePolicy.Fixed, QtWidgets.QSizePolicy.Fixed)
        sizePolicy.setHorizontalStretch(0)
        sizePolicy.setVerticalStretch(0)
        sizePolicy.setHeightForWidth(self.all_tabs.sizePolicy().hasHeightForWidth())
        self.all_tabs.setSizePolicy(sizePolicy)
        self.all_tabs.setMinimumSize(QtCore.QSize(1479, 781))
        self.all_tabs.setMaximumSize(QtCore.QSize(1479, 781))
        self.all_tabs.setObjectName("all_tabs")
        self.all = QtWidgets.QWidget()
        self.all.setObjectName("all")
        self.supliers_and_tasks = QtWidgets.QWidget()
        self.supliers_and_tasks.setObjectName("supliers_and_tasks")
        self.suppliers_and_add_tasks = QtWidgets.QTabWidget(self.supliers_and_tasks)
        self.suppliers_and_add_tasks.setGeometry(QtCore.QRect(0, 0, 1479, 761))
        sizePolicy = QtWidgets.QSizePolicy(QtWidgets.QSizePolicy.Fixed, QtWidgets.QSizePolicy.Fixed)
        sizePolicy.setHorizontalStretch(0)
        sizePolicy.setVerticalStretch(0)
        sizePolicy.setHeightForWidth(self.suppliers_and_add_tasks.sizePolicy().hasHeightForWidth())
        self.suppliers_and_add_tasks.setSizePolicy(sizePolicy)
        self.suppliers_and_add_tasks.setMinimumSize(QtCore.QSize(1479, 761))
        self.suppliers_and_add_tasks.setMaximumSize(QtCore.QSize(1479, 761))
        self.suppliers_and_add_tasks.setObjectName("suppliers_and_add_tasks")
        self.tab = QtWidgets.QWidget()
        self.tab.setObjectName("tab")
        self.suppliers_and_add_tasks.addTab(self.tab, "")
        self.all_tabs.addTab(self.supliers_and_tasks, "")
        MainWindow.setCentralWidget(self.centralwidget)
        self.groupBox_7 = QtWidgets.QGroupBox(self.tab)
        self.groupBox_7.setGeometry(QtCore.QRect(670, 10, 471, 511))
        self.groupBox_7.setObjectName("groupBox_7")
        self.task_name = QtWidgets.QTextBrowser(self.groupBox_7)
        self.task_name.setGeometry(QtCore.QRect(80, 60, 151, 25))
        self.task_name.setLineWrapMode(QtWidgets.QTextEdit.WidgetWidth)
        self.task_name.setReadOnly(False)
        self.task_name.setAcceptRichText(False)
        self.task_name.setObjectName("task_name")
        self.add_task = QtWidgets.QPushButton(self.tab)
        self.add_task.setGeometry(QtCore.QRect(750, 530, 121, 23))
        self.add_task.setObjectName("add_task")

task.py

spoiler

from PyQt5 import QtWidgets, QtGui, QtCore, Qt


class Task:
    def __init__(self):
        super().__init__()
        self.tab_2 = QtWidgets.QTabWidget()
        self.tab_2.setObjectName("tab_2")
        self.start_task = QtWidgets.QPushButton(self.tab_2)
        self.start_task.setGeometry(QtCore.QRect(300, 510, 171, 23))
        font = QtGui.QFont()
        font.setPointSize(9)
        self.start_task.setFont(font)
        self.start_task.setObjectName("start_task")
        self.start_task.clicked.connect(self.test)

    def new_task(self):
        return self.tab_2

    def test(self):
        print('111')

неактуально

и вот теперь почему-то в 1.py при вызове функции класса требует self, и с ним тоже не работает

TypeError: new_task() missing 1 required positional argument: 'self'

Вот, хотя работает в текущем коде
будет создаваться новая вкладка по кнопке с названием из поля, и на созданной вкладке еще 1 кнопка с функцией, которая не работает у меня в текущем коде

spoiler

[TemaKam]

TemaKam, а, тут если в 1.py сделать именно addTab(Task().new_task(),...), то все нормально, создаётся
видимо перепутал что-то

ну значит это весь код, вкладка создается и не работает кнопка на ней

1

и вот эта кнопка не работает из класса

2

P.S.
графический интерфейс компилировал pyuic5

[bbkmzzzz]

TemaKam, уже лучше.
Поехали.
Ремарка: функция класса называется метод класса. Метод отличает от обычное функции то, что он находится в пространстве имен класса и первым аргументом всегда принимает экземпляр класса, тот самый self

TypeError: new_task() missing 1 required positional argument: 'self'

Абсолютно все верно, так как:

self.suppliers_and_add_tasks.addTab(Task.new_task(), self.task_name.toPlainText())

Task.new_task()
метод new_task вызывается не у экземпляра, а у самого класса, соответственно, требуется self
Ощущаем разницу.

class Foo:
    def bar(self):
        print('bar')

x = Foo()
x.bar()

Foo.bar()
<spoiler title="доп">

далее, если все-таки надо вызывать метод именно класса, тогда нужен декоратор @staticmethod, и будет выглядеть так:

class Foo:
    @staticmethod
    def bar():
        print('bar')

x = Foo()
x.bar()

Foo.bar()

elf.suppliers_and_add_tasks.addTab(Task.new_task(), self.task_name.toPlainText())

- экземпляр не создается. В этом одна часть проблемы

def add_task_f(self):
        created_new_task = Task()  # создаем экземпляр класса Task и присваиваем его имени created_new_task 
        self.tasks.append(created_new_task ) # сохраняем ссылку на экземпляр путем добавления в список tasks основного класса.
        #Можно не делать этого, но надо учитывать, что после выхода из метода, все переменные удаляться.
        # Qt заберет владение этими объектами, но вытаскивать их потом тяжело.
        # я подразумеваю, что self.tasks - создан и существует

        # теперь тут мы вызываем метод new_task не класса, а экземпляра класса
        self.suppliers_and_add_tasks.addTab(created_new_task .new_task(), self.task_name.toPlainText())

Далее.
С классом Task все нормально, проблема изначально была не в нем. Наследоваться в данном случае не обязательно, так как сам класс Task никак с сигналами не связан.
super - не нужен, так как он возвращает родительский класс. Ни от чего не наследуемся - super не нужен. (кстати, по умолчанию все объекты унаследованы от object, но это автоматически и скрыто)

Запускаемся, проверяем

spoiler

import sys
from PyQt5 import QtWidgets, QtGui, QtCore
from PyQt5.QtGui import QIcon

class Task:
    def __init__(self):
        super().__init__()
        self.tab_2 = QtWidgets.QTabWidget()
        self.tab_2.setObjectName("tab_2")
        self.start_task = QtWidgets.QPushButton(self.tab_2)
        self.start_task.setGeometry(QtCore.QRect(300, 510, 171, 23))
        font = QtGui.QFont()
        font.setPointSize(9)
        self.start_task.setFont(font)
        self.start_task.setObjectName("start_task")
        self.start_task.clicked.connect(self.test)

    def new_task(self):
        return self.tab_2

    def test(self):
        print('111')


class Ui_MainWindow(object):
    def setupUi(self, MainWindow):
        MainWindow.setObjectName("MainWindow")
        MainWindow.resize(1479, 781)
        sizePolicy = QtWidgets.QSizePolicy(QtWidgets.QSizePolicy.Fixed, QtWidgets.QSizePolicy.Fixed)
        sizePolicy.setHorizontalStretch(0)
        sizePolicy.setVerticalStretch(0)
        sizePolicy.setHeightForWidth(MainWindow.sizePolicy().hasHeightForWidth())
        MainWindow.setSizePolicy(sizePolicy)
        MainWindow.setMinimumSize(QtCore.QSize(1479, 781))
        MainWindow.setMaximumSize(QtCore.QSize(1479, 781))
        font = QtGui.QFont()
        font.setPointSize(9)
        MainWindow.setFont(font)
        self.centralwidget = QtWidgets.QWidget(MainWindow)
        sizePolicy = QtWidgets.QSizePolicy(QtWidgets.QSizePolicy.Fixed, QtWidgets.QSizePolicy.Fixed)
        sizePolicy.setHorizontalStretch(0)
        sizePolicy.setVerticalStretch(0)
        sizePolicy.setHeightForWidth(self.centralwidget.sizePolicy().hasHeightForWidth())
        self.centralwidget.setSizePolicy(sizePolicy)
        self.centralwidget.setMinimumSize(QtCore.QSize(1479, 781))
        self.centralwidget.setMaximumSize(QtCore.QSize(1479, 781))
        self.centralwidget.setObjectName("centralwidget")
        self.all_tabs = QtWidgets.QTabWidget(self.centralwidget)
        self.all_tabs.setGeometry(QtCore.QRect(0, 0, 1479, 781))
        sizePolicy = QtWidgets.QSizePolicy(QtWidgets.QSizePolicy.Fixed, QtWidgets.QSizePolicy.Fixed)
        sizePolicy.setHorizontalStretch(0)
        sizePolicy.setVerticalStretch(0)
        sizePolicy.setHeightForWidth(self.all_tabs.sizePolicy().hasHeightForWidth())
        self.all_tabs.setSizePolicy(sizePolicy)
        self.all_tabs.setMinimumSize(QtCore.QSize(1479, 781))
        self.all_tabs.setMaximumSize(QtCore.QSize(1479, 781))
        self.all_tabs.setObjectName("all_tabs")
        self.all = QtWidgets.QWidget()
        self.all.setObjectName("all")
        self.supliers_and_tasks = QtWidgets.QWidget()
        self.supliers_and_tasks.setObjectName("supliers_and_tasks")
        self.suppliers_and_add_tasks = QtWidgets.QTabWidget(self.supliers_and_tasks)
        self.suppliers_and_add_tasks.setGeometry(QtCore.QRect(0, 0, 1479, 761))
        sizePolicy = QtWidgets.QSizePolicy(QtWidgets.QSizePolicy.Fixed, QtWidgets.QSizePolicy.Fixed)
        sizePolicy.setHorizontalStretch(0)
        sizePolicy.setVerticalStretch(0)
        sizePolicy.setHeightForWidth(self.suppliers_and_add_tasks.sizePolicy().hasHeightForWidth())
        self.suppliers_and_add_tasks.setSizePolicy(sizePolicy)
        self.suppliers_and_add_tasks.setMinimumSize(QtCore.QSize(1479, 761))
        self.suppliers_and_add_tasks.setMaximumSize(QtCore.QSize(1479, 761))
        self.suppliers_and_add_tasks.setObjectName("suppliers_and_add_tasks")
        self.tab = QtWidgets.QWidget()
        self.tab.setObjectName("tab")
        self.suppliers_and_add_tasks.addTab(self.tab, "")
        self.all_tabs.addTab(self.supliers_and_tasks, "")
        MainWindow.setCentralWidget(self.centralwidget)
        self.groupBox_7 = QtWidgets.QGroupBox(self.tab)
        self.groupBox_7.setGeometry(QtCore.QRect(670, 10, 471, 511))
        self.groupBox_7.setObjectName("groupBox_7")
        self.task_name = QtWidgets.QTextBrowser(self.groupBox_7)
        self.task_name.setGeometry(QtCore.QRect(80, 60, 151, 25))
        self.task_name.setLineWrapMode(QtWidgets.QTextEdit.WidgetWidth)
        self.task_name.setReadOnly(False)
        self.task_name.setAcceptRichText(False)
        self.task_name.setObjectName("task_name")
        self.add_task = QtWidgets.QPushButton(self.tab)
        self.add_task.setGeometry(QtCore.QRect(750, 530, 121, 23))
        self.add_task.setObjectName("add_task")





class ExampleApp(QtWidgets.QMainWindow):
    def __init__(self):
        super(ExampleApp, self).__init__()
        self.ui = Ui_MainWindow()

        self.ui.setupUi(self)
        self.ui.add_task.clicked.connect(self.add_task_f)
        self.tasks = []


    # добавление задачи
    def add_task_f(self):
        print('add_task_f')
        created_new_task = Task()
        self.tasks.append(created_new_task)
        
        
        self.ui.suppliers_and_add_tasks.addTab(created_new_task.new_task(), self.ui.task_name.toPlainText())
        

def main():
    app = QtWidgets.QApplication(sys.argv)
    window = ExampleApp()
    window.show()
    app.exec_()


if __name__ == '__main__':
    main()

[TemaKam]

bbkmzzzz, спасибо, с методами понятно
но я не вижу особо отличий от моего кода, объект создавал и присваивал переменной отдельно тоже - но результат тот же, не работала кнопка
вкладки то у меня создаются, но не работает кнопка на них, то есть не выводится '111'

это может быть из-за того, что по разным файлам распределено?
или из-за того, что не сохраняю в список объект?

[TemaKam]

TemaKam, ага, второе
закомментировал строку self.tasks.append(created_new_task)
и перестало работать

а как можно удалить объект из этого списка потом (именно саму ссылку на объект типо закрыть)?

[bbkmzzzz]

отличия тут.

self.suppliers_and_add_tasks.addTab(Task.new_task(), self.task_name.toPlainText())
self.suppliers_and_add_tasks.addTab(Task().new_task(), self.task_name.toPlainText())

вызов метода класса и вызов метода экземпляра класса

[TemaKam]

bbkmzzzz, ну это то я исправил уже давно, я про функцию на созданной вкладке, которая 111 выводит
почему обязательно нужно сохранять куда-то создаваемый объект?

[bbkmzzzz]

1 - потому что сборка мусора. Если на объект никто не ссылается - он будет удален.
2 - в дальнейшем понадобится удалить эти данные, убрать их из интерфейса. Как? Прямых ссылок нет, их не сохранили.

[TemaKam]

bbkmzzzz, понятно, значит буду сохранять
последний вопрос, если можно, как потом при удалении вкладки удалять этот объект (из памяти)
взять элемент из этого списка по имени, а потом?

[TemaKam]

TemaKam, а стоп, имя же вкладки не сохраняется в объекте (ну и в списке соответственно)
если удалю вкладку - объект ведь так и останется?
ничего страшного?

[bbkmzzzz]

продумать зависимости, что и где может ссылаться на него. Возможно, некая сущность, менеджер, которая управляет этими данными, создает их, возвращает, занимается удалением. Больше никто, только он.

или просто извлечь из списка вникуда

lst = [123, '123']
print(lst)
lst.pop(0)
print(lst)

[TemaKam]

bbkmzzzz, ну вот с pop может прокатит, надо индексы задач и списка может сопоставить
спасибо большое

[bbkmzzzz]

TemaKam,

а стоп, имя же вкладки не сохраняется в объекте (ну и в списке соответственно)
если удалю вкладку - объект ведь так и останется?
ничего страшного?

какой именно объект? Task? ссылка на него останется в списке, до тех пор, пока существует хоть одна ссылка ничего удалено не будет. Нет смысле пока заморачиваться с памятью, рано, пока это имеет смысл только для удобства работы с данными. Может, захочется сделать возможность "скрывать" вкладку, и тогда ту часть, которая отбражается нужно скрыть, вкладку удалить, а данные оставить. Ну а потом, когда-нибудь восстановить. А вся информация все еще есть и доступна.
Или сохранять данные из вкладки в файл. вкладку удалили, запустили процесс сохранения информации, удалили данные.

[bbkmzzzz]

TemaKam, словарь например. Заведите в Task поле для имени вкладки, ищите потом по имени. Но это наложит ограничение на уникальность имен. Вариантов масса)

[TemaKam]

bbkmzzzz, да вкладки скрывать не надо будет, только удалять
ну проще сопоставить индексы списка и задач, и при удалении вкладки (задачи) извлекать элемент из списка

[bbkmzzzz]

TemaKam,

да вкладки скрывать не надо будет, только удалять

ну-ну)

640кб хватит всем! © Ленин

[TemaKam]

bbkmzzzz, да точно)