Skip to content

[5주차 과제] 김성재 과제 제출합니다#11

Open
k1mseongjae wants to merge 2 commits intoCAUSWICT-EDU:k1mseongjaefrom
k1mseongjae:dev
Open

[5주차 과제] 김성재 과제 제출합니다#11
k1mseongjae wants to merge 2 commits intoCAUSWICT-EDU:k1mseongjaefrom
k1mseongjae:dev

Conversation

@k1mseongjae
Copy link

ERD

image ----

DB

image image

@k1mseongjae k1mseongjae changed the title Dev 5주차 과제 제출 Nov 20, 2025
@k1mseongjae k1mseongjae reopened this Nov 20, 2025
@k1mseongjae k1mseongjae changed the title 5주차 과제 제출 [5주차 과제] 김성재 과제 제출합니다 Nov 20, 2025
Comment on lines +32 to +33
@Column(name = "anonymous_id")
private Long anonymousId;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

anonymous_id는 댓글에서 익명을 구분하는 번호 같습니다만, Comment에 익명 여부에 대한 필드가 안 보이네요. 혹시 모든 댓글은 익명으로 처리되나요? 만약에 익명 여부를 결정할 수 있다면, 익명이 아닌 댓글에도 번호를 부여하나요?

Comment on lines +19 to +21
@CreatedDate
@Column(updatable = false)
private LocalDateTime createdAt;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

createdAt이 생성 당시 시점을 저장하는 것이니 만큼 업데이트가 안 되게 막으신 점 좋습니다!

private LocalDate duration;

@Column(name = "result")
private Boolean result;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

result라는 필드 이름으로 봐서는 어떤 투표에 대한 결과인 것인지, 또는 투표한 사람의 결과(선택)인지 헷갈릴 여지가 있다고 보입니다. 그리고 타입이 Boolean으로 되어있는데, 혹시 찬반 투표만 제공하는 것인가요?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Board 안에서는 id가 하나인데 id 이름을 board_id로 또 따로 설정한 이유가 있을까요?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

anonymousId는 어떤 것을 위한 필드인지 궁금합니다!

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

중간에 viewCount 필드가 있는 것 같은데 viewCount를 필드로 관리하는 이유가 있을까요?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants