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

상세페이지 API 구현 #7

Open
3 tasks
hanurii opened this issue May 19, 2020 · 0 comments
Open
3 tasks

상세페이지 API 구현 #7

hanurii opened this issue May 19, 2020 · 0 comments
Assignees
Labels
BE feature New feature or request
Milestone

Comments

@hanurii
Copy link
Contributor

hanurii commented May 19, 2020

  • : 필요한 데이터 스키마 구현
  • : 상세페이지 Mockup API 제공
  • : 상세페이지 API 구현
@hanurii hanurii added the BE label May 19, 2020
@hanurii hanurii added the feature New feature or request label May 19, 2020
@hanurii hanurii added this to the BE - 2+1 milestone May 25, 2020
hanurii added a commit that referenced this issue May 25, 2020
hanurii added a commit that referenced this issue May 25, 2020
hanurii added a commit that referenced this issue May 25, 2020
haveagood added a commit that referenced this issue May 26, 2020
- Detail Page의 경우에는 표시해야 할 자료가 너무 많았다.
- 1차적으로 상속을 활용하여 구현하고자 하였는데, table단위로 객체를
  나눠서 상속을 하고자 하였지만, 그럼에도 변수가 너무 많았고, 가독성이
  떨어졌다.
- Component를 활용하여 host,price,location,review,room 으로 객체를
  분리하고 해당되는 변수들을 해당 객체에 이동시켰다.
- DetailDto라는 Main객체에서 하위 컴포넌트를 포함하게 하여 객체를
  생성하였다.

issue : #7

-------------------------
-------------------------
haveagood added a commit that referenced this issue May 26, 2020
- DB에 존재하는 host_type의 형식이 String형식으로 ex)'super'로 되어있다
- 확장성을 고려하여 문자열로 받고자 하였으나, 프로젝트에서는 슈퍼호스트
  여부만 판단하면 될 것 같다고 결정하였다.
- DB의 구조를 바꾸기보다는 CASE WHEN sql구문을 활용하여 값을 변환시켰다.
- 기존의 값은 string이었는데, 1,0값을 가지는 boolean으로 변환하였다.

issue : #7

-------------------------
-------------------------
haveagood added a commit that referenced this issue May 26, 2020
- amenities 데이터가 ""로 이루어져있어 DB에서 가져올때, 깔끔하게
  가져와지지 않았다.
- REPLACE를 활용하여 Data를 정리해서 가져오도록 변환

issue : #7

-------------------------
-------------------------
hanurii added a commit that referenced this issue May 27, 2020
… 페이지와 관련된 소스들 변경

- PropertiesDaoHamill : findByPropertiesId 내용 변경
  - alex 가 구현한 코드를 참고하여 구현

- issue : #7
hanurii added a commit that referenced this issue May 28, 2020
- Arrays.asList(rs.getString("image").split(","))

- issue : #7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE feature New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants