refactor: better use of Optional

Together-Java · May 17, 2024 · e98eb35 · e98eb35
1 parent 0502f83
commit e98eb35
Showing 1 changed file with 6 additions and 7 deletions.
diff --git a/application/src/main/java/org/togetherjava/tjbot/features/cakeday/CakeDayService.java b/application/src/main/java/org/togetherjava/tjbot/features/cakeday/CakeDayService.java
@@ -55,15 +55,14 @@ public CakeDayService(Config config, Database database) {
     }
 
     private Optional<Role> getCakeDayRole(Guild guild) {
-        Role cakeDayRole = getCakeDayRoleFromGuild(guild).orElse(null);
+        Optional<Role> cakeDayRole = getCakeDayRoleFromGuild(guild);
 
-        if (cakeDayRole == null) {
+        if (cakeDayRole.isEmpty()) {
             logger.warn("Cake day role with pattern {} not found for guild: {}",
                     config.rolePattern(), guild.getName());
-            return Optional.empty();
         }
 
-        return Optional.of(cakeDayRole);
+        return cakeDayRole;
     }
 
     /**
@@ -132,13 +131,13 @@ private void addTodayMembersCakeDayRole(Guild guild) {
     protected void addCakeDayRole(Member member) {
         Guild guild = member.getGuild();
         UserSnowflake snowflake = UserSnowflake.fromId(member.getId());
-        Role cakeDayRole = getCakeDayRole(guild).orElse(null);
+        Optional<Role> cakeDayRole = getCakeDayRole(guild);
 
-        if (cakeDayRole == null) {
+        if (cakeDayRole.isEmpty()) {
             return;
         }
 
-        guild.addRoleToMember(snowflake, cakeDayRole).complete();
+        guild.addRoleToMember(snowflake, cakeDayRole.get()).complete();
     }
 
     /**