Tôi có nên sử dụng các loại trả lại khác nhau trên các phương thức quá tải không?

Question

Có bất kỳ phương pháp hay nhất nào về việc trả lại các loại trả lại khác nhau về các phương thức quá tải không? Ví dụ nếu tôi có một phương pháp tải trong DAL của tôi, tôi hoặc muốn tải một mục duy nhất hoặc một loạt các mục. Tôi biết rằng tôi có thể sử dụng một số phương pháp:

tải một đối tượng

MyBusinessObject LoadOne(int id) 
{ 
} 

tải nhiều đối tượng

MyBusinessObject[] LoadMany(params int[] ids) 
{ 
} 

Bây giờ điều mà tôi biết tôi thể làm là quá tải một phương pháp và có các loại trả về khác nhau. Cũng giống như vậy:

MyBusinessObject Load(int id) 
{ 
} 

Và

MyBusinessObject[] Load(params int[] ids) 
{ 
} 

Trong khi có vẻ như không có gì để ngăn cản tôi làm điều này là, và nó giữ mọi thứ gọn gàng từ một góc độ API, điều này có vẻ như là một ý tưởng tốt? Tôi đã đi qua nó đêm qua và một phần của tôi nghĩ rằng tôi không nên làm điều này vì lý do muốn kết hợp các loại trả lại cho phương pháp quá tải.

Tôi cũng có thể có phương thức Tải (int id) trả về bộ sưu tập chỉ chứa một mục. Dường như với tôi rằng điều này vi phạm nguyên tắc ít ngạc nhiên nhất mặc dù trong đó nếu bạn đang mong đợi một mục trở lại, bạn nên trả lại mục đó, bạn không nên trả về một danh sách chứa một mục duy nhất.

Vì vậy, đây là những suy nghĩ mâu thuẫn của tôi xung quanh những ý tưởng này:

• phương pháp quá tải tất cả nên quay lại cùng loại.
• Nếu phương pháp làm tương tự, đừng cho họ một loạt các tên khác nhau, quá tải cùng tên phương thức. Nó làm cho mọi thứ trở nên đơn giản hơn từ quan điểm của người dùng API, họ không phải rà soát thông qua một loạt các phương thức khác nhau mà tất cả về cơ bản đều làm tương tự nhưng với các tham số khác nhau.
• Trả lại loại rõ ràng nhất cho phương pháp, tức là nếu người dùng có khả năng mong đợi một tập hợp các mục, hãy trả về một bộ sưu tập các mặt hàng, nếu họ có khả năng mong đợi một mục duy nhất, hãy trả lại một mục.

Vì vậy, hai ý nghĩ sau này vượt trội hơn lần đầu tiên, nhưng đồng thời, ý nghĩ đầu tiên có vẻ giống như một loại thực hành tốt nhất có lập trình.

Có bất kỳ phương pháp hay nhất nào xung quanh thực tiễn này không? Tôi muốn được nghe những suy nghĩ của người khác về chủ đề này.

Answer 1

tôi có thể bị cám dỗ để làm cho API rõ ràng và sử dụng đa số trong tên:

Customer LoadCustomer(int id) {...} 
Customer[] LoadCustomers(params int[] id) {...} 

Trên thực tế, params hiếm khi hữu ích ở đây - bạn không thường biết id tại thời gian biên dịch.

Answer 2

Ý tưởng cá nhân của tôi là phương pháp sau có vẻ dễ hiểu hơn từ góc độ người dùng API.

Answer 3

Bạn chỉ có thể xem một API hiện có. Ví dụ, hãy lấy LINQ, nó có phương thức "Select" trả về nhiều thực thể và phương thức "Single" chỉ trả về một thực thể. Hầu hết các API hiện có có hai phương thức khác nhau, không phải là các phương thức quá tải và tôi nghĩ điều này là hợp lý và dễ đọc hơn.

Answer 4

Tôi có xu hướng theo điểm đầu tiên trong danh sách "suy nghĩ của bạn vượt qua ý tưởng này" cho biết "quá tải phải trả về cùng loại".

nhưng sau đó bạn có thể quá tải "LoadMany" với một số trường hợp khác nhau;

public Customer Load(int id) 
{ 
    // return just one customer 
} 

public List<Customer> LoadMany() 
{ 
    // return every single customer 
} 

public List<Customer> LoadMany(int statusFilter) 
{ 
    // return a filtered list of customers 
} 

public List<Customer> LoadMany(DateTime InitialContactFrom) 
{ 
    // return a filtered list of customers 
} 

public List<Customer> LoadMany(DateTime InitialContactFrom, DateTime InitialContactBefore) 
{ 
    // return a filtered list of customers 
} 

... bất kỳ kết hợp nào bạn cần rõ ràng có thể được thêm vào nhưng cuối cùng, LoadMany trả về danh sách và Tải trả về một thực thể.

Answer 5

Có thể có ngoại lệ, nhưng trừ khi bạn có lý do thực sự tốt để trả về các loại khác nhau, chức năng và quá tải của nó phải trả về cùng loại để bạn không khiến các nhà phát triển khác phát điên.

Ví dụ:

var a = MyFunc("Some text"); 

và

var a = MyFunc(1); 

cả trông giống như họ nên giải quyết var để cùng loại với tôi. Trước khi tôi đi chệch khỏi việc có tất cả các tình trạng quá tải trở về cùng một loại, tôi sẽ đảm bảo rằng tôi có một lý do rất vững chắc để trả lại các loại khác nhau.