是html.renderAction代码气味[关闭]

“在html.renderAction调用后,错误修复正在通过控制器链进行跟踪。”

哦,上帝。听起来很可怕。

这肯定很臭。渲染只适用于那些“高级小部件”场景。

在对项目和ASP.NET MVC框架进行了一段时间的工作之后,我得出的结论是,不应在生产代码中使用renderAction(也许该规则有一个例外,但我还没有遇到),而是使用renderpartial并向模型提供对它的调用。

这样一来,只有一个控制器阶段可以使维护更加容易,另外,单个控制器阶段可以确保任何昂贵的操作(如数据库交互)都尽可能高效,如果处理跨多个控制器阶段就无法完成。通过使用renderpartial mutliple块视图逻辑,可以遵循dry原理,保持了renderaction的主要优点,但没有两大缺点。

坏的:

Html.RenderAction("ViewName", "PersonName", new { id = Model.Person.ID });

好:

Html.RenderPartial("ViewName", Model.Person.Name);

这是一个古老的问题,但今天仍然适用于代码。应该使用的时间是呈现与当前信息无关的部分视图。例如,如果您有一个在每个页面上呈现但由模型填充的导航。把这个模型放在其他模型上,当它们不相关的时候,这是一个糟糕的mojo。

+---------+ +-----+
|         | |     |
| content | | nav |
|         | |     |
+---------+ +-----+

如果将角色集传递给renderAction()方法,然后使用甜甜圈孔缓存,则可以缓存导航。

我认为最好不要尽可能多地使用所有这些帮助器,集成的或习惯性地编写的。这就像是一个快速解决设计问题的便捷快捷方式,但你真正做的是把事情搞得一团糟。