pygameでスペースを押すと、上に動く四角を表示させたい。

Question

前提・実現したいこと

pygameでシューティングゲームを作っています。

発生している問題・エラーメッセージ

スペースキーを押すと自機から弾が発射されるようにしたいのですが、スペースを押しても反応がありません。
スペースを押すと、任意の自機の位置から白い四角が表示され、x方向に-5ずつ動くようにするにはどうすればいいでしょうか？

該当のソースコード

# -*- coding: utf-8 -*-
import pygame
from pygame.locals import *
import sys

SCREEN = Rect(0, 0, 700, 500)

pygame.init()
screen = pygame.display.set_mode((700, 500))
pygame.display.set_caption("Shooting")


#自機と敵機
class Flyer(pygame.sprite.Sprite):
    def __init__(self, filename, x, y, vx, vy):
        pygame.sprite.Sprite.__init__(self)
        self.x = x
        self.y = y
        self.image = pygame.image.load(filename).convert_alpha()
        w = self.image.get_width()
        h = self.image.get_height()
        self.rect = Rect(x, y, w, h)
        self.vx = vx
        self.vy = vy
    def draw(self, screen):
        screen.blit(self.image, self.rect)

class Player(Flyer):
    def __init__(self, filename, x, y, vx, vy):
        super().__init__(filename, x, y, vx, vy)
    def move(self):
        pressed_key = pygame.key.get_pressed()
        if pressed_key[K_LEFT]:
            self.rect.move_ip(-self.vx, 0)
        if pressed_key[K_RIGHT]:
            self.rect.move_ip(self.vx, 0)
        if pressed_key[K_UP]:
            self.rect.move_ip(0, -self.vy)
        if pressed_key[K_DOWN]:
            self.rect.move_ip(0, self.vy)
        if self.y < -30 or self.y > 500 or self.x < -30 or self.x > 700:
            pygame.sprite.Sprite.remove(self)
    def shoot(self, screen): #弾発射
        bullet = pygame.draw.rect(screen, (255, 255, 255), (self.x, self.y, 20, 20), 0)
        bullet.move_ip(-5, 0)
            
class Enemy(Flyer):
    def __init__(self, filename, x, y, vx, vy):
        super().__init__(filename, x, y, vx, vy)
    def move(self):
        self.rect.move_ip(self.vx, self.vy)
        self.rect = self.rect.clamp(SCREEN)

#メイン
def main():
    player = Player("player.jpg", 350, 400, 5, 5)
    enemy1 = Enemy("enemy.jpg", 100, 100, 5, 0)
    clock = pygame.time.Clock()

    while 1:
        clock.tick(30)
        screen.fill((0, 0, 0))
        
        player.move()
        enemy1.move()
        player.draw(screen)
        enemy1.draw(screen)

        pygame.display.update()

        for event in pygame.event.get():
            if event.type == QUIT:
                pygame.quit()
                sys.exit()
            if event.type == KEYDOWN:
                if event.key == K_SPACE:
                    player.shoot(screen)
                if event.key == K_ESCAPE:
                    pygame.quit()
                    sys.exit()

if __name__ == "__main__":
    main()

試したこと

whileの中にplayer.shoot(screen)を入れてみましたが最初から、playerと同じ位置に動かない白い四角が表示されるだけでした。

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Answer 1

原因は主に2点
0. 移動処理であるbullet.move_ipをショットキー押下時の一度しか行っていないため、一度しか移動されないからです。そのため質問文のような状態になります。
0. move_ipのパラメータは上に移動したいなら、x軸ではなくy軸の値を変更する必要があります。

それ以外に発射したbulletがスクリーンの描画範囲外かの生存管理を行う必要があります。

1. class Bulletを新規追加
2. 変数:bullet_list にてBulletの生存管理をする。

---

2018/03/05 コメント欄の質問を受けて追記

bullet = Bullet()

としていないのに
bullet.move()
bullet.draw()
が動くのはなぜですか？

これはしてますよ。処理の流れをコメント1～5で説明します。
動きが分かりづらい時は図でプログラムの動作を書いてみるといいかもです。

自機を左上に移動してるときだけ弾が発射されないのですが、これはなぜでしょうか？

■条件
・自機を左上方向に移動中
・ショットキーを押下

この時にショットキーを押下しても弾が発射されないという点ですね。
不具合ではないでしょうか、以下のprint文で各処理で正しい値が取れているのかdebugしてみてはどうでしょうか。
少し調査してみた限りでは、event.key == K_SPACE:のスペース押下イベントが発生してませんでした。
キーボードの同時押しの制限(Nキーロールオーバー）に引っかかった可能性もあります

あとはがんばってくださいな。

import time
print('Done' + str(time.perf_counter()))

以下はサンプルコードです。

---

# -*- coding: utf-8 -*-
import pygame
from pygame.locals import *
import sys

SCREEN = Rect(0, 0, 700, 500)

pygame.init()
screen = pygame.display.set_mode((700, 500))
pygame.display.set_caption("Shooting")


#自機と敵機
class Flyer(pygame.sprite.Sprite):
    def __init__(self, filename, x, y, vx, vy):
        pygame.sprite.Sprite.__init__(self)
        self.x = x
        self.y = y
        self.image = pygame.image.load(filename).convert_alpha()
        w = self.image.get_width()
        h = self.image.get_height()
        self.rect = Rect(x, y, w, h)
        self.vx = vx
        self.vy = vy

    def draw(self, screen):
        screen.blit(self.image, self.rect)


class Player(Flyer):
    def __init__(self, filename, x, y, vx, vy):
        super().__init__(filename, x, y, vx, vy)

    def move(self):
        pressed_key = pygame.key.get_pressed()
        if pressed_key[K_LEFT]:
            self.rect.move_ip(-self.vx, 0)
        if pressed_key[K_RIGHT]:
            self.rect.move_ip(self.vx, 0)
        if pressed_key[K_UP]:
            self.rect.move_ip(0, -self.vy)
        if pressed_key[K_DOWN]:
            self.rect.move_ip(0, self.vy)
        if self.y < -30 or self.y > 500 or self.x < -30 or self.x > 700:
            pygame.sprite.Sprite.remove(self)

    def shoot(self, screen): #弾発射
        class Bullet(object):
            def __init__(self, x: int, y: int):
                self.bullet = Rect(x, y, 20, 20)

            def draw(self):
                pygame.draw.rect(screen, (255, 255, 255), self.bullet)

            def move(self, x: int=0, y: int=-5):
                self.bullet.move_ip(x, y)

            @property
            def is_destroy(self) -> bool:
                if self.bullet.x < -30 or self.bullet.y < - 30:
                    return True
                return False

        # 自機の位置が移動しても発射位置が追従するようにself.x/self.y ではなく self.rectを使用
        # 3,Bulletクラスのインスタンス変数を生成してshoot関数の戻り値として返す
        return Bullet(self.rect.x, self.rect.y)


class Enemy(Flyer):
    def __init__(self, filename, x, y, vx, vy):
        super().__init__(filename, x, y, vx, vy)

    def move(self):
        self.rect.move_ip(self.vx, self.vy)
        self.rect = self.rect.clamp(SCREEN)

#メイン
def main():
    player = Player("player.jpg", 350, 400, 5, 5)
    enemy1 = Enemy("enemy.jpg", 100, 100, 5, 0)
    clock = pygame.time.Clock()
    # 1,リストでbulletの生存管理
    bullet_list = []

    while 1:
        clock.tick(30)
        screen.fill((0, 0, 0))

        player.move()
        enemy1.move()
        player.draw(screen)
        enemy1.draw(screen)
        # 5,bullet_listの内容をfor in ループで回す。
        for bullet in bullet_list:
            bullet.move()
            bullet.draw()
        #リストからスクリーン範囲外のbulletをクリーンアップ
        bullet_list = list(filter(lambda x: not x.is_destroy, bullet_list))

        pygame.display.update()

        for event in pygame.event.get():
            if event.type == QUIT:
                pygame.quit()
                sys.exit()
            if event.type == KEYDOWN:
                # 2,ショット：スペースキーを押下時に弾を発射
                if event.key == K_SPACE:
                    # 発射時にbullet_listに追加
                    bullet = player.shoot(screen)
                    # 4,リストにplayer.shootの戻り値である変数:bulletを追加
                    bullet_list.append(bullet)
                if event.key == K_ESCAPE:
                    pygame.quit()
                    sys.exit()


if __name__ == "__main__":
    main()

Comments

[退会済みユーザー]

ありがとうございます。弾を発射することができました。

質問があります。
player = Player("player.jpg", 350, 400, 5, 5)
のように
bullet = Bullet()
としていないのに
bullet.move()
bullet.draw()
が動くのはなぜですか？

[退会済みユーザー]

もう一つ質問があります。
自機を左上に移動してるときだけ弾が発射されないのですが、これはなぜでしょうか？

[退会済みユーザー]

ご回答ありがとうございます。分からないところが何点かありましたが、自分で調べてみようと思います。

[umyu]

もしも友達にコンピュータでの対戦ゲームが好きな人がいれば、
Nキーロールオーバーについては知っているかもしれません、聞いてみてはどうでしょうか。