Merge pull request 'Better code' (#44) from Matheo into master

Reviewed-on: #44

src/DataContractPersistance/DataContractJSON.cs

@@ -13,7 +13,7 @@ namespace DataContractPersistance
     /// <summary>
     /// Implémentation de l'interface IPersistanceManager utilisant la sérialisation avec DataContract.
     /// </summary>
-    public class DataContractJSON : IPersistanceManager
+    public class DataContractJson : IPersistanceManager
     {
         /// <summary>
         /// Obtient ou définit le nom du fichier de sauvegarde JSON.
@@ -40,7 +40,7 @@ namespace DataContractPersistance
                 data = jsonSerializer.ReadObject(stream2) as DataToPersist;
             }
 
-            return (data.Oeuvres, data.Utilisateurs);
+            return (data!.Oeuvres, data.Utilisateurs);
         }
 
         /// <summary>

src/DataContractPersistance/DataContractXML.cs

@@ -14,7 +14,7 @@ namespace DataContractPersistance
     /// <summary>
     /// Implémentation de l'interface IPersistanceManager utilisant la sérialisation avec DataContract.
     /// </summary>
-    public class DataContractXML : IPersistanceManager
+    public class DataContractXml : IPersistanceManager
     {
         /// <summary>
         /// Obtient ou définit le nom du fichier de sauvegarde XML.
@@ -47,7 +47,9 @@ namespace DataContractPersistance
                 data = new DataToPersist(); // Si le fichier n'existe pas, créez une nouvelle liste
             }
 
-            return (data.Oeuvres, data.Utilisateurs);
+            
+            return (data!.Oeuvres, data.Utilisateurs);
+            
         }
 
         /// <summary>

src/MangaMap/App.xaml.cs

@@ -39,8 +39,8 @@ public partial class App : Application
 
         if (File.Exists(Path.Combine(FilePath, FileName)))
         {
-            MyManager = new Manager(new DataContractPersistance.DataContractXML()); // Utilise le DataContract comme moyen de persistance.
+            MyManager = new Manager(new DataContractPersistance.DataContractXml()); // Utilise le DataContract comme moyen de persistance.
-            //MyManager = new Manager(new Stub.DataContractJSON()); // Utilise le DataContract comme moyen de persistance.
+            //MyManager = new Manager(new Stub.DataContractJson()); // Utilise le DataContract comme moyen de persistance.
         }
 
         MyManager.charger();
@@ -50,8 +50,7 @@ public partial class App : Application
 
         if (!File.Exists(Path.Combine(FilePath, FileName)))
         {
-            MyManager.Persistance = new DataContractPersistance.DataContractXML(); // Utilise le Stub comme moyen de persistance.
+            MyManager.Persistance = new DataContractPersistance.DataContractXml(); // Utilise le Stub comme moyen de persistance.
-            //MyManager = new Manager(new Stub.DataContractJSON());
         }
 
         MyManager.sauvegarder();

src/Models/Admin.cs

@@ -27,11 +27,11 @@ namespace Models
         /// <summary>
         /// Méthode permettant d'ajouter un anime.
         /// </summary>
-        public void ajouterAnime() { }
+        static void ajouterAnime() { }
 
         /// <summary>
         /// Méthode permettant de supprimer un anime.
         /// </summary>
-        public void supprimerAnime() { }
+        static void supprimerAnime() { }
     }
 }

src/Models/Manager.cs

@@ -17,7 +17,7 @@ namespace Models
         /// <summary>
         /// Obtient ou définit le gestionnaire de persistance utilisé pour charger et sauvegarder les données.
         /// </summary>
-        public IPersistanceManager Persistance { get; set; }
+        public IPersistanceManager ?Persistance { get; set; }
 
         /// <summary>
         /// Obtient la liste des administrateurs de l'application.
@@ -29,7 +29,7 @@ namespace Models
         /// </summary>
         public List<Utilisateur> Utilisateurs { get; private set; }
 
-        private ObservableCollection<Oeuvre> oeuvres;
+        private ObservableCollection<Oeuvre> ?oeuvres;
 
         /// <summary>
         /// Obtient ou définit la collection observable des oeuvres de l'application.
@@ -38,7 +38,7 @@ namespace Models
         {
             get
             {
-                return oeuvres;
+                return oeuvres ??= new ObservableCollection<Oeuvre>();
             }
             set
             {
@@ -46,7 +46,7 @@ namespace Models
                 OnPropertyChanged();
             }
         }
-
+           
         public event PropertyChangedEventHandler? PropertyChanged;
 
         void OnPropertyChanged([CallerMemberName] string? propertyName = null)
@@ -94,12 +94,19 @@ namespace Models
         /// </summary>
         public void charger()
         {
-            var donne = Persistance.chargeDonne();
+            if (Persistance != null)
-            foreach (var item in donne.Item1)
             {
-                Oeuvres.Add(item);
+
+                var donne = Persistance.chargeDonne();
+
+                foreach (var item in donne.Item1)
+                {
+                    Oeuvres.Add(item);
+                }
+                Utilisateurs.AddRange(donne.Item2);
             }
-            Utilisateurs.AddRange(donne.Item2);
+            
+            
         }
 
         /// <summary>
@@ -107,7 +114,7 @@ namespace Models
         /// </summary>
         public void sauvegarder()
         {
-            Persistance.sauvegarder(Oeuvres, Utilisateurs);
+            Persistance?.sauvegarder(Oeuvres, Utilisateurs);
         }
     }
 }

src/Models/Personne.cs

@@ -38,7 +38,7 @@ namespace Models
         /// <returns>True si la modification a réussi, False sinon.</returns>
         public bool MofifierMotDePasse(string MotDePasse)
         {
-            string test = "";
+            string ?test = "";
             test = Console.ReadLine();
 
             if (test == this.MotDePasse)
@@ -57,7 +57,7 @@ namespace Models
         /// <returns>True si la modification a réussi, False sinon.</returns>
         public bool MofifierEmail(string Email)
         {
-            string test = "";
+            string ?test = "";
             test = Console.ReadLine();
 
             if (test == this.MotDePasse)
@@ -76,7 +76,7 @@ namespace Models
         /// <returns>True si la modification a réussi, False sinon.</returns>
         public bool MofifierPseudo(string Pseudo)
         {
-            string test = "";
+            string ?test = "";
             test = Console.ReadLine();
 
             if (test == this.MotDePasse)

src/Stub/Stub.cs

@@ -38,8 +38,6 @@ namespace Stub
             l1.Add(o1); l1.Add(o2); l1.Add(o3); l1.Add(o4); l1.Add(o5); l1.Add(o6);
             l2.Add(u1); l2.Add(u2); l2.Add(u3);
 
-            //u1.ListeOeuvreEnVisionnage.Add(o1);
-
             return (l1, l2);
         }