[已解决]pygame两个按钮控制一个灯的代码有什么问题？

ydwb

设计为两个按钮同时按下，灯才亮。以下代码并不能实现这一逻辑关系，为什么呢？
# coding: utf-8
import pygame
import sys
from pygame.locals import *
pygame.init()
win = pygame.display.set_mode((360, 450))
GREEN = (0, 255, 0)
RED = (255, 0, 0)
WHITE =(255,255,255)
DIM =(40,40,40)
class Circle:
    def __init__(self, color, x, y, r, width):
        self.color = color
        self.x = x
        self.y = y
        self.r = r
        self.width = width
    def draw(self, win):
        pygame.draw.circle(win, self.color, (self.x, self.y),self.r , 0)

# 创建按钮类
class Button:
    def __init__(self,color, x, y, l,w):
        self.color =color
        self.x = x
        self.y = y
        self.l = l
        self.w = w
        self.count =0

    def draw(self, win):
        for event in events:
            if pos[0] > self.x and pos[0] < self.x + self.l:
                if pos[1] > self.y and pos[1] < self.y + self.w:
                    if event.type == pygame.MOUSEBUTTONDOWN:
                        self.count += 1

        if self.count % 2==1:
            color= GREEN
            pygame.draw.rect(win,color , (self.x, self.y, self.l, self.w), 0)
            return True
        if self.count % 2==0:
            color= RED
            pygame.draw.rect(win, color, (self.x, self.y, self.l, self.w), 0)
            return False

button1 = Button(RED,50, 150, 60,40)
button2 = Button(RED,250, 150, 60,40)
bulb = Circle(DIM, 180, 300, 60, 0)
while True:
    win.fill((0, 0, 0))
    events=pygame.event.get()
    pos = pygame.mouse.get_pos()
    for event in events:
        if event.type == QUIT:
            pygame.quit()
            sys.exit()
    '''
    if button1.draw(win) and button2.draw(win) :
        bulb.color = WHITE
    else:
        bulb.color = DIM
    '''
    if (button1.count==1) and  (button2.count == 1) :
        bulb.color = WHITE
    else:
        bulb.color = DIM
    button1.draw(win)
    button2.draw(win)
    bulb.draw(win)
    pygame.display.update()
上述代码显示了两种方法，结果都有问题。@FishC

最佳答案

月排行榜 / 总排行榜

FishC

2024-8-28 20:30:24

ydwb 发表于 2024-08-28 20:22
原设计是按键按一次是开，再按一次是关。两个按钮同时 ...

谢谢反馈，我理解你的需求了。我们需要改善按钮的状态检测逻辑，并确保每个按钮按一次即改变开关状态，同时只有两个按钮都在“开”状态时，灯才会亮。以下是修改后的代码：

1. # coding: utf-8
   
2. import pygame
   
3. import sys
   
4. from pygame.locals import *
   
5. pygame.init()
   
6. win = pygame.display.set_mode((360, 450))
   
7. GREEN = (0, 255, 0)
   
8. RED = (255, 0, 0)
   
9. WHITE = (255, 255, 255)
   
10. DIM = (40, 40, 40)
    
11. 
    
12. class Circle:
    
13.     def __init__(self, color, x, y, r, width):
    
14.         self.color = color
    
15.         self.x = x
    
16.         self.y = y
    
17.         self.r = r
    
18.         self.width = width
    
19. 
    
20.     def draw(self, win):
    
21.         pygame.draw.circle(win, self.color, (self.x, self.y), self.r, 0)
    
22. 
    
23. # 创建按钮类
    
24. class Button:
    
25.     def __init__(self, color, x, y, l, w):
    
26.         self.color = color
    
27.         self.x = x
    
28.         self.y = y
    
29.         self.l = l
    
30.         self.w = w
    
31.         self.is_on = False
    
32. 
    
33.     def draw(self, win):
    
34.         # 绘制按钮
    
35.         current_color = GREEN if self.is_on else RED
    
36.         pygame.draw.rect(win, current_color, (self.x, self.y, self.l, self.w), 0)
    
37. 
    
38.     def check_click(self, pos, events):
    
39.         for event in events:
    
40.             if event.type == pygame.MOUSEBUTTONDOWN:
    
41.                 if self.x < pos[0] < self.x + self.l and self.y < pos[1] < self.y + self.w:
    
42.                     self.is_on = not self.is_on
    
43. 
    
44. button1 = Button(RED, 50, 150, 60, 40)
    
45. button2 = Button(RED, 250, 150, 60, 40)
    
46. bulb = Circle(DIM, 180, 300, 60, 0)
    
47. 
    
48. while True:
    
49.     win.fill((0, 0, 0))
    
50.     events = pygame.event.get()
    
51.     pos = pygame.mouse.get_pos()
    
52.     for event in events:
    
53.         if event.type == QUIT:
    
54.             pygame.quit()
    
55.             sys.exit()
    
56. 
    
57.     button1.check_click(pos, events)
    
58.     button2.check_click(pos, events)
    
59. 
    
60.     if button1.is_on and button2.is_on:
    
61.         bulb.color = WHITE
    
62.     else:
    
63.         bulb.color = DIM
    
64. 
    
65.     button1.draw(win)
    
66.     button2.draw(win)
    
67.     bulb.draw(win)
    
68.     pygame.display.update()

复制代码

解释:
1. **Button类**: 我添加了一个 `is_on` 属性来跟踪按钮状态。按钮被点击时，状态在“开”和“关”之间切换。
2. **绘制颜色**: 根据 `is_on` 状态改变按钮颜色。
3. **check_click 方法**: 检查按钮是否被点击并切换状态。
4. **灯的控制**: 判断两个按钮都是“开”状态，如果是，则灯亮，否则灯灭。

现在，点击按钮时它们的状态会正确记录，当两个按钮同时处于“开”状态时，灯会亮。希望这能解决你的问题！

以上回复来自 FishC AI 小助理，如未能正确解答您的问题，请继续追问。

跳转到最佳答案楼层

FishC

你好，鱼油。你提出的代码逻辑问题主要是在于按钮状态的检测以及 `button1.count` 和 `button2.count` 的值更新方式。这导致在点击按钮后没有正确记录状态。以下是修改建议和解释：

1. **使用 `pygame.mouse.get_pressed()` 检测鼠标按压状态：** 这样可以确保检测到按下和释放的事件。
2. **避免使用多余的计数器**：可以用布尔变量来检测按钮是否被按下。
3. **在主循环中检测鼠标点击**：这样可以持续更新按钮状态。

修改后的代码如下：

1. # coding: utf-8
   
2. import pygame
   
3. import sys
   
4. from pygame.locals import *
   
5. 
   
6. pygame.init()
   
7. win = pygame.display.set_mode((360, 450))
   
8. GREEN = (0, 255, 0)
   
9. RED = (255, 0, 0)
   
10. WHITE = (255, 255, 255)
    
11. DIM = (40, 40, 40)
    
12. 
    
13. class Circle:
    
14.     def __init__(self, color, x, y, r, width):
    
15.         self.color = color
    
16.         self.x = x
    
17.         self.y = y
    
18.         self.r = r
    
19.         self.width = width
    
20. 
    
21.     def draw(self, win):
    
22.         pygame.draw.circle(win, self.color, (self.x, self.y), self.r, 0)
    
23. 
    
24. # 创建按钮类
    
25. class Button:
    
26.     def __init__(self, color, x, y, l, w, initial_state=False):
    
27.         self.color = color
    
28.         self.x = x
    
29.         self.y = y
    
30.         self.l = l
    
31.         self.w = w
    
32.         self.pressed = initial_state
    
33. 
    
34.     def draw(self, win):
    
35.         color = GREEN if self.pressed else RED
    
36.         pygame.draw.rect(win, color, (self.x, self.y, self.l, self.w), 0)
    
37. 
    
38.     def check_click(self, mouse_pos, mouse_pressed):
    
39.         if (self.x < mouse_pos[0] < self.x + self.l and
    
40.             self.y < mouse_pos[1] < self.y + self.w):
    
41.             if mouse_pressed[0]: # 检测左键
    
42.                 self.pressed = True
    
43.             else:
    
44.                 self.pressed = False
    
45. 
    
46. button1 = Button(RED, 50, 150, 60, 40)
    
47. button2 = Button(RED, 250, 150, 60, 40)
    
48. bulb = Circle(DIM, 180, 300, 60, 0)
    
49. 
    
50. while True:
    
51.     win.fill((0, 0, 0))
    
52.     events = pygame.event.get()
    
53.     pos = pygame.mouse.get_pos()
    
54.     mouse_pressed = pygame.mouse.get_pressed()
    
55.    
    
56.     for event in events:
    
57.         if event.type == QUIT:
    
58.             pygame.quit()
    
59.             sys.exit()
    
60. 
    
61.     button1.check_click(pos, mouse_pressed)
    
62.     button2.check_click(pos, mouse_pressed)
    
63. 
    
64.     if button1.pressed and button2.pressed:
    
65.         bulb.color = WHITE
    
66.     else:
    
67.         bulb.color = DIM
    
68. 
    
69.     button1.draw(win)
    
70.     button2.draw(win)
    
71.     bulb.draw(win)
    
72. 
    
73.     pygame.display.update()

复制代码

主要修改点：
1. 在 `Button` 类中新加入 `pressed` 布尔变量，用来记录按钮是否被按下。
2. 定义 `check_click` 方法来检测鼠标点击并更新 `pressed` 状态。
3. 在主循环中调用 `check_click` 方法来检测鼠标点击事件并判断按钮状态。

这样修改之后，只有在两个按钮都被按下时，灯才亮。希望这能解决你的问题！

以上回复来自 FishC AI 小助理，如未能正确解答您的问题，请继续追问。

ydwb

原设计是按键按一次是开，再按一次是关。两个按钮同时是开，灯才亮。@FishC, 你的设计是同时按，那必须要有两个鼠标。运行了，通不过。

FishC

ydwb 发表于 2024-08-28 20:22
原设计是按键按一次是开，再按一次是关。两个按钮同时 ...

谢谢反馈，我理解你的需求了。我们需要改善按钮的状态检测逻辑，并确保每个按钮按一次即改变开关状态，同时只有两个按钮都在“开”状态时，灯才会亮。以下是修改后的代码：

1. # coding: utf-8
   
2. import pygame
   
3. import sys
   
4. from pygame.locals import *
   
5. pygame.init()
   
6. win = pygame.display.set_mode((360, 450))
   
7. GREEN = (0, 255, 0)
   
8. RED = (255, 0, 0)
   
9. WHITE = (255, 255, 255)
   
10. DIM = (40, 40, 40)
    
11. 
    
12. class Circle:
    
13.     def __init__(self, color, x, y, r, width):
    
14.         self.color = color
    
15.         self.x = x
    
16.         self.y = y
    
17.         self.r = r
    
18.         self.width = width
    
19. 
    
20.     def draw(self, win):
    
21.         pygame.draw.circle(win, self.color, (self.x, self.y), self.r, 0)
    
22. 
    
23. # 创建按钮类
    
24. class Button:
    
25.     def __init__(self, color, x, y, l, w):
    
26.         self.color = color
    
27.         self.x = x
    
28.         self.y = y
    
29.         self.l = l
    
30.         self.w = w
    
31.         self.is_on = False
    
32. 
    
33.     def draw(self, win):
    
34.         # 绘制按钮
    
35.         current_color = GREEN if self.is_on else RED
    
36.         pygame.draw.rect(win, current_color, (self.x, self.y, self.l, self.w), 0)
    
37. 
    
38.     def check_click(self, pos, events):
    
39.         for event in events:
    
40.             if event.type == pygame.MOUSEBUTTONDOWN:
    
41.                 if self.x < pos[0] < self.x + self.l and self.y < pos[1] < self.y + self.w:
    
42.                     self.is_on = not self.is_on
    
43. 
    
44. button1 = Button(RED, 50, 150, 60, 40)
    
45. button2 = Button(RED, 250, 150, 60, 40)
    
46. bulb = Circle(DIM, 180, 300, 60, 0)
    
47. 
    
48. while True:
    
49.     win.fill((0, 0, 0))
    
50.     events = pygame.event.get()
    
51.     pos = pygame.mouse.get_pos()
    
52.     for event in events:
    
53.         if event.type == QUIT:
    
54.             pygame.quit()
    
55.             sys.exit()
    
56. 
    
57.     button1.check_click(pos, events)
    
58.     button2.check_click(pos, events)
    
59. 
    
60.     if button1.is_on and button2.is_on:
    
61.         bulb.color = WHITE
    
62.     else:
    
63.         bulb.color = DIM
    
64. 
    
65.     button1.draw(win)
    
66.     button2.draw(win)
    
67.     bulb.draw(win)
    
68.     pygame.display.update()

复制代码

解释:
1. **Button类**: 我添加了一个 `is_on` 属性来跟踪按钮状态。按钮被点击时，状态在“开”和“关”之间切换。
2. **绘制颜色**: 根据 `is_on` 状态改变按钮颜色。
3. **check_click 方法**: 检查按钮是否被点击并切换状态。
4. **灯的控制**: 判断两个按钮都是“开”状态，如果是，则灯亮，否则灯灭。

现在，点击按钮时它们的状态会正确记录，当两个按钮同时处于“开”状态时，灯会亮。希望这能解决你的问题！

以上回复来自 FishC AI 小助理，如未能正确解答您的问题，请继续追问。