add TODO

Webzine.Repository/DbArtisteRepository.cs

@@ -13,7 +13,7 @@ namespace Webzine.Repository
 
     /// <summary>
     /// Initialise une classe <see cref="DbArtisteRepository"/> qui implémente l'interface <see cref="IArtisteRepository"/> pour gérer les opérations de base de données liées aux artistes.
-    /// Utilise <see cref="IArtisteRepository"/> en injection de dépendances.
+    /// Utilise <see cref="IArtisteRepository"/> en injection de dépendances. // TODO commentaire faux.
     /// </summary>
     public class DbArtisteRepository : IArtisteRepository
     {

Webzine.Repository/DbStyleRepository.cs

@@ -93,10 +93,10 @@ public class DbStyleRepository : IStyleRepository
         {
             this.logger.LogDebug("Recherche du style avec l'ID: {Id}", id);
 
-            if (id <= 0)
+            if (id <= 0) // TODO à supprimer, c'est au controller de vérifier la validité de l'id
             {
                 this.logger.LogWarning("Tentative de recherche d'un style avec un Id invalide: {Id}", id);
-                return new Style();
+                return new Style(); // TODO pas de retour d'objet vide
             }
 
             this.logger.LogDebug("Préparation de la requête avec inclusion des titres");

Webzine.Repository/DbTitreRepository.cs

@@ -33,7 +33,7 @@ public class DbTitreRepository : ITitreRepository
         try
         {
             this.logger.LogInformation("Ajout d'un nouveau titre: {Libelle}", titre.Libelle);
-            this.logger.LogDebug("Début de l'ajout du titre en base de données");
+            this.logger.LogDebug("Début de l'ajout du titre en base de données"); // TODO trop de logs
 
             this.context.Titres.Add(titre);
             this.context.SaveChanges();
@@ -288,7 +288,7 @@ public class DbTitreRepository : ITitreRepository
     {
         try
         {
-            this.logger.LogDebug("Récupération de tous les titres");
+            this.logger.LogDebug("Récupération de tous les titres"); // TODO trop de logs
             this.logger.LogDebug("Préparation de la requête avec les inclusions Artiste et Styles");
 
             var titres = this.context.Titres

Webzine.Repository/LocalArtisteRepository.cs

@@ -11,20 +11,20 @@ namespace Webzine.Repository
 
     /// <summary>
     /// Initialise une classe <see cref="LocalArtisteRepository"/> qui implémente l'interface <see cref="IArtisteRepository"/> pour gérer les opérations de base de données liées aux artistes.
-    /// Utilise <see cref="IArtisteRepository"/> en injection de dépendances.
+    /// Utilise <see cref="IArtisteRepository"/> en injection de dépendances. // TODO faux.
     /// </summary>
     public class LocalArtisteRepository : IArtisteRepository
     {
         private readonly ILogger<LocalArtisteRepository> logger;
 
-        // private readonly List<Artiste> artistes;
+        // private readonly List<Artiste> artistes; // TODO code mort
         private readonly InMemoryDataStore dataStore;
 
         /// <summary>
         /// Initializes a new instance of the <see cref="LocalArtisteRepository"/> class.
         /// Est liéee à une liste d'artistes en local et utilise un logger pour enregistrer les opérations effectuées sur les artistes.
         /// </summary>
-        /// <param name="artistes">La liste des artistes à initialiser. Ne peut pas être null.</param>
+        /// <param name="artistes">La liste des artistes à initialiser. Ne peut pas être null.</param> // TODO pas le bon paramètre
         /// <param name="logger">Le logger à utiliser pour enregistrer les messages de journalisation. Ne peut pas être null.</param>
         public LocalArtisteRepository(InMemoryDataStore dataStore, ILogger<LocalArtisteRepository> logger)
         {
@@ -37,7 +37,7 @@ namespace Webzine.Repository
         /// <inheritdoc/>
         public void Add(Artiste artiste)
         {
-            throw new NotSupportedException("Mode Local");
+            throw new NotSupportedException("Mode Local"); // TODO à implémenter
         }
 
         /// <inheritdoc/>
@@ -52,7 +52,7 @@ namespace Webzine.Repository
             var artiste = this.dataStore.Artistes.First(a => a.IdArtiste == id);
             if (artiste == null)
             {
-                return new Artiste();
+                return new Artiste(); // TODO ne pas retourner un objet vide, FirstOrDefault et gérer le null dans le controller
             }
 
             return artiste;

Webzine.Repository/LocalCommentaireRepository.cs

@@ -38,7 +38,7 @@ namespace Webzine.Repository
         /// <inheritdoc/>
         public void Add(Commentaire commentaire)
         {
-            throw new NotSupportedException("Mode Local");
+            throw new NotSupportedException("Mode Local"); // TODO à implémenter
         }
 
         /// <inheritdoc/>

Webzine.Repository/LocalTitreRepository.cs

@@ -34,7 +34,9 @@ public class LocalTitreRepository : ITitreRepository
     /// <inheritdoc/>
     public int Count()
     {
-        var count = this.dataStore.Titres.Count();
+        var count = this.dataStore.Titres.Count(); // TODO une seule ligne, et attention car les deux méthodes s'appelent pareil,
+
+        // il faut faire attention à ne pas confondre avec la méthode Count() de l'interface ITitreRepository
         return count;
     }
 
@@ -62,14 +64,14 @@ public class LocalTitreRepository : ITitreRepository
     /// <inheritdoc/>
     public void IncrementNbLikes(Titre titre)
     {
-        titre.NbLikes++;
+        titre.NbLikes++; // TODO rien n'est enregistré
     }
 
     /// <inheritdoc/>
     public IEnumerable<Titre> Search(string mot)
     {
         return this.dataStore.Titres
-            .Where(t => t.Libelle != null && t.Libelle.Contains(mot));
+            .Where(t => t.Libelle != null && t.Libelle.Contains(mot)); // TODO attention au null, et à la casse, et à l'indexation pour les performances
     }
 
     /// <inheritdoc/>