Resolve Sonarlint S1155 - Collection.isEmpty() should be used to test for emptiness

[JackPGreen]

Replace checks against .size() with .isEmpty() / CollectionUtil.isEmpty() / CollectionUtil.isNotEmpty() as appropriate

[Patras3]

In CollectionUtil we have also below method:

    /**
     * Returns {@code true} if the given collection is not {@code null} and not empty, otherwise returns {@code false}.
     *
     * @param collection the given collection
     * @return {@code true} if collection is not empty
     */
    public static boolean isNotEmpty(Collection collection) {
        return !isEmpty(collection);
    }

Is there any reason to use !CollectionUtil.isEmpty() instead off CollectionUtil.isNotEmpty()?
For me, second option is more clear.

[JackPGreen]

In CollectionUtil we have also below method:

    /**
     * Returns {@code true} if the given collection is not {@code null} and not empty, otherwise returns {@code false}.
     *
     * @param collection the given collection
     * @return {@code true} if collection is not empty
     */
    public static boolean isNotEmpty(Collection collection) {
        return !isEmpty(collection);
    }

Is there any reason to use !CollectionUtil.isEmpty() instead off CollectionUtil.isNotEmpty()? For me, second option is more clear.

Good spot, I didn't see that method.

fcdb3cf

[Patras3]

LGTM now, thanks!