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

本文关键字：if else 逻辑 添加 视图 | 更新日期: 2023-09-27 18:04:43

我被要求避免在视图中添加像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文件。我希望避免在视图中添加逻辑。

我有什么办法可以做到吗?

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

如果你不想在视图中做，我认为你应该在控制器中做。类似这样的代码可以用于从单个视图中删除条件语句:

// 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")
然而

…"视图逻辑"answers"控制器逻辑"有很大的区别。毕竟，这是我们希望在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>
}

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

控制器: 

model.IsAuthenticated = User.Identity.IsAuthenticated;
model.IsAdministrator = User.IsInRole("administrator");
相关文章：