KeyedCollection 与 KeyNotFoundException

(来自：http://blog.csdn.net/ahbian/article/details/4286076)

实现 KeyedCollection <int, object> 时，容易遇到 KeyNotFoundException。

原来是一个书写习惯引起的。

有时错误，还真不是一眼就能看出来的。

 

 

 class KV
     {
         private int id;
         private string text;

         internal KVCollection OwnerCollection;

         public int Id
         {
             get {}
             set {}
         }

         public string Text
         {
             get {}
             set {}
         }
     }


     class KVCollection : KeyedCollection<int, KV>
     {
         protected override int GetKeyForItem ( KV item )
         {
             return item.Id;
         }

         protected override void SetItem ( int index, KV newItem )
         {
             if (newItem.OwnerCollection != null)
                 throw new ArgumentException("The item already belongs to a collection.");

             // 此处会犯一个习惯性错误，但仅当键的类型恰好为 int 时，才会成为错误。
             // 我们通常会这样写：
             //         KV replaced = this[index];
             // 如果键类型不 int，则按序号访问集合中的值，这是符合意图的。
             // 但如果键类型为 int 时，则只能按键访问了，此时通常会引发 KeyNotFoundException 。
             // 若想安全地按序号访问集合中的值，应该坚持使用：
             KV replaced = this.Items[index];

             base.SetItem(index, newItem);

             newItem.OwnerCollection = this;
             replaced.OwnerCollection = null;
         }

         protected override void RemoveItem ( int index )
         {
             // 此处会犯一个习惯性错误，但仅当键的类型恰好为 int 时，才会成为错误。
             // 我们通常会这样写：
             //         KV removedItem = this[index];
             // 如果键类型不 int，则按序号访问集合中的值，这是符合意图的。
             // 但如果键类型为 int 时，则只能按键访问了，此时通常会引发 KeyNotFoundException 。
             // 若想安全地按序号访问集合中的值，应该坚持使用：

             KV removedItem = this.Items[index];

             base.RemoveItem(index);

             removedItem.OwnerCollection = null;

             OnListChanged(new System.ComponentModel.ListChangedEventArgs(System.ComponentModel.ListChangedType.ItemDeleted, index));
         }

     }