Improve DynamicBindingAdapter safety
[mdsal.git] / binding / mdsal-binding-dom-adapter / src / main / java / org / opendaylight / mdsal / binding / dom / adapter / LazySerializedContainerNode.java
index d684cf11ffdcc56e242c704df13221bae9461ea1..fad68245c2cebd41c14a672e47272f92f6bd583b 100644 (file)
@@ -5,13 +5,12 @@
  * terms of the Eclipse Public License v1.0 which accompanies this distribution,
  * and is available at http://www.eclipse.org/legal/epl-v10.html
  */
-
 package org.opendaylight.mdsal.binding.dom.adapter;
 
-import com.google.common.base.Optional;
-import java.util.Collection;
-import java.util.Map;
-import org.opendaylight.yangtools.binding.data.codec.impl.BindingNormalizedNodeCodecRegistry;
+import static java.util.Objects.requireNonNull;
+
+import org.opendaylight.mdsal.binding.dom.codec.api.BindingNormalizedNodeSerializer;
+import org.opendaylight.mdsal.binding.dom.codec.spi.AbstractBindingLazyContainerNode;
 import org.opendaylight.yangtools.yang.binding.DataObject;
 import org.opendaylight.yangtools.yang.common.QName;
 import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier.NodeIdentifier;
@@ -19,107 +18,54 @@ import org.opendaylight.yangtools.yang.data.api.YangInstanceIdentifier.PathArgum
 import org.opendaylight.yangtools.yang.data.api.schema.ContainerNode;
 import org.opendaylight.yangtools.yang.data.api.schema.DataContainerChild;
 import org.opendaylight.yangtools.yang.data.api.schema.LeafNode;
-import org.opendaylight.yangtools.yang.data.api.schema.NormalizedNode;
-import org.opendaylight.yangtools.yang.model.api.SchemaPath;
 
-/**
- *
- * FIXME: Should be this moved to binding-data-codec?
+/*
+ * FIXME: This is a bit of functionality which should really live in binding-dom-codec, but for to happen we need
+ *        to extend BindingNormalizedNodeSerializer with the concept of a routing context -- which would be deprecated,
+ *        as we want to move to actions in the long term.
  *
+ *        Even then, BindingNormalizedNodeCodecRegistry provides background updates to the context used in
+ *        deserialization, which is currently being used.
  */
-class LazySerializedContainerNode implements ContainerNode {
-
-    private final NodeIdentifier identifier;
-    private final DataObject bindingData;
-
-    private BindingNormalizedNodeCodecRegistry registry;
-    private ContainerNode domData;
+class LazySerializedContainerNode
+        extends AbstractBindingLazyContainerNode<DataObject, BindingNormalizedNodeSerializer> {
 
     private LazySerializedContainerNode(final QName identifier, final DataObject binding,
-            final BindingNormalizedNodeCodecRegistry registry) {
-        this.identifier = new NodeIdentifier(identifier);
-        this.bindingData = binding;
-        this.registry = registry;
-        this.domData = null;
-    }
-
-    static NormalizedNode<?, ?> create(final SchemaPath rpcName, final DataObject data,
-            final BindingNormalizedNodeCodecRegistry codec) {
-        return new LazySerializedContainerNode(rpcName.getLastComponent(), data, codec);
-    }
-
-    static NormalizedNode<?, ?> withContextRef(final SchemaPath rpcName, final DataObject data,
-            final LeafNode<?> contextRef, final BindingNormalizedNodeCodecRegistry codec) {
-        return new WithContextRef(rpcName.getLastComponent(), data, contextRef, codec);
-    }
-
-    @Override
-    public Map<QName, String> getAttributes() {
-        return delegate().getAttributes();
-    }
-
-    private ContainerNode delegate() {
-        if (domData == null) {
-            domData = registry.toNormalizedNodeRpcData(bindingData);
-            registry = null;
-        }
-        return domData;
-    }
-
-    @Override
-    public final QName getNodeType() {
-        return identifier.getNodeType();
-    }
-
-    @Override
-    public final Collection<DataContainerChild<? extends PathArgument, ?>> getValue() {
-        return delegate().getValue();
+            final BindingNormalizedNodeSerializer codec) {
+        super(NodeIdentifier.create(identifier), binding, requireNonNull(codec));
     }
 
-    @Override
-    public final NodeIdentifier getIdentifier() {
-        return identifier;
+    static ContainerNode create(final QName rpcName, final DataObject data,
+            final BindingNormalizedNodeSerializer codec) {
+        return data == null ? null : new LazySerializedContainerNode(rpcName, data, codec);
     }
 
-    @Override
-    public Optional<DataContainerChild<? extends PathArgument, ?>> getChild(final PathArgument child) {
-        return delegate().getChild(child);
+    static ContainerNode withContextRef(final QName rpcName, final DataObject data,
+            final LeafNode<?> contextRef, final BindingNormalizedNodeSerializer serializer) {
+        return new WithContextRef(rpcName, data, contextRef, serializer);
     }
 
     @Override
-    public final Object getAttributeValue(final QName name) {
-        return delegate().getAttributeValue(name);
-    }
-
-    final DataObject bindingData() {
-        return bindingData;
+    protected final ContainerNode computeContainerNode(final BindingNormalizedNodeSerializer context) {
+        return context.toNormalizedNodeRpcData(getDataObject());
     }
 
     /**
      * Lazy Serialized Node with pre-cached serialized leaf holding routing information.
-     *
      */
     private static final class WithContextRef extends LazySerializedContainerNode {
-
         private final LeafNode<?> contextRef;
 
         protected WithContextRef(final QName identifier, final DataObject binding, final LeafNode<?> contextRef,
-                final BindingNormalizedNodeCodecRegistry registry) {
-            super(identifier, binding, registry);
-            this.contextRef = contextRef;
+                final BindingNormalizedNodeSerializer codec) {
+            super(identifier, binding, codec);
+            this.contextRef = requireNonNull(contextRef);
         }
 
         @Override
-        public Optional<DataContainerChild<? extends PathArgument, ?>> getChild(final PathArgument child) {
-            /*
-             * Use precached value of routing field and do not run full serialization if we are
-             * accessing it.
-             */
-            if (contextRef.getIdentifier().equals(child)) {
-                return Optional.<DataContainerChild<? extends PathArgument, ?>>of(contextRef);
-            }
-            return super.getChild(child);
+        public DataContainerChild childByArg(final PathArgument child) {
+            // Use pre-cached value of routing field and do not run full serialization if we are accessing it.
+            return contextRef.getIdentifier().equals(child) ? contextRef : super.childByArg(child);
         }
     }
-
 }