dkforest

commit 43a35b0800a6a5af51d7e22b5f56784167e31b41
parent 249349820ebca21556410098c4c523b99cdc99f3
Author: n0tr1v <n0tr1v@protonmail.com>
Date:   Sun, 17 Dec 2023 06:56:25 -0500

cleanup

Diffstat:
M
pkg/web/handlers/poker/poker.go
 | 
10
++++++----

1 file changed, 6 insertions(+), 4 deletions(-)
diff --git a/pkg/web/handlers/poker/poker.go b/pkg/web/handlers/poker/poker.go
@@ -343,6 +343,10 @@ func (g *PokerGame) getPlayer(username database.Username) (out *SeatedPlayer) {
 func (g *PokerGame) IsSeated(player database.Username) bool {
 	g.PlayersMtx.RLock()
 	defer g.PlayersMtx.RUnlock()
+	return g.isSeated(player)
+}
+
+func (g *PokerGame) isSeated(player database.Username) bool {
 	for _, p := range g.Players {
 		if p != nil && p.Username == player {
 			return true
@@ -454,10 +458,8 @@ func (g *PokerGame) UnSitPlayer1(seatedPlayer *SeatedPlayer) error {
 func (g *PokerGame) SitPlayer(authUser *database.User, pos int, chips database.PokerChip) error {
 	g.PlayersMtx.Lock()
 	defer g.PlayersMtx.Unlock()
-	for _, p := range g.Players {
-		if p != nil && p.Username == authUser.Username {
-			return errors.New("player already seated")
-		}
+	if g.isSeated(authUser.Username) {
+		return errors.New("player already seated")
 	}
 	if g.Players[pos] != nil {
 		return errors.New("seat already taken")