Должна ли вся бизнес-логика быть в моделях домена или?

мы помещаем наше приложение и бизнес-логику в отдельные слои (файл csproj) доменный слой для бизнес-логики и сервисный уровень для логики приложения. Это полностью абстрагирует их от проекта MVC. У этого есть два больших преимущества для нас. Во-первых, бизнес-логика не привязана к шаблону, который может измениться. Несколько лет назад никто из нас не мог представить себе популярность MVC сегодня, и через несколько лет мы не знаем, будет ли какая-то новая вещь, которая придет вместе и замените MVC, чтобы получить подавляющее большинство вашего кода, чтобы быть "не привязанным" к MVC, помогло бы, если вы когда-нибудь захотите отказаться от MVC для чего-то другого.

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

пример Ниже приведен пример кода, что я буду делать. Это быстро и грязно, и должно быть больше, как добавление журнала, если пользователь не сохраняет обратно в базу данных и т. д...

public enum PayoutResult
{
    UserNotFound,
    Success,
    FundsUnavailable,
    DBError
}

public class UserProfile
{
    public float Balance { get; set; }

    public string Username { get; set; }

    // other properties and domain logic you may have

    public bool Withdraw(PayoutModel model)
    {
        if (this.Balance >= model.Amount)
        {
            this.Balance -= model.Amount;
            return true;
        }

        return false;
    }
}


public class PayoutService
{
    IUserRepository userRepository;

    public PayoutService()
    {
        this.userRepository = new UserRepository();
    }

    public PayoutResult Payout(string userName, PayoutModel model)
    {
        var user = this.userRepository.GetAll().SingleOrDefault(u => u.Username == userName);
        if (user == null)
        {
            return PayoutResult.UserNotFound;
        }

        // don't set the model properties until we're ok on the db
        bool hasWithdrawn = user.Withdraw(model);
        if (hasWithdrawn && this.userRepository.SaveUser(user))
        {
            model.Balance = user.Balance;
            model.Amount = 0;

            return PayoutResult.Success;
        }
        else if (hasWithdrawn)
        {
            return PayoutResult.DBError;
        }

        return PayoutResult.FundsUnavailable;
    }
}

ваш контроллер теперь будет выглядеть так

[HttpPost]
public ActionResult Payout(PayoutModel model)
{
    if (ModelState.IsValid)
    {
        var result = service.Payout(User.Identity.Name, model);
        // This part should only be in the MVC project since it deals with 
        // how things should be presented to the user
        switch (result)
        {
            case PayoutResult.UserNotFound:
                ViewBag.Message = "User not found";
                break;
            case PayoutResult.Success:
                ViewBag.Message = string.Format("Successfully withdraw {0:c}", model.Balance);
                break;
            case PayoutResult.FundsUnavailable:
                ViewBag.Message = "Insufficient funds";
                break;
            default:
                break;
        }               
    }

    return View(model);
}

и если вам пришлось выставить выплату в веб-сервисе (я работаю в корпоративной среде, так что это бывает много для меня) Вы делаете следующее...

public class MyWCFService : IMyWCFService
{
    private PayoutService service = new PayoutService();

    public PayoutResult Payout(string username, PayoutModel model)
    {
        return this.service.Payout(username, model);
    }
}