X-Git-Url: https://git.opendaylight.org/gerrit/gitweb?a=blobdiff_plain;f=yang%2Fyang-common%2Fsrc%2Fmain%2Fjava%2Forg%2Fopendaylight%2Fyangtools%2Fyang%2Fcommon%2FQName.java;h=2f6c2bf6c421d4bba45e0b0bb7044fb9380fdfce;hb=7768e923cc12f467ad7e5e32577bdd1a10a4d581;hp=c8b3f0039eb84981730973aafaeb6455387075b5;hpb=15edd1ff6d8dd9e5ff9cafe5cbd69b87db81b5e9;p=yangtools.git diff --git a/yang/yang-common/src/main/java/org/opendaylight/yangtools/yang/common/QName.java b/yang/yang-common/src/main/java/org/opendaylight/yangtools/yang/common/QName.java index c8b3f0039e..2f6c2bf6c4 100644 --- a/yang/yang-common/src/main/java/org/opendaylight/yangtools/yang/common/QName.java +++ b/yang/yang-common/src/main/java/org/opendaylight/yangtools/yang/common/QName.java @@ -10,7 +10,7 @@ package org.opendaylight.yangtools.yang.common; import static com.google.common.base.Preconditions.checkArgument; import static java.util.Objects.requireNonNull; -import com.google.common.base.Strings; +import com.google.common.base.CharMatcher; import com.google.common.collect.Interner; import com.google.common.collect.Interners; import java.io.DataInput; @@ -23,22 +23,24 @@ import java.util.Objects; import java.util.Optional; import java.util.regex.Matcher; import java.util.regex.Pattern; -import javax.annotation.Nonnull; -import javax.annotation.Nullable; import javax.annotation.RegEx; +import org.eclipse.jdt.annotation.NonNull; +import org.eclipse.jdt.annotation.Nullable; import org.opendaylight.yangtools.concepts.Identifier; import org.opendaylight.yangtools.concepts.Immutable; import org.opendaylight.yangtools.concepts.WritableObject; /** - * The QName from XML consists of local name of element and XML namespace, but - * for our use, we added module revision to it. + * The QName from XML consists of local name of element and XML namespace, but for our use, we added module revision to + * it. * *

- * In YANG context QName is full name of defined node, type, procedure or - * notification. QName consists of XML namespace, YANG model revision and local - * name of defined type. It is used to prevent name clashes between nodes with - * same local name, but from different schemas. + * In YANG context QName is full name of defined node, type, procedure or notification. QName consists of XML namespace, + * YANG model revision and local name of defined type. It is used to prevent name clashes between nodes with same local + * name, but from different schemas. + * + *

+ * The local name must conform to RFC7950 Section 6.2. * *

- * - *

- * QName may also have prefix assigned, but prefix does not - * affect equality and identity of two QNames and carry only information - * which may be useful for serializers / deserializers. */ public final class QName implements Immutable, Serializable, Comparable, Identifier, WritableObject { private static final Interner INTERNER = Interners.newWeakInterner(); @@ -70,17 +67,18 @@ public final class QName implements Immutable, Serializable, Comparable, private static final String QNAME_STRING_NO_REVISION = "^\\((.+)\\)(.+)$"; private static final Pattern QNAME_PATTERN_NO_REVISION = Pattern.compile(QNAME_STRING_NO_REVISION); - private static final char[] ILLEGAL_CHARACTERS = new char[] { '?', '(', ')', '&', ':' }; + private static final CharMatcher IDENTIFIER_START = + CharMatcher.inRange('A', 'Z').or(CharMatcher.inRange('a', 'z').or(CharMatcher.is('_'))).precomputed(); + private static final CharMatcher NOT_IDENTIFIER_PART = + IDENTIFIER_START.or(CharMatcher.inRange('0', '9')).or(CharMatcher.anyOf("-.")).negate().precomputed(); - // Non-null - private final QNameModule module; - // Non-null - private final String localName; - private transient int hash; + private final @NonNull QNameModule module; + private final @NonNull String localName; + private transient int hash = 0; - private QName(final QNameModule module, final String localName) { - this.localName = checkLocalName(localName); - this.module = module; + private QName(final QNameModule module, final @NonNull String localName) { + this.module = requireNonNull(module); + this.localName = requireNonNull(localName); } /** @@ -92,23 +90,18 @@ public final class QName implements Immutable, Serializable, Comparable, * YANG schema identifier */ private QName(final URI namespace, final String localName) { - this(QNameModule.create(namespace), localName); + this(QNameModule.create(namespace), checkLocalName(localName)); } - private static String checkLocalName(final String localName) { + private static @NonNull String checkLocalName(final String localName) { checkArgument(localName != null, "Parameter 'localName' may not be null."); - checkArgument(!Strings.isNullOrEmpty(localName), "Parameter 'localName' must be a non-empty string."); - - for (final char c : ILLEGAL_CHARACTERS) { - if (localName.indexOf(c) != -1) { - throw new IllegalArgumentException(String.format( - "Parameter 'localName':'%s' contains illegal character '%s'", localName, c)); - } - } + checkArgument(!localName.isEmpty(), "Parameter 'localName' must be a non-empty string."); + checkArgument(IDENTIFIER_START.matches(localName.charAt(0)) && NOT_IDENTIFIER_PART.indexIn(localName, 1) == -1, + "String '%s' is not a valid identifier", localName); return localName; } - public static QName create(final String input) { + public static @NonNull QName create(final String input) { Matcher matcher = QNAME_PATTERN_FULL.matcher(input); if (matcher.matches()) { final String namespace = matcher.group(1); @@ -125,7 +118,7 @@ public final class QName implements Immutable, Serializable, Comparable, throw new IllegalArgumentException("Invalid input: " + input); } - public static QName create(final QName base, final String localName) { + public static @NonNull QName create(final QName base, final String localName) { return create(base.getModule(), localName); } @@ -138,111 +131,86 @@ public final class QName implements Immutable, Serializable, Comparable, * Local name part of QName. MUST NOT BE null. * @return Instance of QName */ - public static QName create(final QNameModule qnameModule, final String localName) { - return new QName(requireNonNull(qnameModule, "module may not be null"), localName); + public static @NonNull QName create(final QNameModule qnameModule, final String localName) { + return new QName(requireNonNull(qnameModule, "module may not be null"), checkLocalName(localName)); } /** * Creates new QName. * - * @param namespace - * Namespace of QName or null if namespace is undefined. - * @param revision - * Revision of namespace or null if revision is unspecified. - * @param localName - * Local name part of QName. MUST NOT BE null. + * @param namespace Namespace of QName or null if namespace is undefined. + * @param revision Revision of namespace or null if revision is unspecified. + * @param localName Local name part of QName. MUST NOT BE null. * @return Instance of QName */ - public static QName create(final URI namespace, @Nullable final Revision revision, final String localName) { + public static @NonNull QName create(final URI namespace, final @Nullable Revision revision, + final String localName) { return create(QNameModule.create(namespace, revision), localName); } /** * Creates new QName. * - * @param namespace - * Namespace of QName or null if namespace is undefined. - * @param revision - * Revision of namespace. - * @param localName - * Local name part of QName. MUST NOT BE null. + * @param namespace Namespace of QName or null if namespace is undefined. + * @param revision Revision of namespace. + * @param localName Local name part of QName. MUST NOT BE null. * @return Instance of QName */ - public static QName create(final URI namespace, final Optional revision, final String localName) { + public static @NonNull QName create(final URI namespace, final Optional revision, + final String localName) { return create(QNameModule.create(namespace, revision), localName); } /** * Creates new QName. * - * @param namespace - * Namespace of QName or null if namespace is undefined. - * @param revision - * Revision of namespace or null if revision is unspecified. - * @param localName - * Local name part of QName. MUST NOT BE null. + * @param namespace Namespace of QName or null if namespace is undefined. + * @param revision Revision of namespace or null if revision is unspecified. + * @param localName Local name part of QName. MUST NOT BE null. * @return Instance of QName */ - public static QName create(final String namespace, final String localName, final Revision revision) { - final URI namespaceUri = parseNamespace(namespace); - return create(QNameModule.create(namespaceUri, revision), localName); + public static @NonNull QName create(final String namespace, final String localName, final Revision revision) { + return create(QNameModule.create(parseNamespace(namespace), revision), localName); } /** * Creates new QName. * - * @param namespace - * Namespace of QName, MUST NOT BE Null. - * @param revision - * Revision of namespace / YANG module. MUST NOT BE null, MUST BE - * in format YYYY-mm-dd. - * @param localName - * Local name part of QName. MUST NOT BE null. + * @param namespace Namespace of QName, MUST NOT BE Null. + * @param revision Revision of namespace / YANG module. MUST NOT BE null, MUST BE in format {@code YYYY-mm-dd}. + * @param localName Local name part of QName. MUST NOT BE null. * @return A new QName - * @throws NullPointerException - * If any of parameters is null. - * @throws IllegalArgumentException - * If namespace is not valid URI or - * revision is not according to format - * YYYY-mm-dd. + * @throws NullPointerException If any of parameters is null. + * @throws IllegalArgumentException If {@code namespace} is not valid URI or {@code revision} does not conform + * to {@code YYYY-mm-dd}. */ - public static QName create(final String namespace, final String revision, final String localName) { - final URI namespaceUri = parseNamespace(namespace); - final Revision revisionDate = Revision.of(revision); - return create(namespaceUri, revisionDate, localName); + public static @NonNull QName create(final String namespace, final String revision, final String localName) { + return create(parseNamespace(namespace), Revision.of(revision), localName); } /** * Creates new QName. * - * @param namespace - * Namespace of QName, MUST NOT BE Null. - * @param localName - * Local name part of QName. MUST NOT BE null. + * @param namespace Namespace of QName, MUST NOT BE Null. + * @param localName Local name part of QName. MUST NOT BE null. * @return A new QName - * @throws NullPointerException - * If any of parameters is null. - * @throws IllegalArgumentException - * If namespace is not valid URI. + * @throws NullPointerException If any of parameters is null. + * @throws IllegalArgumentException If {@code namespace} is not valid URI. */ - public static QName create(final String namespace, final String localName) { + public static @NonNull QName create(final String namespace, final String localName) { return create(parseNamespace(namespace), localName); } /** * Creates new QName. * - * @param namespace - * Namespace of QName, MUST NOT BE null. - * @param localName - * Local name part of QName. MUST NOT BE null. + * @param namespace Namespace of QName, MUST NOT BE null. + * @param localName Local name part of QName. MUST NOT BE null. * @return A new QName - * @throws NullPointerException - * If any of parameters is null. - * @throws IllegalArgumentException - * If namespace is not valid URI. + * @throws NullPointerException If any of parameters is null. + * @throws IllegalArgumentException If namespace is not valid URI. */ - public static QName create(final URI namespace, final String localName) { + public static @NonNull QName create(final URI namespace, final String localName) { return new QName(namespace, localName); } @@ -255,8 +223,7 @@ public final class QName implements Immutable, Serializable, Comparable, */ public static QName readFrom(final DataInput in) throws IOException { final QNameModule module = QNameModule.readFrom(in); - final String localName = in.readUTF(); - return new QName(module, localName); + return new QName(module, checkLocalName(in.readUTF())); } /** @@ -264,7 +231,7 @@ public final class QName implements Immutable, Serializable, Comparable, * * @return Module component */ - public QNameModule getModule() { + public @NonNull QNameModule getModule() { return module; } @@ -273,7 +240,7 @@ public final class QName implements Immutable, Serializable, Comparable, * * @return XMLNamespace assigned to the YANG module. */ - public URI getNamespace() { + public @NonNull URI getNamespace() { return module.getNamespace(); } @@ -284,7 +251,7 @@ public final class QName implements Immutable, Serializable, Comparable, * @return YANG schema identifier which were defined for this node in the * YANG module */ - public String getLocalName() { + public @NonNull String getLocalName() { return localName; } @@ -293,7 +260,7 @@ public final class QName implements Immutable, Serializable, Comparable, * * @return revision of the YANG module if the module has defined revision. */ - public Optional getRevision() { + public @NonNull Optional getRevision() { return module.getRevision(); } @@ -302,14 +269,14 @@ public final class QName implements Immutable, Serializable, Comparable, * * @return Interned reference, or this object if it was interned. */ - public QName intern() { + public @NonNull QName intern() { // We also want to make sure we keep the QNameModule cached final QNameModule cacheMod = module.intern(); // Identity comparison is here on purpose, as we are deciding whether to potentially store 'qname' into the // interner. It is important that it does not hold user-supplied reference (such a String instance from // parsing of an XML document). - final QName template = cacheMod == module ? this : QName.create(cacheMod, localName.intern()); + final QName template = cacheMod == module ? this : new QName(cacheMod, localName.intern()); return INTERNER.intern(template); } @@ -323,14 +290,12 @@ public final class QName implements Immutable, Serializable, Comparable, } /** - * Compares the specified object with this list for equality. Returns - * true if and only if the specified object is also instance of - * {@link QName} and its {@link #getLocalName()}, {@link #getNamespace()} and + * Compares the specified object with this list for equality. Returns {@code true} if and only if the specified + * object is also instance of {@link QName} and its {@link #getLocalName()}, {@link #getNamespace()} and * {@link #getRevision()} are equals to same properties of this instance. * * @param obj the object to be compared for equality with this QName - * @return true if the specified object is equal to this QName - * + * @return {@code true} if the specified object is equal to this QName */ @Override public boolean equals(final Object obj) { @@ -344,16 +309,16 @@ public final class QName implements Immutable, Serializable, Comparable, return Objects.equals(localName, other.localName) && module.equals(other.module); } - private static URI parseNamespace(final String namespace) { + private static @NonNull URI parseNamespace(final String namespace) { try { return new URI(namespace); } catch (final URISyntaxException ue) { - throw new IllegalArgumentException(String.format("Namespace '%s' is not a valid URI", namespace), ue); + throw new IllegalArgumentException("Namespace '" + namespace + "' is not a valid URI", ue); } } @Override - public String toString() { + public @NonNull String toString() { final StringBuilder sb = new StringBuilder(); if (getNamespace() != null) { sb.append(QNAME_LEFT_PARENTHESIS).append(getNamespace()); @@ -364,32 +329,41 @@ public final class QName implements Immutable, Serializable, Comparable, } sb.append(QNAME_RIGHT_PARENTHESIS); } - sb.append(localName); - return sb.toString(); + return sb.append(localName).toString(); } /** - * Creates copy of this with revision and prefix unset. + * Returns a QName with the specified QNameModule and the same localname as this one. * - * @return copy of this QName with revision and prefix unset. + * @param newModule New QNameModule to use + * @return a QName with specified QNameModule and same local name as this one */ - public QName withoutRevision() { - return create(getNamespace(), localName); + public @NonNull QName withModule(final QNameModule newModule) { + return new QName(newModule, localName); } /** - * Formats {@link Revision} representing revision to format YYYY-mm-dd + * Returns a QName with the same namespace and local name, but with no revision. If this QName does not have + * a Revision, this object is returned. + * + * @return a QName with the same namespace and local name, but with no revision. + */ + public @NonNull QName withoutRevision() { + final QNameModule newModule; + return (newModule = module.withoutRevision()) == module ? this : new QName(newModule, localName); + } + + /** + * Formats {@link Revision} representing revision to format {@code YYYY-mm-dd} * *

- * YANG Specification defines format for revision as - * YYYY-mm-dd. This format for revision is reused accross multiple places - * such as capabilities URI, YANG modules, etc. + * YANG Specification defines format for {@code revision<} as YYYY-mm-dd. This format for revision is reused across + * multiple places such as capabilities URI, YANG modules, etc. * - * @param revision - * Date object to format + * @param revision Date object to format * @return String representation or null if the input was null. */ - public static String formattedRevision(final Optional revision) { + public static @Nullable String formattedRevision(final Optional revision) { return revision.map(Revision::toString).orElse(null); } @@ -397,16 +371,12 @@ public final class QName implements Immutable, Serializable, Comparable, * Compares this QName to other, without comparing revision. * *

- * Compares instance of this to other instance of QName and returns true if - * both instances have equal localName ({@link #getLocalName()} - * ) and namespace ({@link #getNamespace()}). + * Compares instance of this to other instance of QName and returns true if both instances have equal + * {@code localName} ({@link #getLocalName()}) and @{code namespace} ({@link #getNamespace()}). * - * @param other - * Other QName. Must not be null. - * @return true if this instance and other have equals localName and - * namespace. - * @throws NullPointerException - * if other is null. + * @param other Other QName. Must not be null. + * @return true if this instance and other have equals localName and namespace. + * @throws NullPointerException if {@code other} is null. */ public boolean isEqualWithoutRevision(final QName other) { return localName.equals(other.getLocalName()) && Objects.equals(getNamespace(), other.getNamespace()); @@ -416,8 +386,7 @@ public final class QName implements Immutable, Serializable, Comparable, // the impact on iteration order of SortedMap? @Override @SuppressWarnings("checkstyle:parameterName") - public int compareTo(@Nonnull final QName o) { - + public int compareTo(final QName o) { // compare mandatory localName parameter int result = localName.compareTo(o.localName); if (result != 0) { @@ -426,10 +395,13 @@ public final class QName implements Immutable, Serializable, Comparable, return module.compareTo(o.module); } - @Override public void writeTo(final DataOutput out) throws IOException { module.writeTo(out); out.writeUTF(localName); } + + Object writeReplace() { + return new QNv1(this); + } }