Article: Article, Draft 모듈 초기 세팅 #75
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
6 tasks
sweetykr7
requested review from
merge-simpson,
silberbullet,
wowddok99,
jilpoom,
wch-os and
seonghooni
…feature/post # Conflicts: # gradle.properties # monolith/main-runner/build.gradle.kts
merge-simpson
changed the title
merge-simpson
changed the title
seonghooni
reviewed
May 17, 2025
| Objects.requireNonNull(content, "Content cannot be null"); | ||
| Objects.requireNonNull(status, "status cannot be null"); | ||
| Objects.requireNonNull(totalLikes, "totalLikes cannot be null"); | ||
| Objects.requireNonNull(totalLikes, "totalLikes cannot be null"); |
There was a problem hiding this comment.
위 코드와 중복됩니다!
하나는 totalViews로 작성하려고 하셨던 거라고 생각됩니다!
| var existArticle = articleJpaRepository.findById(article.getId()) | ||
| .orElseThrow(ARTICLE_NOT_FOUND::exception); | ||
| existArticle.prepareArticleEntityUpdate() | ||
| .title(existArticle.getTitle()) |
| } | ||
|
|
||
| @Override | ||
| public Article create(Article article) { |
There was a problem hiding this comment.
Adapter (=Repository) 함수 네이밍은 save()가 더 적합하지 않나 생각됩니다!
create -> RDB 또는 usecase 관점
save -> JPA와 같은 ORM 관점 표준 네이밍 이라고 생각합니다.
다른 분들 의견도 한번 여쭤보고 싶습니다!
| .totalShares(existArticle.getTotalShares()) | ||
| .update(); | ||
|
|
||
| return articleEntityMapper.toDomain(articleJpaRepository.save(existArticle)); |
There was a problem hiding this comment.
이미 영속상태인 엔티티(findById로 호출함)가 직접 수정되면,
영속성 컨텍스트에도 반영이 되기 때문에
articleEntityMapper.toDomain(existArticle);
처럼 실행해도 같은 결과를 얻을 수 있습니다!
단, 이 함수를 실행하는 영역에서 @Transactional 이 명시되어야 합니다.
지금 코드를 보니 CommandService에서는 트랜잭션이 적용되지 않은 것 같아요!
Comment on lines
+16
to
+23
| @Override | ||
| public Article createArticle(Article article) { | ||
| return articleCommandPort.create(article); | ||
| } | ||
| @Override | ||
| public Article updateArticle(Article article) { | ||
| return articleCommandPort.update(article); | ||
| } |
There was a problem hiding this comment.
함수와 함수 사이에는 공백 줄을 넣어 주시면 감사합니다!
Comment on lines
+20
to
+30
| private final ArticleQueryPort articleQueryPort; | ||
| @Override | ||
| public Optional<ArticleDetail> getArticle(Long id) { | ||
| return articleQueryPort.findById(id); | ||
| } | ||
| @Override | ||
| public Page<ArticleSummary> getAllArticle(Instant lastCreatedAt, int size) { | ||
| return articleQueryPort.findAllAfter(lastCreatedAt, size); | ||
| } | ||
| @Override | ||
| public Page<ArticleSummary> findByStatuses(Set<ArticleStatus> statuses, Instant lastCreatedAt, int size) { |
There was a problem hiding this comment.
여기도 함수와 함수 사이에는 공백줄을 추가해 주시면 감사합니다!
Comment on lines
+11
to
+12
| Optional<ArticleDetail> getArticle(Long id); | ||
| Page<ArticleSummary> getAllArticle(Instant lastCreatedAt, int size); |
There was a problem hiding this comment.
인터페이스에 내에 정의된 함수 사이에도 함수에는 공백 줄을 추가해주시면 감사합니다!
ex)
public interface ArticleReadUseCase {
Optional<ArticleDetail> getArticle(Long id);
Page<ArticleSummary> getAllArticle(Instant lastCreatedAt, int size);
}
Comment on lines
+15
to
+25
|
|
||
| @Getter | ||
| @DynamicUpdate | ||
| @NoArgsConstructor(access = AccessLevel.PROTECTED) | ||
| @Entity(name = "article") | ||
| public class ArticleEntity extends LongBaseTimeEntity { | ||
| public String title; | ||
| public String content; | ||
| public Integer totalViews = 0; | ||
| public Integer totalLikes = 0; | ||
| public Integer totalShares = 0; |
There was a problem hiding this comment.
💊 Enitity 변수의 접근제어자가 public으로 바뀌었네요! 👍
하기 애노테이션은 수정이 필요합니다! Drafts Enitity도 꼭 수정해 주세요.
@Getter: public으로 인하여 더 이상 Getter 없이도 변수 접근이 가능합니다.@NoArgsConstructor(access = AccessLevel.PROTECTED): Entity 클래스는 기본적으로 기본 생성자를 필요로 하며, 접근제어 세부 설정을 제외해도 좋습니다.@NoArgsConstructor로 수정하기
Comment on lines
+9
to
+10
| // spring | ||
| implementation("org.springframework.boot:spring-boot-starter-data-jpa") |
There was a problem hiding this comment.
💊 Application은 의미론적으로 JPA 한정적이다 보다는 spring-data-commons 라이브러리로 그 의미를 명확하게 해주는게 좋을 것 같습니다.
ex)
// spring
implementation("org.springframework.data:spring-data-commons")
silberbullet
approved these changes
May 18, 2025
seonghooni
reviewed
May 19, 2025
|
|
||
| @Builder | ||
| public record ArticleUpdateCommand( | ||
| @NotBlank(message = "id를 입력하십시오.") |
There was a problem hiding this comment.
Long 타입은 @NotBlank가 아닌 @NotNull로 수정해주세요!
@NotBlank를 쓴다면 컴파일 에러는 발생하지 않더라도 유효성 검사가 정상적으로 이루어지지 않는 것으로 알고 있습니다.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Pull Request
Issues
Description
Post, Draft에 대한 모듈을 제작하였습니다. [MAIN] Post Domain 구현 #56
Draft Table(임시글)을 ERD에 추가하였습니다.(기존에는 임시글이 없었습니다)
조회수,공유수,좋아요수가 제외되었습니다.DRAFT(임시),DONE(완료됨, 포스트 만들어진 상태),DELETED(삭제),PENDING(대기)를 만들었습니다.Exception을 추가 하였습니다.
Read-model을 추가 하였습니다.
Adapter를 추가하였습니다.
Service를 추가하였습니다.
How Has This Been Tested?
Additional Notes
만들면서 발생된 문제들
user id를 고려하지 못한 것 같습니다.
multi-module 관련
Config data resource 'class path resource [post-web.yml]' via location 'post-web.yml' does not existimplementation(project(":post"))post.yml를 추가하였습니다.post.yml을 추가하였습니다.:post:webmvc-adapter와:post:rdb-adapter의 src/main/resources에 해당하는 yml파일을 추가하였습니다.