45

有没有推荐的方法来使用新的

nameof()

ASP.NET MVC 中控制器名称的表达式?

Url.Action("ActionName", "Home")  <------ works

对比

Url.Action(nameof(ActionName), nameof(HomeController)) <----- doesn't work

显然它不起作用,因为nameof(HomeController)转换为"HomeController"而 MVC 需要的只是"Home"

4

8 回答 8

26

我喜欢James 关于使用扩展方法的建议。只有一个问题:尽管您正在使用nameof()并消除了魔术字符串,但仍然存在一个类型安全的小问题:您仍在使用字符串。因此,很容易忘记使用扩展方法,或者提供无效的任意字符串(例如错误输入控制器的名称)。

我认为我们可以通过对 Controller 使用通用扩展方法来改进 James 的建议,其中通用参数是目标控制器:

public static class ControllerExtensions
{
    public static string Action<T>(this Controller controller, string actionName)
        where T : Controller
    {
        var name = typeof(T).Name;
        string controllerName = name.EndsWith("Controller")
            ? name.Substring(0, name.Length - 10) : name;
        return controller.Url.Action(actionName, controllerName);
    }
}

现在的用法更干净了:

this.Action<HomeController>(nameof(ActionName));
于 2014-12-14T00:30:28.273 回答
9

考虑一个扩展方法:

public static string UrlName(this Type controller)
{
  var name = controller.Name;
  return name.EndsWith("Controller") ? name.Substring(0, name.Length - 10) : name;
}

然后你可以使用:

Url.Action(nameof(ActionName), typeof(HomeController).UrlName())
于 2014-12-12T17:06:15.227 回答
4

我需要确保routeValues正确处理,而不是总是像querystring价值观一样对待。但是,我仍然想确保操作与控制器匹配。

我的解决方案是为Url.Action.

<a href="@(Url.Action<MyController>(x=>x.MyAction))">Button Text</a>

我对不同类型的单参数操作有重载。如果我需要通过routeValues...

<a href="@(Url.Action<MyController>(x=>x.MyAction, new { myRouteValue = myValue }))">Button Text</a>

对于我没有显式为其创建重载的具有复杂参数的操作,需要使用控制器类型指定类型以匹配操作定义。

<a href="@(Url.Action<MyController,int,string>(x=>x.MyAction, new { myRouteValue1 = MyInt, MyRouteValue2 = MyString}))">Button Text</a>

当然,大多数时候动作都在同一个控制器中,所以我仍然只使用nameof那些。

<a href="@Url.Action(nameof(MyController.MyAction))">Button Text</a>

由于routeValues不一定与动作参数匹配,因此该解决方案允许这种灵活性。

分机代码

namespace System.Web.Mvc {
    public static class UrlExtensions {

    // Usage : <a href="@(Url.Action<MyController>(x=>x.MyActionNoVars, new {myroutevalue = 1}))"></a>
    public static string Action<T>(this UrlHelper helper,Expression<Func<T,Func<ActionResult>>> expression,object routeValues = null) where T : Controller
        => helper.Action<T>((LambdaExpression)expression,routeValues);

    // Usage : <a href="@(Url.Action<MyController,vartype1>(x=>x.MyActionWithOneVar, new {myroutevalue = 1}))"></a>
    public static string Action<T, P1>(this UrlHelper helper,Expression<Func<T,Func<P1,ActionResult>>> expression,object routeValues = null) where T : Controller
        => helper.Action<T>(expression,routeValues);

    // Usage : <a href="@(Url.Action<MyController,vartype1,vartype2>(x=>x.MyActionWithTwoVars, new {myroutevalue = 1}))"></a>
    public static string Action<T, P1, P2>(this UrlHelper helper,Expression<Func<T,Func<P1,P2,ActionResult>>> expression,object routeValues = null) where T : Controller
        => helper.Action<T>(expression,routeValues);

    // Usage : <a href="@(Url.Action<MyController>(x=>x.MyActionWithOneInt, new {myroutevalue = 1}))"></a>
    public static string Action<T>(this UrlHelper helper,Expression<Func<T,Func<int,ActionResult>>> expression,object routeValues = null) where T : Controller
        => helper.Action<T>((LambdaExpression)expression,routeValues);

    // Usage : <a href="@(Url.Action<MyController>(x=>x.MyActionWithOneString, new {myroutevalue = 1}))"></a>
    public static string Action<T>(this UrlHelper helper,Expression<Func<T,Func<string,ActionResult>>> expression,object routeValues = null) where T : Controller
        => helper.Action<T>((LambdaExpression)expression,routeValues);

    //Support function
    private static string Action<T>(this UrlHelper helper,LambdaExpression expression,object routeValues = null) where T : Controller
        => helper.Action(
                ((MethodInfo)((ConstantExpression)((MethodCallExpression)((UnaryExpression)expression.Body).Operand).Object).Value).Name,
                typeof(T).Name.Replace("Controller","").Replace("controller",""),
                routeValues);
    }
}
于 2017-11-03T16:14:09.717 回答
3

到目前为止我看到的所有解决方案都有一个缺点:虽然它们使更改控制器或动作的名称安全,但它们不能保证这两个实体之间的一致性。您可以指定来自不同控制器的操作:

public class HomeController : Controller
{
    public ActionResult HomeAction() { ... }
}

public class AnotherController : Controller
{
    public ActionResult AnotherAction() { ... }

    private void Process()
    {
        Url.Action(nameof(AnotherAction), nameof(HomeController));
    }
}

更糟糕的是,这种方法无法考虑可能应用于控制器和/或动作以更改路由的众多属性,例如RouteAttributeRoutePrefixAttribute,因此对基于属性的路由的任何更改都可能被忽视。

最后,它Url.Action()本身并不能确保 action 方法与其构成 URL 的参数之间的一致性:

public class HomeController : Controller
{
    public ActionResult HomeAction(int id, string name) { ... }

    private void Process()
    {
        Url.Action(nameof(HomeAction), new { identity = 1, title = "example" });
    }
}

我的解决方案基于Expression元数据:

public static class ActionHelper<T> where T : Controller
{
    public static string GetUrl(Expression<Func<T, Func<ActionResult>>> action)
    {
        return GetControllerName() + '/' + GetActionName(GetActionMethod(action));
    }

    public static string GetUrl<U>(
        Expression<Func<T, Func<U, ActionResult>>> action, U param)
    {
        var method = GetActionMethod(action);
        var parameters = method.GetParameters();

        return GetControllerName() + '/' + GetActionName(method) +
            '?' + GetParameter(parameters[0], param);
    }

    public static string GetUrl<U1, U2>(
        Expression<Func<T, Func<U1, U2, ActionResult>>> action, U1 param1, U2 param2)
    {
        var method = GetActionMethod(action);
        var parameters = method.GetParameters();

        return GetControllerName() + '/' + GetActionName(method) +
            '?' + GetParameter(parameters[0], param1) +
            '&' + GetParameter(parameters[1], param2);
    }

    private static string GetControllerName()
    {
        const string SUFFIX = nameof(Controller);
        string name = typeof(T).Name;
        return name.EndsWith(SUFFIX) ? name.Substring(0, name.Length - SUFFIX.Length) : name;
    }

    private static MethodInfo GetActionMethod(LambdaExpression expression)
    {
        var unaryExpr = (UnaryExpression)expression.Body;
        var methodCallExpr = (MethodCallExpression)unaryExpr.Operand;
        var methodCallObject = (ConstantExpression)methodCallExpr.Object;
        var method = (MethodInfo)methodCallObject.Value;

        Debug.Assert(method.IsPublic);
        return method;
    }

    private static string GetActionName(MethodInfo info)
    {
        return info.Name;
    }

    private static string GetParameter<U>(ParameterInfo info, U value)
    {
        return info.Name + '=' + Uri.EscapeDataString(value.ToString());
    }
}

这可以防止您传递错误的参数来生成 URL:

ActionHelper<HomeController>.GetUrl(controller => controller.HomeAction, 1, "example");

因为它是一个 lambda 表达式,所以动作总是绑定到它的控制器上。(而且您还拥有 Intellisense!)一旦选择了动作,它就会强制您指定其所有正确类型的参数。

给定的代码仍然没有解决路由问题,但至少可以修复它,因为有控制器Type.AttributesMethodInfo.Attributes可用的。

编辑:

正如@CarterMedlin 指出的那样,非原始类型的操作参数可能没有与查询参数的一对一绑定。目前,这是通过ToString()专门为此目的而在参数类中覆盖的调用来解决的。然而,该方法可能并不总是适用,它也不控制参数名称。

要解决此问题,您可以声明以下接口:

public interface IUrlSerializable
{
    Dictionary<string, string> GetQueryParams();
}

并在参数类中实现:

public class HomeController : Controller
{
    public ActionResult HomeAction(Model model) { ... }
}

public class Model : IUrlSerializable
{
    public int Id { get; set; }
    public string Name { get; set; }

    public Dictionary<string, string> GetQueryParams()
    {
        return new Dictionary<string, string>
        {
            [nameof(Id)] = Id,
            [nameof(Name)] = Name
        };
    }
}

以及相应的更改ActionHelper

public static class ActionHelper<T> where T : Controller
{
    ...

    private static string GetParameter<U>(ParameterInfo info, U value)
    {
        var serializableValue = value as IUrlSerializable;

        if (serializableValue == null)
            return GetParameter(info.Name, value.ToString());

        return String.Join("&",
            serializableValue.GetQueryParams().Select(param => GetParameter(param.Key, param.Value)));
    }

    private static string GetParameter(string name, string value)
    {
        return name + '=' + Uri.EscapeDataString(value);
    }
}

如您所见,ToString()当参数类未实现接口时,它仍然有一个回退。

用法:

ActionHelper<HomeController>.GetUrl(controller => controller.HomeAction, new Model
{
    Id = 1,
    Name = "example"
});
于 2017-10-18T13:07:33.673 回答
1

基于Gigi 的回答(它为控制器引入了类型安全),我又迈出了一步。我非常喜欢 T4MVC,但我从不喜欢运行 T4 一代。我喜欢代码生成,但它不是 MSBuild 原生的,所以构建服务器很难使用它。

我重新使用了通用概念并添加了一个Expression参数:

public static class ControllerExtensions
{
    public static ActionResult RedirectToAction<TController>(
        this Controller controller, 
        Expression<Func<TController, ActionResult>> expression)
        where TController : Controller
    {
        var fullControllerName = typeof(TController).Name;
        var controllerName = fullControllerName.EndsWith("Controller")
            ? fullControllerName.Substring(0, fullControllerName.Length - 10)
            : fullControllerName;

        var actionCall = (MethodCallExpression) expression.Body;
        return controller.RedirectToAction(actionCall.Method.Name, controllerName);
    }
}

上面的示例调用如下所示:

    public virtual ActionResult Index()
    {
        return this.RedirectToAction<JobController>( controller => controller.Index() );
    }

如果JobController没有Index,你会遇到编译器错误。这可能是与先前答案相比的唯一优势 - 所以这是另一个愚蠢检查。JobController如果JobController没有,它会帮助您停止使用Index。此外,它还会在寻找动作时为您提供智能感知。

--

我还在这个签名中添加了:

    public static ActionResult RedirectToAction<TController>(this TController controller, Expression<Func<TController, ActionResult>> expression)
        where TController : Controller

这允许以更简单的方式为当前控制器键入操作,而无需指定类型。两者可以并排使用:

    public virtual ActionResult Index()
    {
        return this.RedirectToAction(controller => controller.Test());
    }
    public virtual ActionResult Test()
    {
         ...
    }

--

我在评论中被问到是否支持参数。上面的答案是否定的。但是,我很快就创建了一个可以解析参数的版本。这是调整后的方法:

    public static ActionResult RedirectToAction<TController>(this Controller controller, Expression<Func<TController, ActionResult>> expression)
        where TController : Controller
    {
        var fullControllerName = typeof(TController).Name;
        var controllerName = fullControllerName.EndsWith("Controller")
            ? fullControllerName.Substring(0, fullControllerName.Length - 10)
            : fullControllerName;

        var actionCall = (MethodCallExpression)expression.Body;

        var routeValues = new ExpandoObject();
        var routeValuesDictionary = (IDictionary<String, Object>)routeValues;
        var parameters = actionCall.Method.GetParameters();
        for (var i = 0; i < parameters.Length; i++)
        {
            var arugmentLambda = Expression.Lambda(actionCall.Arguments[i], expression.Parameters);
            var arugmentDelegate = arugmentLambda.Compile();
            var argumentValue = arugmentDelegate.DynamicInvoke(controller);
            routeValuesDictionary[parameters[i].Name] = argumentValue;
        }
        return controller.RedirectToAction(actionCall.Method.Name, controllerName, routeValues);
    }

我没有亲自测试过它(但 Intellisense 让它看起来可以编译)。总而言之,代码查看该方法的所有参数,并创建一个包含所有参数的 ExpandoObject。这些值由传入的表达式确定,通过使用主表达式的原始参数将每个值调用为独立的 lambda 表达式。然后编译并调用表达式,并将结果值存储在 ExpandoObject 中。然后将结果传递到内置帮助程序中。

于 2019-10-02T17:31:24.117 回答
0

对于那些寻找如何在 ASP.NET Core 中执行此操作的人,请尝试以下操作:https ://github.com/ivaylokenov/AspNet.Mvc.TypedRouting

@(Html.ActionLink<HomeController>("Home page", c => c.Index()))
于 2021-03-05T23:32:37.380 回答
0

对@James 的回答:

相反,使用字符串扩展方法:返回控制器名称前缀,否则返回传入的参数。

    /// <summary>
    /// Gets the prefix of the controller name.
    /// <para> <see langword="Usage:"/>
    /// <code>var <paramref name="controllerNamePrefix"/> = 
    /// <see langword="nameof"/>(ExampleController).
    /// <see cref="GetControllerPrefix()"/>;
    /// </code>
    /// </para>
    /// </summary>
    /// <param name="fullControllerName"></param>
    /// <returns></returns>
    public static string GetControllerPrefix(this string fullControllerName)
    {
        const string Controller = nameof(Controller);

        if (string.IsNullOrEmpty(fullControllerName) || !fullControllerName.EndsWith(Controller))
            return fullControllerName;

        return fullControllerName.Substring(0, fullControllerName.Length - Controller.Length);
    }
于 2020-03-25T02:42:59.830 回答
0

我使用这样的东西并且它有效: Url.Action(nameof(ActionName), nameof(HomeController).Replace("Controller", string.Empty))

于 2021-07-07T11:18:29.140 回答