3

我被要求避免在视图中添加诸如 if/else 之类的逻辑。目前，我正在开发页面的登录/注册功能，如果用户处于角色中，则必须显示一组链接，如果他是角色，则必须显示另一个链接在另一个角色。

这是我到目前为止所做的：

<ul id="menu">
    <li>@Html.ActionLink("Products", "Books", "Home")</li>
    @if (User.Identity.IsAuthenticated)
    { 
         <li>@Html.ActionLink("Log Out" ,"LogOut","Account")</li>
    }
    else
    {
         <li>@Html.ActionLink("Log In" ,"LogIn","Account")</li>
    }
    @if(User.IsInRole("administrator"))
    {
       <li>@Html.ActionLink("Product Manager", "Books", "ProductManager")</li>
    }
</ul>

此代码存储在 _Layout.cshtml 文件中。我想避免将逻辑添加到视图中。

有什么办法可以做到吗？

4

3 回答 3

6

如果您不想在视图中执行此操作，我认为您应该在控制器中执行此操作。像这样的东西适用于从单个视图中删除条件语句：

// controller
ActionResult MyAction()
{
    if (!User.Identity.IsAuthenticated)
    {
        ViewBag.MenuControl = "Menu/NotLoggedIn"
    } 
    else if (User.IsInRole("Administrator"))
    {
        ViewBag.MenuControl = "Menu/Administrator"
    } 
    else
    {
        ViewBag.MenuControl = "Menu/LoggedIn"
    }

    ...
}

// view
@Html.Partial(ViewBag.MenuControl);

或者要在多个视图中共享此逻辑，我建议您创建一个特定的 MenuController 来容纳此逻辑。

ActionResult RenderMenu()
{
    string template;
    if (!User.Identity.IsAuthenticated)
    {
        template = "Menu/NotLoggedIn"
    } 
    else if (User.IsInRole("Administrator"))
    {
        template = "Menu/Administrator"
    } 
    else
    {
        template = "Menu/LoggedIn"
    }

    return View(template);
}

// view
@Html.Action("RenderMenu", "MenuController")

但是......“视图逻辑”和“控制器逻辑”之间存在很大差异。毕竟这是我们希望在 MVC 架构中将视图与控制器分开的主要原因之一。像“避免视图中的所有条件语句”这样的简单规则确实忽略了 MVC 的设计方式。

我真的更喜欢在视图中这样做，因为实际上它更多地与如何格式化视图有关，而不是控制器应该如何运行。我会坚持使用您当前的代码。

于 2013-03-20T07:37:33.180 回答
5

if/else我认为在您的观点中发表声明并没有错。问题在于实际的条件语句本身。

以此为例：

@if(User.IsInRole("administrator"))
{
   <li>@Html.ActionLink("Product Manager", "Books", "ProductManager")</li>
}

在这里，您将业务逻辑与视图逻辑混合在一起。您正在引用角色的名称。这与视图无关。为了解决这个问题，试试这个：

型号代码

public class MyModel
{
    public bool IsAdministrator { get; }
}

控制器代码

myModel.IsAdministrator = User.IsInRole("administrator");

查看代码

@if(this.Model.IsAdministrator)
{
   <li>@Html.ActionLink("Product Manager", "Books", "ProductManager")</li>
}
于 2013-03-20T07:47:41.613 回答
1

MVC 的漏洞在于划分职责区域。模型必须是完全独立的，即只知道它自己。在模型中包含与表示相关的逻辑是很正常的，除非您将它与业务逻辑混合使用。在您的示例中，明显的改进可能是将所有授权信息移动到模型对象中并保留 if/else 逻辑。

控制器：

model.IsAuthenticated = User.Identity.IsAuthenticated;
model.IsAdministrator = User.IsInRole("administrator");
于 2013-03-20T08:26:24.870 回答