diff --git a/Sources/apiLOL/Controllers/ControllerChampions.cs b/Sources/apiLOL/Controllers/ControllerChampions.cs index cfc0d31..0817c85 100644 --- a/Sources/apiLOL/Controllers/ControllerChampions.cs +++ b/Sources/apiLOL/Controllers/ControllerChampions.cs @@ -6,38 +6,38 @@ using Model; namespace apiLOL.Controllers { - [ApiController] + [ApiController] [Route("api/v1/[controller]")] [ApiVersion("1.0")] public class ControllerChampions : Controller { - private readonly IDataManager data; - // EFdata manager qui implémente l'interface IDataManager - // coté client : Refaire un APIdata manager qui implémente l'interface IDataManager + private readonly IDataManager _data; private readonly ILogger _logger; public ControllerChampions(IDataManager manager, ILogger log) { - data = manager; + _data = manager; _logger = log; } - // GET: api/ [HttpGet] - public async Task Get([FromQuery]int index = 0, int count = 10, string? name = "") + [ProducesResponseType(typeof(ChampionPageDTO), 200)] + public async Task Get([FromQuery] int index = 0, int count = 10, string? name = "") { + _logger.LogInformation($"methode Get de ControllerChampions appelée index:{index}, count: {count} et name:{name}"); - int nbChampions = await data.ChampionsMgr.GetNbItems(); + int nbChampions = await _data.ChampionsMgr.GetNbItems(); _logger.LogInformation($"Nombre de champions : {nbChampions}"); - //var champs = (await data.ChampionsMgr.GetItems(index, count)).Where(Model => Model.Name.Contains(name)).Select(Model => Model.ToDTO()); - var champs = (await data.ChampionsMgr.GetItems(index, await data.ChampionsMgr.GetNbItems())).Where(Model => Model.Name.Contains(name)).Skip(index * count).Take(count).Select(Model => Model.ToDTO()); + var champions = await _data.ChampionsMgr.GetItems(index, nbChampions); + var filteredChampions = champions.Where(Model => Model.Name.Contains(name)).Skip(index * count).Take(count); + var championDTO = filteredChampions.Select(Model => Model.ToDTO()); var page = new ChampionPageDTO { - Data = (IEnumerable)champs, + Data = championDTO, Index = index, Count = count, TotalCount = nbChampions @@ -46,26 +46,25 @@ namespace apiLOL.Controllers } - // GET api//Charle [HttpGet] [Route("{name}")] + [ProducesResponseType(typeof(ChampionDTO), 200)] public async Task GetChampion(string name) { _logger.LogInformation($"methode GetChampion de ControllerChampions appelée avec le paramètre {name}"); try { - var champs = (await data.ChampionsMgr.GetItemsByName(name, 0, 1)); + var champs = (await _data.ChampionsMgr.GetItemsByName(name, 0, 1)); return Ok(champs.First().ToDTO()); } - catch(Exception ex) - { + catch (Exception ex) + { _logger.LogError($"erreur methode Get de ControllerChampions: {ex}"); return BadRequest("erreur de nom de champion"); - } + } } - // POST api/ [HttpPost] public async Task Post(ChampionDTO champDTO) { @@ -73,9 +72,9 @@ namespace apiLOL.Controllers try { Champion tmp = champDTO.ToModel(); - Champion champion = await data.ChampionsMgr.AddItem(tmp); + Champion champion = await _data.ChampionsMgr.AddItem(tmp); ChampionDTO dtoChamp = champion.ToDTO(); - return CreatedAtAction(nameof(GetChampion), new { name = dtoChamp.Name }, dtoChamp); + return CreatedAtAction(nameof(GetChampion), new {name = dtoChamp.Name}, dtoChamp); } catch (Exception ex) { @@ -84,15 +83,15 @@ namespace apiLOL.Controllers } } - // PUT api//5 [HttpPut("{name}")] public async Task Put(string name, string bio) { - _logger.LogInformation($"methode Put de ControllerChampions appelée avec le paramètre name: {name} et bio: {bio}"); + _logger.LogInformation( + $"methode Put de ControllerChampions appelée avec le paramètre name: {name} et bio: {bio}"); try { - var champs = (await data.ChampionsMgr.GetItemsByName(name, 0, 1)).First(); + var champs = (await _data.ChampionsMgr.GetItemsByName(name, 0, 1)).First(); champs.Bio = bio; return Ok(champs.ToDTO()); } @@ -103,7 +102,6 @@ namespace apiLOL.Controllers } } - // DELETE api//5 [HttpDelete("{name}")] public async Task Delete(string name) { @@ -111,8 +109,8 @@ namespace apiLOL.Controllers try { - var champ = (await data.ChampionsMgr.GetItemsByName(name, 0, 1)).First(); - await data.ChampionsMgr.DeleteItem(champ); + var champ = (await _data.ChampionsMgr.GetItemsByName(name, 0, 1)).First(); + await _data.ChampionsMgr.DeleteItem(champ); return Ok(champ.ToDTO()); } catch (Exception ex) @@ -123,4 +121,41 @@ namespace apiLOL.Controllers } } + + [ApiController] + [Route("api/v2/[controller]")] + [ApiVersion("2.0")] + + public class ControllerChampionsSecondVersion : Controller + { + private readonly ILogger _logger; + + public ControllerChampionsSecondVersion(ILogger log) + { + _logger = log; + } + + + [HttpGet()] + public string Get(int id) + { + return "Version 2 of GET"; + } + + [HttpPost] + public void Post([FromBody] string value) + { + } + + [HttpPut("{id}")] + public void Put(int id, [FromBody] string value) + { + } + + [HttpDelete("{id}")] + public void Delete(int id) + { + } + + } } \ No newline at end of file diff --git a/Sources/apiLOL/Program.cs b/Sources/apiLOL/Program.cs index 4535ba4..80d2be9 100644 --- a/Sources/apiLOL/Program.cs +++ b/Sources/apiLOL/Program.cs @@ -1,3 +1,5 @@ +using Microsoft.AspNetCore.Mvc.Versioning; +using Microsoft.AspNetCore.Mvc; using Model; using StubLib; @@ -10,18 +12,27 @@ builder.Services.AddControllers(); builder.Services.AddEndpointsApiExplorer(); builder.Services.AddSwaggerGen(); -builder.Services.AddSingleton(); +builder.Services.AddSingleton(); // Scoped et Transient sont des types de cycle de vie // Transient : une instance a chaque contruction d'objet // Scoped : une instance par requete client +// Configure API versioning +builder.Services.AddApiVersioning(options => +{ + options.ReportApiVersions = true; + options.AssumeDefaultVersionWhenUnspecified = true; + options.DefaultApiVersion = new ApiVersion(1, 0); + options.ApiVersionReader = new HeaderApiVersionReader("api-version"); +}); + var app = builder.Build(); // Configure the HTTP request pipeline. if (app.Environment.IsDevelopment()) { - app.UseSwagger(); - app.UseSwaggerUI(); + app.UseSwagger(); + app.UseSwaggerUI(); } app.UseHttpsRedirection(); @@ -30,4 +41,4 @@ app.UseAuthorization(); app.MapControllers(); -app.Run(); +app.Run(); \ No newline at end of file