如何重构这个 switch 语句？

Question

昨天我在玩 jQGrid 插件和 ASP.NET。 一切结果都很好，我的网格现在正在工作，但我有两种方法，让我的代码有味道。

臭方法：

private IOrderedEnumerable<Employee> GetOrderedEmployees(Column sortColumn, bool ascending)
    {
        switch (sortColumn)
        {
            case Column.Name:
                {
                    return GetOrderedEmployees(e => e.Name, ascending);
                }
            case Column.Salary:
                {
                    return GetOrderedEmployees(e => e.Salary, ascending);
                }
            default:
                {
                    return GetOrderedEmployees(e => e.ID, ascending);
                }
        }
    }

    private IOrderedEnumerable<Employee> GetOrderedEmployees<TSortKey>(Func<Employee, TSortKey> func, bool ascending)
    {
        return ascending ? Context.Employees.OrderBy(func) : Context.Employees.OrderByDescending(func);
    }

我不知道如何正确重构它们。 看起来最好的解决方案是在 switch 语句中仅返回 lambda (fe return e=>e.Name)，但如何才能做到呢？

在 switch 语句中，ascending 参数被传递 3 次。 不是重复吗？

Answer 1

我认为你可能有点过分了。 返回 lambda (IMO) 比简单的 switch 语句或 if;else if;else 块更令人困惑。 可能有更好的方法，但有时您确实需要检查条件，特别是对于列（我讨厌使用 ListViews，但它们是必要的并且通常需要这种代码。）

我不认为您处于您需要重构任何内容的点。 如果该 switch 语句成为维护难题，那么就继续吧，但并非所有 switch 语句都构成“代码异味”。

为了解决您的代码重复问题，您可以使用开关来获取 e.?? 首先将值保存下来，然后在方法结束时调用该函数一次。

Answer 2

有一些模式可能在这里起作用，但是提供者怎么样？ 您可以创建一个提供 GetOrderEmployees 的 Provider，然后传入您想要使用的 Provider 而不是排序列。

编辑 -
但我也认为第一个发帖者的评论有很多优点——无论如何，不​​要为小事制定复杂的解决方案。

Answer 3

如果您只是想减少代码中的重复：（

private IOrderedEnumerable<Employee> GetOrderedEmployees(Column sortColumn, bool ascending)
{
    Func<Employee, TSortKey> func = e => e.ID;

    switch (sortColumn)
    {
        case Column.Name:
            {
                func = e => e.Name;
            }
        case Column.Salary:
            {
                func = e => e.Salary;
            }
    }

    return GetOrderedEmployees(func, ascending);
}

private IOrderedEnumerable<Employee> GetOrderedEmployees<TSortKey>(Func<Employee, TSortKey> func, bool ascending)
{
    return ascending ? Context.Employees.OrderBy(func) : Context.Employees.OrderByDescending(func);
}

我不太精通 .NET；请原谅任何语法错误。）