Упрощение контроллера C # MVC.Net

Question

Я пытаюсь избежать сценария «жирного контроллера».

Существует много примеров, когда контроллеры вызывают один API, но что, если вашему контроллеру нужно вызвать два или более API?

Как вы упростите такой сценарий?Кажется, должен быть более чистый путь:

private ServiceTwo serviceTwo = new ServiceTwo();
private MailService mailService = new MailService();

[HttpPost]
public async Task<ActionResult> Index(Customer formData) 
{
    if (!ModelState.IsValid) 
    {
        return View();
    }

    var apiOneService = new ApiOneService(formData);

    if (apiOneService.ExistingUserCheck()) 
    {
        ModelState.AddModelError("Email", "* Email address already exists.");
        return View(formData);
    }

    var apiTwoResult = serviceTwo.CreateTrial(formData);

    var emailResult = await mailService.SendMailAsync((formData));

    return RedirectToAction("Index", "TrialConfirmation");
}

Answer 1

Я склонен реализовывать сервисный уровень в некоторых подобных случаях и выставлять интерфейсы с внедрением зависимостей.Логика находится на уровне обслуживания, и контроллер просто предоставляет эту логику представлению.

Answer 2

Вы можете упростить контроллер до этого:

https://github.com/DashboardCode/Routines/blob/master/AdminkaV1/Injected.AspCore.MvcApp/Controllers/UsersController.cs

Здесь вы можете увидеть только определение метаданных: что такое id, в какие поля нужно загружать данные как для одного ко многимчто касается многих ко многим и т. д.

На самом деле вы спрашиваете, как написать Внутренний DSL, который может определять мои контроллеры в «больших блоках».Просто напиши это.

Возможно, но не приносит полного удовлетворения.Люди придерживаются общего кода, потому что он ... общий.Даже если многословно.

Answer 3

Для чего-то такого маленького, вы можете просто извлечь логику в другой метод контроллера:

[HttpPost]
public async Task<ActionResult> Index(Customer formData) {
    if (!ModelState.IsValid) {
        return View();
    }

    var ok = await CreateTrial(formData);
    return ok 
        ? (ActionResult)RedirectToAction("Index", "TrialConfirmation")
        : View(formData);
    }
}

protected async Task<bool> CreateTrial(Customer formData) {
    var apiOneService = new ApiOneService(formData);
    if (apiOneService.ExistingUserCheck()) return false;

    var emailTask = mailService.SendMailAsync((formData));
    var apiTwoResult = serviceTwo.CreateTrial(formData);
    await emailTask;

    return true;
}

Для более сложного метода вы также можете извлечь логику в отдельный класс или слой..