From b555f8b849e9a424eb660b9ed9abda8d76d52af1 Mon Sep 17 00:00:00 2001
From: "jules.lascret" <jules.lascret@etu.uca.fr>
Date: Thu, 6 Jun 2024 16:45:50 +0200
Subject: [PATCH] Game save + Leaderboard save + Leaderboard load

---
 .../Players/Leaderboard.cs                    | 10 ++---
 Qwirkle/QwirkleClassLibrary/Tiles/TileBag.cs  | 38 ++++++++-----------
 Qwirkle/QwirkleViews/App.xaml.cs              |  9 ++++-
 Qwirkle/QwirkleViews/MainPage.xaml            |  4 +-
 Qwirkle/QwirkleViews/MainPage.xaml.cs         |  3 +-
 Qwirkle/QwirkleViews/Pages/GameBoard.xaml.cs  | 15 +++++++-
 Qwirkle/QwirkleViews/Pages/Leaderboard.xaml   | 34 +++++++++--------
 .../QwirkleViews/Pages/Leaderboard.xaml.cs    |  2 +-
 Qwirkle/TestBase/TestLeaderboard.cs           |  4 +-
 9 files changed, 64 insertions(+), 55 deletions(-)

diff --git a/Qwirkle/QwirkleClassLibrary/Players/Leaderboard.cs b/Qwirkle/QwirkleClassLibrary/Players/Leaderboard.cs
index 8d40223..d85d4ee 100644
--- a/Qwirkle/QwirkleClassLibrary/Players/Leaderboard.cs
+++ b/Qwirkle/QwirkleClassLibrary/Players/Leaderboard.cs
@@ -16,7 +16,7 @@ namespace QwirkleClassLibrary.Players
     public class Leaderboard : INotifyPropertyChanged
     {
         [DataMember]
-        private readonly ObservableCollection<Score> leaderboard = new();
+        private ObservableCollection<Score> leaderboard = new();
 
         public event PropertyChangedEventHandler? PropertyChanged;
 
@@ -26,12 +26,7 @@ namespace QwirkleClassLibrary.Players
         }
 
         public ReadOnlyObservableCollection<Score> Lb => new(leaderboard);
-
-
-
-
-
-
+        
         /// <summary>
         /// Returns the index of the player in the leaderboard, -1 if the player is not in the leaderboard
         /// </summary>
@@ -96,6 +91,7 @@ namespace QwirkleClassLibrary.Players
                 first = false;
             }
 
+            leaderboard = new ObservableCollection<Score>(leaderboard.OrderByDescending(x => x.Points).ThenBy(x => x.Victories));
         }
     }
 }
diff --git a/Qwirkle/QwirkleClassLibrary/Tiles/TileBag.cs b/Qwirkle/QwirkleClassLibrary/Tiles/TileBag.cs
index 8f096dc..05d5c92 100644
--- a/Qwirkle/QwirkleClassLibrary/Tiles/TileBag.cs
+++ b/Qwirkle/QwirkleClassLibrary/Tiles/TileBag.cs
@@ -15,7 +15,7 @@ namespace QwirkleClassLibrary.Tiles
         private readonly List<Tile> tiles = [];
         
         public ReadOnlyCollection<Tile>? TilesBag { get; private set; }
-        
+
 
         /// <summary>
         /// This is the constructor for the TileBag. It will create a tile of each of the possibilities among the Color and Shape Enums.
@@ -24,29 +24,23 @@ namespace QwirkleClassLibrary.Tiles
         /// <exception cref="ArgumentException">Throw an exception if the number of copies is negative (impossible) or superior to 3 (contradiction with the rules).</exception>
         public TileBag(int nbSet)
         {
-            /*            if (nbSet < 0 || nbSet > 3)
-                        {
-                            throw new ArgumentException(nbSet.ToString());
-                        }
-
-                        for (int i = 0; i < nbSet; i++)
-                        {
-                            foreach (Shape s in Enum.GetValues(typeof(Shape)))
-                            {
-                                foreach (Color c in Enum.GetValues(typeof(Color)))
-                                {
-                                    Tile t = new(s, c);
-                                    tiles.Add(t);
-                                }
-                            }
-                        }*/
+            if (nbSet < 0 || nbSet > 3)
+            {
+                throw new ArgumentException(nbSet.ToString());
+            }
 
-            Tile t1 = new Tile(Shape.Club, Color.Yellow);
-            Tile t2 = new Tile(Shape.Round, Color.Orange);
+            for (int i = 0; i < nbSet; i++)
+            {
+                foreach (Shape s in Enum.GetValues(typeof(Shape)))
+                {
+                    foreach (Color c in Enum.GetValues(typeof(Color)))
+                    {
+                        Tile t = new(s, c);
+                        tiles.Add(t);
+                    }
+                }
+            }
 
-            tiles.Add(t1);
-            tiles.Add(t2);
-            
             Init();
         }
 
diff --git a/Qwirkle/QwirkleViews/App.xaml.cs b/Qwirkle/QwirkleViews/App.xaml.cs
index 12606b5..5a8252b 100644
--- a/Qwirkle/QwirkleViews/App.xaml.cs
+++ b/Qwirkle/QwirkleViews/App.xaml.cs
@@ -2,7 +2,9 @@
 using Microsoft.Maui.Controls;
 using Qwirkle.Pages;
 using QwirkleClassLibrary.Games;
+using QwirkleClassLibrary.Persistences;
 using QwirkleClassLibrary.Players;
+using Leaderboard = QwirkleClassLibrary.Players.Leaderboard;
 
 namespace Qwirkle
 {
@@ -10,6 +12,7 @@ namespace Qwirkle
     {
         public App()
         {
+            Directory.SetCurrentDirectory(Path.Combine(Directory.GetCurrentDirectory(), "..\\..\\..\\Files"));
             InitializeComponent();
 
             MainPage = new AppShell();
@@ -20,10 +23,12 @@ namespace Qwirkle
             Routing.RegisterRoute(nameof(MainPage), typeof(MainPage));
             Routing.RegisterRoute(nameof(Qwirkle.Pages.Leaderboard), typeof(Qwirkle.Pages.Leaderboard));
 
+            ILeaderboardPersistence leaderboardLoad = new LeaderboardPersistenceJson();
+            Ld = leaderboardLoad.LoadLeaderboard();
         }
 
+        
         public Game Game { get; set; } = new();
-        public QwirkleClassLibrary.Players.Leaderboard LD { get; set; } = new();
-
+        public Leaderboard Ld { get; set; }
     }
 }
diff --git a/Qwirkle/QwirkleViews/MainPage.xaml b/Qwirkle/QwirkleViews/MainPage.xaml
index 607867c..c8eb72a 100644
--- a/Qwirkle/QwirkleViews/MainPage.xaml
+++ b/Qwirkle/QwirkleViews/MainPage.xaml
@@ -6,7 +6,7 @@
 
     <ScrollView>
         <VerticalStackLayout
-            Padding="30,0"
+            Padding="30, 0, 30, 30"
             Spacing="25">
 
             <Label
@@ -25,6 +25,8 @@
                 Text="Play"
                   InfoClicked="OnInfoClicked"
                 />
+            
+            <controls:ButtonShadow Text="Continue"/>
 
             <controls:ButtonShadow Text="Leaderboard"
                                    InfoClicked="OnCLeaderboardClicked"/>
diff --git a/Qwirkle/QwirkleViews/MainPage.xaml.cs b/Qwirkle/QwirkleViews/MainPage.xaml.cs
index c36e134..fa9d69b 100644
--- a/Qwirkle/QwirkleViews/MainPage.xaml.cs
+++ b/Qwirkle/QwirkleViews/MainPage.xaml.cs
@@ -21,8 +21,7 @@ namespace Qwirkle
 
             Navigation.PushAsync(new SetPlayers());
         }
-
-
+        
         public void OnRulesClicked(object sender, EventArgs e)
         {
             Navigation.PushAsync(new Rules());
diff --git a/Qwirkle/QwirkleViews/Pages/GameBoard.xaml.cs b/Qwirkle/QwirkleViews/Pages/GameBoard.xaml.cs
index 0ed74ae..c09a48c 100644
--- a/Qwirkle/QwirkleViews/Pages/GameBoard.xaml.cs
+++ b/Qwirkle/QwirkleViews/Pages/GameBoard.xaml.cs
@@ -9,6 +9,7 @@ using System.ComponentModel;
 using Cell = QwirkleClassLibrary.Boards.Cell;
 using Color = Microsoft.Maui.Graphics.Color;
 using System.Drawing;
+using QwirkleClassLibrary.Persistences;
 using CommunityToolkit.Maui.Views;
 
 namespace Qwirkle.Pages;
@@ -111,6 +112,11 @@ public partial class Gameboard : ContentPage
             game.EmptyCellUsed();
             game.DrawTiles(game.GetPlayingPlayer());
         }
+        
+        game.CheckGameOver(game.GetPlayingPlayer());
+
+        IGamePersistence gameIntermediateSave = new GamePersistenceXml();
+        gameIntermediateSave.SaveGame(game);
 
         if (!game.CheckGameOver(game.GetPlayingPlayer()))
         {
@@ -119,7 +125,14 @@ public partial class Gameboard : ContentPage
         }
         else
         {
-            ((App)Application.Current!).LD.AddScoreInLead(game.ScoreBoard);
+            ((App)Application.Current!).Ld.AddScoreInLead(game.ScoreBoard);
+
+            IGamePersistence gameEndSave = new GamePersistenceXml();
+            gameEndSave.SaveGame(game);
+            
+            ILeaderboardPersistence leaderboardSave = new LeaderboardPersistenceJson();
+            leaderboardSave.SaveLeaderboard(((App)Application.Current!).Ld);
+            
             game.ClearGame();
         }
         game.NextPlayerNotified -= Game_NextPlayerNotified;
diff --git a/Qwirkle/QwirkleViews/Pages/Leaderboard.xaml b/Qwirkle/QwirkleViews/Pages/Leaderboard.xaml
index 62ef969..09747cd 100644
--- a/Qwirkle/QwirkleViews/Pages/Leaderboard.xaml
+++ b/Qwirkle/QwirkleViews/Pages/Leaderboard.xaml
@@ -1,4 +1,5 @@
-<?xml version="1.0" encoding="utf-8" ?>
+<?xml version="1.0" encoding="utf-8"?>
+
 <ContentPage xmlns="http://schemas.microsoft.com/dotnet/2021/maui"
              xmlns:x="http://schemas.microsoft.com/winfx/2009/xaml"
              x:Class="Qwirkle.Pages.Leaderboard"
@@ -10,55 +11,56 @@
         <VerticalStackLayout Spacing="25" Padding="5, 5, 5, 10">
             <Grid Style="{StaticResource GridMain}">
                 <controls:GoBack></controls:GoBack>
-                <Label Text="Leaderboard" 
-                        Style="{StaticResource Title}"/>
+                <Label Text="Leaderboard"
+                       Style="{StaticResource Title}" />
             </Grid>
 
             <Border Style="{StaticResource TabBorder}">
 
                 <Border.Shadow>
-                    <Shadow/>
+                    <Shadow />
                 </Border.Shadow>
                 <Border.StrokeShape>
-                    <RoundRectangle CornerRadius="3"/>
+                    <RoundRectangle CornerRadius="3" />
                 </Border.StrokeShape>
 
                 <StackLayout>
                     <Grid ColumnDefinitions="4*, auto, 2*, auto, 2*, auto, 2*"
                           RowDefinitions="50">
-                        <Label 
+                        <Label
                             Text="Player tag"
-                            Style="{StaticResource ContentTab}"/>
+                            Style="{StaticResource ContentTab}" />
                         <Rectangle
                             Style="{StaticResource RectangleTab}"
-                            Grid.Column="1"/>
+                            Grid.Column="1" />
                         <Label
                             Grid.Column="2"
                             Text="Date"
-                            Style="{StaticResource ContentTab}"/>
+                            Style="{StaticResource ContentTab}" />
                         <Rectangle
                             Style="{StaticResource RectangleTab}"
-                            Grid.Column="3"/>
+                            Grid.Column="3" />
                         <Label
                             Grid.Column="4"
                             Text="Points"
-                            Style="{StaticResource ContentTab}"/>
+                            Style="{StaticResource ContentTab}" />
                         <Rectangle
                             Style="{StaticResource RectangleTab}"
-                            Grid.Column="5"/>
+                            Grid.Column="5" />
                         <Label
                             Grid.Column="6"
                             Style="{StaticResource ContentTab}"
-                            Text="Victories"/>
+                            Text="Victories" />
                     </Grid>
 
                     <CollectionView ItemsSource="{Binding Lb}">
                         <CollectionView.ItemsLayout>
-                            <GridItemsLayout Orientation="Vertical"/>
+                            <GridItemsLayout Orientation="Vertical" />
                         </CollectionView.ItemsLayout>
                         <CollectionView.ItemTemplate>
                             <DataTemplate>
-                                <controls:LeaderboardLine PlayerName="{Binding PlayerName}" Date="{Binding Date}" Points="{Binding Points}" Victories="{Binding Victories}" />
+                                <controls:LeaderboardLine PlayerName="{Binding PlayerName}" Date="{Binding Date}"
+                                                          Points="{Binding Points}" Victories="{Binding Victories}" />
                             </DataTemplate>
                         </CollectionView.ItemTemplate>
                     </CollectionView>
@@ -68,4 +70,4 @@
 
         </VerticalStackLayout>
     </ScrollView>
-</ContentPage>
+</ContentPage>
\ No newline at end of file
diff --git a/Qwirkle/QwirkleViews/Pages/Leaderboard.xaml.cs b/Qwirkle/QwirkleViews/Pages/Leaderboard.xaml.cs
index 85be440..83f3139 100644
--- a/Qwirkle/QwirkleViews/Pages/Leaderboard.xaml.cs
+++ b/Qwirkle/QwirkleViews/Pages/Leaderboard.xaml.cs
@@ -5,6 +5,6 @@ public partial class Leaderboard : ContentPage
 	public Leaderboard()
 	{
 		InitializeComponent();
-		BindingContext = ((App)Application.Current!).LD;
+		BindingContext = ((App)Application.Current!).Ld;
     }
 }
\ No newline at end of file
diff --git a/Qwirkle/TestBase/TestLeaderboard.cs b/Qwirkle/TestBase/TestLeaderboard.cs
index ee860c5..6c0d9fe 100644
--- a/Qwirkle/TestBase/TestLeaderboard.cs
+++ b/Qwirkle/TestBase/TestLeaderboard.cs
@@ -50,8 +50,6 @@ public class TestLeaderboard
 
         leaderboard.AddScoreInLead(game.ScoreBoard);
 
-        var game2 = new Game();
-
         game.AddPlayerInGame(playerstest);
 
         game.StartGame();
@@ -61,7 +59,7 @@ public class TestLeaderboard
 
         leaderboard.AddScoreInLead(game.ScoreBoard);
 
-        Assert.Equal(2, leaderboard.Lb[0].Victories);
+        Assert.Equal(2, leaderboard.Lb[1].Victories);
     }