If (items! = Null) có thừa trước foreach (T item trong các item) không?

Tôi thường gặp mã như sau:

if ( items != null)
{
   foreach(T item in items)
   {
        //...
   }
}

Về cơ bản, ifđiều kiện đảm bảo rằng foreachkhối sẽ thực thi chỉ khi itemskhông rỗng. Tôi đang tự hỏi liệu ifđiều kiện có thực sự cần thiết hay foreachsẽ xử lý trường hợp này nếu items == null.

Ý tôi là, tôi có thể viết đơn giản không

foreach(T item in items)
{
    //...
}

mà không cần lo lắng về việc liệu itemscó null hay không? Là ifthừa điều kiện? Hoặc này phụ thuộc vào loại của itemshoặc có thể trên Tkhông?

Bạn vẫn cần kiểm tra xem (items! = Null) nếu không bạn sẽ nhận được NullReferenceException. Tuy nhiên, bạn có thể làm như sau:

List<string> items = null;  
foreach (var item in items ?? new List<string>())
{
    item.Dump();
}

nhưng bạn có thể kiểm tra hiệu suất của nó. Vì vậy, tôi vẫn thích có if (items! = Null) trước.

Dựa trên gợi ý Lippert của Eric, tôi đã đổi mã thành:

List<string> items = null;  
foreach (var item in items ?? Enumerable.Empty<string>())
{
    item.Dump();
}

Sử dụng C # 6, bạn có thể sử dụng toán tử có điều kiện mới cùng với List<T>.ForEach(Action<T>)(hoặc IEnumerable<T>.ForEachphương thức mở rộng của riêng bạn ).

List<string> items = null;
items?.ForEach(item =>
{
    // ...
});

Bài học thực sự ở đây phải là một chuỗi hầu như không bao giờ được để trống ngay từ đầu . Đơn giản chỉ cần làm cho nó trở thành bất biến trong tất cả các chương trình của bạn mà nếu bạn có một chuỗi, nó không bao giờ là null. Nó luôn được khởi tạo là dãy trống hoặc một số dãy chính hãng khác.

Nếu một chuỗi không bao giờ là null thì rõ ràng bạn không cần kiểm tra nó.

Trên thực tế, có một yêu cầu tính năng trên @Connect đó: http://connect.microsoft.com/VisualStudio/feedback/details/93497/foreach-should-check-for-null

Và phản ứng khá hợp lý:

Tôi nghĩ rằng hầu hết các vòng lặp foreach được viết với mục đích lặp lại một tập hợp không rỗng. Nếu bạn thử lặp lại qua null, bạn sẽ nhận được ngoại lệ của mình để có thể sửa mã của mình.

Bạn luôn có thể kiểm tra nó với danh sách rỗng ... nhưng đây là những gì tôi tìm thấy trên trang web msdn

foreach-statement:
    foreach   (   type   identifier   in   expression   )   embedded-statement 

Nếu biểu thức có giá trị null, một System.NullReferenceException được ném ra.

Nó không phải là superflous. Trong thời gian chạy, các mục sẽ được chuyển sang IEnumerable và phương thức GetEnumerator của nó sẽ được gọi. Điều đó sẽ gây ra việc tham khảo các mục sẽ không thành công

Bạn có thể đóng gói kiểm tra null trong một phương thức mở rộng và sử dụng lambda:

public static class EnumerableExtensions {
  public static void ForEach<T>(this IEnumerable<T> self, Action<T> action) {
    if (self != null) {
      foreach (var element in self) {
        action(element);
      }
    }
  }
}

Mã trở thành:

items.ForEach(item => { 
  ...
});

Nếu có thể ngắn gọn hơn nữa nếu bạn chỉ muốn gọi một phương thức nhận một mục và trả về void:

items.ForEach(MethodThatTakesAnItem);

Bạn cần điều này. Bạn sẽ nhận được một ngoại lệ khiforeach truy cập vùng chứa để thiết lập lặp lại.

Dưới các bìa, foreachsử dụng một giao diện được triển khai trên lớp bộ sưu tập để thực hiện lặp lại. Giao diện tương đương chung là ở đây .

Câu lệnh foreach của ngôn ngữ C # (cho mỗi ngôn ngữ trong Visual Basic) ẩn sự phức tạp của các điều tra viên. Do đó, nên sử dụng foreach thay vì thao tác trực tiếp với điều tra viên.

Việc kiểm tra là cần thiết, bởi vì nếu bộ sưu tập là null, foreach sẽ ném ra một NullReferenceException. Nó thực sự khá đơn giản để thử nó.

List<string> items = null;
foreach(var item in items)
{
   Console.WriteLine(item);
}

thứ hai sẽ ném một NullReferenceExceptiontin nhắnObject reference not set to an instance of an object.

Như đã đề cập ở đây, bạn cần phải kiểm tra xem nó có phải không.

Không sử dụng một biểu thức đánh giá là null.

Trong C # 6, bạn có thể viết sth như thế này:

// some string from file or UI, i.e.:
// a) string s = "Hello, World!";
// b) string s = "";
// ...
var items = s?.Split(new char[] { ',', '!', ' ' }) ?? Enumerable.Empty<string>();  
foreach (var item in items)
{
    //..
}

Về cơ bản đó là giải pháp của Vlad Bezden nhưng sử dụng dấu ?? biểu thức để luôn tạo ra một mảng không rỗng và do đó tồn tại trước foreach hơn là có dấu kiểm này bên trong dấu ngoặc foreach.