2017-12-14 10 views
2

私は基本的なタンクゲームを作成していますが、私は現在タンクのガンで作業しています。私は銃を軸回りに回転させ、 "a"と "d"キーを押すたびに移動しますが、銃を回転させて移動させたときに何らかの理由で右に移動します。ここでは、コードは次のとおりPygameスプライトが回転して移動した後にジャンプします

class tankGun(pygame.sprite.Sprite): 
    def __init__(self): 
     super().__init__() 
     self.image = tankGunImage 
     self.rect = self.image.get_rect() 
     self.x = 400 
     self.xx = 410 
     self.y = 400 
     self.rect.x = self.x 
     self.rect.y = self.y 
     self.angle = 0 
     self.original = self.image 
     tankGunList.add(self) 
     tankGunList.draw(gameDisplay) 

def rotate(self, chk): 
    test = False 
    if chk == 1: 
     self.angle += 1 
    elif chk == 2: 
     self.angle -= 1 
    if self.angle > 0 and self.angle < 180: 
     test = True 
    else: 
     if chk == 1: 
      self.angle = 180 
     elif chk == 2: 
      self.angle = 0 
    if test == True: 
     self.image = pygame.transform.rotate(self.original, self.angle) 
     tankGunList.clear(gameDisplay, background) 
     tankGunList.draw(gameDisplay) 
     self.rect = self.image.get_rect(center = self.rect.center) 

def move(self, chk): 
    self.x += chk 
    self.xx = self.x + 10 
    self.rect.x = self.x 
    tankGunList.clear(gameDisplay, background) 
    tankGunList.draw(gameDisplay)   

(サイドノート:CHKは、回転のために+1または-1移動および1または2のいずれかとして送信されるそれはキーが押された検出だけ方法です。)。

+0

:あなたが唯一の 'self.rect.x'を使用し、' self.x'削除することができますので、 'self.x'がself.rect.x''と常に同じ値を持っている - 'と同じself.y' – furas

+0

'tankGunList.draw(gameDisplay)'を3か所で使用しているようです。あなたはメインループで一度だけ描くべきです – furas

答えて

0

あなたはなぜ問題があるのか​​わかりません - 私は問題を見るためにそれを実行することはできません。

しかし、私は何を変えるかを示します。多分それが助けになるでしょう。

まず、chkの代わりにわかりやすい名前を使用します。 directionstep

class tankGun(pygame.sprite.Sprite): 

    def __init__(self): 
     super().__init__() 

     self.original = tankGunImage 
     self.image = self.original 

     self.rect = self.image.get_rect() 
     self.rect.x = 400 
     self.rect.y = 400 

     self.xx = self.rect.x + 10 

     self.angle = 0 

     tankGunList.add(self) 

    def rotate(self, direction): 
     rotate_image = False 

     if direction == 1: 
      if self.angle < 180 
       self.angle += 1 
       rotate_image = True 
     elif direction == 2: 
      if self.angle > 0 
       self.angle -= 1 
       rotate_image = True 

     if rotate_image == True: 
      self.image = pygame.transform.rotate(self.original, self.angle) 
      self.rect = self.image.get_rect(center=self.rect.center) 

    def move(self, step): 
     self.rect.x += step 
     self.xx = self.rect.x + 10 

# --- in mainloop --- 

tankGunList.clear(gameDisplay, background) 
tankGunList.draw(gameDisplay) 
0

あなたは回転後self.x位置を更新しないために発生します。画像を回転すると、その枠のサイズが変更され、新しい矩形のx self.rect.x位置が変更されます。ただし、self.x属性は同じままで、次回にタンクを移動するとself.xself.rect.xが再び同期してジャンプします。あなたがする必要がないように、

​​

私はむしろ、中心COORDSためxy属性を使用して、self.rect.center = (self.x, self.y)を設定したい:

だけ回転した後 self.xを更新し、問題を修正するには回転後に self.xを更新してください。ところで

import pygame as pg 


pg.init() 
screen = pg.display.set_mode((640, 480)) 
tankGunImage = pg.Surface((30, 50), pg.SRCALPHA) 
tankGunImage.fill((70, 200, 30)) 


class tankGun(pg.sprite.Sprite): 
    def __init__(self): 
     super().__init__() 
     self.image = tankGunImage 
     self.rect = self.image.get_rect() 
     self.x = 300 # Center x. 
     self.xx = 310 
     self.y = 200 # Center y. 
     # Set the center attributes. 
     self.rect.centerx = self.x 
     self.rect.centery = self.y 
     self.angle = 0 
     self.original = self.image 

    def rotate(self, chk): 
     turn = False 
     if chk == 1: 
      self.angle += 4 
     elif chk == 2: 
      self.angle -= 4 
     if self.angle > 0 and self.angle < 180: 
      turn = True 
     else: 
      if chk == 1: 
       self.angle = 180 
      elif chk == 2: 
       self.angle = 0 

     if turn == True: 
      self.image = pg.transform.rotate(self.original, self.angle) 
      self.rect = self.image.get_rect(center=self.rect.center) 
      # No need to update the `self.x` now, since the center stays the same. 

    def move(self, chk): 
     self.x += chk 
     self.xx = self.x + 10 
     self.rect.center = (self.x, self.y) 


def main(): 
    clock = pg.time.Clock() 
    color = pg.Color(30, 30, 30) 
    all_sprites = pg.sprite.Group() 
    tank = tankGun() 
    all_sprites.add(tank) 

    done = False 
    while not done: 
     for event in pg.event.get(): 
      if event.type == pg.QUIT: 
       done = True 

     keys = pg.key.get_pressed() 
     if keys[pg.K_a]: 
      tank.rotate(1) 
     if keys[pg.K_d]: 
      tank.rotate(2) 
     if keys[pg.K_j]: 
      tank.move(-4) 
     if keys[pg.K_l]: 
      tank.move(4) 
     all_sprites.update() 

     screen.fill(color) 
     all_sprites.draw(screen) 
     pg.display.flip() 
     clock.tick(30) 


if __name__ == '__main__': 
    main() 
    pg.quit() 
関連する問題