C# 避免在视图中添加if/else逻辑

我被要求避免添加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>

@ActionLink（“产品”、“书籍”、“家庭”）
@if（User.Identity.IsAuthenticated）
{ 
@ActionLink（“注销”、“注销”、“帐户”）
}
其他的
{
@ActionLink（“登录”、“登录”、“帐户”）
}
@if（User.IsInRole（“管理员”））
{
@ActionLink（“产品经理”、“书籍”、“产品经理”）
}

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

---

有什么方法可以做到吗？

如果您不想在视图中执行此操作，我想您应该在控制器中执行此操作。类似这样的操作可以从单个视图中删除条件语句：

// 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的设计要点

---

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

我认为在您的视图中使用

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>
}

查看代码

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

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

@if(this.Model.IsAdministrator)
{
   <li>@Html.ActionLink("Product Manager", "Books", "ProductManager")</li>
}

@if（this.Model.IsAdministrator）
{
@ActionLink（“产品经理”、“书籍”、“产品经理”）
}

MVC的孔点是将责任区分开。模型必须是完全独立的，也就是说只知道它自己。在模型中具有与表示相关的逻辑是很正常的，除非您将其与业务逻辑混合使用。在您的示例中，明显的改进可能是将所有授权信息移动到模型对象中，并保留if/else逻辑

控制器：

model.IsAuthenticated = User.Identity.IsAuthenticated;
model.IsAdministrator = User.IsInRole("administrator");

---

为什么视图中这样的逻辑是一件坏事？mvc的漏洞是分离逻辑、标记和数据，所以我认为这是一种不好的做法。我不认为这一定是非mvc的，这个逻辑只与视图相关。可能只是从视图中删除业务逻辑就足够了？如果

If/else

block-inside-view违背了框架的概念，为什么视图引擎会支持它们？+1指出IsAdministrator测试应该在模型中。