Config no longer implements Identifiable because this was in violation of the contrac...

[xmlrouter] / impl / src / main / java / org / wamblee / xmlrouter / impl / CompositeConfig.java

diff --git a/impl/src/main/java/org/wamblee/xmlrouter/impl/CompositeConfig.java b/impl/src/main/java/org/wamblee/xmlrouter/impl/CompositeConfig.java
index b378052c207b05015b78d427870070c733e7dcbf..3b45a3ef25b5d35d395e8703bafec1db0f1946cd 100644 (file)
--- a/impl/src/main/java/org/wamblee/xmlrouter/impl/CompositeConfig.java
+++ b/impl/src/main/java/org/wamblee/xmlrouter/impl/CompositeConfig.java
@@ -15,44 +15,89 @@
  */
 package org.wamblee.xmlrouter.impl;
 
-import java.util.LinkedHashMap;
-import java.util.Map;
+import static org.wamblee.xmlrouter.impl.MessageUtil.*;
+
+import java.util.ArrayList;
+import java.util.HashSet;
+import java.util.List;
+import java.util.Set;
 
 import org.wamblee.xmlrouter.common.Id;
-import org.wamblee.xmlrouter.config.Config;
+import org.wamblee.xmlrouter.config.Identifiable;
+
+/**
+ * Composite config. The composite config
+ * 
+ * @author Erik Brakkee
+ * 
+ * @param <T>
+ */
+public class CompositeConfig<T extends Identifiable<T>> implements
+    ExtendedConfig<T> {
 
-public class CompositeConfig<T> implements ExtendedConfig<T> {
+    private static final String PREFIX = "compositeconfig";
+    private static final String READ_ONLY_INSTANCE = "read only instance";
 
-    private Map<Id<T>, T> configs;
+    private Class<T> type;
+    private Set<String> prefixes;
+    private List<Id<T>> valueIds;
+    private List<T> values;
 
-    public CompositeConfig() {
-        configs = new LinkedHashMap<Id<T>, T>();
+    public CompositeConfig(Class<T> aType) {
+        type = aType;
+        prefixes = new HashSet<String>();
+        valueIds = new ArrayList<Id<T>>();
+        values = new ArrayList<T>();
     }
 
-    public void add(Config<T> aConfig) {
-        for (Id<T> id : aConfig.map().keySet()) {
-            configs.put(id, aConfig.map().get(id));
+    @Override
+    public Class<T> getType() {
+        return type;
+    }
+
+    @Override
+    public String getPrefix() {
+        return PREFIX;
+    }
+
+    public void addConfig(Config<T> aConfig) {
+        notNull("aConfig", aConfig);
+        if (prefixes.contains(aConfig.getPrefix())) {
+            throw new ConfigException("duplicate prefix '" +
+                aConfig.getPrefix() + "'");
+        }
+        String prefix = aConfig.getPrefix() + ".";
+        for (T item : aConfig.values()) {
+            Id<T> newId = new Id<T>(prefix + item.getId());
+            if (valueIds.contains(newId)) {
+                throw new ConfigException("duplicate id '" +
+                    item.getId().toString() + "'");
+            }
+        }
+
+        prefixes.add(aConfig.getPrefix());
+
+        for (T item : aConfig.values()) {
+            Id<T> newId = new Id<T>(prefix + item.getId());
+            valueIds.add(newId);
+            values.add(IdentifiablePrefixProxyFactory.getProxy(prefix, item,
+                Identifiable.class, aConfig.getType()));
         }
     }
 
     @Override
-    public Map<Id<T>, T> map() {
-        return configs;
+    public List<T> values() {
+        return values;
     }
 
     @Override
     public boolean remove(Id<T> aId) {
-        notSupported();
+        notSupported(READ_ONLY_INSTANCE);
         return false;
     }
 
-    private void notSupported() {
-        throw new RuntimeException("readonly instance");
-    }
-
     @Override
-    public Id<T> add(T aT) {
-        notSupported();
-        return null;
+    public void add(T aT) {
+        notSupported(READ_ONLY_INSTANCE);
     }
 }