namespace Webzine.WebApplication.Controllers
{
    using Microsoft.AspNetCore.Mvc;

    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>
        /// Initializes a new instance of the <see cref="ArtisteController"/> class.
        /// 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,
            IArtisteRepository artisteRepository)
        {
            this.logger = logger;
            this.logger.LogDebug("Initialisation du ArtisteController");
            this.artisteRepository = artisteRepository;
        }

        /// <summary>
        /// Prend en paramètre le nom de l'artiste (ex: "fatal-bazooka"), utilise la factory pour trouver l'artiste correspondant, et affiche sa page dédiée.
        /// </summary>
        /// <param name="nom">Le nom de l'artiste à rechercher, formaté en kebab-case (ex: "fatal-bazooka").</param>
        /// <returns>La vue de l'artiste avec son ViewModel, ou une redirection vers l'accueil si le nom est vide, ou une erreur 404 si l'artiste n'est pas trouvé.</returns>
        [HttpGet("/artiste/{nom}")]
        public IActionResult Index(string nom)
        {
            this.logger.LogInformation("Tentative d'accès à l'artiste avec le nom : {NomArtiste}", nom);

            if (string.IsNullOrEmpty(nom))
            {
                this.logger.LogWarning("Nom de l'artiste manquant dans la requête.");
                return this.RedirectToAction("Index", "Accueil");
            }

            // On transforme "fatal-bazooka" en "Fatal Bazooka" pour la factory
            string nomPropre = System.Globalization.CultureInfo.CurrentCulture.TextInfo
                .ToTitleCase(nom.Replace("-", " "));

            // On appelle la factory pour obtenir l'artiste unique
            var artiste = this.artisteRepository.FindByName(nomPropre);

            // Check if artiste was found
            if (artiste == null)
            {
                this.logger.LogWarning("Artiste non trouvé avec le nom : {NomArtiste}", nomPropre);
                return this.RedirectToAction("Index", "Accueil");
            }

            var viewModel = new ArtisteDetailsViewModel
            {
                IdArtiste = artiste.IdArtiste,
                Nom = artiste.Nom,
                Biographie = artiste.Biographie,
                AlbumsGroupes = artiste.Titres
                    .OrderBy(t => t.Libelle)
                    .GroupBy(t => t.Album)
                    .OrderBy(g => g.Key),
            };

            this.logger.LogInformation("Artiste trouvé : {NomArtiste}", nom);
            return this.View(viewModel);
        }
    }
}