Uploaded image for project: 'WildFly Core'
  1. WildFly Core
  2. WFCORE-5285

Utilize org.wildfly.common.Assert for Null-Checks

XMLWordPrintable

    Currently there are different approaches to have null checks of parameters and throw a NullPointerException or IllegalArgumentException. Most of them do not offer the parameter name which is null or causing a null lookup. Some of them offer the parameter name but are not I18N. Some offer both, but don't reuse the common solution.
    There is an existing solution for these cases:
    org.wildfly.common.Assert.checkNotNullParamWithNullPointerException;
    org.wildfly.common.Assert.checkNotNullParam;

    The approach is to have the same method signature, keep NullPointerException and/or IllegalArgumentException in the same way as now. Only the check and throw is changed with the mentioned methods.
    Some modules require org.wildfly.common as new dependency.

     

    To ease the pull request verification process, several pull requests (by sub-project) are provided.

            jmesnil1@redhat.com Jeff Mesnil
            xf01213 Boris Unckel (Inactive)
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved: