use && in TimeZoneStorageHelper for readability

Author: gavinking

hibernate-core/src/main/java/org/hibernate/boot/model/internal/TimeZoneStorageHelper.java

@@ -33,7 +33,7 @@ static Class<? extends CompositeUserType<?>> resolveTimeZoneStorageCompositeUser
 			ClassDetails returnedClass,
 			MetadataBuildingContext context) {
 		if ( useColumnForTimeZoneStorage( attributeMember, context ) ) {
-			String returnedClassName = returnedClass.getName();
+			final String returnedClassName = returnedClass.getName();
 			if ( OFFSET_DATETIME_CLASS.equals( returnedClassName ) ) {
 				return OffsetDateTimeCompositeUserType.class;
 			}
@@ -49,24 +49,20 @@ else if ( OFFSET_TIME_CLASS.equals( returnedClassName ) ) {
 
 	private static boolean isTemporalWithTimeZoneClass(String returnedClassName) {
 		return OFFSET_DATETIME_CLASS.equals( returnedClassName )
-				|| ZONED_DATETIME_CLASS.equals( returnedClassName )
-				|| isOffsetTimeClass( returnedClassName );
+			|| ZONED_DATETIME_CLASS.equals( returnedClassName )
+			|| isOffsetTimeClass( returnedClassName );
 	}
 
 	public static boolean isOffsetTimeClass(AnnotationTarget element) {
-		if ( element instanceof MemberDetails memberDetails ) {
-			return isOffsetTimeClass( memberDetails );
-		}
-		return false;
+		return element instanceof MemberDetails memberDetails
+			&& isOffsetTimeClass( memberDetails );
 	}
 
 	public static boolean isOffsetTimeClass(MemberDetails element) {
 		final TypeDetails type = element.getType();
-		if ( type == null ) {
-			return false;
-		}
+		return type != null
+			&& isOffsetTimeClass( type.determineRawClass().getClassName() );
 
-		return isOffsetTimeClass( type.determineRawClass().getClassName() );
 	}
 
 	private static boolean isOffsetTimeClass(String returnedClassName) {
@@ -76,14 +72,10 @@ private static boolean isOffsetTimeClass(String returnedClassName) {
 	static boolean useColumnForTimeZoneStorage(AnnotationTarget element, MetadataBuildingContext context) {
 		final TimeZoneStorage timeZoneStorage = element.getDirectAnnotationUsage( TimeZoneStorage.class );
 		if ( timeZoneStorage == null ) {
-			if ( element instanceof MemberDetails attributeMember ) {
-				return isTemporalWithTimeZoneClass( attributeMember.getType().getName() )
-						//no @TimeZoneStorage annotation, so we need to use the default storage strategy
-						&& context.getBuildingOptions().getDefaultTimeZoneStorage() == COLUMN;
-			}
-			else {
-				return false;
-			}
+			return element instanceof MemberDetails attributeMember
+				&& isTemporalWithTimeZoneClass( attributeMember.getType().getName() )
+				//no @TimeZoneStorage annotation, so we need to use the default storage strategy
+				&& context.getBuildingOptions().getDefaultTimeZoneStorage() == COLUMN;
 		}
 		else {
 			return switch ( timeZoneStorage.value() ) {