Я очищаю старый код.
Поделитесь своим мнением об использовании UpdateDataAsyncInternal. Повысит ли это безопасность кода? Должны ли мы удалить UpdateDataAsyncInternal и вместо этого объединить всю логику в UpdateDataAsync? Такой подход может привести к более чистому коду и упростить создание тестовых примеров.
Любой вклад будет принят с благодарностью. Спасибо!
public class GetStuffFromDBService(IConfiguration config) : IGetStuffFromDBService
{
private readonly IConfiguration _config = config;
public async Task UpdateDataAsync(DetailsOutputDTO detailsInputDTO)
=> await UpdateDataAsyncInternal(detailsInputDTO);
private static async Task UpdateDataAsyncInternal(DetailsOutputDTO detailsInputDTO)
{
try
{
using SqlConnection con = new(config.ConnectionStrings.ConnectionString1);
await con.OpenAsync();
using SqlCommand com = ...
var results = await com.ExecuteNonQueryAsync();
message = "Good";
IsSuccess = true;
}
catch (Exception ex)
{
await new ErrorLogService().LogError(
new ErrorLog()
{
...
});
message = "something went wrong"; //ex.Message;
IsSuccess = false;
}
return new DetailsOutputDTO() { IsSuccess = IsSuccess, Message = message};
}
}
Подробнее здесь: https://stackoverflow.com/questions/790 ... on-c-sharp
