X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=blobdiff_plain;f=binding%2Fmdsal-binding-java-api-generator%2Fsrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Fmdsal%2Fbinding%2Fjava%2Fapi%2Fgenerator%2FClassTemplate.xtend;h=be2f6e8e034c345e3395036b13c957f869e4ad87;hb=4c2990d640ee5cd0811e1c7af6f2664822f95cb8;hp=35dc26bd7c9fd91bbc071a9f3810c8adeae76164;hpb=f65d2bb69acd2495a2ee063d6264e742f64ffa59;p=mdsal.git diff --git a/binding/mdsal-binding-java-api-generator/src/main/java/org/opendaylight/mdsal/binding/java/api/generator/ClassTemplate.xtend b/binding/mdsal-binding-java-api-generator/src/main/java/org/opendaylight/mdsal/binding/java/api/generator/ClassTemplate.xtend index 35dc26bd7c..be2f6e8e03 100644 --- a/binding/mdsal-binding-java-api-generator/src/main/java/org/opendaylight/mdsal/binding/java/api/generator/ClassTemplate.xtend +++ b/binding/mdsal-binding-java-api-generator/src/main/java/org/opendaylight/mdsal/binding/java/api/generator/ClassTemplate.xtend @@ -8,21 +8,38 @@ package org.opendaylight.mdsal.binding.java.api.generator import static java.util.Objects.requireNonNull -import static org.opendaylight.mdsal.binding.model.util.Types.BOOLEAN; -import static org.opendaylight.mdsal.binding.model.util.Types.BYTE_ARRAY; -import static org.opendaylight.mdsal.binding.model.util.Types.STRING; +import static org.opendaylight.mdsal.binding.model.ri.BaseYangTypes.BINARY_TYPE +import static org.opendaylight.mdsal.binding.model.ri.BaseYangTypes.BOOLEAN_TYPE +import static org.opendaylight.mdsal.binding.model.ri.BaseYangTypes.DECIMAL64_TYPE +import static org.opendaylight.mdsal.binding.model.ri.BaseYangTypes.EMPTY_TYPE +import static org.opendaylight.mdsal.binding.model.ri.BaseYangTypes.INSTANCE_IDENTIFIER +import static org.opendaylight.mdsal.binding.model.ri.BaseYangTypes.INT16_TYPE +import static org.opendaylight.mdsal.binding.model.ri.BaseYangTypes.INT32_TYPE +import static org.opendaylight.mdsal.binding.model.ri.BaseYangTypes.INT64_TYPE +import static org.opendaylight.mdsal.binding.model.ri.BaseYangTypes.INT8_TYPE +import static org.opendaylight.mdsal.binding.model.ri.BaseYangTypes.STRING_TYPE +import static org.opendaylight.mdsal.binding.model.ri.BaseYangTypes.UINT16_TYPE +import static org.opendaylight.mdsal.binding.model.ri.BaseYangTypes.UINT32_TYPE +import static org.opendaylight.mdsal.binding.model.ri.BaseYangTypes.UINT64_TYPE +import static org.opendaylight.mdsal.binding.model.ri.BaseYangTypes.UINT8_TYPE +import static org.opendaylight.mdsal.binding.model.ri.BindingTypes.SCALAR_TYPE_OBJECT +import static org.opendaylight.mdsal.binding.model.ri.Types.BOOLEAN +import static org.opendaylight.mdsal.binding.model.ri.Types.STRING; import static extension org.apache.commons.text.StringEscapeUtils.escapeJava +import static extension org.opendaylight.mdsal.binding.model.ri.BindingTypes.isBitsType import com.google.common.base.Preconditions import com.google.common.collect.ImmutableList import com.google.common.collect.Lists +import edu.umd.cs.findbugs.annotations.SuppressFBWarnings import java.beans.ConstructorProperties import java.util.ArrayList import java.util.Base64; +import java.util.Collection import java.util.Comparator import java.util.List import java.util.Map -import java.util.regex.Pattern +import java.util.Set import javax.management.ConstructorParameters import org.gaul.modernizer_maven_annotations.SuppressModernizer import org.opendaylight.mdsal.binding.model.api.ConcreteType @@ -32,14 +49,9 @@ import org.opendaylight.mdsal.binding.model.api.GeneratedProperty import org.opendaylight.mdsal.binding.model.api.GeneratedTransferObject import org.opendaylight.mdsal.binding.model.api.Restrictions import org.opendaylight.mdsal.binding.model.api.Type -import org.opendaylight.mdsal.binding.model.util.TypeConstants -import org.opendaylight.yangtools.yang.binding.CodeHelpers +import org.opendaylight.mdsal.binding.model.ri.TypeConstants +import org.opendaylight.mdsal.binding.spec.naming.BindingMapping import org.opendaylight.yangtools.yang.common.Empty -import org.opendaylight.yangtools.yang.common.Uint16 -import org.opendaylight.yangtools.yang.common.Uint32 -import org.opendaylight.yangtools.yang.common.Uint64 -import org.opendaylight.yangtools.yang.common.Uint8 -import org.opendaylight.yangtools.yang.model.api.type.BitsTypeDefinition /** * Template for generating JAVA class. @@ -47,6 +59,17 @@ import org.opendaylight.yangtools.yang.model.api.type.BitsTypeDefinition @SuppressModernizer class ClassTemplate extends BaseTemplate { static val Comparator PROP_COMPARATOR = Comparator.comparing([prop | prop.name]) + static val VALUEOF_TYPES = Set.of( + BOOLEAN_TYPE, + DECIMAL64_TYPE, + INT8_TYPE, + INT16_TYPE, + INT32_TYPE, + INT64_TYPE, + UINT8_TYPE, + UINT16_TYPE, + UINT32_TYPE, + UINT64_TYPE) protected val List properties protected val List finalProperties @@ -100,7 +123,7 @@ class ClassTemplate extends BaseTemplate { this.consts = genType.constantDefinitions if (restrictions !== null && restrictions.rangeConstraint.present) { - rangeGenerator = requireNonNull(AbstractRangeGenerator.forType(findProperty(genType, "value").returnType)) + rangeGenerator = requireNonNull(AbstractRangeGenerator.forType(TypeUtils.encapsulatedValueType(genType))) } else { rangeGenerator = null } @@ -126,8 +149,11 @@ class ClassTemplate extends BaseTemplate { * @return string with class source code in JAVA format */ def protected generateBody(boolean isInnerClass) ''' - «wrapToDocumentation(formatDataForJavaDoc(type))» + «type.formatDataForJavaDoc.wrapToDocumentation» «annotationDeclaration» + «IF !isInnerClass» + «generatedAnnotation» + «ENDIF» «generateClassDeclaration(isInnerClass)» { «suidDeclaration» «innerClassesDeclarations» @@ -137,7 +163,8 @@ class ClassTemplate extends BaseTemplate { «IF restrictions !== null» «IF restrictions.lengthConstraint.present» - «LengthGenerator.generateLengthChecker("_value", findProperty(genTO, "value").returnType, restrictions.lengthConstraint.get, this)» + «LengthGenerator.generateLengthChecker("_value", TypeUtils.encapsulatedValueType(genTO), + restrictions.lengthConstraint.get, this)» «ENDIF» «IF restrictions.rangeConstraint.present» «rangeGenerator.generateRangeChecker("_value", restrictions.rangeConstraint.get, this)» @@ -148,14 +175,9 @@ class ClassTemplate extends BaseTemplate { «defaultInstance» - «FOR field : properties SEPARATOR "\n"» - «field.getterMethod» - «IF !field.readOnly» - «field.setterMethod» - «ENDIF» - «ENDFOR» + «propertyMethods» - «IF (genTO.isTypedef() && genTO.getBaseType instanceof BitsTypeDefinition)» + «IF genTO.isBitsType» «generateGetValueForBitsTypeDef» «ENDIF» @@ -168,6 +190,38 @@ class ClassTemplate extends BaseTemplate { ''' + def private propertyMethods() { + if (properties.empty) { + return "" + } + isScalarTypeObject ? scalarTypeObjectValue(properties.get(0)) : defaultProperties + } + + def private isScalarTypeObject() { + for (impl : genTO.implements) { + if (SCALAR_TYPE_OBJECT.identifier.equals(impl.identifier)) { + return true + } + } + return false + } + + def private defaultProperties() ''' + «FOR field : properties SEPARATOR "\n"» + «field.getterMethod» + «IF !field.readOnly» + «field.setterMethod» + «ENDIF» + «ENDFOR» + ''' + + def private scalarTypeObjectValue(GeneratedProperty field) ''' + @«OVERRIDE.importedName» + public «field.returnType.importedName» «BindingMapping.SCALAR_TYPE_OBJECT_GET_VALUE_NAME»() { + return «field.fieldName»«field.cloneCall»; + } + ''' + /** * Template method which generates the method getValue() for typedef, * which base type is BitsDefinition. @@ -201,7 +255,7 @@ class ClassTemplate extends BaseTemplate { def protected constructors() ''' «IF genTO.unionType» «genUnionConstructor» - «ELSEIF genTO.typedef && allProperties.size == 1 && allProperties.get(0).name.equals("value")» + «ELSEIF genTO.typedef && allProperties.size == 1 && allProperties.get(0).name.equals(TypeConstants.VALUE_PROP)» «typedefConstructor» «ELSE» «allValuesConstructor» @@ -215,9 +269,9 @@ class ClassTemplate extends BaseTemplate { «ENDIF» ''' - def private allValuesConstructor() ''' + def allValuesConstructor() ''' public «type.name»(«allProperties.asArgumentsDeclaration») { - «IF false == parentProperties.empty» + «IF !parentProperties.empty» super(«parentProperties.asArguments»); «ENDIF» «FOR p : allProperties» @@ -236,10 +290,10 @@ class ClassTemplate extends BaseTemplate { ''' def private typedefConstructor() ''' - @«ConstructorParameters.importedName»("value") - @«ConstructorProperties.importedName»("value") + @«ConstructorParameters.importedName»("«TypeConstants.VALUE_PROP»") + @«ConstructorProperties.importedName»("«TypeConstants.VALUE_PROP»") public «type.name»(«allProperties.asArgumentsDeclaration») { - «IF false == parentProperties.empty» + «IF !parentProperties.empty» super(«parentProperties.asArguments»); «ENDIF» «FOR p : allProperties» @@ -249,33 +303,14 @@ class ClassTemplate extends BaseTemplate { * If we have patterns, we need to apply them to the value field. This is a sad consequence of how this code is * structured. */» - «CodeHelpers.importedName».requireValue(_value); + «CODEHELPERS.importedName».requireValue(_value); «genPatternEnforcer("_value")» «FOR p : properties» «val fieldName = p.fieldName» - «IF p.returnType.name.endsWith("[]")» - this.«fieldName» = «fieldName».clone(); - «ELSE» - this.«fieldName» = «fieldName»; - «ENDIF» + this.«fieldName» = «fieldName»«p.cloneCall»; «ENDFOR» } - «val propType = allProperties.get(0).returnType» - «val uintType = UINT_TYPES.get(propType)» - «IF uintType !== null» - - /** - * Utility migration constructor. - * - * @param value Wrapped value in legacy type - * @deprecated Use {#link «type.name»(«propType.importedName»)} instead. - */ - @Deprecated(forRemoval = true) - public «type.getName»(final «uintType.importedName» value) { - this(«CodeHelpers.importedName».compatUint(value)); - } - «ENDIF» ''' def protected genUnionConstructor() ''' @@ -289,7 +324,7 @@ class ClassTemplate extends BaseTemplate { def protected genConstructor(GeneratedProperty property, Iterable other) ''' public «type.name»(«property.returnType.importedName + " " + property.name») { - «IF false == parentProperties.empty» + «IF !parentProperties.empty» super(«parentProperties.asArguments»); «ENDIF» @@ -306,7 +341,7 @@ class ClassTemplate extends BaseTemplate { def private genPatternEnforcer(String ref) ''' «FOR c : consts» «IF c.name == TypeConstants.PATTERN_CONSTANT_NAME» - «CodeHelpers.importedName».checkPattern(«ref», «Constants.MEMBER_PATTERN_LIST», «Constants.MEMBER_REGEX_LIST»); + «CODEHELPERS.importedName».checkPattern(«ref», «Constants.MEMBER_PATTERN_LIST», «Constants.MEMBER_REGEX_LIST»); «ENDIF» «ENDFOR» ''' @@ -319,7 +354,7 @@ class ClassTemplate extends BaseTemplate { } } - def private generateRestrictions(Type type, String paramName, Type returnType) ''' + def generateRestrictions(Type type, String paramName, Type returnType) ''' «val restrictions = type.restrictions» «IF restrictions !== null» «IF restrictions.lengthConstraint.present || restrictions.rangeConstraint.present» @@ -342,7 +377,7 @@ class ClassTemplate extends BaseTemplate { * @param source Source object */ public «type.name»(«type.name» source) { - «IF false == parentProperties.empty» + «IF !parentProperties.empty» super(source); «ENDIF» «FOR p : properties» @@ -367,52 +402,38 @@ class ClassTemplate extends BaseTemplate { def protected defaultInstance() ''' «IF genTO.typedef && !allProperties.empty && !genTO.unionType» «val prop = allProperties.get(0)» - «IF !("org.opendaylight.yangtools.yang.binding.InstanceIdentifier".equals(prop.returnType.fullyQualifiedName))» - public static «genTO.name» getDefaultInstance(String defaultValue) { - «IF BYTE_ARRAY.equals(prop.returnType)» - return new «genTO.name»(«Base64.importedName».getDecoder().decode(defaultValue)); - «ELSEIF STRING.equals(prop.returnType)» + «val propType = prop.returnType» + «IF !(INSTANCE_IDENTIFIER.identifier.equals(propType.identifier))» + public static «genTO.name» getDefaultInstance(final String defaultValue) { + «IF allProperties.size > 1» + «bitsArgs» + «ELSEIF VALUEOF_TYPES.contains(propType)» + return new «genTO.name»(«propType.importedName».valueOf(defaultValue)); + «ELSEIF STRING_TYPE.equals(propType)» return new «genTO.name»(defaultValue); - «ELSEIF Constants.EMPTY.equals(prop.returnType)» + «ELSEIF BINARY_TYPE.equals(propType)» + return new «genTO.name»(«Base64.importedName».getDecoder().decode(defaultValue)); + «ELSEIF EMPTY_TYPE.equals(propType)» «Preconditions.importedName».checkArgument(defaultValue.isEmpty(), "Invalid value %s", defaultValue); - return new «genTO.name»(«Empty.importedName».getInstance()); - «ELSEIF allProperties.size > 1» - «bitsArgs» - «ELSEIF BOOLEAN.equals(prop.returnType)» - return new «genTO.name»(«Boolean.importedName».valueOf(defaultValue)); - «ELSEIF "java.lang.Byte".equals(prop.returnType.fullyQualifiedName)» - return new «genTO.name»(«Byte.importedName».valueOf(defaultValue)); - «ELSEIF "java.lang.Short".equals(prop.returnType.fullyQualifiedName)» - return new «genTO.name»(«Short.importedName».valueOf(defaultValue)); - «ELSEIF "java.lang.Integer".equals(prop.returnType.fullyQualifiedName)» - return new «genTO.name»(«Integer.importedName».valueOf(defaultValue)); - «ELSEIF "java.lang.Long".equals(prop.returnType.fullyQualifiedName)» - return new «genTO.name»(«Long.importedName».valueOf(defaultValue)); - «ELSEIF "org.opendaylight.yangtools.yang.common.Uint8".equals(prop.returnType.fullyQualifiedName)» - return new «genTO.name»(«Uint8.importedName».valueOf(defaultValue)); - «ELSEIF "org.opendaylight.yangtools.yang.common.Uint16".equals(prop.returnType.fullyQualifiedName)» - return new «genTO.name»(«Uint16.importedName».valueOf(defaultValue)); - «ELSEIF "org.opendaylight.yangtools.yang.common.Uint32".equals(prop.returnType.fullyQualifiedName)» - return new «genTO.name»(«Uint32.importedName».valueOf(defaultValue)); - «ELSEIF "org.opendaylight.yangtools.yang.common.Uint64".equals(prop.returnType.fullyQualifiedName)» - return new «genTO.name»(«Uint64.importedName».valueOf(defaultValue)); + return new «genTO.name»(«Empty.importedName».value()); «ELSE» - return new «genTO.name»(new «prop.returnType.importedName»(defaultValue)); + return new «genTO.name»(new «propType.importedName»(defaultValue)); «ENDIF» } «ENDIF» «ENDIF» ''' + @SuppressFBWarnings(value = "DLS_DEAD_LOCAL_STORE", justification = "FOR with SEPARATOR, not needing for value") def protected bitsArgs() ''' - «List.importedName»<«String.importedName»> properties = «Lists.importedName».newArrayList(«allProperties.propsAsArgs»); + «JU_LIST.importedName»<«STRING.importedName»> properties = «Lists.importedName».newArrayList(«allProperties.propsAsArgs»); if (!properties.contains(defaultValue)) { - throw new «IllegalArgumentException.importedName»("invalid default parameter"); + throw new «IAE.importedName»("invalid default parameter"); } int i = 0; return new «genTO.name»( «FOR prop : allProperties SEPARATOR ","» - properties.get(i++).equals(defaultValue) ? «Boolean.importedName».TRUE : null + properties.get(i++).equals(defaultValue) ? «BOOLEAN.importedName».TRUE : null «ENDFOR» ); ''' @@ -486,13 +507,14 @@ class ClassTemplate extends BaseTemplate { «FOR c : consts» «IF c.name == TypeConstants.PATTERN_CONSTANT_NAME» «val cValue = c.value as Map» - public static final «List.importedName» «TypeConstants.PATTERN_CONSTANT_NAME» = «ImmutableList.importedName».of(« + «val jurPatternRef = JUR_PATTERN.importedName» + public static final «JU_LIST.importedName» «TypeConstants.PATTERN_CONSTANT_NAME» = «ImmutableList.importedName».of(« FOR v : cValue.keySet SEPARATOR ", "»"«v.escapeJava»"«ENDFOR»); «IF cValue.size == 1» - private static final «Pattern.importedName» «Constants.MEMBER_PATTERN_LIST» = «Pattern.importedName».compile(«TypeConstants.PATTERN_CONSTANT_NAME».get(0)); + private static final «jurPatternRef» «Constants.MEMBER_PATTERN_LIST» = «jurPatternRef».compile(«TypeConstants.PATTERN_CONSTANT_NAME».get(0)); private static final String «Constants.MEMBER_REGEX_LIST» = "«cValue.values.iterator.next.escapeJava»"; «ELSE» - private static final «Pattern.importedName»[] «Constants.MEMBER_PATTERN_LIST» = «CodeHelpers.importedName».compilePatterns(«TypeConstants.PATTERN_CONSTANT_NAME»); + private static final «jurPatternRef»[] «Constants.MEMBER_PATTERN_LIST» = «CODEHELPERS.importedName».compilePatterns(«TypeConstants.PATTERN_CONSTANT_NAME»); private static final String[] «Constants.MEMBER_REGEX_LIST» = { « FOR v : cValue.values SEPARATOR ", "»"«v.escapeJava»"«ENDFOR» }; «ENDIF» @@ -531,13 +553,17 @@ class ClassTemplate extends BaseTemplate { return "" } return ''' - @«Override.importedName» + @«OVERRIDE.importedName» public int hashCode() { «IF size != 1» - «hashCodeResult(genTO.hashCodeIdentifiers)» + final int prime = 31; + int result = 1; + «FOR property : genTO.hashCodeIdentifiers» + result = prime * result + «property.importedUtilClass».hashCode(«property.fieldName»); + «ENDFOR» return result; «ELSE» - return «CodeHelpers.importedName».wrapperHashCode(«genTO.hashCodeIdentifiers.get(0).fieldName»); + return «CODEHELPERS.importedName».wrapperHashCode(«genTO.hashCodeIdentifiers.get(0).fieldName»); «ENDIF» } ''' @@ -550,7 +576,7 @@ class ClassTemplate extends BaseTemplate { */ def private generateEquals() ''' «IF !genTO.equalsIdentifiers.empty» - @«Override.importedName» + @«OVERRIDE.importedName» public final boolean equals(java.lang.Object obj) { if (this == obj) { return true; @@ -570,13 +596,25 @@ class ClassTemplate extends BaseTemplate { «ENDIF» ''' + def private generateToString(Collection properties) ''' + «IF !properties.empty» + @«OVERRIDE.importedName» + public «STRING.importedName» toString() { + final var helper = «MOREOBJECTS.importedName».toStringHelper(«type.importedName».class); + «FOR property : properties» + «CODEHELPERS.importedName».appendValue(helper, "«property.fieldName»", «property.fieldName»); + «ENDFOR» + return helper.toString(); + } + «ENDIF» + ''' + def GeneratedProperty getPropByName(String name) { for (GeneratedProperty prop : allProperties) { if (prop.name.equals(name)) { return prop; } } - return null; + return null } - }