Skip to content

Commit c26c489

Browse files
author
github-actions
committed
chore: Java 스타일 수정
1 parent 098dc1d commit c26c489

File tree

7 files changed

+44
-35
lines changed

7 files changed

+44
-35
lines changed

backend/src/main/java/io/f1/backend/domain/quiz/api/QuizController.java

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,9 +3,9 @@
33
import io.f1.backend.domain.quiz.app.QuizService;
44
import io.f1.backend.domain.quiz.dto.QuizCreateRequest;
55
import io.f1.backend.domain.quiz.dto.QuizCreateResponse;
6-
76
import io.f1.backend.domain.quiz.dto.QuizListPageResponse;
87
import io.f1.backend.domain.quiz.dto.QuizUpdateRequest;
8+
99
import jakarta.validation.Valid;
1010

1111
import lombok.RequiredArgsConstructor;
@@ -20,7 +20,6 @@
2020
import org.springframework.web.bind.annotation.PathVariable;
2121
import org.springframework.web.bind.annotation.PostMapping;
2222
import org.springframework.web.bind.annotation.PutMapping;
23-
import org.springframework.web.bind.annotation.RequestBody;
2423
import org.springframework.web.bind.annotation.RequestMapping;
2524
import org.springframework.web.bind.annotation.RequestParam;
2625
import org.springframework.web.bind.annotation.RequestPart;

backend/src/main/java/io/f1/backend/domain/quiz/app/QuizService.java

Lines changed: 21 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
import static io.f1.backend.domain.quiz.mapper.QuizMapper.quizCreateRequestToQuiz;
55
import static io.f1.backend.domain.quiz.mapper.QuizMapper.quizToQuizCreateResponse;
66
import static io.f1.backend.domain.quiz.mapper.QuizMapper.toQuizListPageResponse;
7+
78
import static java.nio.file.Files.deleteIfExists;
89

910
import io.f1.backend.domain.question.app.QuestionService;
@@ -18,7 +19,6 @@
1819
import io.f1.backend.domain.user.dao.UserRepository;
1920
import io.f1.backend.domain.user.entity.User;
2021

21-
import java.util.NoSuchElementException;
2222
import lombok.RequiredArgsConstructor;
2323

2424
import org.springframework.beans.factory.annotation.Value;
@@ -32,6 +32,7 @@
3232
import java.nio.file.Path;
3333
import java.nio.file.Paths;
3434
import java.util.List;
35+
import java.util.NoSuchElementException;
3536
import java.util.UUID;
3637

3738
@Service
@@ -104,10 +105,12 @@ private String getExtension(String filename) {
104105
@Transactional
105106
public void deleteQuiz(Long quizId) {
106107

107-
Quiz quiz = quizRepository.findById(quizId)
108-
.orElseThrow(() -> new NoSuchElementException("존재하지 않는 퀴즈입니다."));
108+
Quiz quiz =
109+
quizRepository
110+
.findById(quizId)
111+
.orElseThrow(() -> new NoSuchElementException("존재하지 않는 퀴즈입니다."));
109112

110-
if(1L != quiz.getCreator().getId()) {
113+
if (1L != quiz.getCreator().getId()) {
111114
throw new RuntimeException("권한이 없습니다.");
112115
}
113116

@@ -117,20 +120,22 @@ public void deleteQuiz(Long quizId) {
117120

118121
@Transactional
119122
public void updateQuiz(Long quizId, MultipartFile thumbnailFile, QuizUpdateRequest request)
120-
throws IOException {
123+
throws IOException {
121124

122-
Quiz quiz = quizRepository.findById(quizId)
123-
.orElseThrow(() -> new NoSuchElementException("존재하지 않는 퀴즈입니다."));
125+
Quiz quiz =
126+
quizRepository
127+
.findById(quizId)
128+
.orElseThrow(() -> new NoSuchElementException("존재하지 않는 퀴즈입니다."));
124129

125-
if(request.title() != null) {
130+
if (request.title() != null) {
126131
quiz.changeTitle(request.title());
127132
}
128133

129-
if(request.description() != null) {
134+
if (request.description() != null) {
130135
quiz.changeDescription(request.description());
131136
}
132137

133-
if(thumbnailFile !=null && !thumbnailFile.isEmpty()) {
138+
if (thumbnailFile != null && !thumbnailFile.isEmpty()) {
134139
validateImageFile(thumbnailFile);
135140
String newThumbnailPath = convertToThumbnailPath(thumbnailFile);
136141

@@ -140,7 +145,7 @@ public void updateQuiz(Long quizId, MultipartFile thumbnailFile, QuizUpdateReque
140145
}
141146

142147
private void deleteOldThumbnailFileIfNeeded(String oldFilename) {
143-
if(oldFilename.contains("default")) {
148+
if (oldFilename.contains("default")) {
144149
return;
145150
}
146151

@@ -151,7 +156,7 @@ private void deleteOldThumbnailFileIfNeeded(String oldFilename) {
151156

152157
try {
153158
boolean deleted = deleteIfExists(filePath);
154-
if( deleted ) {
159+
if (deleted) {
155160
System.out.println("기존 썸네일 삭제 완료 : " + filePath);
156161
} else {
157162
System.out.println("기존 썸네일 존재 X : " + filePath);
@@ -162,15 +167,15 @@ private void deleteOldThumbnailFileIfNeeded(String oldFilename) {
162167
}
163168
}
164169

165-
@Transactional(readOnly=true)
170+
@Transactional(readOnly = true)
166171
public QuizListPageResponse getQuizzes(String title, String creator, Pageable pageable) {
167172

168173
Page<Quiz> quizzes;
169174

170175
// 검색어가 있을 때
171-
if(title != null && !title.isBlank()) {
176+
if (title != null && !title.isBlank()) {
172177
quizzes = quizRepository.findQuizzesByTitleContaining(title, pageable);
173-
} else if(creator !=null && !creator.isBlank()) {
178+
} else if (creator != null && !creator.isBlank()) {
174179
quizzes = quizRepository.findQuizzesByCreator_NicknameContaining(creator, pageable);
175180
} else { // 검색어가 없을 때 혹은 빈 문자열일 때
176181
quizzes = quizRepository.findAll(pageable);
@@ -180,4 +185,4 @@ public QuizListPageResponse getQuizzes(String title, String creator, Pageable pa
180185

181186
return toQuizListPageResponse(quizListResponses);
182187
}
183-
}
188+
}

backend/src/main/java/io/f1/backend/domain/quiz/dao/QuizRepository.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,10 +5,10 @@
55
import org.springframework.data.domain.Page;
66
import org.springframework.data.domain.Pageable;
77
import org.springframework.data.jpa.repository.JpaRepository;
8-
import org.springframework.data.jpa.repository.Query;
98

109
public interface QuizRepository extends JpaRepository<Quiz, Long> {
1110

1211
Page<Quiz> findQuizzesByTitleContaining(String title, Pageable pageable);
12+
1313
Page<Quiz> findQuizzesByCreator_NicknameContaining(String creator, Pageable pageable);
1414
}

backend/src/main/java/io/f1/backend/domain/quiz/dto/QuizListPageResponse.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,4 +2,5 @@
22

33
import java.util.List;
44

5-
public record QuizListPageResponse(int totalPages, int currentPage, long totalElements, List<QuizListResponse> quiz) { }
5+
public record QuizListPageResponse(
6+
int totalPages, int currentPage, long totalElements, List<QuizListResponse> quiz) {}
Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,9 @@
11
package io.f1.backend.domain.quiz.dto;
22

3-
public record QuizListResponse(Long quizId, String title, String description, String creatorNickname, int numberOfQuestion, String thumbnailUrl) { }
3+
public record QuizListResponse(
4+
Long quizId,
5+
String title,
6+
String description,
7+
String creatorNickname,
8+
int numberOfQuestion,
9+
String thumbnailUrl) {}
Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,3 @@
11
package io.f1.backend.domain.quiz.dto;
22

3-
public record QuizUpdateRequest(String title, String description) {}
3+
public record QuizUpdateRequest(String title, String description) {}

backend/src/main/java/io/f1/backend/domain/quiz/mapper/QuizMapper.java

Lines changed: 11 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66
import io.f1.backend.domain.quiz.dto.QuizListResponse;
77
import io.f1.backend.domain.quiz.entity.Quiz;
88
import io.f1.backend.domain.user.entity.User;
9-
import java.util.List;
9+
1010
import org.springframework.data.domain.Page;
1111

1212
public class QuizMapper {
@@ -37,22 +37,20 @@ public static QuizCreateResponse quizToQuizCreateResponse(Quiz quiz) {
3737

3838
public static QuizListResponse quizToQuizListResponse(Quiz quiz) {
3939
return new QuizListResponse(
40-
quiz.getId(),
41-
quiz.getTitle(),
42-
quiz.getDescription(),
43-
quiz.getCreator().getNickname(),
44-
quiz.getQuestions().size(),
45-
quiz.getThumbnailUrl()
46-
);
40+
quiz.getId(),
41+
quiz.getTitle(),
42+
quiz.getDescription(),
43+
quiz.getCreator().getNickname(),
44+
quiz.getQuestions().size(),
45+
quiz.getThumbnailUrl());
4746
}
4847

4948
public static QuizListPageResponse toQuizListPageResponse(Page<QuizListResponse> quizzes) {
5049
return new QuizListPageResponse(
51-
quizzes.getTotalPages(),
52-
quizzes.getNumber() + 1,
53-
quizzes.getTotalElements(),
54-
quizzes.getContent()
55-
);
50+
quizzes.getTotalPages(),
51+
quizzes.getNumber() + 1,
52+
quizzes.getTotalElements(),
53+
quizzes.getContent());
5654
}
5755

5856
public static Page<QuizListResponse> pageQuizToPageQuizListResponse(Page<Quiz> quizzes) {

0 commit comments

Comments
 (0)