Merge #225

225: Make seat number optional in Players constructor r=charleskawczynski a=charleskawczynski



Co-authored-by: Charles Kawczynski <kawczynski.charles@gmail.com>

Project.toml

@@ -1,7 +1,7 @@
 name = "TexasHoldem"
 uuid = "6cef90fc-eb55-4a2a-97d0-7ecce2b738fe"
 authors = ["Charles Kawczynski <kawczynski.charles@gmail.com>"]
-version = "0.4.1"
+version = "0.4.2"
 
 [deps]
 Logging = "56ddb016-857b-54e1-b83d-db4d58db5568"

src/player_type.jl

@@ -70,7 +70,7 @@ function Base.show(io::IO, player::Player)
     print(io, "$(name(player)): $(player.cards)")
 end
 
-function Player(strategy, seat_number, cards = nothing; bank_roll = 200)
+function Player(strategy, seat_number = -1, cards = nothing; bank_roll = 200)
     action_required = true
     all_in = false
     round_bank_roll = bank_roll

src/players.jl

@@ -34,10 +34,17 @@ coincide.
 struct Players{PS<:Union{Tuple,AbstractArray}}
     players::PS
     function Players(players)
+        n = length(players)
+        if all(x->seat_number(x)==-1, players)
+            for i in 1:n
+                players[i].seat_number = i
+            end
+        end
         @assert allunique(map(x->seat_number(x), players))
         spbsn = sortperm_by_seat_number(players)
         splayers = map(sp->players[sp], spbsn)
         @assert issorted(map(x->seat_number(x), splayers))
+        @assert seat_number(splayers[n]) ≤ n
         return new{typeof(splayers)}(splayers)
     end
 end