X-Git-Url: http://wamblee.org/gitweb/?a=blobdiff_plain;f=impl%2Fsrc%2Fmain%2Fjava%2Forg%2Fwamblee%2Fxmlrouter%2Fimpl%2FConfigImpl.java;h=8088be30e31df69d8bf87280f68d7bfef94bbf42;hb=d7938dd34e22ba8d34b4a89840c6efbb5d866380;hp=75b8222c04363038f3ed289eb5386a9c7843aded;hpb=06368da424915190303bcb501f0cc4cd1b74a2f2;p=xmlrouter

diff --git a/impl/src/main/java/org/wamblee/xmlrouter/impl/ConfigImpl.java b/impl/src/main/java/org/wamblee/xmlrouter/impl/ConfigImpl.java
index 75b8222..8088be3 100644
--- a/impl/src/main/java/org/wamblee/xmlrouter/impl/ConfigImpl.java
+++ b/impl/src/main/java/org/wamblee/xmlrouter/impl/ConfigImpl.java
@@ -17,13 +17,14 @@ package org.wamblee.xmlrouter.impl;
 
 import static org.wamblee.xmlrouter.impl.MessageUtil.*;
 
-import java.util.ArrayList;
+import java.util.Collection;
 import java.util.Collections;
-import java.util.Iterator;
-import java.util.List;
+import java.util.HashMap;
+import java.util.Map;
 
 import org.wamblee.xmlrouter.common.Id;
 import org.wamblee.xmlrouter.config.Config;
+import org.wamblee.xmlrouter.config.ConfigException;
 import org.wamblee.xmlrouter.config.Identifiable;
 
 /**
@@ -33,13 +34,11 @@ import org.wamblee.xmlrouter.config.Identifiable;
  * 
  * @param <T>
  */
-// TODO make sure that each item inside this config is prefixed with the id of
-// the config.
 public abstract class ConfigImpl<T extends Identifiable<T>> implements
     ExtendedConfig<T> {
 
     private Id<Config> id;
-    private List<T> registered;
+    private Map<Id<T>, T> registered;
 
     /**
      * Constructs the object.
@@ -47,7 +46,22 @@ public abstract class ConfigImpl<T extends Identifiable<T>> implements
     public ConfigImpl(Id<Config> aId) {
         notNull("id", aId);
         id = aId;
-        registered = new ArrayList<T>();
+        registered = new HashMap<Id<T>, T>();
+    }
+
+    /**
+     * Copies the config object.
+     * 
+     * @param aConfig
+     *            Config to copy.
+     */
+    public ConfigImpl(ConfigImpl<T> aConfig) {
+        notNull("config", aConfig);
+        id = aConfig.id;
+        registered = new HashMap<Id<T>, T>();
+        for (Map.Entry<Id<T>, T> entry : aConfig.registered.entrySet()) {
+            registered.put(entry.getKey(), entry.getValue());
+        }
     }
 
     @Override
@@ -63,7 +77,10 @@ public abstract class ConfigImpl<T extends Identifiable<T>> implements
     @Override
     public synchronized void add(T aT) {
         notNull("aT", aT);
-        registered.add(wrap(id.getId() + ".", aT));
+        if (registered.containsKey(aT.getId())) {
+            throw new ConfigException("Duplicate id '" + aT.getId() + "'");
+        }
+        registered.put(aT.getId(), wrap(id.getId() + ".", aT));
     }
 
     /**
@@ -85,19 +102,33 @@ public abstract class ConfigImpl<T extends Identifiable<T>> implements
     @Override
     public synchronized boolean remove(Id<T> aId) {
         notNull("aId", aId);
-        Iterator<T> i = registered.iterator();
-        while (i.hasNext()) {
-            T t = i.next();
-            if (t.getId().equals(aId)) {
-                i.remove();
-                return true;
-            }
+        T value = registered.get(aId);
+        if (value != null) {
+            registered.remove(aId);
+            return true;
         }
         return false;
     }
 
     @Override
-    public List<T> values() {
-        return Collections.unmodifiableList(registered);
+    public Collection<T> values() {
+        return Collections.unmodifiableCollection(registered.values());
+    }
+
+    @Override
+    public boolean equals(Object aObj) {
+        if (aObj == null) {
+            return false;
+        }
+        if (!(aObj instanceof ConfigImpl)) {
+            return false;
+        }
+        ConfigImpl obj = (ConfigImpl) aObj;
+        return registered.keySet().equals(obj.registered.keySet());
+    }
+
+    @Override
+    public int hashCode() {
+        return registered.keySet().hashCode();
     }
 }