Question

我使用pygame绘制了Particle和Explosion类爆炸代表一堆飞行的粒子。粒子最终消失;当它的ttl属性变为0时，它不应该是可见的，应该从Explosion.particles中删除。我希望程序删除死粒子，以便它们不会更新。

我遇到的问题是使用Explosion.update()方法。它似乎没有删除任何粒子。我正在经历一个据称已经死亡的错误＆＃39;粒子仍然被绘制，但它们的移动没有更新。我已经在python的列表中进行了实验，并验证了迭代“死亡”的前提。要从其他列表中删除的列表有效。
对于此代码中的错误位置的任何建议将不胜感激。

编辑：我已经附加并缩短了两个源文件以获得更好的上下文。

explosion.py

import sys
import pygame
from particleac import *

class App:
    def __init__(self):
        pygame.init()
        self.screen = pygame.display.set_mode(SCREEN_SIZE, pygame.HWSURFACE|pygame.DOUBLEBUF)                    
        self.clock = pygame.time.Clock()

        self.running = True
        self.explosions = []
        self.frame_no = 0

    def check_input(self):
        for event in pygame.event.get():
            if event.type == pygame.QUIT:
                self.running = False
            if event.type == pygame.MOUSEBUTTONDOWN or event.type == pygame.MOUSEMOTION:
                x, y = pygame.mouse.get_pos()
                self.explosions.append(Explosion(x, y))

    def update_screen(self):
        self.screen.fill(BLACK)
        dead_explosions = []
        # remove explosion if particles all gone
        for e in self.explosions:
            if len(e.particles) == 0:
                dead_explosions.append(e)
            else:
                for p in e.particles:
                    pygame.draw.circle(self.screen, p.colour, [p.x, p.y], p.size)
                    p.update()

        for e in dead_explosions:
            self.explosions.remove(e)

        pygame.display.flip()
        self.frame_no += 1
        self.frame_no %= 60
        self.clock.tick(FRAMERATE)

    def run(self):
        while self.running:
            self.check_input()
            self.update_screen()

        pygame.quit()       

app = App()
app.run()

particleac.py

import random
import sys

BLACK = [  0,   0,   0]
WHITE = [255, 255, 255]
BLUE = [0, 0, 255]
SCREEN_WIDTH = 600
SCREEN_HEIGHT = 400
SCREEN_CENTRE = [SCREEN_WIDTH/2, SCREEN_HEIGHT/2]
FRAMERATE = 40
SCREEN_SIZE = [SCREEN_WIDTH, SCREEN_HEIGHT]

GRAVITY = 1
TERMINAL_VELOCITY = 10

class Particle:
    def __init__(self, x=SCREEN_CENTRE[0], y=SCREEN_CENTRE[1], colour=WHITE):
        self.x = x
        self.y = y
        self.colour = colour
        self.brightness = 255
        self.size = 2
        self.x_velocity = random.randrange(-4, 4)
        self.y_velocity = random.randrange(-8, 3)
        self.ttl = random.randrange(50, 200)
        self.decay = (self.ttl / FRAMERATE) * 2

    def update(self):            
        if self.ttl > 0:
            self.ttl -= 1

            self.brightness -= self.decay
            if self.brightness < 0:
                self.brightness = 0

            self.colour[0] = self.colour[1] = self.colour[2] = self.brightness 

            self.y_velocity += GRAVITY
            self.y += self.y_velocity
            self.x += self.x_velocity

            if self.y > SCREEN_HEIGHT:
                self.y -= SCREEN_HEIGHT

class Explosion:
    MAX_NUM_PARTICLES = 2

    def __init__(self, x=0, y=0, colour=WHITE):
        self.x = x
        self.y = y
        self.colour = colour
        self.x_velocity = random.randrange(-4, 4)
        self.y_velocity = random.randrange(-6, -1)
        self.num_particles = random.randrange(1, self.MAX_NUM_PARTICLES)
        self.particles = []

        for i in range(self.MAX_NUM_PARTICLES):
            p = Particle(self.x, self.y)
            p.colour = self.colour
            p.x_velocity += self.x_velocity
            p.y_velocity += self.y_velocity

            self.particles.append(p)

    def update(self):
        for p in self.particles:
            p.update()
        self.particles = [p for p in self.particles if p.ttl > 0]
        sys.stdout.write("len(self.particles) == {}".format(len(self.particles)))
        sys.stdout.flush()

Answer 1

问题可能来自self.particles在班级定义的事实。这可能会在您的代码中的其他地方导致一些问题，即您创建Explosion类的多个实例。尝试移动构造函数中的self.particle，看看会发生什么。

（作为建议）既然你建立了一个列表，为什么不复制那些活着的呢？

def update(self):
  particles_alive = []
  for p in self.particles:
    p.update()
    sys.out.write("p.ttl == {}\n".format(p.ttl))
    if p.ttl == 0:
      sys.out.write("Particle died.\n")
    else:
      particles_alive.append(p)
  self.particles = particles_alive

Answer 2

一旦sys.out.write()更改为sys.stdout.write()，您的代码就适合我。由于您的代码中存在错误，您确定帖子中的代码与失败的代码相同吗？

您可以将Explosion.update()简化为：

    def update(self):
        for p in self.particles:
            p.update()
        self.particles = [p for p in self.particles if p.ttl > 0]

可能这个更改可能会解决问题，因为您现在只处理一个列表，但我相信您的代码应该可以正常工作。

Answer 3

解决方案：我从未调用过e.update()！

因此，在迭代它们时爆炸没有更新，这意味着没有删除死亡的粒子。 Aaargh！

我将e.update()添加到app.update_screen()并删除了不正确的p.update()（e.update()中为每个粒子调用的那个）。

def update_screen(self):
    self.screen.fill(BLACK)
    dead_explosions = []
    for e in self.explosions:
        if len(e.particles) == 0:
            dead_explosions.append(e)
        else:
            e.update()
            for p in e.particles:
                pygame.draw.circle(self.screen, p.colour, [p.x, p.y], p.size)

    for e in dead_explosions:
        self.explosions.remove(e)

    pygame.display.flip()
    self.clock.tick(FRAMERATE)

我现在知道在发布代码时提供更好的上下文。谢谢你们麻烦的家伙们！

pygame particle：从列表中删除问题

3 个答案: