fix: start next player's timer after pass

Recursion-Group-P / blokee

https://blockee.netlify.app

3 stars 0 forks source link

fix: start next player's timer after pass #58

Closed hayato0309 closed 2 years ago

hayato0309 commented 2 years ago

Issue: #56

目的

以下の問題を解消

Pass後に次のPlayerのTimerが自動で開始しない
最後に残ったPlayerのTimerが0になっても止まらない（マイナスになってもカウントダウンし続ける）

実装概要

Timerの制御をPlayerAreaに移した（元々はRoomPageでやっていた）
- PlayerAreaでwatchを使って、currentPlayerIdとそのエリアのplayerIdを比較しTimerを制御
- currentPlayerIdとplayerIdが一致すればTimer開始
- 一致しなければTimer停止
0になったらTimer停止

netlify[bot] commented 2 years ago

Deploy Preview for blockee ready!

Name	Link
Latest commit	c5739fd2ebe0b1119dc5d8429d5ea9dbabd75c5c
Latest deploy log	https://app.netlify.com/sites/blockee/deploys/62a730d193d11100096da61a
Deploy Preview	https://deploy-preview-58--blockee.netlify.app/
Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

tkwonn commented 2 years ago

@hayato0309 一度 #55 のプルリクを修正した後に、こちらマージします！ありがとうございます！

tkwonn commented 2 years ago

@hayato0309 できたらハルキさんがコメントしてくださったfor ofを使って修正できそうな箇所がしてもらえると可読性があがって良いと思います！

横からすみません
for...of使うともう少しスッキリします
既知でしたらすみません

for(const player of this.players){
    const remainingPieces = player.remainingPieces;
    ...
}
「 i 」がどこで使われるか、考えなくて済むので楽だと思ってます。

https://developer.mozilla.org/ja/docs/Web/JavaScript/Reference/Statements/for...of

evaulation.js とPieceAlter? PieceSelectorらへんに修正できそうな部分があったような気がします

hayato0309 commented 2 years ago

@tkwonn すみません！分からなかったので質問させてください。

このプルリクでは evaulation.js, PieceAlter, PieceSelector を編集していないのですが、頂いているコメントはこのプルリクに関するものでしょうか？別でブランチ切って修正するとよいってことでしょうか？（確かに使えそうなところありそうです！）

よろしくお願いします！

tkwonn commented 2 years ago

@hayato0309 たしかにややこしかったですね。今日のMTGで編集しましょう！とりあえずこちらは問題ないので、マージします

Recursion-Group-P / blokee