Не работает метод button_click_check. Почему?

Question

[cntfrgthr]

Метод необходим для того, чтобы обрабатывать клики по кнопке и получать определенный callback. Вызывается в методе update, который в свою очередь вызывается в mainloop(while True).

Сейчас в качестве колбека печатается в консоль 1(так было легче проверить работоспособность)

ПРОБЛЕМА ТОЧНО ДОЛЖНА БЫТЬ В ООП, В ИСПОЛЬЗОВАНИИ БИБЛИОТЕЧНЫХ ФУНКЦИЙ ТОЧНО ВСЕ ВЕРНО

from settings import *
import pygame

pygame.init()



display = pygame.display.set_mode((width, height))
clock = pygame.time.Clock()
pygame.display.update()






class Interface:
    def button_click_check(self, button):
        for event in pygame.event.get():
            if event.type == pygame.MOUSEBUTTONDOWN:
                self.coords = pygame.mouse.get_pos()
                if (self.button[0] < self.coords[0] <= (self.button[0] + self.button[2])) & (self.button[1] < self.coords[1] <= (self.button[1] + self.button[3])):
                    print('1')
                else:
                    pygame.draw.rect(display, (0,0,255), self.button)

    def is_cursor_on_button(coords):
        if event.type == pygame.MOUSEMOTION:
                coords = pygame.mouse.get_pos()
                if (coords[0] < coords[0] <= (coords[0] + coords[2])) & (coords[1] < coords[1] <= (coords[1] + coords[3])):
                    pygame.draw.rect(display, (255,0,0), coords)
                else:
                    pygame.draw.rect(display, (0,0,255), coords)



class Button(Interface):
    def __init__(self, x, y, W, H):
        objects.append(self)
        self.x = x
        self.y = y 
        self.W = W
        self.H = H
        
    def get_coords(self):
        return (self.x, self.y, self.W, self.H)    
        #Interface.is_cursor_on_button(self.coords)

    def update(self):
        self.button_click_check(self.get_coords())

    def draw(self):
        pygame.draw.rect(display, (0,0,255), self.get_coords())       
        
objects = []
Button1 = Button(0,0,1920,1080) 


while True:
    for event in pygame.event.get():
        if event.type == pygame.QUIT:
            exit()

    for object in objects:
        object.update()    
    
    display.fill('black')    
    
    for object in objects:
        object.draw()

        
    pygame.display.update()    
    clock.tick(60)

Comments

[Vindicar]

1. Что значит "не работает"? Не работать программа может многими способами: выкидывать исключение; не выдавать ошибок, но вести себя не так, как ожидалось; прерывать программу; форматировать диск; поджигать твой дом и стрелять в твою собаку, и т.п.
2. Почему у этого метода параметр называется button, а ты при вызове передаёшь туда выхлоп метода под названием get_coords()? Ты уж реши, что он принимает: кнопку или координаты. И если на то пошло, зачем этот параметр, если ты его не используешь?
3. Ты обращаешься к свойствам self.coords и self.button, но они нигде не объявлены и не заданы.

[cntfrgthr]

Vindicar, программа запускается, метод не выдает ожидаемый результат.

[cntfrgthr]

Vindicar, self coords объявлен внутри метода

[cntfrgthr]

Vindicar, увидел что нигде не объявил self.button, исправил, но метод все еще не выдает ожидаемый результат

[shurshur]

cntfrgthr, я тоже ни фига не понимаю, что тут имелось в виду и какое поведение ожидалось.

В метод button_click_check передаётся button, но в его теле используется вместо него self.button.

Далее, в классе Interface используется self.button, который инициализируется только в его потомке Button. Что не соответствует элементарным принципам наследования. И вообще, интерфейс это интерфейс, а методы внутри него предполагают, что это уже кнопка, непорядок...

У Interface нет конкструктора - тоже плохо. По-хорошему все поля класса должны появиться после конструктора (self.coords, self.button - все поля класса, которые я тут вижу).

В button_click_check используется self.coords, хотя в реальности за пределами этого метода поле coords не нужно, поэтому имеет смысл использовать локальную переменную, а не поле класса, как в is_cursor_on_button.