Browse Source

Simplify conditionals

See gh-39259
pull/39285/head
Tobias Lippert 2 years ago committed by Phillip Webb
parent
commit
65a1ff84e6
  1. 2
      spring-boot-project/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/endpoint/invoke/reflect/OperationMethodParameter.java
  2. 2
      spring-boot-project/spring-boot-devtools/src/main/java/org/springframework/boot/devtools/restart/classloader/ClassLoaderFile.java
  3. 2
      spring-boot-project/spring-boot-test-autoconfigure/src/main/java/org/springframework/boot/test/autoconfigure/OverrideAutoConfigurationContextCustomizerFactory.java
  4. 6
      spring-boot-project/spring-boot-tools/spring-boot-loader/src/main/java/org/springframework/boot/loader/net/protocol/jar/JarUrlConnection.java
  5. 4
      spring-boot-project/spring-boot/src/main/java/org/springframework/boot/context/properties/NotConstructorBoundInjectionFailureAnalyzer.java
  6. 2
      spring-boot-project/spring-boot/src/main/java/org/springframework/boot/context/properties/bind/DefaultBindConstructorProvider.java
  7. 2
      spring-boot-project/spring-boot/src/main/java/org/springframework/boot/logging/log4j2/SpringProfileArbiter.java
  8. 2
      spring-boot-project/spring-boot/src/main/java/org/springframework/boot/web/embedded/tomcat/NestedJarResourceSet.java

2
spring-boot-project/spring-boot-actuator/src/main/java/org/springframework/boot/actuate/endpoint/invoke/reflect/OperationMethodParameter.java

@ -68,7 +68,7 @@ class OperationMethodParameter implements OperationParameter { @@ -68,7 +68,7 @@ class OperationMethodParameter implements OperationParameter {
if (!ObjectUtils.isEmpty(this.parameter.getAnnotationsByType(Nullable.class))) {
return false;
}
return (jsr305Present) ? new Jsr305().isMandatory(this.parameter) : true;
return !jsr305Present || new Jsr305().isMandatory(this.parameter);
}
@Override

2
spring-boot-project/spring-boot-devtools/src/main/java/org/springframework/boot/devtools/restart/classloader/ClassLoaderFile.java

@ -55,7 +55,7 @@ public class ClassLoaderFile implements Serializable { @@ -55,7 +55,7 @@ public class ClassLoaderFile implements Serializable {
*/
public ClassLoaderFile(Kind kind, long lastModified, byte[] contents) {
Assert.notNull(kind, "Kind must not be null");
Assert.isTrue((kind != Kind.DELETED) ? contents != null : contents == null,
Assert.isTrue((kind != Kind.DELETED) == (contents != null),
() -> "Contents must " + ((kind != Kind.DELETED) ? "not " : "") + "be null");
this.kind = kind;
this.lastModified = lastModified;

2
spring-boot-project/spring-boot-test-autoconfigure/src/main/java/org/springframework/boot/test/autoconfigure/OverrideAutoConfigurationContextCustomizerFactory.java

@ -44,7 +44,7 @@ class OverrideAutoConfigurationContextCustomizerFactory implements ContextCustom @@ -44,7 +44,7 @@ class OverrideAutoConfigurationContextCustomizerFactory implements ContextCustom
}
OverrideAutoConfiguration overrideAutoConfiguration = TestContextAnnotationUtils.findMergedAnnotation(testClass,
OverrideAutoConfiguration.class);
boolean enabled = (overrideAutoConfiguration != null) ? overrideAutoConfiguration.enabled() : true;
boolean enabled = overrideAutoConfiguration == null || overrideAutoConfiguration.enabled();
return !enabled ? new DisableAutoConfigurationContextCustomizer() : null;
}

6
spring-boot-project/spring-boot-tools/spring-boot-loader/src/main/java/org/springframework/boot/loader/net/protocol/jar/JarUrlConnection.java

@ -207,7 +207,7 @@ final class JarUrlConnection extends java.net.JarURLConnection { @@ -207,7 +207,7 @@ final class JarUrlConnection extends java.net.JarURLConnection {
@Override
public boolean getAllowUserInteraction() {
return (this.jarFileConnection != null) ? this.jarFileConnection.getAllowUserInteraction() : false;
return this.jarFileConnection != null && this.jarFileConnection.getAllowUserInteraction();
}
@Override
@ -219,7 +219,7 @@ final class JarUrlConnection extends java.net.JarURLConnection { @@ -219,7 +219,7 @@ final class JarUrlConnection extends java.net.JarURLConnection {
@Override
public boolean getUseCaches() {
return (this.jarFileConnection != null) ? this.jarFileConnection.getUseCaches() : true;
return this.jarFileConnection == null || this.jarFileConnection.getUseCaches();
}
@Override
@ -231,7 +231,7 @@ final class JarUrlConnection extends java.net.JarURLConnection { @@ -231,7 +231,7 @@ final class JarUrlConnection extends java.net.JarURLConnection {
@Override
public boolean getDefaultUseCaches() {
return (this.jarFileConnection != null) ? this.jarFileConnection.getDefaultUseCaches() : true;
return this.jarFileConnection == null || this.jarFileConnection.getDefaultUseCaches();
}
@Override

4
spring-boot-project/spring-boot/src/main/java/org/springframework/boot/context/properties/NotConstructorBoundInjectionFailureAnalyzer.java

@ -61,8 +61,8 @@ class NotConstructorBoundInjectionFailureAnalyzer @@ -61,8 +61,8 @@ class NotConstructorBoundInjectionFailureAnalyzer
}
private boolean isConstructorBindingConfigurationProperties(InjectionPoint injectionPoint) {
return (injectionPoint != null && injectionPoint.getMember() instanceof Constructor<?> constructor)
? isConstructorBindingConfigurationProperties(constructor) : false;
return injectionPoint != null && injectionPoint.getMember() instanceof Constructor<?> constructor
&& isConstructorBindingConfigurationProperties(constructor);
}
private boolean isConstructorBindingConfigurationProperties(Constructor<?> constructor) {

2
spring-boot-project/spring-boot/src/main/java/org/springframework/boot/context/properties/bind/DefaultBindConstructorProvider.java

@ -127,7 +127,7 @@ class DefaultBindConstructorProvider implements BindConstructorProvider { @@ -127,7 +127,7 @@ class DefaultBindConstructorProvider implements BindConstructorProvider {
return true;
}
Class<?> userClass = ClassUtils.getUserClass(type);
return (userClass != type) ? isAutowiredPresent(userClass) : false;
return userClass != type && isAutowiredPresent(userClass);
}
private static Constructor<?>[] getCandidateConstructors(Class<?> type) {

2
spring-boot-project/spring-boot/src/main/java/org/springframework/boot/logging/log4j2/SpringProfileArbiter.java

@ -53,7 +53,7 @@ final class SpringProfileArbiter implements Arbiter { @@ -53,7 +53,7 @@ final class SpringProfileArbiter implements Arbiter {
@Override
public boolean isCondition() {
return (this.environment != null) ? this.environment.acceptsProfiles(this.profiles) : false;
return this.environment != null && this.environment.acceptsProfiles(this.profiles);
}
@PluginBuilderFactory

2
spring-boot-project/spring-boot/src/main/java/org/springframework/boot/web/embedded/tomcat/NestedJarResourceSet.java

@ -124,7 +124,7 @@ class NestedJarResourceSet extends AbstractSingleArchiveResourceSet { @@ -124,7 +124,7 @@ class NestedJarResourceSet extends AbstractSingleArchiveResourceSet {
// JarFile.isMultiRelease() is final so we must go to the manifest
Manifest manifest = getManifest();
Attributes attributes = (manifest != null) ? manifest.getMainAttributes() : null;
this.multiRelease = (attributes != null) ? attributes.containsKey(MULTI_RELEASE) : false;
this.multiRelease = attributes != null && attributes.containsKey(MULTI_RELEASE);
}
}
}

Loading…
Cancel
Save