Question

目前我有一个像这样的文件夹结构：

Area (folder)
 - Toolkit (folder)
    - Controllers (folder)
        - AdminController.cs
    - Views (folder)
        - Admin (folder)
           - Privledges (folder)
              - Create.cshtml
              - Edit.cshtml
              - Delete.cshtml

转换为

/Toolkit/{controller}/{action}/{tool}/{id}

根据传递给操作的字符串{tool}参数和参数{id}，将操作设置为像控制器一样提供视图是不是一种不好的做法？ < / p>

我正在谈论的实施：

    private const string FOLDER_PRIVILEGES = "./Privileges/";

    public ActionResult Privileges(string tool, string id = "")
    {
        dynamic viewModel = null;
        ToolViews view; // enum for the views
        // Parse the tool name to get the enum representation of the view requested
        bool isParsed = Enum.TryParse(tool, out view);

        if (!isParsed)
        {
            return HttpNotFound();
        }

        switch (view)
        {
            case ToolViews.Index:
                viewModel = GetIndexViewModel(); // call a function that gets the VM
                break;
            case ToolViews.Edit:
                viewModel = GetEditViewModelById(int.Parse(id)); // sloppy parse
                break;
            default:
                viewModel = GetIndexViewModel();
                break;
        }
        // The folder path is needed to reach the correct view, is this bad?
        // Should I just create a more specific controller even though it would
        // require making about 15-20 controllers?
        return View(FOLDER_PRIVILEGES + tool, viewModel);
    }

当我编写View时，我必须确保Path名称用于文件夹

@Html.ActionLink("Edit", "./Toolkit/Admin/Priveleges/Edit", "Admin", new { id = item.id })

这似乎是一种糟糕的做法，因为如果文件夹结构发生变化，则需要大量维护。

但是，如果我必须将操作分解为控制器，那么它们中就会有很多（随着时间的推移，将近20个添加更多）。

如果我正在做的是一个不好的做法，那么提供这样的路线的最佳方式是什么？

/Toolkit/Admin/Privileges/Edit/1

我想避免执行以下操作：

/Toolkit/Admin/CreatePrivileges/1
/Toolkit/Admin/EditPrivileges/1
/Toolkit/Admin/DeletePrivileges/1

如果我没有任何意义，请告诉我，因为我很难将这个问题写成文字。

Answer 1

我认为你试图将一个约定强制转换为违反其原始意图的MVC。

使用MVC，您的控制器是名词，您的动作是动词。使用您的示例，您有：

工具包（名词） - 区域
- Admin（名词？） - 子区域？＆lt; - 这是一个有点时髦的
  - 特权（名词） - 控制器
    - 创建（动词） - 动作
    - 编辑（动词） - 行动
    - 删除（动词） - 操作

正如您所看到的，如果您可以将Toolkit + Admin视为区域+子区域，或将它们组合到一个区域（TookitAdmin），它将使您回到控制器和操作的最初目的。

根据评论，听起来你可能已经决定采用这种方式。但我想指出，你以一种全面的方式得出的结论是回到MVC的根源。

作为附注，您是否考虑过转向MVC4？它的Web API为RESTful API提供了更好的支持，听起来你可能会尝试这样做。

Answer 2

不是原始问题的答案，但OP要求提供Enum约束的样本，而不是必须检查每个操作中的枚举。即：

// Parse the tool name to get the enum representation of the view requested
bool isParsed = Enum.TryParse(tool, out view);

if (!isParsed)
{
    return HttpNotFound();
}

您不必将枚举值（在本例中为工具）接受为字符串，而是可以强制该值进入已作为相应枚举转换的操作。另外一个好处是MVC框架将负责在这种情况下返回正确的响应（HttpNotFound）。

这是你的约束方法。它接受任何类型的枚举。无需为每个枚举创建单独的约束。

public class EnumConstraint<T> : IRouteConstraint where T : struct
{
    private readonly HashSet<string> enumNames;
    public EnumConstraint()
    {
        string[] names = Enum.GetNames(typeof(T));
        this.enumNames = new HashSet<string>(from name in names select name.ToLowerInvariant());
    }

    public bool Match(HttpContextBase httpContext, Route route, string parameterName, RouteValueDictionary values, RouteDirection routeDirection)
    {
        return this.enumNames.Contains(values[parameterName].ToString().ToLowerInvariant());
    }

}

然后，在您的RegisterRoutes方法（MVC4）或global.asax.cs页面（MVC3）中，您只需注册以下路线：

routes.MapRoute(
    url: "/Toolkit/Admin/{Action}/{id}",
    constraints: new { Action = new EnumConstraint<ToolViews>(), id = @"\d+" }
);

我还在id参数上添加了一个数字约束，以节省你必须解析它。

让我知道这对你有用。

MVC区域路由 - 带文件夹的控制器文件夹

2 个答案: