diff --git a/src/main/java/com/genius/gitget/topic/domain/Topic.java b/src/main/java/com/genius/gitget/topic/domain/Topic.java index 18e6ec33..7758d313 100644 --- a/src/main/java/com/genius/gitget/topic/domain/Topic.java +++ b/src/main/java/com/genius/gitget/topic/domain/Topic.java @@ -60,7 +60,6 @@ public Topic(String title, String description, String tags, String notice, int p this.pointPerPerson = pointPerPerson; } - //== 비즈니스 로직 ==// public void updateExistInstance(String description) { this.description = description; } diff --git a/src/main/java/com/genius/gitget/topic/dto/TopicCreateRequest.java b/src/main/java/com/genius/gitget/topic/dto/TopicCreateRequest.java index 7e00659d..4af0369b 100644 --- a/src/main/java/com/genius/gitget/topic/dto/TopicCreateRequest.java +++ b/src/main/java/com/genius/gitget/topic/dto/TopicCreateRequest.java @@ -1,5 +1,6 @@ package com.genius.gitget.topic.dto; +import com.genius.gitget.topic.domain.Topic; import lombok.Builder; @Builder @@ -10,4 +11,13 @@ public record TopicCreateRequest( int pointPerPerson, String notice ) { + public static Topic from(TopicCreateRequest topicCreateRequest) { + return Topic.builder() + .title(topicCreateRequest.title()) + .description(topicCreateRequest.description()) + .tags(topicCreateRequest.tags()) + .pointPerPerson(topicCreateRequest.pointPerPerson()) + .notice(topicCreateRequest.notice()) + .build(); + } } diff --git a/src/main/java/com/genius/gitget/topic/service/TopicService.java b/src/main/java/com/genius/gitget/topic/service/TopicService.java index 5c2519a6..9a450145 100644 --- a/src/main/java/com/genius/gitget/topic/service/TopicService.java +++ b/src/main/java/com/genius/gitget/topic/service/TopicService.java @@ -3,7 +3,6 @@ import com.genius.gitget.global.util.exception.BusinessException; import com.genius.gitget.global.util.exception.ErrorCode; import com.genius.gitget.topic.domain.Topic; -import com.genius.gitget.topic.dto.TopicCreateRequest; import com.genius.gitget.topic.repository.TopicRepository; import lombok.RequiredArgsConstructor; import lombok.extern.slf4j.Slf4j; @@ -29,8 +28,8 @@ public Topic findOne(Long id) { } @Transactional - public Long create(Topic byTopicCreateDto) { - Topic savedTopic = topicRepository.save(byTopicCreateDto); + public Long create(Topic topic) { + Topic savedTopic = topicRepository.save(topic); return savedTopic.getId(); } @@ -40,14 +39,4 @@ public void delete(Long id) { .orElseThrow(() -> new BusinessException(ErrorCode.TOPIC_NOT_FOUND)); topicRepository.delete(topic); } - - public Topic createTopicByTopicCreateRequest(TopicCreateRequest topicCreateRequest) { - return Topic.builder() - .title(topicCreateRequest.title()) - .description(topicCreateRequest.description()) - .tags(topicCreateRequest.tags()) - .pointPerPerson(topicCreateRequest.pointPerPerson()) - .notice(topicCreateRequest.notice()) - .build(); - } } diff --git a/src/main/java/com/genius/gitget/topic/serviceFacade/TopicFacadeImpl.java b/src/main/java/com/genius/gitget/topic/serviceFacade/TopicFacadeImpl.java index 00d7f363..f91e2a07 100644 --- a/src/main/java/com/genius/gitget/topic/serviceFacade/TopicFacadeImpl.java +++ b/src/main/java/com/genius/gitget/topic/serviceFacade/TopicFacadeImpl.java @@ -37,7 +37,7 @@ public TopicDetailResponse findOne(Long id) { @Override public Long create(TopicCreateRequest topicCreateRequest) { - Topic topic = topicService.createTopicByTopicCreateRequest(topicCreateRequest); + Topic topic = TopicCreateRequest.from(topicCreateRequest); return topicService.create(topic); }