Hành vi dự kiến ​​khi một yêu cầu cho bộ sưu tập sẽ không có mục nào

Giả sử bạn được cung cấp ...

List<Thing> theThings = fubar.Things.All();

Nếu không có gì để trở lại, bạn sẽ mong chờ điều gì ở fubar.Things.All () sẽ trở lại?

Chỉnh sửa: Cảm ơn các ý kiến. Tôi sẽ đợi một chút và chấp nhận mục có nhiều nhất.

Tôi đồng ý với các câu trả lời cho đến nay, đặc biệt là những người đề xuất một bộ sưu tập trống. Một nhà cung cấp đã cung cấp một API với một số cuộc gọi tương tự như ví dụ trên. Một nhà cung cấp đã kiếm được 4,6 triệu đô la doanh thu thông qua (các) API của họ vào năm ngoái, BTW. Họ làm điều gì đó về cơ bản tôi không đồng ý - họ ném một ngoại lệ.

Trong hai khả năng (nghĩa là trả lại một nullhoặc trả lại một bộ sưu tập trống) tôi sẽ chọn trả về một bộ sưu tập trống, bởi vì nó cho phép người gọi bỏ qua kiểm tra giá trị được trả về. Thay vì viết cái này

List<Thing> theThings = fubar.Things.All();
if (theThings != null) {
    for (Thing t : theThings) {
        t.doSomething();
    }
}

họ sẽ có thể viết điều này:

List<Thing> theThings = fubar.Things.All();
for (Thing t : theThings) {
    t.doSomething();
}

Đoạn mã thứ hai này ngắn hơn và dễ đọc hơn, bởi vì mức lồng nhau thấp hơn một.

Tôi sẽ mong đợi một danh sách trống. theThingsvẫn sẽ là một đối tượng, nhưng theThings.Counthoặc theThings.size()sẽ trở lại 0.

Các vấn đề thiết kế như thế được giải quyết bằng mẫu Null Object

... Thay vì sử dụng tham chiếu null để truyền đạt sự vắng mặt của một đối tượng (ví dụ: khách hàng không tồn tại), người ta sử dụng một đối tượng thực hiện giao diện dự kiến, nhưng phần thân phương thức trống. Ưu điểm của phương pháp này so với triển khai mặc định đang hoạt động là đối tượng Null rất dễ đoán và không có tác dụng phụ: nó không làm gì cả.

Ví dụ, một hàm có thể truy xuất danh sách các tệp trong một thư mục và thực hiện một số hành động trên mỗi tệp. Trong trường hợp thư mục trống, một phản hồi có thể là ném ngoại lệ hoặc trả về tham chiếu null thay vì danh sách. Do đó, mã mong đợi một danh sách phải xác minh rằng trên thực tế nó có một mã trước khi tiếp tục, điều này có thể làm phức tạp thiết kế ...

Đề xuất đặc biệt áp dụng trong trường hợp của bạn (trở lại Listkhi không có Things) là:

... Thay vào đó, bằng cách trả về một đối tượng null (tức là một danh sách trống ), không cần phải xác minh rằng giá trị trả về thực tế là một danh sách. Hàm gọi có thể đơn giản lặp lại danh sách như bình thường, không có hiệu quả. Tuy nhiên, vẫn có thể kiểm tra xem giá trị trả về có phải là đối tượng null hay không (ví dụ: danh sách trống) và phản ứng khác nhau nếu muốn.

Bạn nên, IMHO, trả về giá trị EMPTY. Tôi không biết về C #, nhưng trong Java chúng ta có điều này:

  List list = Collections.EMPTY_LIST;
  Set set = Collections.EMPTY_SET;
  Map map = Collections.EMPTY_MAP;

  // For the type-safe 
  List<String> s = Collections.emptyList();
  Set<Long> l = Collections.emptySet();
  Map<Date> d = Collections.emptyMap();

http://docs.oracle.com/javase/1.4.2/docs/api/java/util/Collections.html

Tôi sẽ trả về một bộ sưu tập trống thay vì trả về giá trị null vì theo cách đó bạn có thể tránh việc viết xác minh null trong mã gọi.

Hai giải pháp họ có nghĩa là những thứ khác nhau.

Nếu không có thứ gì bạn trả lại, bạn LUÔN trả lại một bộ sưu tập trống! Lấy trường hợp của một danh sách thư mục. Nếu không có tệp nào trong thư mục, bạn trả về một tập hợp các tệp trống.

Mặt khác, nếu thư mục không tồn tại, điều đó không thực sự phù hợp. "Tôi không thể trả lại bất cứ điều gì" có nghĩa là một cái gì đó hoàn toàn khác. Trong trường hợp đó, bạn nên trả về null hoặc ném ngoại lệ tùy theo tình huống, đừng trả lại một bộ sưu tập trống như không có gì sai.