#84 Implémentation du repository Artiste. Le controler Artiste pour la partie utilisateur et la partie admin. Ajout de commentaires sur toutes les méthodes.

Webzine.Repository.Contracts/IArtisteRepository.cs

@@ -1,17 +1,46 @@
-// using Webzine.Entity;
+namespace Webzine.Repository.Contracts
-
-namespace Webzine.Repository.Contracts
 {
+    using Webzine.Entity;
+    /// <summary>
+    /// Défini une interface <see cref="IArtisteRepository"/> pour gérer les opérations de base de données liées aux artistes.  
+    /// </summary>
     public interface IArtisteRepository
     {
-        // void Add(Artiste artiste);
+        /// <summary>
+        /// Ajoute un nouvel artiste.
+        /// </summary>
+        /// <param name="artiste">L'artiste à ajouter à la collection. Ne peut pas être null.</param>
+        void Add(Artiste artiste);
 
-        // void Delete(Artiste artiste);
+        /// <summary>
+        /// Supprime un artiste.
+        /// </summary>
+        /// <param name="artiste">L'artiste à supprimer.</param>
+        void Delete(Artiste artiste);
 
-        // Artiste Find(int id);
+        /// <summary>
+        /// Récupère un artiste par son identifiant unique. Si aucun artiste n'est trouvé, retourne null.
+        /// </summary>
+        /// <param name="id">L'identifiant de l'artiste.</param>
+        /// <returns></returns>
+        Artiste Find(int id);
+        /// <summary>
+        /// Récupère un artiste par son nom. Si aucun artiste n'est trouvé, retourne null.
+        /// </summary>
+        /// <param name="name">Le nom de l'artiste.</param>
+        /// <returns>L'artiste recherché ou null.</returns>
+        Artiste FindByName(string name);
 
-        // IEnumerable<Artiste> FindAll();
+        /// <summary>
+        /// Récupère tous les artistes disponibles dans la collection. Si aucun artiste n'est trouvé, retourne une collection vide.
+        /// </summary>
+        /// <returns>Retourne une collection d'artistes.</returns>
+        IEnumerable<Artiste> FindAll();
 
-        // void Update(Artiste artiste);
+        /// <summary>
+        /// Met à jour les informations d'un artiste existant dans la collection.
+        /// </summary>
+        /// <param name="artiste">L'artiste à mettre à jour.</param>
+        void Update(Artiste artiste);
     }
 }

Webzine.Repository/LocalArtisteRepository.cs

@@ -0,0 +1,92 @@
+namespace Webzine.Repository
+{
+    using Microsoft.EntityFrameworkCore;
+    using Webzine.EntitiesContext;
+    using Webzine.Entity;
+    using Webzine.Repository.Contracts;
+
+    /// <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.
+    /// </summary>
+    public class LocalArtisteRepository : IArtisteRepository
+    {
+        /// <summary>
+        ///  Contexte de la base de données utilisé pour accéder aux entités et effectuer des opérations de persistance. Injecté via le constructeur.
+        /// </summary>
+        private WebzineDbContext _context;
+
+        /// <summary>
+        /// Initializes a new instance of the <see cref="LocalArtisteRepository"/> class.
+        /// </summary>
+        /// <param name="context">Le contexte de base de données à utiliser pour accéder aux entités et effectuer des opérations de
+        /// persistance. Ne peut pas être null.</param>
+        public LocalArtisteRepository(WebzineDbContext context)
+        {
+            this._context = context;
+        }
+
+        /// <summary>
+        /// Méthode pour ajouter un nouvel artiste à la base de données.
+        /// </summary>
+        /// <param name="artiste">L'artiste à ajouter. Ne peut pas être null.</param>
+        public void Add(Artiste artiste)
+        {
+            this._context.Artistes.Add(artiste);
+            this._context.SaveChanges();
+        }
+
+        /// <summary>
+        /// Méthode pour supprimer un artiste de la base de données.
+        /// </summary>
+        /// <param name="artiste">L'artiste à supprimer.</param>
+        public void Delete(Artiste artiste)
+        {
+            this._context.Artistes.Remove(artiste);
+            this._context.SaveChanges();
+        }
+
+        /// <summary>
+        /// Récupère un artiste avec son id.
+        /// </summary>
+        /// <param name="id"> L'id de l'artiste.</param>
+        /// <returns>L'artiste correspondant à l'identifiant fourni, ou null si aucun artiste trouvé.</returns>
+        public Artiste Find(int id)
+        {
+            return this._context.Artistes
+                .Include(a => a.Titres)
+                .FirstOrDefault(a => a.IdArtiste == id);
+        }
+
+        /// <summary>
+        /// Récupère un artiste avec son nom.
+        /// </summary>
+        /// <param name="nom">Nom de l'artiste.</param>
+        /// <returns>L'artiste correspondant à l'identifiant fourni, ou null si aucun artiste trouvé.</returns>
+        public Artiste FindByName(string nom)
+        {
+            return this._context.Artistes
+                .Include(a => a.Titres)
+                .FirstOrDefault(a => a.Nom == nom);
+        }
+
+        /// <summary>
+        /// Retourne tous les artistes présents dans la base de données.
+        /// </summary>
+        /// <returns>La liste des artistes.</returns>
+        public IEnumerable<Artiste> FindAll()
+        {
+            return this._context.Artistes.ToList();
+        }
+
+        /// <summary>
+        /// Mets à jour un artiste existant dans la base de données avec les nouvelles informations fournies dans l'objet artiste.
+        /// </summary>
+        /// <param name="artiste">L'artiste à mettre à jour.</param>
+        public void Update(Artiste artiste)
+        {
+            this._context.Artistes.Update(artiste);
+            this._context.SaveChanges();
+        }
+    }
+}

Webzine.Repository/Webzine.Repository.csproj

@@ -24,6 +24,7 @@
     </ItemGroup>
 
     <ItemGroup>
+      <ProjectReference Include="..\Webzine.EntitiesContext\Webzine.EntitiesContext.csproj" />
       <ProjectReference Include="..\Webzine.Entity\Webzine.Entity.csproj" />
       <ProjectReference Include="..\Webzine.Repository.Contracts\Webzine.Repository.Contracts.csproj" />
     </ItemGroup>

Webzine.WebApplication/Areas/Administration/Controllers/ArtisteController.cs

@@ -1,6 +1,7 @@
 using Microsoft.AspNetCore.Mvc;
 using Webzine.Entity;
 using Webzine.Entity.Fixtures;
+using Webzine.Repository.Contracts;
 using Webzine.WebApplication.Areas.Administration.ViewModels.Artiste;
 
 namespace Webzine.WebApplication.Areas.Administration.Controllers;
@@ -10,16 +11,17 @@ public class ArtisteController : Controller
 {
     // Injection du logger via le constructeur
     private readonly ILogger<ArtisteController> _logger;
-    private readonly List<Artiste> _artistes;
+    private readonly IArtisteRepository _artisteRepository;
+    private readonly List<Artiste> _artistes = new List<Artiste>();
 
 
-    public ArtisteController(ILogger<ArtisteController> logger)
+    public ArtisteController(ILogger<ArtisteController> logger,
+        IArtisteRepository artisteRepository)
     {
         this._logger = logger;
         this._logger.LogDebug(1, "initialisation du ArtisteController d'administration");
-        var factory = new DataFactory();
+        this._artisteRepository = artisteRepository;
-
+        this._artistes.AddRange(this._artisteRepository.FindAll());
-        _artistes = factory.GenerateArtists(10);
     }
     
     /// <summary>
@@ -30,7 +32,9 @@ public class ArtisteController : Controller
     public IActionResult Index()
     {
 
-        var _artistes_ordre = _artistes.OrderBy(t => t.Nom).ToList();
+        this._logger.LogInformation("ID 1", this._artisteRepository.Find(1));
+        var _artistes_ordre = this._artistes.OrderBy(t => t.Nom).ToList();
+        this._logger.LogInformation("LISTE ARTISTES ORDRE", _artistes_ordre);
 
         this._logger.LogInformation("Initialisation du contrôleur TitreController pour l'Administration.");
 
@@ -60,7 +64,7 @@ public class ArtisteController : Controller
     /// <returns>Redirection.</returns>
     public IActionResult Edit(int id)
     {
-        var artiste = _artistes.First(t => t.IdArtiste == id);
+        var artiste = this._artistes.First(t => t.IdArtiste == id);
 
         var model = new AdminArtisteForm
         {
@@ -79,7 +83,7 @@ public class ArtisteController : Controller
     /// <returns>Redirection.></returns>
     public IActionResult Delete(int id)
     {
-        var artiste = _artistes.First(t => t.IdArtiste == id);
+        var artiste = this._artistes.First(t => t.IdArtiste == id);
         var model = new AdminArtisteForm
         {
             Id = id,

Webzine.WebApplication/Controllers/ArtisteController.cs

@@ -1,21 +1,26 @@
-using Microsoft.AspNetCore.Mvc;
+namespace Webzine.WebApplication.Controllers
-using Webzine.Entity.Fixtures;
-
-namespace Webzine.WebApplication.Controllers
 {
+    using Microsoft.AspNetCore.Mvc;
+    using Webzine.Entity.Fixtures;
+    using Webzine.Repository.Contracts;
+    using Webzine.WebApplication.ViewModels.Artiste;
+
     public class ArtisteController : Controller
     {
         // Injection du logger via le constructeur
         private readonly ILogger<ArtisteController> _logger;
+        private readonly IArtisteRepository _artisteRepository;
 
         /// <summary>
-        /// Initialise une nouvelle instance du <see cref="ArtisteController"/> avec un service de journalisation injecté.
+        /// Initialise une nouvelle instance du <see cref="ArtisteController"/>. avec un service de journalisation injecté.
         /// </summary>
         /// <param name="logger">Service de journalisation injecté pour enregistrer les événements et les erreurs.</param>
-        public ArtisteController(ILogger<ArtisteController> logger)
+        public ArtisteController(ILogger<ArtisteController> logger,
+            IArtisteRepository artisteRepository)
         {
             this._logger = logger;
-            this._logger.LogDebug(1, "initialisation du ArtisteController");
+            this._logger.LogDebug("Initialisation du ArtisteController");
+            this._artisteRepository = artisteRepository;
         }
 
         /// <summary>
@@ -31,7 +36,7 @@ namespace Webzine.WebApplication.Controllers
             if (string.IsNullOrEmpty(nom))
             {
                 this._logger.LogWarning("Nom de l'artiste manquant dans la requête.");
-                return RedirectToAction("Index", "Accueil");
+                return RedirectToAction("Index");
             }
 
             // On transforme "fatal-bazooka" en "Fatal Bazooka" pour la factory
@@ -39,17 +44,28 @@ namespace Webzine.WebApplication.Controllers
                 .ToTitleCase(nom.Replace("-", " "));
 
             // On appelle la factory pour obtenir l'artiste unique
-            var artiste = ArtisteFactory.SeedArtisteByName(nomPropre);
+            var artiste = this._artisteRepository.FindByName(nomPropre);
 
             if (artiste == null)
             {
                 this._logger.LogWarning("Artiste non trouvé pour le nom : {NomArtiste}", nomPropre);
                 return RedirectToAction("Index");
             }
+            var viewModel = new ArtisteDetailsViewModel
+            {
+                IdArtiste = artiste.IdArtiste,
+                Nom = artiste.Nom,
+                Biographie = artiste.Biographie,
+                // On effectue le groupement ici une bonne fois pour toutes
+                AlbumsGroupes = artiste.Titres
+                .OrderBy(t => t.Libelle)
+                .GroupBy(t => t.Album)
+                .OrderBy(g => g.Key),
+            };
 
             this._logger.LogInformation("Artiste trouvé : {NomArtiste}", nom);
 
-            return View(artiste);
+            return View(viewModel);
         }
     }
 }

Webzine.WebApplication/Program.cs

@@ -22,6 +22,7 @@ try
         .AddRazorRuntimeCompilation();
 
     builder.Services.AddSingleton<ITitreRepository, LocalEntityRepository>();
+    builder.Services.AddScoped<IArtisteRepository, LocalArtisteRepository>();
 
     builder.Services.AddDbContext<WebzineDbContext>(options =>
         options.UseSqlite(builder.Configuration.GetConnectionString("DefaultConnection")));

Webzine.WebApplication/Webzine.WebApplication.csproj

@@ -17,7 +17,6 @@
     </ItemGroup>
 
     <ItemGroup>
-      <Folder Include="Data\" />
       <Folder Include="wwwroot\data\" />
       <Folder Include="wwwroot\lib\" />
     </ItemGroup>