issues
search
Recursion-Group-P
/
blokee
https://blockee.netlify.app
3
stars
0
forks
source link
refactor: To manage currentPlayerId in store-game
#39
Closed
hayato0309
closed
2 years ago
hayato0309
commented
2 years ago
Issue: #38
目的
currentPlayerIdをgame-storeで管理するようリファクタリング(現在はRoomPageにて管理されている)
達成条件
curretPlayerIdをgame-storeで管理し、既存の機能が今まで通り動くこと
実装概要
stateにcurrentPlayerIdを作成
それに伴うmutation, action, getterを作成
RoomPageでcurretPlayerIdを使う場合game-storeから取得
その他気づき
PlayerArea, PieceSelector, PieceAlterにてcurrentPlayerId(もしくはplayerId)としているものは、今回game-storeに移行したものとは異なり、そのコンポーネントが持っているplayerIdのこと
なので、例えばgame-storeのcurrentPlayerIdが0でも、Player2(idは1)のPlayerArea内のcurrentPlayerIdは常に1
もし PlayerArea, PieceSelector, PieceAlter もgame-storeのcurrentPlayerIdを参照するようにすると、他のPlayerのターンの時に自分のピースをフリップ・回転させたりすると、他のPlayerのピースに影響が出そう(通信プレイを想定)
結論、PlayerArea, PieceSelector, PieceAlterの中で使用するcurrentPlayerId(もしくはplayerId)は今のまま変更なしで、そのコンポーネントのplayerIdを参照する方法が安全でいいと思いました
tkwonn
commented
2 years ago
家に戻って確認しますね!
Issue: #38
目的
達成条件
実装概要
その他気づき