improve null handling

AppDirect · Jul 31, 2020 · 8f4c4c9 · 8f4c4c9
1 parent dffff4d
commit 8f4c4c9
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/src/main/java/com/appdirect/sdk/meteredusage/service/MeteredUsageApiClientServiceImpl.java b/src/main/java/com/appdirect/sdk/meteredusage/service/MeteredUsageApiClientServiceImpl.java
@@ -3,7 +3,6 @@
 import java.io.IOException;
 import java.util.Collections;
 import java.util.List;
-import java.util.Objects;
 import java.util.UUID;
 
 import lombok.extern.slf4j.Slf4j;
@@ -140,10 +139,10 @@ private APIResult processResponse(Response<MeteredUsageResponse> response) throw
 			return new APIResult(true, response.body().toString());
 		}
 		String errorBodyMessage = null;
-		if (!Objects.isNull(response.errorBody())) {
+		if (response.errorBody() != null) {
 			errorBodyMessage = response.errorBody().string();
 		}
-		String errorMessage = !StringUtils.isEmpty(errorBodyMessage) ? response.message().concat(" " + errorBodyMessage) : response.message();
+		String errorMessage = !StringUtils.isEmpty(errorBodyMessage) ? response.message().concat(" ".concat(errorBodyMessage)) : response.message();
 		log.error("Metered Usage API Client failed with error={}", errorMessage);
 		return APIResult.failure(response.code(), String.format("Failed to inform Usage with errorCode=%s, message=%s", response.code(), errorMessage));
 	}