BUG-4688: align Optional/nullable Date usage
[yangtools.git] / yang / yang-common / src / main / java / org / opendaylight / yangtools / yang / common / QName.java
index ed05140174f03cbf3932a1f6bd0c0ed4dfb19bdc..dec755e68c3956ed698c6060106a591b6c4a3cc5 100644 (file)
@@ -7,8 +7,11 @@
  */
 package org.opendaylight.yangtools.yang.common;
 
+import static com.google.common.base.Preconditions.checkArgument;
+import static java.util.Objects.requireNonNull;
 import static org.opendaylight.yangtools.yang.common.SimpleDateFormatUtil.getRevisionFormat;
-import com.google.common.base.Preconditions;
+
+import com.google.common.base.Strings;
 import com.google.common.collect.Interner;
 import com.google.common.collect.Interners;
 import java.io.Serializable;
@@ -17,16 +20,19 @@ import java.net.URISyntaxException;
 import java.text.ParseException;
 import java.util.Date;
 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.opendaylight.yangtools.concepts.Immutable;
-import org.opendaylight.yangtools.objcache.ObjectCache;
-import org.opendaylight.yangtools.objcache.ObjectCacheFactory;
 
 /**
  * The QName from XML consists of local name of element and XML namespace, but
  * for our use, we added module revision to it.
  *
+ * <p>
  * 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
@@ -41,64 +47,40 @@ import org.opendaylight.yangtools.objcache.ObjectCacheFactory;
  * node in the YANG module</li>
  * </ul>
  *
+ * <p>
  * QName may also have <code>prefix</code> 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<QName> {
     private static final Interner<QName> INTERNER = Interners.newWeakInterner();
-    private static final ObjectCache CACHE = ObjectCacheFactory.getObjectCache(QName.class);
     private static final long serialVersionUID = 5398411242927766414L;
 
     static final String QNAME_REVISION_DELIMITER = "?revision=";
     static final String QNAME_LEFT_PARENTHESIS = "(";
     static final String QNAME_RIGHT_PARENTHESIS = ")";
 
-    private static final Pattern QNAME_PATTERN_FULL = Pattern.compile("^\\((.+)\\" + QNAME_REVISION_DELIMITER
-            + "(.+)\\)(.+)$");
-    private static final Pattern QNAME_PATTERN_NO_REVISION = Pattern.compile("^\\((.+)\\)(.+)$");
-    private static final Pattern QNAME_PATTERN_NO_NAMESPACE_NO_REVISION = Pattern.compile("^(.+)$");
-    private static final char[] ILLEGAL_CHARACTERS = new char[] { '?', '(', ')', '&' };
+    @RegEx
+    private static final String QNAME_STRING_FULL = "^\\((.+)\\?revision=(.+)\\)(.+)$";
+    private static final Pattern QNAME_PATTERN_FULL = Pattern.compile(QNAME_STRING_FULL);
+
+    @RegEx
+    private static final String QNAME_STRING_NO_REVISION = "^\\((.+)\\)(.+)$";
+    private static final Pattern QNAME_PATTERN_NO_REVISION = Pattern.compile(QNAME_STRING_NO_REVISION);
 
-    // Mandatory
+    private static final char[] ILLEGAL_CHARACTERS = new char[] { '?', '(', ')', '&', ':' };
+
+    // Non-null
     private final QNameModule module;
-    // Mandatory
+    // Non-null
     private final String localName;
+    private transient int hash;
 
     private QName(final QNameModule module, final String localName) {
         this.localName = checkLocalName(localName);
         this.module = module;
     }
 
-    /**
-     * Look up specified QName in the global cache and return a shared reference.
-     *
-     * @param qname QName instance
-     * @return Cached instance, according to {@link ObjectCache} policy.
-     *
-     * @deprecated Use {@link #intern()} instead.
-     */
-    @Deprecated
-    public static QName cachedReference(final QName qname) {
-        // We also want to make sure we keep the QNameModule cached
-        final QNameModule myMod = qname.getModule();
-        final QNameModule cacheMod = QNameModule.cachedReference(myMod);
-
-        final QName what;
-        // Identity comparison is here on purpose, as we are deciding whether to potentially store 'qname'
-        // into cache. It is important that it does not hold user-supplied reference (such a String instance from
-        // XML parser.
-        if (cacheMod == myMod) {
-            what = qname;
-        } else {
-            what = QName.create(cacheMod, qname.localName);
-        }
-
-        return CACHE.getReference(what);
-    }
-
     /**
      * QName Constructor.
      *
@@ -107,17 +89,13 @@ public final class QName implements Immutable, Serializable, Comparable<QName> {
      * @param localName
      *            YANG schema identifier
      */
-    public QName(final URI namespace, final String localName) {
-        this(QNameModule.create(namespace, null), localName);
+    private QName(final URI namespace, final String localName) {
+        this(QNameModule.create(namespace), localName);
     }
 
     private static String checkLocalName(final String localName) {
-        if (localName == null) {
-            throw new IllegalArgumentException("Parameter 'localName' may not be null.");
-        }
-        if (localName.length() == 0) {
-            throw new IllegalArgumentException("Parameter 'localName' must be a non-empty string.");
-        }
+        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) {
@@ -142,12 +120,128 @@ public final class QName implements Immutable, Serializable, Comparable<QName> {
             final String localName = matcher.group(2);
             return new QName(namespace, localName);
         }
-        matcher = QNAME_PATTERN_NO_NAMESPACE_NO_REVISION.matcher(input);
-        if (matcher.matches()) {
-            final String localName = matcher.group(1);
-            return new QName((URI) null, localName);
-        }
-        throw new IllegalArgumentException("Invalid input:" + input);
+        throw new IllegalArgumentException("Invalid input: " + input);
+    }
+
+    public static QName create(final QName base, final String localName) {
+        return create(base.getModule(), localName);
+    }
+
+    /**
+     * Creates new QName.
+     *
+     * @param qnameModule
+     *            Namespace and revision enclosed as a QNameModule
+     * @param localName
+     *            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);
+    }
+
+    /**
+     * 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.
+     * @return Instance of QName
+     */
+    public static QName create(final URI namespace, @Nullable final Date 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.
+     * @return Instance of QName
+     */
+    public static QName create(final URI namespace, final Optional<Date> 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.
+     * @return Instance of QName
+     */
+    public static QName create(final String namespace, final String localName, final Date revision) {
+        final URI namespaceUri = parseNamespace(namespace);
+        return create(QNameModule.create(namespaceUri, 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 <code>YYYY-mm-dd</code>.
+     * @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 <code>namespace</code> is not valid URI or
+     *             <code>revision</code> is not according to format
+     *             <code>YYYY-mm-dd</code>.
+     */
+    public static QName create(final String namespace, final String revision, final String localName) {
+        final URI namespaceUri = parseNamespace(namespace);
+        final Date revisionDate = parseRevision(revision);
+        return create(namespaceUri, revisionDate, localName);
+    }
+
+    /**
+     * Creates new QName.
+     *
+     * @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 <code>namespace</code> is not valid URI.
+     */
+    public static 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.
+     * @return A new QName
+     * @throws NullPointerException
+     *             If any of parameters is null.
+     * @throws IllegalArgumentException
+     *             If <code>namespace</code> is not valid URI.
+     */
+    public static QName create(final URI namespace, final String localName) {
+        return new QName(namespace, localName);
     }
 
     /**
@@ -170,7 +264,7 @@ public final class QName implements Immutable, Serializable, Comparable<QName> {
 
     /**
      * Returns YANG schema identifier which were defined for this node in the
-     * YANG module
+     * YANG module.
      *
      * @return YANG schema identifier which were defined for this node in the
      *         YANG module
@@ -180,13 +274,11 @@ public final class QName implements Immutable, Serializable, Comparable<QName> {
     }
 
     /**
-     * Returns revision of the YANG module if the module has defined revision,
-     * otherwise returns <code>null</code>
+     * Returns revision of the YANG module if the module has defined revision.
      *
-     * @return revision of the YANG module if the module has defined revision,
-     *         otherwise returns <code>null</code>
+     * @return revision of the YANG module if the module has defined revision.
      */
-    public Date getRevision() {
+    public Optional<Date> getRevision() {
         return module.getRevision();
     }
 
@@ -202,22 +294,20 @@ public final class QName implements Immutable, Serializable, Comparable<QName> {
         // 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);
+        final QName template = cacheMod == module ? this : QName.create(cacheMod, localName.intern());
 
         return INTERNER.intern(template);
     }
 
     @Override
     public int hashCode() {
-        final int prime = 31;
-        int result = 1;
-        result = prime * result + Objects.hashCode(localName);
-        result = prime * result + module.hashCode();
-        return result;
+        if (hash == 0) {
+            hash = Objects.hash(module, localName);
+        }
+        return hash;
     }
 
     /**
-     *
      * Compares the specified object with this list for equality.  Returns
      * <tt>true</tt> if and only if the specified object is also instance of
      * {@link QName} and its {@link #getLocalName()}, {@link #getNamespace()} and
@@ -239,63 +329,6 @@ public final class QName implements Immutable, Serializable, Comparable<QName> {
         return Objects.equals(localName, other.localName) && module.equals(other.module);
     }
 
-    public static QName create(final QName base, final String localName) {
-        return create(base.getModule(), localName);
-    }
-
-    /**
-     * Creates new QName.
-     *
-     * @param qnameModule
-     *            Namespace and revision enclosed as a QNameModule
-     * @param localName
-     *            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(Preconditions.checkNotNull(qnameModule,"module may not be null"), 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.
-     * @return Instance of QName
-     */
-    public static QName create(final URI namespace, final Date revision, final String localName) {
-        return create(QNameModule.create(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 <code>YYYY-mm-dd</code>.
-     * @param localName
-     *            Local name part of QName. MUST NOT BE null.
-     * @return
-     * @throws NullPointerException
-     *             If any of parameters is null.
-     * @throws IllegalArgumentException
-     *             If <code>namespace</code> is not valid URI or
-     *             <code>revision</code> is not according to format
-     *             <code>YYYY-mm-dd</code>.
-     */
-    public static QName create(final String namespace, final String revision, final String localName) {
-        final URI namespaceUri = parseNamespace(namespace);
-        final Date revisionDate = parseRevision(revision);
-        return create(namespaceUri, revisionDate, localName);
-    }
-
     private static URI parseNamespace(final String namespace) {
         try {
             return new URI(namespace);
@@ -304,31 +337,14 @@ public final class QName implements Immutable, Serializable, Comparable<QName> {
         }
     }
 
-    /**
-     * Creates new QName.
-     *
-     * @param namespace
-     *            Namespace of QName, MUST NOT BE Null.
-     * @param localName
-     *            Local name part of QName. MUST NOT BE null.
-     * @return
-     * @throws NullPointerException
-     *             If any of parameters is null.
-     * @throws IllegalArgumentException
-     *             If <code>namespace</code> is not valid URI.
-     */
-    public static QName create(final String namespace, final String localName) {
-        return create(parseNamespace(namespace), null, localName);
-    }
-
     @Override
     public String toString() {
         final StringBuilder sb = new StringBuilder();
         if (getNamespace() != null) {
-            sb.append(QNAME_LEFT_PARENTHESIS + getNamespace());
+            sb.append(QNAME_LEFT_PARENTHESIS).append(getNamespace());
 
             if (getFormattedRevision() != null) {
-                sb.append(QNAME_REVISION_DELIMITER + getFormattedRevision());
+                sb.append(QNAME_REVISION_DELIMITER).append(getFormattedRevision());
             }
             sb.append(QNAME_RIGHT_PARENTHESIS);
         }
@@ -337,9 +353,9 @@ public final class QName implements Immutable, Serializable, Comparable<QName> {
     }
 
     /**
-     * Return string representation of revision in format
-     * <code>YYYY-mm-dd</code>
+     * Return string representation of revision in format <code>YYYY-mm-dd</code>
      *
+     * <p>
      * YANG Specification defines format for <code>revision</code> as
      * YYYY-mm-dd. This format for revision is reused accross multiple places
      * such as capabilities URI, YANG modules, etc.
@@ -357,9 +373,10 @@ public final class QName implements Immutable, Serializable, Comparable<QName> {
      * @return copy of this QName with revision and prefix unset.
      */
     public QName withoutRevision() {
-        return create(getNamespace(), null, localName);
+        return create(getNamespace(), localName);
     }
 
+    @SuppressWarnings("checkstyle:illegalCatch")
     public static Date parseRevision(final String formatedDate) {
         try {
             return getRevisionFormat().parse(formatedDate);
@@ -373,25 +390,23 @@ public final class QName implements Immutable, Serializable, Comparable<QName> {
      * Formats {@link Date} representing revision to format
      * <code>YYYY-mm-dd</code>
      *
+     * <p>
      * YANG Specification defines format for <code>revision</code> as
      * YYYY-mm-dd. This format for revision is reused accross multiple places
      * such as capabilities URI, YANG modules, etc.
      *
      * @param revision
-     *            Date object to format or null
+     *            Date object to formatl
      * @return String representation or null if the input was null.
      */
-    public static String formattedRevision(final Date revision) {
-        if (revision == null) {
-            return null;
-        }
-        return getRevisionFormat().format(revision);
+    public static String formattedRevision(final Optional<Date> revision) {
+        return revision.map(rev -> getRevisionFormat().format(rev)).orElse(null);
     }
 
     /**
-     *
      * Compares this QName to other, without comparing revision.
      *
+     * <p>
      * Compares instance of this to other instance of QName and returns true if
      * both instances have equal <code>localName</code> ({@link #getLocalName()}
      * ) and <code>namespace</code> ({@link #getNamespace()}).
@@ -408,44 +423,25 @@ public final class QName implements Immutable, Serializable, Comparable<QName> {
     }
 
     @Override
-    public int compareTo(final QName other) {
+    public int compareTo(@Nonnull final QName other) {
         // compare mandatory localName parameter
         int result = localName.compareTo(other.localName);
         if (result != 0) {
             return result;
         }
-
-        // compare nullable namespace parameter
-        if (getNamespace() == null) {
-            if (other.getNamespace() != null) {
-                return -1;
-            }
-        } else {
-            if (other.getNamespace() == null) {
-                return 1;
-            }
-            result = getNamespace().compareTo(other.getNamespace());
-            if (result != 0) {
-                return result;
-            }
+        result = getNamespace().compareTo(other.getNamespace());
+        if (result != 0) {
+            return result;
         }
 
+        final Date myRev = getRevision().orElse(null);
+        final Date otherRev = other.getRevision().orElse(null);
+
         // compare nullable revision parameter
-        if (getRevision() == null) {
-            if (other.getRevision() != null) {
-                return -1;
-            }
-        } else {
-            if (other.getRevision() == null) {
-                return 1;
-            }
-            result = getRevision().compareTo(other.getRevision());
-            if (result != 0) {
-                return result;
-            }
+        if (myRev != null) {
+            return otherRev == null ? 1 : myRev.compareTo(otherRev);
         }
 
-        return result;
+        return otherRev == null ? 0 : -1;
     }
-
 }