Question

Sorry to ask this yet again. I’ve been reading the related topics here for hours and still don’t quite get some of the issues with casting and generics. I’m trying to create our own HashSet class, because we can’t use .net 3.5 yet.

The error is in the RetainAll method, below. It compiles fine but I get a runtime error ‘Unable to cast object of type ‘System.Collections.DictionaryEntry’ to type ‘mycompany.UnivID’ on the foreach statement.

Does this mean I can’t cast a generic object to its implementing type? I’ve tried a couple things like

foreach ((T) obj in set) 

but they don’t compile.

Relevant code is:

public sealed class HashSet<T> : ISet<T> {    private readonly Hashtable set;    public HashSet() {     this.set = new Hashtable();   }    public void RetainAll(ISet<T> c) {     lock (set) {       foreach (T obj in set) {         if (!c.Contains(obj))           this.set.Remove(obj);       }     }   } } 

The implementing class snippet is

public sealed class UnivIDSet : ISet {   private readonly ISet<UnivID> univIDs;    public UnivIDSet() : base() {     univIDs = new HashSet<UnivID>();   } } 

and then we’ve got our UnivID class which are actually the objects in the sets.

Answer 1

I think

foreach (T obj in set) { 

should be

foreach (T obj in set.Values) {