Предложение кода для рефакторинга и оптимизации частной внутренней функции Код C# [закрыто] ⇐ C#

[Anonymous]

Я очищаю старый код.
Поделитесь своим мнением об использовании 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