如何从另一个控制器引用控制器功能

Question

试图学习来自 Linux/LAMP 背景的 ASP MVC（换句话说，我是一个新手）...

出于某种原因，我似乎无法在另一个控制器中使用一个控制器中定义的函数。

这是我的 MessagesController.cs 文件中的函数：

    public List<Message> GetMessagesById(string username)
    {
        return db.Messages.Where(p => p.user == username).ToList();
    }

当我尝试引用它时：

    using LemonadeTrader.Models;
    using LemonadeTrader.Controllers;     // added this to pull the Messages::getMesssagesById
    ...

    ViewBag.messages = lemondb.Messages.GetMessagesById(Membership.GetUser().ProviderUserKey.ToString());

我得到了类似于柠檬数据库的内容。Messages 不包含名为 GetMesssagesById 的方法。

我该如何引用它？

Answer 1

您不应该像这样链接控制器方法，更不用说控制器不应该直接执行数据访问。我建议您将此函数外部化到一个单独的类/存储库中，两个控制器都可以使用它。

示例：

public class MessagesRepository
{
    public List<Message> GetMessagesById(string username)
    {
        return db.Messages.Where(p => p.user == username).ToList();
    }
}

然后：

public class FooController: Controller
{
    public ActionResult Index()
    {
        var db = new MessagesRepository()
        ViewBag.messages = db.GetMessagesById(Membership.GetUser().ProviderUserKey.ToString());
        return View();
    }
}

public class BarController: Controller
{
    public ActionResult Index()
    {
        var db = new MessagesRepository()
        ViewBag.messages = db.GetMessagesById(Membership.GetUser().ProviderUserKey.ToString());
        return View();
    }
}

好的，这是第一步。可以通过引入此存储库的抽象来将控制器与存储库解耦来改进此代码：

public interface IMessagesRepository
{
    List<Message> GetMessagesById(string username);
}

public class MessagesRepository: IMessagesRepository
{
    public List<Message> GetMessagesById(string username)
    {
        return db.Messages.Where(p => p.user == username).ToList();
    }
}

然后您可以对这些控制器使用构造函数注入：

public class FooController: Controller
{
    private readonly IMessagesRepository _repository;
    public class FooController(IMessagesRepository repository)
    {
        _repository = repository;
    }

    public ActionResult Index()
    {
        ViewBag.messages = _repository.GetMessagesById(Membership.GetUser().ProviderUserKey.ToString());
        return View();
    }
}

public class BarController: Controller
{
    private readonly IMessagesRepository _repository;
    public class BarController(IMessagesRepository repository)
    {
        _repository = repository;
    }

    public ActionResult Index()
    {
        ViewBag.messages = _repository.GetMessagesById(Membership.GetUser().ProviderUserKey.ToString());
        return View();
    }
}

最后您将配置您的 DI 框架以将相应的实现传递到这些控制器中。

我还建议您用强类型视图模型替换此 ViewBag：

public class MyViewModel
{
    public List<Message> Messages { get; set; }
}

然后：

public ActionResult Index()
{
    var model = new MyViewModel
    {
        Messages = _repository.GetMessagesById(Membership.GetUser().ProviderUserKey.ToString())
    };
    return View(model);
}

Answer 2

将 GetMessageById （以及访问消息所需的所有其他方法）放置在单独的类中，并在需要获取 Message 数据的任何地方使用该类。

MessageService service = new MessageService();
ViewBag.messages = service.GetMessagesById(...);