From 960f65e83d87003dfc755cf4223a8cee12175f3d Mon Sep 17 00:00:00 2001 From: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com> Date: Wed, 3 Dec 2025 19:37:56 +0000 Subject: [PATCH] refactor: use isEmpty() instead of size() > 0 for better readability and performance Co-Authored-By: Sam Fertig --- .../user/service/service/implementation/UserServiceImpl.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/User-Service/src/main/java/org/training/user/service/service/implementation/UserServiceImpl.java b/User-Service/src/main/java/org/training/user/service/service/implementation/UserServiceImpl.java index 7e416ba..5d887fa 100644 --- a/User-Service/src/main/java/org/training/user/service/service/implementation/UserServiceImpl.java +++ b/User-Service/src/main/java/org/training/user/service/service/implementation/UserServiceImpl.java @@ -62,7 +62,7 @@ public class UserServiceImpl implements UserService { public Response createUser(CreateUser userDto) { List userRepresentations = keycloakService.readUserByEmail(userDto.getEmailId()); - if(userRepresentations.size() > 0) { + if(!userRepresentations.isEmpty()) { log.error("This emailId is already registered as a user"); throw new ResourceConflictException("This emailId is already registered as a user"); }