方法重载。你可以过度使用它吗？

Question

定义几种使用不同过滤器返回相同数据形状的方法时，哪种方法更好？显式方法名称还是重载方法？

例如。如果我有一些产品，并且要从数据库中提取

显式方式：

public List<Product> GetProduct(int productId) {    // return a List    }
public List<Product> GetProductByCategory(Category category) {    // return a List    }
public List<Product> GetProductByName(string Name ) {    // return a List    }

重载方式：

public List<Product> GetProducts() {    // return a List of all products    }
public List<Product> GetProducts(Category category) { // return a List by Category }
public List<Product> GetProducts(string searchString ) { // return a List by search string }

我意识到您可能会遇到 类似签名 的问题，但是如果您要传递对象而不是基本类型（字符串，整数，字符，日期时间等），那么这将不是问题。所以…
重载一个方法 以减少您拥有的方法数量并为清楚起见是一个好主意， 还是 每个 以不同方式过滤数据的 方法 都应
具有不同的方法名称 ？

Answer 1

是的，过载很容易被过度使用。

我发现确定是否需要重载的关键是要考虑受众-不是编译器，而是维护程序员，他们将在几周/几个月/几年内出现，并且必须了解代码是什么试图实现。

一个简单的方法名称（如GetProducts（））清晰易懂，但确实有很多说法。

在很多情况下，如果传递给GetProducts（）的参数名称正确，则维护人员将能够弄清楚重载的作用-
但这依赖于使用时的良好命名规则，而您无法执行。您可以强制执行的是他们正在调用的方法的名称。

我遵循的准则是仅在方法可互换时重载方法-如果它们做相同的事情。这样，我不介意类的使用者调用哪个版本，因为它们是等效的。

为了说明这一点，我很乐意对DeleteFile（）方法使用重载：

void DeleteFile(string filePath);
void DeleteFile(FileInfo file);
void DeleteFile(DirectoryInfo directory, string fileName);

但是，对于您的示例，我将使用单独的名称：

public IList<Product> GetProductById(int productId) {...}
public IList<Product> GetProductByCategory(Category category) {...}
public IList<Product> GetProductByName(string Name ) {...}

具有全名可以使代码对于维护人员（可能是我）更加明确。它避免了签名冲突的问题：

// No collisions, even though both methods take int parameters
public IList<Employee> GetEmployeesBySupervisor(int supervisorId);
public IList<Employee> GetEmployeesByDepartment(int departmentId);

也有机会针对每种目的引入重载：

// Examples for GetEmployees

public IList<Employee> GetEmployeesBySupervisor(int supervisorId);
public IList<Employee> GetEmployeesBySupervisor(Supervisor supervisor);
public IList<Employee> GetEmployeesBySupervisor(Person supervisor);

public IList<Employee> GetEmployeesByDepartment(int departmentId);
public IList<Employee> GetEmployeesByDepartment(Department department);

// Examples for GetProduct

public IList<Product> GetProductById(int productId) {...}
public IList<Product> GetProductById(params int[] productId) {...}

public IList<Product> GetProductByCategory(Category category) {...}
public IList<Product> GetProductByCategory(IEnumerable<Category> category) {...}
public IList<Product> GetProductByCategory(params Category[] category) {...}

读取的代码远比编写的要多-即使在初次签入源代码控制后再也没有回到代码上，编写代码时，您仍将读取该行代码数十次随后的代码。

最后，除非您要编写一次性代码，否则您需要允许其他人使用其他语言调用您的代码。看来，大多数业务系统最终都会在其按日期使用之前远远超过其生产期限。可能是2016年消耗您的类的代码最终用VB.NET，C＃6.0，F＃或尚未发明的全新东西编写。该语言可能不支持重载。