Исправления после ревью

AsbCloudApp/Requests/GtrWithGetDataRequest.cs

@@ -0,0 +1,29 @@
+using System;
+
+namespace AsbCloudApp.Requests;
+
+/// <summary>
+/// Параметры запроса для получения загруженных данных ГТИ по скважине
+/// </summary>
+public class GtrWithGetDataRequest
+{
+	/// <summary>
+	/// Id скважины
+	/// </summary>
+	public int IdWell { get; set; }
+	
+	/// <summary>
+	/// Дата начала выборки.По умолчанию: текущее время - IntervalSec
+	/// </summary>
+	public DateTime? Begin { get; set; }
+
+	/// <summary>
+	/// Интервал времени даты начала выборки, секунды
+	/// </summary>
+	public int IntervalSec { get; set; } = 600;
+
+	/// <summary>
+	/// Желаемое количество точек. Если в выборке точек будет больше, то выборка будет прорежена.
+	/// </summary>
+	public int ApproxPointsCount { get; set; } = 1024;
+}

AsbCloudApp/Requests/ReportCreateRequest.cs

@@ -0,0 +1,34 @@
+using System;
+
+namespace AsbCloudApp.Requests;
+
+/// <summary>
+/// Ïàðàìåòðû çàïðîñà äëÿ ñîçäàíèÿ îò÷¸òà
+/// </summary>
+public class ReportCreateRequest
+{
+	/// <summary>
+	/// Id ñêâàæèíû
+	/// </summary>
+	public int IdWell { get; set; }
+	
+	/// <summary>
+	/// Øàã èíòåðâàëà
+	/// </summary>
+	public int StepSeconds { get; set; }
+	
+	/// <summary>
+	/// ôîðìàò îò÷åòà (0-PDF, 1-LAS)
+	/// </summary>
+	public int Format { get; set; }
+
+	/// <summary>
+	/// Äàòà íà÷àëà èíòåðâàëà
+	/// </summary>
+	public DateTime Begin { get; set; } = default;
+
+	/// <summary>
+	/// Äàòà îêîí÷àíèÿ èíòåðâàëà
+	/// </summary>
+	public DateTime End { get; set; } = default;
+}

AsbCloudApp/Requests/ReportGetSizeRequest.cs

@@ -0,0 +1,34 @@
+using System;
+
+namespace AsbCloudApp.Requests;
+
+/// <summary>
+/// Ïàðàìåòðû çàïðîñà äëÿ ïîëó÷åíèÿ ïðîãíîçèðóåìîãî êîëè÷åñòâî ñòðàíèö áóäóùåãî îò÷åòà
+/// </summary>
+public class ReportGetSizeRequest
+{
+	/// <summary>
+	/// Id ñêâàæèíû
+	/// </summary>
+	public int IdWell { get; set; }
+	
+	/// <summary>
+	/// Øàã èíòåðâàëà
+	/// </summary>
+	public int StepSeconds { get; set; }
+	
+	/// <summary>
+	/// Ôîðìàò îò÷åòà (0-PDF, 1-LAS)
+	/// </summary>
+	public int Format { get; set; }
+
+	/// <summary>
+	/// Äàòà íà÷àëà èíòåðâàëà
+	/// </summary>
+	public DateTime Begin { get; set; } = default;
+
+	/// <summary>
+	/// Äàòà îêîí÷àíèÿ èíòåðâàëà
+	/// </summary>
+	public DateTime End { get; set; } = default;
+}

AsbCloudWebApi/Controllers/ReportController.cs

@@ -3,10 +3,10 @@ using AsbCloudApp.Services;
 using AsbCloudWebApi.SignalR;
 using Microsoft.AspNetCore.Mvc;
 using Microsoft.AspNetCore.SignalR;
-using System;
 using System.Collections.Generic;
 using System.Threading;
 using System.Threading.Tasks;
+using AsbCloudApp.Requests;
 
 namespace AsbCloudWebApi.Controllers
 {
@@ -34,19 +34,14 @@ namespace AsbCloudWebApi.Controllers
         /// <summary>
         /// Создает отчет по скважине с указанными параметрами
         /// </summary>
-        /// <param name="idWell">id скважины</param>
+        /// <param name="request">Параметры запроса</param>
-        /// <param name="stepSeconds">шаг интервала</param>
-        /// <param name="format">формат отчета (0-PDF, 1-LAS)</param>
-        /// <param name="begin">дата начала интервала</param>
-        /// <param name="end">дата окончания интервала</param>
         /// <param name="token">Токен для отмены задачи</param>
         /// <returns>id фоновой задачи формирования отчета</returns>
         [HttpPost]
         [Permission]
         [ProducesResponseType(typeof(int), (int)System.Net.HttpStatusCode.OK)]
-        public async Task<IActionResult> CreateReportAsync(int idWell, int stepSeconds, int format,
+        public async Task<IActionResult> CreateReportAsync(ReportCreateRequest request, 
-            CancellationToken token,
+            CancellationToken token)
-            DateTime begin = default, DateTime end = default)
         {
             var idCompany = User.GetCompanyId();
             var idUser = User.GetUserId();
@@ -55,7 +50,7 @@ namespace AsbCloudWebApi.Controllers
                 return Forbid();
 
             if (!await wellService.IsCompanyInvolvedInWellAsync((int)idCompany,
-                idWell, token).ConfigureAwait(false))
+                request.IdWell, token).ConfigureAwait(false))
                 return Forbid();
 
             void HandleReportProgressAsync(object progress, string id) =>
@@ -68,8 +63,8 @@ namespace AsbCloudWebApi.Controllers
                     ).ConfigureAwait(false);
                 }, token);
 
-            var id = reportService.EnqueueCreateReportWork(idWell, (int)idUser,
+            var id = reportService.EnqueueCreateReportWork(request.IdWell, (int)idUser,
-                stepSeconds, format, begin, end, HandleReportProgressAsync);
+                request.StepSeconds, request.Format, request.Begin, request.End, HandleReportProgressAsync);
 
             return Ok(id);
         }
@@ -102,35 +97,27 @@ namespace AsbCloudWebApi.Controllers
         /// <summary>
         /// Возвращает прогнозируемое количество страниц будущего отчета
         /// </summary>
-        /// <param name="idWell">id скважины</param>
+        /// <param name="request">Параметры запроса</param>
-        /// <param name="begin">дата начала интервала</param>
-        /// <param name="end">дата окончания интервала</param>
-        /// <param name="stepSeconds">шаг интервала</param>
-        /// <param name="format">формат отчета (0-PDF, 1-LAS)</param>
         /// <param name="token">Токен для отмены задачи</param>
         /// <returns>прогнозируемое кол-во страниц отчета</returns>
         [HttpGet]
         [Route("reportSize")]
         [Permission]
         [ProducesResponseType(typeof(string), (int)System.Net.HttpStatusCode.OK)]
-        public async Task<IActionResult> GetReportSizeAsync(int idWell,
+        public async Task<IActionResult> GetReportSizeAsync(ReportGetSizeRequest request,
-            int stepSeconds, 
+            CancellationToken token)
-            int format,
-            CancellationToken token,
-            DateTime begin = default,
-            DateTime end = default)
         {
             int? idCompany = User.GetCompanyId();
 
             if (idCompany is null)
                 return Forbid();
 
-            if (!await wellService.IsCompanyInvolvedInWellAsync((int)idCompany,
+            if (!await wellService.IsCompanyInvolvedInWellAsync((int)idCompany, 
-                idWell, token).ConfigureAwait(false))
+                    request.IdWell, token).ConfigureAwait(false))
                 return Forbid();
 
-            int reportSize = reportService.GetReportPagesCount(idWell,
+            int reportSize = reportService.GetReportPagesCount(request.IdWell,
-                begin, end, stepSeconds, format);
+                request.Begin, request.End, request.StepSeconds, request.Format);
 
             return Ok(reportSize);
         }

AsbCloudWebApi/Controllers/SAUB/GtrWitsController.cs

@@ -4,10 +4,10 @@ using AsbCloudApp.Services;
 using AsbCloudWebApi.SignalR;
 using Microsoft.AspNetCore.Mvc;
 using Microsoft.AspNetCore.SignalR;
-using System;
 using System.Collections.Generic;
 using System.Threading;
 using System.Threading.Tasks;
+using AsbCloudApp.Requests;
 
 namespace AsbCloudWebApi.Controllers.SAUB
 {
@@ -39,16 +39,13 @@ namespace AsbCloudWebApi.Controllers.SAUB
         /// <summary>
         /// Получить загруженные данные ГТИ по скважине
         /// </summary>
-        /// <param name = "idWell" > id скважины</param>
+        /// <param name="request">Параметры запроса</param>
-        /// <param name = "begin" > дата начала выборки.По умолчанию: текущее время - intervalSec</param>
+        /// <param name = "token" >Токен завершения задачи</param>
-        /// <param name = "intervalSec" > интервал времени даты начала выборки, секунды</param>
-        /// <param name = "approxPointsCount" > желаемое количество точек. Если в выборке точек будет больше, то выборка будет прорежена.</param>
-        /// <param name = "token" > Токен завершения задачи</param>
         /// <returns></returns>
         [HttpGet("{idWell}")]
         [Permission]
-        public async Task<ActionResult<IEnumerable<WitsRecordDto>>> GetDataAsync(int idWell, DateTime? begin, 
+        public async Task<ActionResult<IEnumerable<WitsRecordDto>>> GetDataAsync(GtrWithGetDataRequest request,
-            CancellationToken token, int intervalSec = 600, int approxPointsCount = 1024)
+            CancellationToken token)
         {
             int? idCompany = User.GetCompanyId();
 
@@ -56,13 +53,13 @@ namespace AsbCloudWebApi.Controllers.SAUB
                 return Forbid();
 
             bool isCompanyOwnsWell = await wellService.IsCompanyInvolvedInWellAsync((int)idCompany,
-                idWell, token).ConfigureAwait(false);
+                request.IdWell, token).ConfigureAwait(false);
 
             if (!isCompanyOwnsWell)
                 return Forbid();
 
-            var content = await gtrRepository.GetAsync(idWell, begin,
+            var content = await gtrRepository.GetAsync(request.IdWell, request.Begin,
-              intervalSec, approxPointsCount, token).ConfigureAwait(false);
+              request.IntervalSec, request.ApproxPointsCount, token).ConfigureAwait(false);
 
             return Ok(content);
         }

AsbCloudWebApi/Controllers/SAUB/TelemetryDataBaseController.cs

@@ -72,10 +72,12 @@ namespace AsbCloudWebApi.Controllers.SAUB
         /// <returns></returns>
         [HttpGet("{idWell}")]
         [Permission]
-        public virtual async Task<ActionResult<IEnumerable<TDto>>> GetDataAsync(int idWell, CancellationToken token,
+        public virtual async Task<ActionResult<IEnumerable<TDto>>> GetDataAsync(int idWell,
             DateTime begin = default,
             int intervalSec = 600, 
-            int approxPointsCount = 1024)
+            int approxPointsCount = 1024,
+            //TODO: сделать cancellationToken обязательным 
+            CancellationToken token = default)
         {
             int? idCompany = User.GetCompanyId();