add goToNextPlayer in Game + changed constructors and inheritance from LightGame
3 unresolved threads
3 unresolved threads
Merge request reports
Activity
requested review from @pirousse
assigned to @fordenin
3 3 using System.Linq; 4 4 using System.Text; 5 5 using System.Threading.Tasks; 6 using Shared.data; changed this line in version 3 of the diff
19 21 public int nbNoRise { set; get; } 20 22 public List<ChatMessage> chat { set; get; } 21 23 22 public Game(int smallBlind, int bingBLind) 23 { 24 this.rounds = new List<Round>(); 24 // Default constructor for the derived class. 25 26 public Game(int id):base(id) 27 { 28 } 29 30 31 // public Game(int smallBlind, int bingBLind, int id) 32 public Game(int id, string status, int indexRound, List<Player> players, List<User> spectators, List<User> lobby, int smallBlind, 33 int bingBLind) : base(id) Au regard de cela: https://learn.microsoft.com/en-us/dotnet/csharp/fundamentals/tutorials/inheritance Je suis pas sur que ton heritage soit correct. Je ferai:
public Game(int id, int smallBlind, int bingBLind) : base(id)
Avec ca tu n as pas besoin de les affecter dans le constructeur de la classEdited by Pierre Rousselchanged this line in version 3 of the diff
56 77 this.highestBet = highestBet; 57 78 this.nbNoRise = nbNoRise; 58 79 this.chat = chat; 59 } 80 }*/ 81 82 public int GoToNextPlayer() 83 { 84 85 int nbOfPlayers = this.players.Count; 86 int tmp = this.currentPLayerIndex + 1; 87 this.currentPLayerIndex = tmp < nbOfPlayers ? tmp : 0; changed this line in version 3 of the diff
added 4 commits
-
1aa37f6f...f3c55b48 - 2 commits from branch
data
- 546ffa0b - add goToNextPlayer in Game + changed constructors and inheritance from LightGame
- e3f96486 - review_v1
-
1aa37f6f...f3c55b48 - 2 commits from branch
mentioned in commit adf66f0c
Please register or sign in to reply