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

Question

Я недавно начал делать Atari Breakout в Pygame. Я столкнулся с проблемой, которую не знаю, как исправить. Всякий раз, когда я запускаю свой код, и мяч вступает в контакт с блоком, блок, который сталкивается с ним, исчезает не так, как должен, а последний блок в списке. (Я составил список из 21 pygame.Surfaces, перебрал его и проколол каждую). Очевидно, что если мяч касается последнего блока из списка, правильный исчезает. Пожалуйста, вы можете мне помочь?

Это мой код:

import pygame, random, math

pygame.init()

screen = pygame.display.set_mode((800,600))
pygame.display.set_caption('Atari Breakout')

player = pygame.image.load('player copy.png')
ball = pygame.image.load('poland.png')
blocks = [pygame.image.load('block.png') for i in range(21)]


x, y = 25,25
blockx, blocky = [], []
for i in range(21):
    if i % 7 == 0:
        y += 52
        x = 25
    blockx.append(x)
    blocky.append(y)
    x += (50 + 64)

ballx, bally = 400, 300

balldx, balldy = 4,4

score = 0
score_font = pygame.font.Font('freesansbold.ttf', 32)

def show_score():
    score_text = score_font.render('Score : {}'.format(score), True, (255,255,255))
    screen.blit(score_text, (0,0))

def isCollision(x1,y1,x2,y2):
    ballRect = ball.get_rect(topleft = (x1, y1))
    playerRect = player.get_rect(topleft = (x2, y2))
    return ballRect.colliderect(playerRect)

def isCollision2(x1,y1,x2,y2):
    ballRect = ball.get_rect(topleft = (x1,y1))
    blockRect = blocks[i].get_rect(topleft = (x2,y2))
    return ballRect.colliderect(blockRect)

def blit_blocks():
    for i in range(len(blocks)):
        screen.blit(blocks[i], (blockx[i], blocky[i]))
running = True

while running:

    if score >= 21:
        running = False
    screen.fill((0,0,0))


    for event in pygame.event.get():
        if event.type == pygame.QUIT:
            running = False

    pos = pygame.mouse.get_pos()

    for i, block in enumerate(blocks):

        if isCollision2(ballx, bally, blockx[i], blocky[i]):
            balldy *= -1
            score += 1
            blocks.pop(i)

    if isCollision(ballx, bally, pos[0], 525):
        balldy *= -1
    if bally <= 0:
        balldy *= -1
    if bally >= 570:
        running = False
    if ballx <= 0:
        balldx *= -1
    if ballx >= 770:
        balldx *= -1

    ballx += balldx
    bally += balldy

    screen.blit(player, (pos[0], 525))
    screen.blit(ball, (ballx,bally))
    blit_blocks()
    show_score()
    pygame.display.update()

Answer 1

Что вы на самом деле делаете, так это манипулируете списком (pop), пока вы его повторяете Я рекомендую обходить копию списка (blocks[:]) и манипулировать исходным списком. Например:

i = 0
for x, y, block in zip(blockx[:], blocky[:], blocks[:]):

        if isCollision2(ballx, bally, x, y):
            blockx.pop(i)
            blocky.pop(i)
            blocks.pop(i)
            balldy *= -1
            score += 1
        else:
            i += 1

---

В любом случае, я рекомендую создать класс Block:

class Block():
    def __init__(self, image, x, y)
        self.image = image
        self.x = x
        self.y = y

image = pygame.image.load('block.png')
x, y = 25,25
blocks = []
for i in range(21)
    if i % 7 == 0:
        y += 52
        x = 25
    blocks.append(Block(image, x, y))
    x += (50 + 64)

i = 0
for block in block[:]:
    if isCollision2(ballx, bally, block.x, block.y):
        balldy *= -1
        score += 1
        block.pop(i)
    else:
        i += 1

Answer 2

Это плохая практика, если pop из списка, пока вы выполняете итерацию, это приведет к неожиданному поведению, здесь:

for i, block in enumerate(blocks):

        if isCollision2(ballx, bally, blockx[i], blocky[i]):
            balldy *= -1
            score += 1
            blocks.pop(i)

вы можете использовать:

new_blocks = []
for i, block in enumerate(blocks):

        if isCollision2(ballx, bally, blockx[i], blocky[i]):
            balldy *= -1
            score += 1
        else:
            new_blocks.append(block)

blocks = new_blocks