Est-il correct de transmettre un dictionnaire générique en tant que param à une méthode attendue IDictionary?

Question

AssemblyInstaller.Install attend un System.Collections.IDictionary. Ai-je raison d'être «allergique» à l'utilisation de collections non génériques comme Hashtable ou devrais-je m'en remettre?

par exemple.

using System.Collections.Generic; 
using System.Configuration.Install; 
using System.Reflection; 
using AssemblyWithInstaller; 

namespace InstallerDemo 
{ 
    class InstallerDemo 
    { 
     static void Main(string[] args) 
     { 
      var savedState = new Dictionary<object, object>(); 
      // i.e. as opposed to something that implements IDictionary: 
      //var savedState = new System.Collections.Hashtable() 
      var assembly = Assembly.GetAssembly(typeof (MyInstaller)); 
      var ai = new AssemblyInstaller(assembly, new[] {"/LogFile=install.log"}); 
      ai.Install(savedState); 
      ai.Commit(savedState); 
     } 
    } 
} 

De plus, le compilateur n'a pas de problème avec ce decalration:

var savedState = new Dictionary<string, object>(); 

Mais sera quelque chose de mauvais se lors de l'exécution si quelqu'un utilise autre chose que des chaînes comme clés?

---

Mise à jour [réflecteur à la rescousse]

var savedState = new Dictionary<string, object>(); 

Confirmant ce que dit Jon, Dictionnaire implémente IDictionary comme suit:

void IDictionary.Add(object key, object value) 
{ 
    Dictionary<TKey, TValue>.VerifyKey(key); 
    Dictionary<TKey, TValue>.VerifyValueType(value); 
    this.Add((TKey) key, (TValue) value); 
} 

... donc à vérifier la clé, il lancera une exception lorsque le type de la clé ne correspond pas à celui utilisé lors de la déclaration de la spécialisation particulière du dictionnaire générique (et de même pour la pe de la valeur):

private static void VerifyKey(object key) 
{ 
    if (key == null) 
    { 
     ThrowHelper.ThrowArgumentNullException(ExceptionArgument.key); 
    } 
    if (!(key is TKey)) 
    { 
     ThrowHelper.ThrowWrongKeyTypeArgumentException(key, typeof(TKey)); 
    } 
} 

Answer 1

Cela fonctionne parce que Dictionary<TKey, TValue> implémente IDictionary - mais il sera en effet l'échec au moment de l'exécution si quelqu'un appelle IDictionary.Add(object, object) avec une clé non-string - vous obtiendrez un ArgumentException.

Notez que lui-même ne couvre pas l'interface IDictionary<TKey, TValue>IDictionary - il est juste que la mise en œuvre Dictionary<TKey, TValue>également implémente IDictionary.

Answer 2

Depuis le programme d'installation est l'acceptation d'une IDictionary-objet, il devrait être en mesure de traiter tout Dictionnaire passé à lui. Au moins c'est mon opinion ... si je prends une interface, je dois être capable de gérer n'importe quelle implémentation.

En outre, je suggère que vous l'essayiez juste.