removed copy method and implemented clone.

Signed-off-by: thiyagu06 <[email protected]>

Author: thiyagu06

src/main/java/dev/openfeature/sdk/ImmutableStructure.java

@@ -39,7 +39,7 @@ public ImmutableStructure() {
     public ImmutableStructure(Map<String, Value> attributes) {
         Map<String, Value> copy = attributes.entrySet()
                 .stream()
-                .collect(Collectors.toMap(Map.Entry::getKey, e -> copy(e.getValue())));
+                .collect(Collectors.toMap(Map.Entry::getKey, e -> e.getValue().clone()));
         this.attributes = new HashMap<>(copy);
     }
 
@@ -52,7 +52,7 @@ public Set<String> keySet() {
     @Override
     public Value getValue(String key) {
         Value value = this.attributes.get(key);
-        return copy(value);
+        return value.clone();
     }
 
     /**
@@ -86,30 +86,4 @@ public Map<String, Object> asObjectMap() {
                         e -> convertValue(getValue(e.getKey()))
                 ));
     }
-
-    /**
-     * Perform deep copy of value object.
-     *
-     * @param value value object
-     * @return new copy of the given value object
-     */
-    @SneakyThrows
-    private Value copy(Value value) {
-        if (value.isList()) {
-            List<Value> copy = value.asList().stream().map(Value::new).collect(Collectors.toList());
-            return new Value(copy);
-        }
-        if (value.isStructure()) {
-            Map<String, Value> copy = value.asStructure().asMap().entrySet().stream().collect(Collectors.toMap(
-                    Map.Entry::getKey,
-                    e -> copy(e.getValue())
-            ));
-            return new Value(new ImmutableStructure(copy));
-        }
-        if (value.isInstant()) {
-            Instant copy = Instant.ofEpochMilli(value.asInstant().toEpochMilli());
-            return new Value(copy);
-        }
-        return new Value(value.asObject());
-    }
 }

src/main/java/dev/openfeature/sdk/Value.java

@@ -2,8 +2,11 @@
 
 import java.time.Instant;
 import java.util.List;
+import java.util.Map;
+import java.util.stream.Collectors;
 
 import lombok.EqualsAndHashCode;
+import lombok.SneakyThrows;
 import lombok.ToString;
 
 /**
@@ -14,7 +17,7 @@
 @ToString
 @EqualsAndHashCode
 @SuppressWarnings({"PMD.BeanMembersShouldSerialize", "checkstyle:MissingJavadocType"})
-public class Value {
+public class Value implements Cloneable{
 
     private final Object innerObject;
 
@@ -238,4 +241,31 @@ public Instant asInstant() {
         }
         return null;
     }
+
+    /**
+     * Perform deep clone of value object.
+     *
+     * @return Value
+     */
+
+    @SneakyThrows
+    @Override
+    protected Value clone() {
+        if (this.isList()) {
+            List<Value> copy = this.asList().stream().map(Value::new).collect(Collectors.toList());
+            return new Value(copy);
+        }
+        if (this.isStructure()) {
+            Map<String, Value> copy = this.asStructure().asMap().entrySet().stream().collect(Collectors.toMap(
+                    Map.Entry::getKey,
+                    e -> e.getValue().clone()
+            ));
+            return new Value(new ImmutableStructure(copy));
+        }
+        if (this.isInstant()) {
+            Instant copy = Instant.ofEpochMilli(this.asInstant().toEpochMilli());
+            return new Value(copy);
+        }
+        return new Value(this.asObject());
+    }
 }