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

임의 데이터 추가함. 관계 설정 안 된 상태. #30

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

Shin-Yu-1
Copy link
Collaborator

@Shin-Yu-1 Shin-Yu-1 commented May 25, 2023

Motivation 🥳 (코드를 추가/변경하게 된 이유)

pin 생성 시 입력된 정보를 바탕으로 core data에 저장되도록 작성 중임.

Key Changes 🔥 (주요 구현/변경 사항)

core data 추가

ToDo 📆 (남은 작업)

  • 릴레이션 수정
  • core data �저장 작업 진행 중

ScreenShot 📷 (참고 사진)

To Reviewers 🙏 (리뷰어에게 전달하고 싶은 말)

  • MapView 내에 지도뷰를 추가했더니 빌드 시 xcode가 꺼집니다. 코드는 계속 수정 중이다보니 확인을 위해 중간 중간 커밋 하고자 합니다.

Reference 🔗

없음

Close Issues 🔒 (닫을 Issue)

Close #No.29

Copy link
Collaborator

@taek0622 taek0622 left a comment

Choose a reason for hiding this comment

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

코어데이터는 코드로 리뷰 하기가 애매해서 자세한 내용은 회의에서 다시 말씀 드려보겠습니다.
우선, 테마 추가에 대해서 다 같이 이야기 해야할 부분이 하나 생겨서 이에 따라 엔티티가 하나 더 추가될 수도 있을 것 같습니다.
또, 현재 엔티티 이름에 약간의 오타가 보이는 것 같아요. 해당 부분은 회의에서 다시 이야기 해보겠습니다.

@LiSeul LiSeul requested a review from taek0622 June 8, 2023 10:21
Copy link
Collaborator

@taek0622 taek0622 left a comment

Choose a reason for hiding this comment

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

수정 확인했습니다.
현재 상황에서 일부 수정하면 좋을 것 같은 이름과 관계를 조금 구성해봤습니다.

우선 현재 상황은 아래와 같은데요.

CoordinateAddress와 1:1 매칭 관계라고 생각이 됩니다. 그렇다면 이 둘은 하나로 통합하는 것이 더 나을 것이라고 판단됩니다. 이 두개를 나눠놓았을 때는 좌표나 주소가 수정되었을 경우, id를 얻기위한 PinInfo와 주소를 수정하기 위한 Address, 그리고 좌표를 수정하기 위한 Coordinate에 모두 접근해야 하지만, 이 둘을 통합하면 PinInfo와 좌표와 주소를 가지고 있는 총 2개의 엔티티에만 접근하면 되니 이쪽이 더 효율적일 것 같네요.

또, Pictures는 Picture가 되면 더 좋을 것 같아요. Pictures라는 이름이 뭔가 하나의 엔티티에 여러 사진을 갖고 있는 것 처럼 느껴져서 어차피 1:M 관계로 매핑하기 때문에 Picture가 되는 쪽이 더 좋을 것 같네요.

아래는 제가 생각했을 때 수정되면 좋을 것 같은 방향입니다. 참고만 해보시고 더 좋을 것 같은 방향으로 진행해주세요~

@Shin-Yu-1 Shin-Yu-1 closed this Jul 8, 2023
@Shin-Yu-1 Shin-Yu-1 reopened this Jul 8, 2023
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.

2 participants