From d227242b4940fabc5a39da75f0ed3cce2c44cb62 Mon Sep 17 00:00:00 2001 From: Gary Wu Date: Thu, 13 Apr 2017 16:37:37 -0700 Subject: Simplify AbstractOperation methods Change-Id: I58fa346425ab996ed7b9550bd8f21be679b08d0a Signed-off-by: Gary Wu --- .../model/operations/impl/AbstractOperation.java | 47 +++++----------------- .../model/operations/impl/GroupTypeOperation.java | 4 +- .../model/operations/impl/PolicyTypeOperation.java | 4 +- 3 files changed, 11 insertions(+), 44 deletions(-) (limited to 'catalog-model') diff --git a/catalog-model/src/main/java/org/openecomp/sdc/be/model/operations/impl/AbstractOperation.java b/catalog-model/src/main/java/org/openecomp/sdc/be/model/operations/impl/AbstractOperation.java index 0f36f09ca8..2847367999 100644 --- a/catalog-model/src/main/java/org/openecomp/sdc/be/model/operations/impl/AbstractOperation.java +++ b/catalog-model/src/main/java/org/openecomp/sdc/be/model/operations/impl/AbstractOperation.java @@ -21,11 +21,11 @@ package org.openecomp.sdc.be.model.operations.impl; import java.lang.reflect.Type; -import java.util.ArrayList; import java.util.List; import java.util.Map; import java.util.function.Function; import java.util.function.Supplier; +import java.util.stream.Collectors; import org.apache.commons.lang3.tuple.ImmutablePair; import org.openecomp.sdc.be.config.BeEcompErrorManager; @@ -118,7 +118,6 @@ public abstract class AbstractOperation { log.debug("Before adding {} to graph. data = {}", defName, someData); - @SuppressWarnings("unchecked") Either eitherSomeData = titanGenericDao.createNode(someData); log.debug("After adding {} to graph. status is = {}", defName, eitherSomeData); @@ -284,27 +283,15 @@ public abstract class AbstractOperation { PropertyTypeValidator validator = type.getValidator(); - boolean isValid = validator.isValid(value, innerType, dataTypes); - if (true == isValid) { - return true; - } else { - return false; - } - + return validator.isValid(value, innerType, dataTypes); } public boolean isEmptyValue(String value) { - if (value == null) { - return true; - } - return false; + return value == null; } public boolean isNullParam(String value) { - if (value == null) { - return true; - } - return false; + return value == null; } protected StorageOperationStatus validateAndUpdateComplexValue(IComplexDefaultValue propertyDefinition, String propertyType, @@ -339,11 +326,7 @@ public abstract class AbstractOperation { if (jsonElement == null || jsonElement.isJsonNull()) { value = EMPTY_VALUE; } else { - if (jsonElement.toString().isEmpty()) { - value = ""; - } else { - value = jsonElement.toString(); - } + value = jsonElement.toString(); } return value; @@ -376,18 +359,11 @@ public abstract class AbstractOperation { */ public List convertConstraintsToString(List constraints) { - List result = null; - - if (constraints != null && false == constraints.isEmpty()) { - result = new ArrayList(); - for (PropertyConstraint propertyConstraint : constraints) { - String constraint = gson.toJson(propertyConstraint); - result.add(constraint); - } - + if (constraints == null || constraints.isEmpty()) { + return null; } - return result; + return constraints.stream().map(gson::toJson).collect(Collectors.toList()); } public List convertConstraints(List constraints) { @@ -396,18 +372,13 @@ public abstract class AbstractOperation { return null; } - List list = new ArrayList(); Type constraintType = new TypeToken() { }.getType(); Gson gson = new GsonBuilder().registerTypeAdapter(constraintType, new PropertyConstraintDeserialiser()).create(); - for (String constraintJson : constraints) { - PropertyConstraint propertyConstraint = gson.fromJson(constraintJson, constraintType); - list.add(propertyConstraint); - } + return constraints.stream().map(c -> gson.fromJson(c, PropertyConstraint.class)).collect(Collectors.toList()); - return list; } } diff --git a/catalog-model/src/main/java/org/openecomp/sdc/be/model/operations/impl/GroupTypeOperation.java b/catalog-model/src/main/java/org/openecomp/sdc/be/model/operations/impl/GroupTypeOperation.java index 2fcdf695be..f9057217a6 100644 --- a/catalog-model/src/main/java/org/openecomp/sdc/be/model/operations/impl/GroupTypeOperation.java +++ b/catalog-model/src/main/java/org/openecomp/sdc/be/model/operations/impl/GroupTypeOperation.java @@ -25,7 +25,6 @@ import java.util.HashSet; import java.util.List; import java.util.Map; import java.util.Set; -import java.util.function.Function; import org.apache.commons.lang3.tuple.ImmutablePair; import org.openecomp.sdc.be.config.BeEcompErrorManager; @@ -155,8 +154,7 @@ public class GroupTypeOperation extends AbstractOperation implements IGroupTypeO @Override public Either getGroupType(String uniqueId, boolean inTransaction) { - Function> groupTypeGetter = uId -> getGroupTypeByUid(uId); - return getElementType(groupTypeGetter, uniqueId, inTransaction); + return getElementType(this::getGroupTypeByUid, uniqueId, inTransaction); } diff --git a/catalog-model/src/main/java/org/openecomp/sdc/be/model/operations/impl/PolicyTypeOperation.java b/catalog-model/src/main/java/org/openecomp/sdc/be/model/operations/impl/PolicyTypeOperation.java index 796fc8dd34..adea89b9ea 100644 --- a/catalog-model/src/main/java/org/openecomp/sdc/be/model/operations/impl/PolicyTypeOperation.java +++ b/catalog-model/src/main/java/org/openecomp/sdc/be/model/operations/impl/PolicyTypeOperation.java @@ -23,7 +23,6 @@ package org.openecomp.sdc.be.model.operations.impl; import java.util.HashMap; import java.util.List; import java.util.Map; -import java.util.function.Function; import javax.annotation.Resource; @@ -167,8 +166,7 @@ public class PolicyTypeOperation extends AbstractOperation implements IPolicyTyp @Override public Either getPolicyType(String uniqueId, boolean inTransaction) { - Function> policyTypeGetter = uId -> getPolicyTypeByUid(uId); - return getElementType(policyTypeGetter, uniqueId, inTransaction); + return getElementType(this::getPolicyTypeByUid, uniqueId, inTransaction); } -- cgit 1.2.3-korg