ASP.NET Core中替换Controller与View引用硬编码字符串的最佳实践有哪些？

你的枚举实现是否属于不良代码？ #

不属于不良代码，这是非常实用的强类型优化思路，核心收益你已经提到了：

• 避免硬编码字符串的拼写错误，把运行时报错提前到编译期
• 控制器/视图名改动时只需修改枚举定义，不需要全局搜索替换
  你提到的两个缺点确实存在：需要手动维护枚举和实际业务代码的同步，新增控制器、Action、视图时都要先改枚举，多人协作时很容易出现两边不一致的问题。

更优的硬编码替换最佳实践 #

1. 用nameof运算符替代手动枚举 #

不需要自己维护枚举定义，直接用C#自带的nameof获取符号的字符串名称：

// 自动获取Action名，修改方法名时IDE会自动同步所有引用
@Url.Action(nameof(PractitionerController.UploadVideo), nameof(PractitionerController).Replace("Controller", ""))

可以写个简单的静态工具类处理控制器名的后缀截取，避免每次手动替换：

public static class ControllerHelper
{
    public static string GetShortName<T>() where T : ControllerBase
    {
        return typeof(T).Name.Replace("Controller", string.Empty);
    }
}
// 使用方式
@Url.Action(nameof(PractitionerController.UploadVideo), ControllerHelper.GetShortName<PractitionerController>())

2. 泛型表达式扩展（最推荐的强类型方案） #

给IUrlHelper写泛型扩展方法，直接通过Lambda表达式解析控制器和Action，完全不需要手动维护任何字符串或者枚举：

using Microsoft.AspNetCore.Mvc;
using Microsoft.AspNetCore.Mvc.Routing;
using System.Linq.Expressions;
using System.Reflection;

public static class UrlHelperExtensions
{
    public static string ActionFor<TController>(this IUrlHelper url, Expression<Func<TController, IActionResult>> actionExpr) where TController : ControllerBase
    {
        // 解析表达式获取Action方法信息
        var methodCall = (MethodCallExpression)actionExpr.Body;
        var actionName = methodCall.Method.Name;
        // 获取控制器短名称
        var controllerName = typeof(TController).Name.Replace("Controller", string.Empty);
        // 解析Action参数
        var routeValues = new RouteValueDictionary();
        foreach (var param in methodCall.Method.GetParameters().Zip(methodCall.Arguments, (p, a) => new { p.Name, Value = Expression.Lambda(a).Compile().DynamicInvoke() }))
        {
            routeValues.Add(param.Name, param.Value);
        }
        return url.Action(actionName, controllerName, routeValues);
    }
}

使用方式完全强类型，重构时代码自动同步，拼写错误编译期直接报错：

// 无参数Action
@Url.ActionFor<PractitionerController>(c => c.UploadVideo())
// 带参数的Action也自动解析参数
@Url.ActionFor<PractitionerController>(c => c.GetVideo(123))

3. 视图名的强类型引用 #

如果是返回视图的场景，同样可以用nameof，或者给控制器写扩展方法获取枚举对应的视图名，和你原本的枚举方案配合即可。

内容的提问来源于stack exchange，提问作者TheKlinto