Recursion-Group-C / card-game

🎮 Card Game Stadium
https://card-game-drab-ten.vercel.app/
3 stars 0 forks source link

[Bug]: デッキの初期位置が指定した座標と異なる #110

Closed y0uk1 closed 1 year ago

y0uk1 commented 1 year ago

どんな不具合が出る?

deck classからインスタンスを作成すると、引数x, yに指定した座標とは異なる位置にデッキが配置される。

どんな動作を期待していた?

コンストラクタの引数x, yに指定した座標にデッキが配置される。

不具合の再現方法

No response

原因

https://github.com/Recursion-Group-C/card-game/blob/b847142b2899bb72fd58c60510f17fa66e4e57ce/games/common/Factories/tableScene.ts#L79

resetAndShuffleDeck関数の以下の部分でデッキの座標をgameZoneの中心に設定しているため

 this.deck.cardList.forEach((card) => {
      Phaser.Display.Align.In.Center(
        card as Card,
        this.gameZone as Zone,
        x ?? 0,
        y ?? 0
      );
    });

修正案

resetAndShuffleDeck関数の以下の部分を削除し、xやy方向の中心に置きたい場合は、引数のx, yにそれぞれ、this.config.width/2, this.config.height/2を指定する。

 this.deck.cardList.forEach((card) => {
      Phaser.Display.Align.In.Center(
        card as Card,
        this.gameZone as Zone,
        x ?? 0,
        y ?? 0
      );
    });