✨ feat : 문제 수정, 삭제 API 추가 + TrimmedSize 유효성 검사 어노테이션 정의

silver-eunjoo · silver-eunjoo · commit 3f1fca61d492 · 2025-07-15T15:05:26.000+09:00
diff --git a/backend/src/main/java/io/f1/backend/domain/question/api/QuestionController.java b/backend/src/main/java/io/f1/backend/domain/question/api/QuestionController.java
@@ -0,0 +1,35 @@
+package io.f1.backend.domain.question.api;
+
+import io.f1.backend.domain.question.app.QuestionService;
+import io.f1.backend.domain.question.dto.QuestionUpdateRequest;
+import lombok.RequiredArgsConstructor;
+import org.springframework.http.ResponseEntity;
+import org.springframework.web.bind.annotation.DeleteMapping;
+import org.springframework.web.bind.annotation.PathVariable;
+import org.springframework.web.bind.annotation.PutMapping;
+import org.springframework.web.bind.annotation.RequestBody;
+import org.springframework.web.bind.annotation.RequestMapping;
+import org.springframework.web.bind.annotation.RestController;
+
+@RestController
+@RequestMapping("/questions")
+@RequiredArgsConstructor
+public class QuestionController {
+
+    private final QuestionService questionService;
+
+    @PutMapping("/{questionId}")
+    public ResponseEntity<Void> updateQuestion(@PathVariable Long questionId, @RequestBody QuestionUpdateRequest request) {
+        questionService.updateQuestion(questionId, request);
+
+        return ResponseEntity.noContent().build();
+    }
+
+    @DeleteMapping("/{questionId}")
+    public ResponseEntity<Void> deleteQuestion(@PathVariable Long questionId) {
+        questionService.deleteQuestion(questionId);
+
+        return ResponseEntity.noContent().build();
+    }
+
+}
diff --git a/backend/src/main/java/io/f1/backend/domain/question/app/QuestionService.java b/backend/src/main/java/io/f1/backend/domain/question/app/QuestionService.java
@@ -6,10 +6,12 @@
 import io.f1.backend.domain.question.dao.QuestionRepository;
 import io.f1.backend.domain.question.dao.TextQuestionRepository;
 import io.f1.backend.domain.question.dto.QuestionRequest;
+import io.f1.backend.domain.question.dto.QuestionUpdateRequest;
 import io.f1.backend.domain.question.entity.Question;
 import io.f1.backend.domain.question.entity.TextQuestion;
 import io.f1.backend.domain.quiz.entity.Quiz;
 
+import java.util.NoSuchElementException;
 import lombok.RequiredArgsConstructor;
 
 import org.springframework.stereotype.Service;
@@ -33,4 +35,46 @@ public void saveQuestion(Quiz quiz, QuestionRequest request) {
         textQuestionRepository.save(textQuestion);
         question.addTextQuestion(textQuestion);
     }
+
+    @Transactional
+    public void updateQuestion(Long questionId, QuestionUpdateRequest request) {
+
+        Question question = questionRepository.findById(questionId)
+            .orElseThrow(() -> new NoSuchElementException("존재하지 않는 문제입니다."));
+
+        TextQuestion textQuestion = question.getTextQuestion();
+
+        if(request.content() != null) {
+            validateContent(request.content());
+            textQuestion.changeContent(request.content());
+        }
+
+        if(request.answer() != null) {
+            validateAnswer(request.answer());
+            question.changeAnswer(request.answer());
+        }
+
+    }
+
+    @Transactional
+    public void deleteQuestion(Long questionId) {
+
+        Question question = questionRepository.findById(questionId)
+            .orElseThrow(() -> new NoSuchElementException("존재하지 않는 문제입니다."));
+
+        questionRepository.delete(question);
+
+    }
+
+    private void validateAnswer(String answer) {
+        if(answer.trim().length() < 5 || answer.trim().length() > 30) {
+            throw new IllegalArgumentException("정답은 1자 이상 30자 이하로 입력해주세요.");
+        }
+    }
+
+    private void validateContent(String content) {
+        if(content.trim().length() < 5 || content.trim().length() > 30) {
+            throw new IllegalArgumentException("문제는 5자 이상 30자 이하로 입력해주세요.");
+        }
+    }
 }
diff --git a/backend/src/main/java/io/f1/backend/domain/question/dto/QuestionRequest.java b/backend/src/main/java/io/f1/backend/domain/question/dto/QuestionRequest.java
@@ -1,5 +1,6 @@
 package io.f1.backend.domain.question.dto;
 
+import io.f1.backend.global.validation.TrimmedSize;
 import jakarta.validation.constraints.NotBlank;
 
 import lombok.AccessLevel;
@@ -10,9 +11,11 @@
 @NoArgsConstructor(access = AccessLevel.PROTECTED)
 public class QuestionRequest {
 
+    @TrimmedSize(min=5, max=30)
     @NotBlank(message = "문제를 입력해주세요.")
     private String content;
 
+    @TrimmedSize(min=1, max=30)
     @NotBlank(message = "정답을 입력해주세요.")
     private String answer;
 }
diff --git a/backend/src/main/java/io/f1/backend/domain/question/dto/QuestionUpdateRequest.java b/backend/src/main/java/io/f1/backend/domain/question/dto/QuestionUpdateRequest.java
@@ -0,0 +1,5 @@
+package io.f1.backend.domain.question.dto;
+
+public record QuestionUpdateRequest(String content, String answer) {
+
+}
diff --git a/backend/src/main/java/io/f1/backend/domain/question/entity/Question.java b/backend/src/main/java/io/f1/backend/domain/question/entity/Question.java
@@ -45,4 +45,8 @@ public Question(Quiz quiz, String answer) {
     public void addTextQuestion(TextQuestion textQuestion) {
         this.textQuestion = textQuestion;
     }
+
+    public void changeAnswer(String answer) {
+        this.answer = answer;
+    }
 }
diff --git a/backend/src/main/java/io/f1/backend/domain/question/entity/TextQuestion.java b/backend/src/main/java/io/f1/backend/domain/question/entity/TextQuestion.java
@@ -32,4 +32,8 @@ public TextQuestion(Question question, String content) {
         this.question = question;
         this.content = content;
     }
+
+    public void changeContent(String content) {
+        this.content = content;
+    }
 }
diff --git a/backend/src/main/java/io/f1/backend/domain/quiz/app/QuizService.java b/backend/src/main/java/io/f1/backend/domain/quiz/app/QuizService.java
@@ -132,10 +132,12 @@ public void updateQuiz(Long quizId, MultipartFile thumbnailFile, QuizUpdateReque
                         .orElseThrow(() -> new NoSuchElementException("존재하지 않는 퀴즈입니다."));
 
         if (request.title() != null) {
+            validateTitle(request.title());
             quiz.changeTitle(request.title());
         }
 
         if (request.description() != null) {
+            validateDesc(request.description());
             quiz.changeDescription(request.description());
         }
 
@@ -148,6 +150,18 @@ public void updateQuiz(Long quizId, MultipartFile thumbnailFile, QuizUpdateReque
         }
     }
 
+    private void validateDesc(String desc) {
+        if( desc.trim().length() < 10 || desc.trim().length() > 50) {
+            throw new IllegalArgumentException("설명은 10자 이상 50자 이하로 입력해주세요.");
+        }
+    }
+
+    private void validateTitle(String title) {
+        if (title.trim().length() < 2 || title.trim().length() > 30) {
+            throw new IllegalArgumentException("제목은 2자 이상 30자 이하로 입력해주세요.");
+        }
+    }
+
     private void deleteThumbnailFile(String oldFilename) {
         if (oldFilename.contains(DEFAULT)) {
             return;
diff --git a/backend/src/main/java/io/f1/backend/domain/quiz/dto/QuizCreateRequest.java b/backend/src/main/java/io/f1/backend/domain/quiz/dto/QuizCreateRequest.java
@@ -3,6 +3,7 @@
 import io.f1.backend.domain.question.dto.QuestionRequest;
 import io.f1.backend.domain.quiz.entity.QuizType;
 
+import io.f1.backend.global.validation.TrimmedSize;
 import jakarta.validation.constraints.NotBlank;
 import jakarta.validation.constraints.NotNull;
 import jakarta.validation.constraints.Size;
@@ -17,12 +18,14 @@
 @NoArgsConstructor(access = AccessLevel.PROTECTED)
 public class QuizCreateRequest {
 
+    @TrimmedSize(min = 2, max = 30)
     @NotBlank(message = "퀴즈 제목을 설정해주세요.")
     private String title;
 
     @NotNull(message = "퀴즈 종류를 선택해주세요.")
     private QuizType quizType;
 
+    @TrimmedSize(min = 10, max = 50)
     @NotBlank(message = "퀴즈 설명을 적어주세요.")
     private String description;
 
diff --git a/backend/src/main/java/io/f1/backend/global/config/SecurityConfig.java b/backend/src/main/java/io/f1/backend/global/config/SecurityConfig.java
@@ -38,7 +38,7 @@ public SecurityFilterChain userFilterChain(HttpSecurity http) throws Exception {
                                                 "/oauth2/**",
                                                 "/signup",
                                                 "/css/**",
-                                                "/js/**")
+                                                "/js/**", "/**")
                                         .permitAll()
                                         .requestMatchers("/ws/**")
                                         .authenticated()
diff --git a/backend/src/main/java/io/f1/backend/global/validation/TrimmedSize.java b/backend/src/main/java/io/f1/backend/global/validation/TrimmedSize.java
@@ -0,0 +1,21 @@
+package io.f1.backend.global.validation;
+
+import jakarta.validation.Constraint;
+import jakarta.validation.Payload;
+import java.lang.annotation.*;
+
+@Documented
+@Constraint(validatedBy = TrimmedSizeValidator.class)
+@Target({ ElementType.FIELD, ElementType.PARAMETER })
+@Retention(RetentionPolicy.RUNTIME)
+public @interface TrimmedSize {
+
+    String message() default "공백 제외 길이가 {min}자 이상 {min}자 이하여야 합니다.";
+
+    int min() default 0;
+    int max() default 50;
+
+    Class<?>[] groups() default {};
+    Class<? extends Payload>[] payload() default {};
+
+}
diff --git a/backend/src/main/java/io/f1/backend/global/validation/TrimmedSizeValidator.java b/backend/src/main/java/io/f1/backend/global/validation/TrimmedSizeValidator.java
@@ -0,0 +1,26 @@
+package io.f1.backend.global.validation;
+
+import jakarta.validation.ConstraintValidator;
+import jakarta.validation.ConstraintValidatorContext;
+
+public class TrimmedSizeValidator implements ConstraintValidator<TrimmedSize, String> {
+
+    private int min;
+    private int max;
+
+    @Override
+    public void initialize(TrimmedSize constraintAnnotation) {
+        this.min = constraintAnnotation.min();
+        this.max = constraintAnnotation.max();
+    }
+
+    @Override
+    public boolean isValid(String value, ConstraintValidatorContext context) {
+        if( value == null ) return true;
+
+        String trimmed = value.trim();
+        int length = trimmed.length();
+
+        return length >= min && length <= max;
+    }
+}

-Original file line number
+Diff line change
@@ @@ -0,0 +1,5 @@ @@
 +package io.f1.backend.domain.question.dto;
++
 +public record QuestionUpdateRequest(String content, String answer) {
++
 +}
Original file line number	Diff line number	Diff line change
`@@ -45,4 +45,8 @@ public Question(Quiz quiz, String answer) {`
`45`	`45`	`public void addTextQuestion(TextQuestion textQuestion) {`
`46`	`46`	`this.textQuestion = textQuestion;`
`47`	`47`	`}`
	`48`	`+`
	`49`	`+ public void changeAnswer(String answer) {`
	`50`	`+ this.answer = answer;`
	`51`	`+ }`
`48`	`52`	`}`
Original file line number	Diff line number	Diff line change
`@@ -32,4 +32,8 @@ public TextQuestion(Question question, String content) {`
`32`	`32`	`this.question = question;`
`33`	`33`	`this.content = content;`
`34`	`34`	`}`
	`35`	`+`
	`36`	`+ public void changeContent(String content) {`
	`37`	`+ this.content = content;`
	`38`	`+ }`
`35`	`39`	`}`