ASP.NET MVC 5 更多控制器的常用操作

Question

我有一些控制器（并且会更多），它们共享一些类似的操作：

public ActionResult DeleteConfirmed(int id)
{
    Supplier s = db.Suppliers.Find(id);
    s.Deleted = true;
    db.SaveChanges();
    return RedirectToAction("Index");
}

public ActionResult RestoreConfirmed(int id)
{
    Supplier s = db.Suppliers.Find(id);
    s.Deleted = false;
    db.SaveChanges();
    return RedirectToAction("Index");
}

这些操作是SuppliersController 的一部分。这样做的目的是，当我删除或恢复一个对象时，它会将数据库中的对象标记为已删除字段的真（恢复时为假）。

CurrenciesController、ProductsController 等许多其他控制器共享相同的行为...

在我展示的代码中，您应该看到明确指定了我的数据库实体 (Supplier) 以及存储库 (Suppliers)。

我想找到一种通用的方法。我想创建一个自定义控制器，并且所有其他具有相同行为的控制器都将对其进行扩展。在这种情况下，ProductsController 将扩展我的 DeleteRestoreController。

我怎样才能以“通用”的方式做到这一点？

db 是 DbContext

public partial class LE: DbContext
{
public LE()
    : base("name=LE")
{
}

protected override void OnModelCreating(DbModelBuilder modelBuilder)
{
    throw new UnintentionalCodeFirstException();
}

public virtual DbSet<Category> Categories { get; set; }
public virtual DbSet<CategoryText> CategoryTexts { get; set; }
...
}

Categories 也有相同的行为。

Answer 1

更进一步

public abstract class DeleteRestoreController<T> : Controller
{

public virtual Action DeleteConfirmed(int id)
{
    var dbset = db.Set<T>();

    var s = dbset.Find(id);
    s.Deleted = true;
    db.SaveChanges();
    return RedirectToAction("Index");
}
}

然后在定义控制器时添加实体类型

public class ProductsController : DeleteRestoreController<Supplier>
{
////blah
}

Answer 2

您可以将DeleteRestoreController 实现为抽象类。

public abstract class DeleteRestoreController : Controller
{
    private IRepository : Repository;
    public DeleteRestoreController() { ... }
    public DeleteRestoreController(IRepository Repository) { ... }

    public virtual Action DeleteConfirmed(int id)
    {
        Supplier s = db.Suppliers.Find(id);
        s.Deleted = true;
        db.SaveChanges();
        return RedirectToAction("Index");
    }
}

如果您需要与 ProductsController 中的该行为不同，您可以简单地覆盖该方法。

public class ProductsController : DeleteRestoreController
{
    public override void DeleteConfirmed()
    {
        //override the logic
    }
}

您总是可以更进一步并实现一个通用存储库，但我的应用程序中从未超过 6-8 个控制器，也没有创建过一次。

编辑我刚刚在 cmets 中读到，实体会从控制器中的 Suppliers 更改，因此如果您不实现，实现基本控制器将没有多大意义一个通用接口也是如此。罗伯特·哈维（Robert Harvey）提出了一个很好的观点，他指出复杂性必须在某个地方。