Почему мой шаровой вектор сохраняет свои компоненты после повторного запуска?

Question

Я пишу игру Pong с Pygame, и это код моего класса Ball:

class Ball(pygame.sprite.Sprite):

    def __init__(self, game, vector=Vec2D.Vec2D()):
        super(Ball, self).__init__()

        self.image = pygame.Surface((BALL_RADIUS*2, BALL_RADIUS*2))
        self.rect = self.image.get_rect()
        self.__draw_ball()

        screen = pygame.display.get_surface()
        self.area = screen.get_rect().inflate(-GAP*2, 0)

        self.vector = vector
        self.game = game
        self.reinit()

    def __draw_ball(self):
        self.image.fill(BLACK)
        self.image.set_colorkey(BLACK, RLEACCEL)
        pygame.draw.circle(self.image, WHITE, (self.rect.centerx, self.rect.centery), BALL_RADIUS)

    def reinit(self):
        self.rect.centerx = self.area.centerx
        self.rect.centery = self.area.centery
        self.vector = Vec2D.Vec2D.from_magn_and_angle(BALL_SPEED, 0)

    def update(self):
        self.rect = self.calcnewpos()
        self.handle_collision()

    def calcnewpos(self):
        (dx, dy) = self.vector.get_xy()
        return self.rect.move(dx, dy)

    def handle_collision(self):
        (dx, dy) = self.vector.get_xy()

        if not self.area.contains(self.rect):
            if self.__hit_topbottom():
                dy = -dy
            elif self.__hit_leftright():
                self.game.increase_score()
                self.reinit()
        else:
            for paddle in self.hit_paddle(dx):
                if dx < 0: self.rect.left = GAP + PADDLE_WIDTH
                elif dx > 0: self.rect.right = SCREEN_WIDTH - (GAP + PADDLE_WIDTH)

                dx = -dx
                dy = paddle.hitpos / 4
                paddle.collided = True

        self.vector = Vec2D.Vec2D(dx, dy)

    def _hit_topbottom(self):
        return self.rect.top < 0 or self.rect.bottom > SCREEN_HEIGHT

    def _hit_leftright(self):
        return self.rect.left < self.area.left or self.rect.right > self.area.right

    def hit_paddle(self, dx):
        if dx < 0: paddle = self.game.paddles['left']
        elif dx > 0: paddle = self.game.paddles['right']

        if self.rect.colliderect(paddle.rect): return [paddle]
        else: return []  

Что ж, после того, как игрок (или ИИ) забьет, мяч вызывает метод его повторной установки, который помещает мяч в центр экрана и сбрасывает вектор:

def reinit(self):
    self.rect.centerx = self.area.centerx
    self.rect.centery = self.area.centery
    self.vector = Vec2D.Vec2D.from_magn_and_angle(BALL_SPEED, 0)

Но, каким-то образом, шар все еще поддерживает вектор, который был до того, как был вызван метод reinit. Поэтому, когда мяч проходит через левую сторону с вектором, подобным (-5,0 -2,0), он быстро меняется в вызове повторного набора, а затем возвращается к (-5,0, -2,0). Может кто-нибудь сказать, пожалуйста, почему это происходит?

Answer 1

Проблема в твоем handle_collision метод.

Короткая история:
Первое, что делает функция, это устанавливает dx и dy в текущий вектор. Тогда это вызывает reinit(), Затем он возвращает их к тому, что было первым в конце функции.

Чинить:
+ Изменить

def handle_collision(self):
    (dx, dy) = self.vector.get_xy() # <-- It first sets (dx, dy) to the old vector

    if not self.area.contains(self.rect):
        if self.__hit_topbottom():
            ...
        elif self.__hit_leftright():
            ...  # <-- here is where the reinit gets called, which changes the vector to new values.
    else:
        for paddle in self.hit_paddle(dx):
            ...

    self.vector = Vec2D.Vec2D(dx, dy)  # <-- Then the vector gets changed again, to the OLD vector saved above in (dx, dy)

к

def handle_collision(self):
    (dx, dy) = self.vector.get_xy()

    if not self.area.contains(self.rect):
        if self.__hit_topbottom():
            ...
        elif self.__hit_leftright():
            ...
            return  # <-- With the return here, the parsing never gets to change it back
    else:
        for paddle in self.hit_paddle(dx):
            ...

    self.vector = Vec2D.Vec2D(dx, dy)

Это предотвратит возврат вектора к тому, что он был, и в то же время установит его, когда это необходимо.