Resharper ConcurrentBag<T> AssignNullToNotNull，但不使用 List<T>

Question

我对 ConcurrentBag<T> 和 List<T> 作为存储一些值的底层集合存在理解问题。

我正在编写一个应该是可枚举的类，我想遍历底层 ConcurrentBag<T>，但 ReSharper 告诉我 return _items.GetEnumerator(); 是：

可能对标有“NotNull”属性的实体进行“空”赋值

这是什么原因？

代码示例：

public class MyClass : IEnumerable<Item> {
    private readonly ConcurrentBag<Item> _items;

    public MyClass() {
        _items = new ConcurrentBag<Item>();
    }

    public IEnumerator<Item> GetEnumerator() {
        if(_items == null)
            throw new InvalidOperationException("Error.");
        return _items.GetEnumerator(); // AssignNullToNotNull
    }

    IEnumerator IEnumerable.GetEnumerator() {
        return GetEnumerator();
    }
}

ReSharper 告诉我一切都很好：

• List<T> 而不是 ConcurrentBag<T>。
• return ((IEnumerable<ITranslationItem>)_items).GetEnumerator();

我的方法有问题吗？

谢谢！

Answer 1

你的方法没有错。这是由 ReSharper 的外部注释引起的。 IEnumerable<T>.GetEnumerator() 标有[NotNullAttribute]。但是，IEnumerable 的 ConcurrentBag<T> 实现不是。

ReSharper 发现 MyClass.GetEnumerator() 不能为 null，并在发现 _items.GetEnumerator() 没有相同的约束时引发警告。

你可以

1. “禁止检查“可能的“空”赋值 [...] 并带有注释”

   // ReSharper disable AssignNullToNotNullAttribute
       return _items.GetEnumerator(); // AssignNullToNotNull
   // ReSharper restore AssignNullToNotNullAttribute
   

2. 将_items更改为private readonly IEnumerable<Item> _items

3. 在_items.GetEnumerator()上添加一个空检查

   IEnumerator<Item> retVal = _items.GetEnumerator();
   if (retVal == null)
       throw new InvalidOperationException("Error.");
   return retVal;