Как обрабатывать изменение универсального типа при перемещении вверх по стеку вызовов при использовании шаблона результа ⇐ C#

[Anonymous]

Я изучаю различные способы обработки ошибок в API-интерфейсах .NET REST. Я увидел, как появился шаблон результатов, и теперь пытаюсь реализовать его в личном проекте. Однако я не могу не чувствовать, что чего-то не хватает, поскольку пользоваться им очень неудобно. Я попытаюсь объяснить на примере:
Здесь, в моем репозитории, я использую простой список в памяти, в котором хранятся мои пользователи:

Код: Выделить всё

    public ResultAdd(User user)
{
var userExists = CheckIfUserNameExists(user.GetUserName());

if (userExists)
{
var error = UserErrors.UserAlreadyExistsError(user.GetUserName());
return Result.Failure(error);
}

_users.Add(user);

return Result.Success(user.GetId());
}

Если имя пользователя существует, я возвращаю результат с ошибкой и возвращаю ошибку. Если пользователь успешно добавлен в список, я хочу вернуть идентификатор пользователя вызывающему абоненту, чтобы он мог позже получить пользователя по идентификатору. Поэтому я заставляю свой результат принимать универсальный тип, который будет типом значения, которое он возвращает вызывающему объекту:

Код: Выделить всё

public class Result
{
private Result(T value, bool isSuccess, Error error)
{
if(isSuccess && error != Error.None || !isSuccess && error == Error.None)
{
throw new ArgumentException("Invalid error", nameof(error));
}

IsSuccess = isSuccess;
Value = value;
Error = error;
}
public T Value { get;  }
public bool IsSuccess { get; }
public Error Error { get; }
public static Result Success(T value) => new(value, true, Error.None);
public static Result Failure(Error error) => new(default!, false, error);
}

Отчасти неловкость заключается в том, что на уровне обслуживания я хочу вернуть Result другого типа, где SignUpResponse является моим DTO.

Код: Выделить всё

public Result SignUp(SignUpRequest request)
{
var user = new User(
request.FirstName,
request.LastName,
request.UserName,
request.Password
);

var idResult = _userRepository.Add(user);

return idResult.Map(id => new SignUpResponse(id));
}

Чтобы обойти эту проблему, я создал расширение Map для результата:

Код: Выделить всё

public static Result Map(this Result result, Func mapper)
{
return result.IsSuccess
? Result.Success(mapper(result.Value))
: Result.Failure(result.Error);
}

Что мне не нравится в этом подходе:

• Функция Map скрывает тот факт, что она обрабатывает сценарии успеха и неудачи, поэтому делает код менее читабельным.
• В результатах сбоя мне все равно нужно пропустить тип T, даже если он не используется, что кажется странным
• I Могу себе представить, что по мере масштабирования это станет еще более неудобным, поскольку мне, возможно, придется продолжать картографирование в нескольких слоях.

Итак, мой вопрос: если бы я воспользовался этим подходом, что думают люди о том, как обойти эти вещи? Не уверен, что я неправильно реализовал шаблон, выявило ли это какое-то большее непонимание дизайна программного обеспечения, которое у меня есть, или это хорошо известный компромисс. Будем очень признательны за любые советы о том, как использовать функциональный подход к обработке ошибок и делать это хорошо!

Подробнее здесь: https://stackoverflow.com/questions/798 ... -using-the