Skip to main content
情報追加
ソース リンク
hata
  • 2,764
  • 1
  • 12
  • 22

このコードはあくまでも今回の問題に即した解決策ですので、これが最良ではありません。pygame.sprite.spritecollideanypygame.sprite.spritecollide を使えば、そもそも下のように全ブロックを for ループでチェックすること自体が冗長になります。が、今回の問題の主旨からは外れますので、詳しい説明は割愛しますが、

self.rect.x += dx
for block in block_group.sprites():
    block.collide_x()
self.rect.y += dy
for block in block_group.sprites():
    block.collide_y()

この部分を次のようにしてブロックグループ全体で 1 回ずつ実行します:

self.rect.x += dx
Block.collide_x()
self.rect.y += dy
Block.collide_y()

Block#collide_xBlock#collide_y は、スタティックメソッドに書き換えます。

@staticmethod
def collide_x():
    # 略

@staticmethod
def collide_y():
    # 略

このコードはあくまでも今回の問題に即した解決策ですので、これが最良ではありません。pygame.sprite.spritecollideanypygame.sprite.spritecollide を使えば、そもそも下のように全ブロックを for ループでチェックすること自体が冗長になります。が、今回の問題の主旨からは外れますので、割愛します

for block in block_group.sprites():
    block.collide_x()

for block in block_group.sprites():
    block.collide_y()

このコードはあくまでも今回の問題に即した解決策ですので、これが最良ではありません。pygame.sprite.spritecollideanypygame.sprite.spritecollide を使えば、そもそも下のように全ブロックを for ループでチェックすること自体が冗長になります。が、今回の問題の主旨からは外れますので、詳しい説明は割愛しますが、

self.rect.x += dx
for block in block_group.sprites():
    block.collide_x()
self.rect.y += dy
for block in block_group.sprites():
    block.collide_y()

この部分を次のようにしてブロックグループ全体で 1 回ずつ実行します:

self.rect.x += dx
Block.collide_x()
self.rect.y += dy
Block.collide_y()

Block#collide_xBlock#collide_y は、スタティックメソッドに書き換えます。

@staticmethod
def collide_x():
    # 略

@staticmethod
def collide_y():
    # 略
本文に 2 文字追加
ソース リンク
hata
  • 2,764
  • 1
  • 12
  • 22

collide_xcollide_y の定義に問題があります。

pygame.sprite.spritecollideany はどれかのブロックと衝突している場合に真になりますが、Y 方向に衝突している場合でも collide_x 中で真になるので、そのまま collide_x の中の、どのブロックにめりこもうとしているのかという部分に進み、L 字型に右に飛び出した部分に X 方向的に左側からめり込んでいるものと判断され、右に飛び出したブロックの右端まで押し戻されます。

その後、Y 方向の移動処理が引き続き行われ、下に進んでいくわけです。

今回の質問に即した応急処置的には、pygame.sprite.spritecollideany で真になった場合、さらに、どのブロックと衝突しているのか? を pygame の命令を利用して行います。pygame.sprite.spritecollide がそれですので、例えば次のようになります:

def collide_x(self):
    if pygame.sprite.spritecollideany(player, block_group):
        colliders = pygame.sprite.spritecollide(player, block_group, False)
        if colliders[0].rect.left < player.rect.left < colliders[0].rect.right:
            player.rect.left = colliders[0].rect.right
        elif colliders[0].rect.left < player.rect.right < colliders[0].rect.right:
            player.rect.right = colliders[0].rect.left

def collide_y(self):
    if pygame.sprite.spritecollideany(player, block_group):
        colliders = pygame.sprite.spritecollide(player, block_group, False)
        if colliders[0].rect.top < player.rect.top < colliders[0].rect.bottom:
            player.rect.top = colliders[0].rect.bottom
        ifelif colliders[0].rect.top < player.rect.bottom < colliders[0].rect.bottom:
            player.rect.bottom = colliders[0].rect.top

このコードはあくまでも今回の問題に即した解決策ですので、これが最良ではありません。pygame.sprite.spritecollideanypygame.sprite.spritecollide を使えば、そもそも下のように全ブロックを for ループでチェックすること自体が冗長になります。が、今回の問題の主旨からは外れますので、割愛します。

for block in block_group.sprites():
    block.collide_x()

for block in block_group.sprites():
    block.collide_y()

collide_xcollide_y の定義に問題があります。

pygame.sprite.spritecollideany はどれかのブロックと衝突している場合に真になりますが、Y 方向に衝突している場合でも collide_x 中で真になるので、そのまま collide_x の中の、どのブロックにめりこもうとしているのかという部分に進み、L 字型に右に飛び出した部分に X 方向的に左側からめり込んでいるものと判断され、右に飛び出したブロックの右端まで押し戻されます。

その後、Y 方向の移動処理が引き続き行われ、下に進んでいくわけです。

今回の質問に即した応急処置的には、pygame.sprite.spritecollideany で真になった場合、さらに、どのブロックと衝突しているのか? を pygame の命令を利用して行います。pygame.sprite.spritecollide がそれですので、例えば次のようになります:

def collide_x(self):
    if pygame.sprite.spritecollideany(player, block_group):
        colliders = pygame.sprite.spritecollide(player, block_group, False)
        if colliders[0].rect.left < player.rect.left < colliders[0].rect.right:
            player.rect.left = colliders[0].rect.right
        elif colliders[0].rect.left < player.rect.right < colliders[0].rect.right:
            player.rect.right = colliders[0].rect.left

def collide_y(self):
    if pygame.sprite.spritecollideany(player, block_group):
        colliders = pygame.sprite.spritecollide(player, block_group, False)
        if colliders[0].rect.top < player.rect.top < colliders[0].rect.bottom:
            player.rect.top = colliders[0].rect.bottom
        if colliders[0].rect.top < player.rect.bottom < colliders[0].rect.bottom:
            player.rect.bottom = colliders[0].rect.top

このコードはあくまでも今回の問題に即した解決策ですので、これが最良ではありません。pygame.sprite.spritecollideanypygame.sprite.spritecollide を使えば、そもそも下のように全ブロックを for ループでチェックすること自体が冗長になります。が、今回の問題の主旨からは外れますので、割愛します。

for block in block_group.sprites():
    block.collide_x()

for block in block_group.sprites():
    block.collide_y()

collide_xcollide_y の定義に問題があります。

pygame.sprite.spritecollideany はどれかのブロックと衝突している場合に真になりますが、Y 方向に衝突している場合でも collide_x 中で真になるので、そのまま collide_x の中の、どのブロックにめりこもうとしているのかという部分に進み、L 字型に右に飛び出した部分に X 方向的に左側からめり込んでいるものと判断され、右に飛び出したブロックの右端まで押し戻されます。

その後、Y 方向の移動処理が引き続き行われ、下に進んでいくわけです。

今回の質問に即した応急処置的には、pygame.sprite.spritecollideany で真になった場合、さらに、どのブロックと衝突しているのか? を pygame の命令を利用して行います。pygame.sprite.spritecollide がそれですので、例えば次のようになります:

def collide_x(self):
    if pygame.sprite.spritecollideany(player, block_group):
        colliders = pygame.sprite.spritecollide(player, block_group, False)
        if colliders[0].rect.left < player.rect.left < colliders[0].rect.right:
            player.rect.left = colliders[0].rect.right
        elif colliders[0].rect.left < player.rect.right < colliders[0].rect.right:
            player.rect.right = colliders[0].rect.left

def collide_y(self):
    if pygame.sprite.spritecollideany(player, block_group):
        colliders = pygame.sprite.spritecollide(player, block_group, False)
        if colliders[0].rect.top < player.rect.top < colliders[0].rect.bottom:
            player.rect.top = colliders[0].rect.bottom
        elif colliders[0].rect.top < player.rect.bottom < colliders[0].rect.bottom:
            player.rect.bottom = colliders[0].rect.top

このコードはあくまでも今回の問題に即した解決策ですので、これが最良ではありません。pygame.sprite.spritecollideanypygame.sprite.spritecollide を使えば、そもそも下のように全ブロックを for ループでチェックすること自体が冗長になります。が、今回の問題の主旨からは外れますので、割愛します。

for block in block_group.sprites():
    block.collide_x()

for block in block_group.sprites():
    block.collide_y()
ソース リンク
hata
  • 2,764
  • 1
  • 12
  • 22

collide_xcollide_y の定義に問題があります。

pygame.sprite.spritecollideany はどれかのブロックと衝突している場合に真になりますが、Y 方向に衝突している場合でも collide_x 中で真になるので、そのまま collide_x の中の、どのブロックにめりこもうとしているのかという部分に進み、L 字型に右に飛び出した部分に X 方向的に左側からめり込んでいるものと判断され、右に飛び出したブロックの右端まで押し戻されます。

その後、Y 方向の移動処理が引き続き行われ、下に進んでいくわけです。

今回の質問に即した応急処置的には、pygame.sprite.spritecollideany で真になった場合、さらに、どのブロックと衝突しているのか? を pygame の命令を利用して行います。pygame.sprite.spritecollide がそれですので、例えば次のようになります:

def collide_x(self):
    if pygame.sprite.spritecollideany(player, block_group):
        colliders = pygame.sprite.spritecollide(player, block_group, False)
        if colliders[0].rect.left < player.rect.left < colliders[0].rect.right:
            player.rect.left = colliders[0].rect.right
        elif colliders[0].rect.left < player.rect.right < colliders[0].rect.right:
            player.rect.right = colliders[0].rect.left

def collide_y(self):
    if pygame.sprite.spritecollideany(player, block_group):
        colliders = pygame.sprite.spritecollide(player, block_group, False)
        if colliders[0].rect.top < player.rect.top < colliders[0].rect.bottom:
            player.rect.top = colliders[0].rect.bottom
        if colliders[0].rect.top < player.rect.bottom < colliders[0].rect.bottom:
            player.rect.bottom = colliders[0].rect.top

このコードはあくまでも今回の問題に即した解決策ですので、これが最良ではありません。pygame.sprite.spritecollideanypygame.sprite.spritecollide を使えば、そもそも下のように全ブロックを for ループでチェックすること自体が冗長になります。が、今回の問題の主旨からは外れますので、割愛します。

for block in block_group.sprites():
    block.collide_x()

for block in block_group.sprites():
    block.collide_y()