Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feature/1 : 도메인 설정 및 기본 설정 추가 #2

Merged
merged 3 commits into from
Sep 28, 2024
Merged

Conversation

soleu
Copy link
Collaborator

@soleu soleu commented Sep 25, 2024

  1. 도메인 설정

고객, 상품, 입찰 히스토리 엔티티 설정

  1. h2 기본 연결 기본 설정 추가
image
  1. pr template 설정

고민 중인 부분

  1. DB에서 관계를 맺지 않고, id 값으로 연동하도록 설정하였습니다. 각 엔티티간의 결합도를 낮추기 위해서 사용하였는데,연관관계를 맺게 하는 방식이 더 나을지 의견 부탁드립니다.

@soleu soleu added the setting label Sep 25, 2024
@soleu soleu requested a review from mason136 September 25, 2024 10:49
@soleu soleu self-assigned this Sep 25, 2024
Copy link

sonarcloud bot commented Sep 25, 2024

Quality Gate Failed Quality Gate failed

Failed conditions
E Security Rating on New Code (required ≥ A)

See analysis details on SonarCloud

Catch issues before they fail your Quality Gate with our IDE extension SonarLint

@soleu soleu merged commit dfa4c29 into main Sep 28, 2024
1 of 2 checks passed
@mason136
Copy link
Collaborator

DB에서 관계를 맺지 않고, id 값으로 연동하도록 설정하였습니다. 각 엔티티간의 결합도를 낮추기 위해서 사용하였는데,연관관계를 맺게 하는 방식이 더 나을지 의견 부탁드립니다.

늦게 봤네요ㅠ
연관관계 맺는 방식이 엔터티간 결합도도 많아지기도 하고, 쿼리를 직관적으로 예측하기가 애매한 부분이 있어서
저도 id 값으로 연동하는 것을 선호하고 자주 씁니다.

@mason136
Copy link
Collaborator

엔터티 pk id도 생성자 안에 넣어도 되는데 왜 생성자에서는 빼신건가요??

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants