src/LittleTown.Core.Tests/MatchTesting.cs

@@ -17,6 +17,7 @@ public class MatchTesting
         Match match2 = new Match(getter);
         match2.AddPlayer("Player1");
         match2.AddPlayer("Player2");
+        Assert.Throws<MatchConfigException>(() => match2.AddPlayer("Player2"));
         match2.Init();
 
         Match match3 = new Match(getter);
@@ -51,10 +52,23 @@ public class MatchTesting
 
         PlayerZone player1 = match.GetPlayerZone("Player1");
         PlayerZone player1_3 = match.GetPlayerZone("Player2");
+
         Assert.Equal(3, player1.Ressources[Enums.ResourceType.Piece]);
         player1.AddRessources(ResourceType.Piece, 1);
         Assert.Equal(3, player1_3.Ressources[Enums.ResourceType.Piece]);
 
         Assert.Equal(4, player1.Objectives.Count);
     }
+
+    [Fact]
+    public void MatchGetters()
+    {
+        StaticDataGetter getter = new();
+        Match match = new Match(getter);
+        match.AddPlayer("Player1");
+        match.AddPlayer("Player2");
+        match.Init();
+
+        Assert.Throws<ArgumentException>(() => match.GetPlayerZone("UnknownPlayer"));
+    }
 }

src/LittleTown.Core.Tests/MatchWorkflowTesting.cs

@@ -0,0 +1,34 @@
+using LittleTown.Core.Actions;
+using LittleTown.Core.Exceptions;
+using LittleTown.StaticDataAcces;
+
+namespace LittleTown.Core.Tests;
+
+public class MatchWorkflowTesting
+{
+
+
+    [Fact]
+    public void Simple2PlayerGame()
+    {
+        StaticDataGetter getter = new StaticDataGetter();
+
+        Match match = new Match(getter);
+        match.AddPlayer("Player1");
+        match.AddPlayer("Player2");
+        match.Init();
+
+        int count = 0;
+        while (!match.IsDone)
+        {
+            EmptyAction action = new EmptyAction();
+            match.ExecuteAction(action);
+            count++;
+            if (count > 40)
+            {
+                Assert.Fail("Trop d'action pour une partie vide");
+            }
+        }
+        Assert.Throws<MatchFinishedException>(() => match.ExecuteAction(new EmptyAction()));
+    }
+}

src/LittleTown.Core/Exceptions/MatchFinishedException.cs

@@ -0,0 +1,21 @@
+namespace LittleTown.Core.Exceptions;
+
+/// <summary>
+/// Exception levee quand on essais d'executer une actin dans un match terminé
+/// </summary>
+public class MatchFinishedException : Exception
+{
+    /// <summary> constructeur de base </summary>
+    public MatchFinishedException() : base() { }
+
+    /// <summary> Constructeur avec un message d'erreur </summary>
+    /// <param name="message">le message decrivant l'exception en detail</param>
+    public MatchFinishedException(string message) : base(message) { }
+
+    /// <summary>
+    /// Constructeur avec un message et une exception interne
+    /// </summary>
+    /// <param name="message">Le message de l'erreur</param>
+    /// <param name="innerException">l'exception encapsulée</param>
+    public MatchFinishedException(string message, Exception innerException) : base(message, innerException) { }
+}

src/LittleTown.Core/LittleTown.Core.csproj

@@ -9,8 +9,6 @@
     <EnableNETAnalyzers>true</EnableNETAnalyzers>
     <EnforceCodeStyleInBuild>true</EnforceCodeStyleInBuild>
     <AnalysisMode>All</AnalysisMode>
-    <DebugType>full</DebugType>
-    <DebugSymbols>true</DebugSymbols>
   </PropertyGroup>
 
 </Project>

src/LittleTown.Core/MatchAggregate/Actions/EmptyAction.cs

@@ -0,0 +1,15 @@
+namespace LittleTown.Core.Actions;
+
+/// <summary>
+/// Un action de test qui permet de faire le déroulé d'une partie, cette action ne fait rien qu'utiliser un ouvrier du joueur et passer au suivant
+/// </summary>
+public class EmptyAction : IAction
+{
+    /// <inheritdoc/>
+    public void Execute(Match match)
+    {
+        ArgumentNullException.ThrowIfNull(match);
+
+        match.NextPlayer();
+    }
+}

src/LittleTown.Core/MatchAggregate/Actions/IAction.cs

@@ -0,0 +1,13 @@
+namespace LittleTown.Core.Actions;
+
+/// <summary>
+/// Interface qui représente ce que doivent implémenter les actions
+/// </summary>
+public interface IAction
+{
+    /// <summary>
+    /// Methode demandant a l'action d'appliquer ses changement au match
+    /// </summary>
+    /// <param name="match">le match a modifier avec cette action</param>
+    public void Execute(Match match);
+}

src/LittleTown.Core/MatchAggregate/Match.cs

@@ -1,3 +1,4 @@
+using LittleTown.Core.Actions;
 using LittleTown.Core.Exceptions;
 using LittleTown.Core.Ports;
 
@@ -8,6 +9,18 @@ namespace LittleTown.Core;
 /// </summary>
 public class Match
 {
+    /// <summary> LE numero du tour en cours (Partant de 1) </summary>
+    public int CurrentTurn { get; private set; } = 1;
+
+    /// <summary> l'id du joueur a qui c'est le tour de jouer</summary>
+    public string CurrentPlayer { get => _players[_playerTurnsOrder[_currentPlayerIndex]].PlayerName; }
+
+    /// <summary> Indique si le match est terminé </summary>
+    public bool IsDone { get; private set; }
+
+    /// <summary>la liste indiquant l'ordre des joueurs, _playerTurnsOrder[0] donne l'index du 1er joueur, _playerTurnsOrder[1] du second..... </summary>
+    private List<int> _playerTurnsOrder = new List<int>();
+
     private const int _minPlayerCount = 2;
     private const int _maxPlayerCount = 4;
 
@@ -20,18 +33,12 @@ public class Match
 
     private Random _random = new Random();
 
-    private Dictionary<string, PlayerZone> _players = new();
-
-    /// <summary> LE numero du tour en cours (Partant de 1) </summary>
-    public int CurrentTurn { get; private set; } = 1;
-
-    /// <summary>la liste indiquant l'ordre des joueurs, _playerTurnsOrder[0] donne l'index du 1er joueur, _playerTurnsOrder[1] du second..... </summary>
-    private List<int> _playerTurnsOrder = new List<int>();
+    private List<PlayerZone> _players = new();
+    private int _currentPlayerIndex;
 
     /// <summary>
     /// Constructeur d'une nouvelle partie avec un nombre de joueurs données en parametres
     /// </summary>
-    /// <param name="nbPlayer"></param>
     /// <param name="staticData">un objet permettant de récupérer les données statiques du jeu</param>
     public Match(IStaticDataGetter staticData)
     {
@@ -48,11 +55,14 @@ public class Match
     {
         if (_players.Count < _maxPlayerCount)
         {
-            if (_players.ContainsKey(playerName))
+            if (_players.Any(p => p.PlayerName == playerName))
             {
                 throw new MatchConfigException("Un joueur existe déjà avec ce nom");
             }
-            _players.Add(playerName, new PlayerZone());
+            _players.Add(new PlayerZone()
+            {
+                PlayerName = playerName
+            });
         }
         else
         {
@@ -60,6 +70,25 @@ public class Match
         }
     }
 
+
+    /// <summary>
+    /// Demander au match d'executer une action si elle est autorisée
+    /// </summary>
+    /// <param name="action">l'action a réaliser</param>
+    public void ExecuteAction(IAction action)
+    {
+        ArgumentNullException.ThrowIfNull(action);
+
+        //quelques vérification génériques pour savoir si l'action peut être jouée
+        if (IsDone)
+        {
+            throw new MatchFinishedException("Impossible d'effectuer une action sur un match terminé");
+        }
+
+        //autoriser l'action a s'executer en lui donner les getters dont elle a besoin
+        action.Execute(this);
+    }
+
     /// <summary> Initialiser la partie, il faut avoir ajouté les joueurs au préalable </summary>
     /// <exception cref="MatchConfigException"></exception>
     public void Init()
@@ -70,13 +99,14 @@ public class Match
         ArgumentOutOfRangeException.ThrowIfGreaterThan(nbPlayer, _maxPlayerCount);
 
         List<int> freeObjectiveIndexs = Enumerable.Range(0, _objectives.Count).ToList();
-        foreach (PlayerZone zone in _players.Values)
+        foreach (PlayerZone zone in _players)
         {
             zone.AddObjectives(GetRandomObjectives(nbPlayer switch
             {
                 2 => 4,
                 3 => 3,
                 4 => 2,
+                _ => throw new MatchConfigException("Mauvais nombre de joueurs lors Workers")
             }, freeObjectiveIndexs));
             zone.AddRessources(Enums.ResourceType.Piece, 3);
         }
@@ -106,6 +136,8 @@ public class Match
                 index = 0;
         }
 
+        _currentPlayerIndex = 0;
+
     }
 
     /// <summary> Permet de récuperer une player zone(une copie) </summary>
@@ -113,7 +145,10 @@ public class Match
     /// <returns></returns>
     public PlayerZone GetPlayerZone(string playerName)
     {
-        if (!_players.TryGetValue(playerName, out PlayerZone? value))
+        var value = _players.Where(p => p.PlayerName == playerName).FirstOrDefault();
+
+
+        if (null == value)
             throw new ArgumentException("playerID is out of bound");
 
         return value.Clone() as PlayerZone ?? throw new ArgumentException("Cast exception in GetPlayerZone"); ;
@@ -131,4 +166,17 @@ public class Match
         }
         return result;
     }
+    public void NextPlayer()
+    {
+        _currentPlayerIndex++;
+        if (_currentPlayerIndex >= _players.Count)
+        {
+            _currentPlayerIndex = 0;
+            CurrentTurn++;
+        }
+        if (CurrentTurn >= 4)
+        {
+            IsDone = true;
+        }
+    }
 }

src/LittleTown.Core/PlayerZone/PlayerZone.cs

@@ -7,7 +7,8 @@ public class PlayerZone : ICloneable
     /// <summary> Les ressources que possede le joueur </summary>
     public IDictionary<ResourceType, int> Ressources { get; init; } = new Dictionary<ResourceType, int>();
 
-
+    /// <summary> l'id du joueur a cette zone </summary>
+    public required string PlayerName { get; init; }
 
     /// <summary> La liste des objectifs que le joueur possede/// </summary>
     public IReadOnlyCollection<Objective> Objectives { get => _objectives.AsReadOnly(); init => _objectives = new List<Objective>(value); }
@@ -32,6 +33,10 @@ public class PlayerZone : ICloneable
         }
     }
 
+    /// <summary>
+    /// Assigner des objectifs au joueur
+    /// </summary>
+    /// <param name="objectives"></param>
     public void AddObjectives(ICollection<Objective> objectives)
     {
         _objectives.AddRange(objectives);
@@ -45,7 +50,8 @@ public class PlayerZone : ICloneable
         {
             Ressources = new Dictionary<ResourceType, int>(Ressources),
             Objectives = new List<Objective>(Objectives),
-            ScoreMarker = ScoreMarker
+            ScoreMarker = ScoreMarker,
+            PlayerName = PlayerName
         };
 
         return result;