如何解决方法重载你可以过度使用它吗?
是的,过载很容易被过度使用。
我发现确定是否需要重载的关键是要考虑受众-不是编译器,而是维护程序员,他们将在几周/几个月/几年内出现,并且必须了解代码是什么试图实现。
一个简单的方法名称(如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#或尚未发明的全新东西编写。该语言可能不支持重载。
解决方法
定义几种使用不同过滤器返回相同数据形状的方法时,哪种方法更好?显式方法名称还是重载方法?
例如。如果我有一些产品,并且要从数据库中提取
显式方式:
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 }
我意识到您可能会遇到 类似签名 的问题,但是如果您要传递对象而不是基本类型(字符串,整数,字符,日期时间等),那么这将不是问题。所以…
重载一个方法 以减少您拥有的方法数量并为清楚起见是一个好主意, 还是 每个 以不同方式过滤数据的 方法 都应
具有不同的方法名称 ?
版权声明:本文内容由互联网用户自发贡献,该文观点与技术仅代表作者本人。本站仅提供信息存储空间服务,不拥有所有权,不承担相关法律责任。如发现本站有涉嫌侵权/违法违规的内容, 请发送邮件至 dio@foxmail.com 举报,一经查实,本站将立刻删除。