MVC5 비동기 ActionResult. 그게 가능하니?

Question

신청서에는 CQRS가 있습니다 : IAsyncCommand은 IAsyncCommandHandler<IAsyncCommand>입니다.

보통 명령은 다음과 같이 중재자를 통해 처리됩니다 잘 작동

var mediator = //get mediator injected into MVC controller via constructor 
var asyncCommand = // construct AsyncCommand 
// mediator runs ICommandValidator and that returns a list of errors if any 
var errors = await mediator.ProcessCommand(asyncCommand); 

합니다. 이제 나는 컨트롤러 액션에서 반복적 인 코드를 많이 수행한다는 것을 알게되었습니다 :

public async virtual Task<ActionResult> DoStuff(DoStuffAsyncCommand command) 
{ 
    if (!ModelState.IsValid) 
    { 
     return View(command); 
    } 

    var result = await mediator.ProcessCommandAsync(command); 

    if (!result.IsSuccess()) 
    { 
     AddErrorsToModelState(result); 
     return View(command); 
    } 
    return RedirectToAction(MVC.HomePage.Index()); 
} 

그리고이 패턴은 많은 컨트롤러에서 반복해서 반복됩니다. 그래서 단일 스레드 명령에 내가했던 단순화 :

public class ProcessCommandResult<T> : ActionResult where T : ICommand 
{ 
    private readonly T command; 
    private readonly ActionResult failure; 
    private readonly ActionResult success; 
    private readonly IMediator mediator; 


    public ProcessCommandResult(T command, ActionResult failure, ActionResult success) 
    { 
     this.command = command; 
     this.success = success; 
     this.failure = failure; 

     mediator = DependencyResolver.Current.GetService<IMediator>(); 
    } 

    public override void ExecuteResult(ControllerContext context) 
    { 
     if (!context.Controller.ViewData.ModelState.IsValid) 
     { 
      failure.ExecuteResult(context); 
      return; 
     } 

     var handlingResult = mediator.ProcessCommand(command); 

     if (handlingResult.ConainsErrors()) 
     { 
      AddErrorsToModelState(handlingResult); 
      failure.ExecuteResult(context); 
     } 

     success.ExecuteResult(context); 
    } 
    // plumbing code 
} 

그리고 일부 배관 공사 완료 후, 내 컨트롤러 액션은 다음과 같습니다

가 :

public virtual ActionResult Create(DoStuffCommand command) 
{ 
    return ProcessCommand(command, View(command), RedirectToAction(MVC.HomePage.Index())); 
} 

이 내가 돈 동기화-명령에 대해 잘 작동 ' 패턴을 async-await 할 필요가 없습니다. async 작업을 시도하자마자 MVC에 AsyncActionResult이 없기 때문에 컴파일되지 않으며 MVC 프레임 워크에서 비동기 작업을 void ExecuteResult(ControllerContext context)에서 사용할 수 없습니다.

그럼, 내가 어떻게 질문의 맨 위에 인용 컨트롤러 작업의 일반적인 구현을 만들 수있는 아이디어?

Answer 1

Action과 같은 것처럼 보이지만 ActionResult 대신 논리를 처리하는 것이 가장 좋습니다.

코드가 중복되는 경우 보호 된 도우미 메서드로 기본 클래스를 사용하는 것이 좋습니다 ...?

public class BaseCommandController : Controller 
{ 
    protected IMediator Mediator { get { return DependencyResolver.Current.GetService(typeof (IMediator)) as IMediator; } } 

    public async virtual Task<ActionResult> BaseDoStuff<TCommand>(TCommand command, Func<ActionResult> success, Func<ActionResult> failure) 
    { 
     if (!ModelState.IsValid) 
     { 
      return failure(); 
     } 

     var result = await Mediator.ProcessCommand(command); 

     if (!result.IsSuccess()) 
     { 
      AddErrorsToModelState(result); 
      return failure(); 
     } 

     return success(); 
    } 

    private void AddErrorsToModelState(IResponse result) 
    { 
    } 

} 

컨트롤러의 행동이 그 다음으로 렌더링됩니다

...

public class DefaultController : BaseCommandController 
{ 
    protected async virtual Task<ActionResult> DoStuff(DoStuffAsyncCommand command) 
    { 
     return await BaseDoStuff(command,() => RedirectToAction("Index"),() => View(command)); 
    } 
} 

Answer 2

귀하의 솔루션 (서비스 위치 및 기타 냄새를 모두 포함) 매우 냄새, 지나치게 복잡 보인다 시점을 놓칠 것 같다

무엇 ActionResults는 (명령 객체 그 자체입니다.)

실제로 이것은 The XY Problem의 좋은 예입니다. 행동 방법에서 일반적인 코드를 비동기 친화적 인 방식으로 리팩터링하는 실제 문제에 대해 묻는 대신 문제를 해결한다고 생각하는 지나치게 복잡한 솔루션을 생각해냅니다. 불행히도, 당신은 그것을 작동하게 만드는 방법을 알아낼 수 없으므로, 당신은 당신의 진짜 문제보다는 그 문제에 대해 물어볼 것입니다.

간단한 도우미 기능으로 원하는 것을 얻을 수 있습니다.

public async virtual Task<ActionResult> DoStuff(DoStuffAsyncCommand command) 
{ 
    return await ControllerHelper.Helper(command, ModelState, _mediator, 
     RedirectToAction(MVC.HomePage.Index()), View(command), View(command)); 
} 

public static class ControllerHelper 
{ 
    // You may need to constrain this to where T : class, didn't test it 
    public static async Task<ActionResult> Helper<T>(T command, 
     ModelStateDictionary ModelState, IMediator mediator, ActionResult returnAction, 
     ActionResult successAction, ActionResult failureAction) 
    { 
     if (!ModelState.IsValid) 
     { 
      return failureResult; 
     } 

     var result = await mediator.ProcessCommandAsync(command); 

     if (!result.IsSuccess()) 
     { 
      ModelState.AddErrorsToModelState(result); 
      return successResult; 
     } 

     return returnAction; 
    } 

    public static void AddErrorsToModelState(this ModelStateDictionary ModelState, ...) 
    { 
     // add your errors to the ModelState 
    } 
} 

또는 상태 저장 객체로 만들고 생성자 삽입을 통해 계단식 종속성을 통해 조정자를 삽입 할 수 있습니다. ModelState를 삽입하는 것은 쉽지 않습니다. 불행히도 ModelState를 매개 변수로 전달해야합니다.

또한 ActionResults의 문자열을 전달할 수도 있지만 RedirectToActionResult 객체가 새로 추가되지 않으므로 RedirectToRoute 객체를 초기화하는 것이 엉망이며 ActionResult를 전달하기가 더 쉽습니다. 새로운 ViewResult를 직접 생성하는 것보다 Controller View() 함수를 사용하는 것이 훨씬 쉽습니다.

Sambo가 사용하는 Func<ActionResult> 접근법을 사용할 수도 있습니다.이 방법은 게으르게 평가되므로 필요할 때만 RedirectToAction 메서드를 호출합니다. RedirectToAction에 충분한 가치가있는 오버 헤드가 있다고 생각하지 않습니다.