use isEmpty check

Author: iddeepak

integration-tests/src/test/java/org/springframework/beans/factory/xml/ComponentFactoryBean.java

@@ -35,7 +35,7 @@ public void setChildren(List<Component> children) {
 
 	@Override
 	public Component getObject() {
-		if (this.children != null && this.children.size() > 0) {
+		if (this.children != null && !this.children.isEmpty()) {
 			for (Component child : children) {
 				this.parent.addComponent(child);
 			}

spring-context/src/main/java/org/springframework/context/support/ReloadableResourceBundleMessageSource.java

@@ -379,18 +379,18 @@ protected List<String> calculateFilenamesForLocale(String basename, Locale local
 		StringBuilder temp = new StringBuilder(basename);
 
 		temp.append('_');
-		if (language.length() > 0) {
+		if (!language.isEmpty()) {
 			temp.append(language);
 			result.add(0, temp.toString());
 		}
 
 		temp.append('_');
-		if (country.length() > 0) {
+		if (!country.isEmpty()) {
 			temp.append(country);
 			result.add(0, temp.toString());
 		}
 
-		if (variant.length() > 0 && (language.length() > 0 || country.length() > 0)) {
+		if (!variant.isEmpty() && (!language.isEmpty() || !country.isEmpty())) {
 			temp.append('_').append(variant);
 			result.add(0, temp.toString());
 		}

spring-context/src/testFixtures/java/org/springframework/context/testfixture/jndi/SimpleNamingContext.java

@@ -326,7 +326,7 @@ private AbstractNamingEnumeration(SimpleNamingContext context, String proot) thr
 					}
 				}
 			}
-			if (contents.size() == 0) {
+			if (contents.isEmpty()) {
 				throw new NamingException("Invalid root: [" + context.root + proot + "]");
 			}
 			this.iterator = contents.values().iterator();

spring-core/src/main/java/org/springframework/asm/TypePath.java

@@ -117,7 +117,7 @@ public int getStepArgument(final int index) {
    * @return the corresponding TypePath object, or {@literal null} if the path is empty.
    */
   public static TypePath fromString(final String typePath) {
-    if (typePath == null || typePath.length() == 0) {
+    if (typePath == null || typePath.isEmpty()) {
       return null;
     }
     int typePathLength = typePath.length();

spring-core/src/main/java/org/springframework/cglib/proxy/CallbackHelper.java

@@ -63,7 +63,7 @@ public CallbackHelper(Class superclass, Class[] interfaces) {
     abstract protected Object getCallback(Method method);
 
     public Callback[] getCallbacks() {
-        if (callbacks.size() == 0) {
+        if (callbacks.isEmpty()) {
             return new Callback[0];
         }
         if (callbacks.get(0) instanceof Callback) {
@@ -75,7 +75,7 @@ public Callback[] getCallbacks() {
     }
 
     public Class[] getCallbackTypes() {
-        if (callbacks.size() == 0) {
+        if (callbacks.isEmpty()) {
             return new Class[0];
         }
         if (callbacks.get(0) instanceof Callback) {

spring-core/src/main/java/org/springframework/cglib/proxy/Enhancer.java

@@ -755,7 +755,7 @@ public void generateClass(ClassVisitor v) throws Exception {
 	 */
 	protected void filterConstructors(Class sc, List constructors) {
 		CollectionUtils.filter(constructors, new VisibilityPredicate(sc, true));
-		if (constructors.size() == 0) {
+		if (constructors.isEmpty()) {
 			throw new IllegalArgumentException("No visible constructors in " + sc);
 		}
 	}

spring-core/src/main/java/org/springframework/core/io/support/LocalizedResourceHelper.java

@@ -100,20 +100,20 @@ public Resource findLocalizedResource(String name, String extension, @Nullable L
 			String variant = locale.getVariant();
 
 			// Check for file with language, country and variant localization.
-			if (variant.length() > 0) {
+			if (!variant.isEmpty()) {
 				String location =
 						name + this.separator + lang + this.separator + country + this.separator + variant + extension;
 				resource = this.resourceLoader.getResource(location);
 			}
 
 			// Check for file with language and country localization.
-			if ((resource == null || !resource.exists()) && country.length() > 0) {
+			if ((resource == null || !resource.exists()) && !country.isEmpty()) {
 				String location = name + this.separator + lang + this.separator + country + extension;
 				resource = this.resourceLoader.getResource(location);
 			}
 
 			// Check for document with language localization.
-			if ((resource == null || !resource.exists()) && lang.length() > 0) {
+			if ((resource == null || !resource.exists()) && !lang.isEmpty()) {
 				String location = name + this.separator + lang + extension;
 				resource = this.resourceLoader.getResource(location);
 			}

spring-core/src/main/java/org/springframework/core/io/support/PropertySourceProcessor.java

@@ -80,7 +80,7 @@ public void processPropertySource(PropertySourceDescriptor descriptor) throws IO
 		String name = descriptor.name();
 		String encoding = descriptor.encoding();
 		List<String> locations = descriptor.locations();
-		Assert.isTrue(locations.size() > 0, "At least one @PropertySource(value) location is required");
+		Assert.isTrue(!locations.isEmpty(), "At least one @PropertySource(value) location is required");
 		boolean ignoreResourceNotFound = descriptor.ignoreResourceNotFound();
 		PropertySourceFactory factory = (descriptor.propertySourceFactory() != null ?
 				instantiateClass(descriptor.propertySourceFactory()) : defaultPropertySourceFactory);

spring-core/src/main/java/org/springframework/util/AntPathMatcher.java

@@ -348,7 +348,7 @@ private boolean isPotentialMatch(String path, String[] pattDirs) {
 				pos += skipped;
 				skipped = skipSegment(path, pos, pattDir);
 				if (skipped < pattDir.length()) {
-					return (skipped > 0 || (pattDir.length() > 0 && isWildcardChar(pattDir.charAt(0))));
+					return (skipped > 0 || (!pattDir.isEmpty() && isWildcardChar(pattDir.charAt(0))));
 				}
 				pos += skipped;
 			}

spring-core/src/main/java/org/springframework/util/CommonsLogWriter.java

@@ -44,7 +44,7 @@ public CommonsLogWriter(Log logger) {
 
 
 	public void write(char ch) {
-		if (ch == '\n' && this.buffer.length() > 0) {
+		if (ch == '\n' && !this.buffer.isEmpty()) {
 			logger.debug(this.buffer.toString());
 			this.buffer.setLength(0);
 		}