是的,过载很容易被过度使用。
我发现确定是否需要重载的关键是要考虑受众-不是编译器,而是维护程序员,他们将在几周/几个月/几年内出现,并且必须了解代码是什么试图实现。
一个简单的方法名称(如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 parameterspublic IList<Employee> GetEmployeesBySupervisor(int supervisorId);public IList<Employee> GetEmployeesByDepartment(int departmentId);
也有机会针对每种目的引入重载:
// Examples for GetEmployeespublic 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 GetProductpublic 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#或尚未发明的全新东西编写。该语言可能不支持重载。



