refactor: 람다식 수정

kgu-clab · Jul 19, 2024 · 32d2a44 · 32d2a44
1 parent a1a139e
commit 32d2a44
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 14 deletions.
diff --git a/src/main/java/page/clab/api/domain/activity/activitygroup/domain/ActivityGroupBoard.java b/src/main/java/page/clab/api/domain/activity/activitygroup/domain/ActivityGroupBoard.java
@@ -84,9 +84,7 @@ public void update(ActivityGroupBoardUpdateRequestDto requestDto, UploadedFileSe
         Optional.ofNullable(requestDto.getTitle()).ifPresent(this::setTitle);
         Optional.ofNullable(requestDto.getContent()).ifPresent(this::setContent);
         Optional.ofNullable(requestDto.getDueDateTime()).ifPresent(this::setDueDateTime);
-        Optional.ofNullable(requestDto.getFileUrls()).ifPresent(urls -> {
-            this.setUploadedFiles(uploadedFileService.getUploadedFilesByUrls(urls));
-        });
+        Optional.ofNullable(requestDto.getFileUrls()).ifPresent(urls -> this.setUploadedFiles(uploadedFileService.getUploadedFilesByUrls(urls)));
     }
 
     public void addChild(ActivityGroupBoard child) {

diff --git a/src/main/java/page/clab/api/global/common/email/api/EmailController.java b/src/main/java/page/clab/api/global/common/email/api/EmailController.java
@@ -34,10 +34,7 @@ public ApiResponse<List<String>> broadcastEmail(
             EmailDto emailDto,
             @RequestParam(name = "multipartFile", required = false) List<MultipartFile> files
     ) {
-        CompletableFuture<List<String>> emailTask = CompletableFuture.supplyAsync(() -> {
-            return emailService.broadcastEmail(emailDto, files);
-        });
-
+        CompletableFuture<List<String>> emailTask = CompletableFuture.supplyAsync(() -> emailService.broadcastEmail(emailDto, files));
         List<String> successfulAddresses = emailTask.join();
         return ApiResponse.success(successfulAddresses);
     }
@@ -49,10 +46,7 @@ public ApiResponse<List<String>> broadcastEmailToAllMember(
             EmailDto emailDto,
             @RequestParam(name = "multipartFile", required = false) List<MultipartFile> files
     ) {
-        CompletableFuture<List<String>> emailTask = CompletableFuture.supplyAsync(() -> {
-            return emailService.broadcastEmailToAllMember(emailDto, files);
-        });
-
+        CompletableFuture<List<String>> emailTask = CompletableFuture.supplyAsync(() -> emailService.broadcastEmailToAllMember(emailDto, files));
         List<String> successfulEmails = emailTask.join();
         return ApiResponse.success(successfulEmails);
     }

diff --git a/src/main/java/page/clab/api/global/util/IPInfoUtil.java b/src/main/java/page/clab/api/global/util/IPInfoUtil.java
@@ -31,9 +31,8 @@ public static IPInfoResponse getIpInfo(String ipAddress) {
                 .header("Authorization", "Bearer " + accessToken)
                 .accept(MediaType.APPLICATION_JSON)
                 .retrieve()
-                .onStatus(HttpStatusCode::is4xxClientError, ((request, response) -> {
-                    log.warn("4xx error occurred while getting ip info. Status code: {}", response.getStatusCode());
-                }))
+                .onStatus(HttpStatusCode::is4xxClientError, ((request, response) ->
+                    log.warn("4xx error occurred while getting ip info. Status code: {}", response.getStatusCode())))
                 .body(IPInfoResponse.class);
     }