[TEAM-11][BE][민지노&&필] 3주차 PR 입니다 #283

PhilSoGooood · 2022-06-09T03:07:32Z

안녕하세요 브라이언! 오랜만에 다시 PR 보내드립니다...
그동안 인프라에 집중하기도 하였고 집중해서 코딩을 하다보니 PR 시간에 늦어 그동안 보내지 못했습니다 🥲
그래도 시작할때 한번 끝날때 한번 보내는건 아닌 것 같아서 오늘 한번 보내게 되었습니다! 죄송하구 감사합니다!

ps. 커밋 내역이 민지노만 있는 것은 저희가 항상 만나서 짝코딩을 해서 그렇습니당ㅎㅎㅎ 절대 싸운것 아니에요...🥹

API 명세서

방 상세정보 API
테스트 서버 URL: http://144.24.86.236/room/1

메인 홈화면 API
테스트 서버 URL: http://144.24.86.236/home

여행지 거리 계산 API
테스트 서버 URL: http://144.24.86.236/distance/times?x=127.033466&y=37.490868

현재 상황

인프라 구축 (지난 PR에서 보내드린 설계도대로 구축을 하였습니다)
숙소 상세 페이지 조회
메인 화면 조회( iOS팀과 상의하여 Home Data라는 클래스안에 다 담아서 보내기로 하였습니다.)
카카오 api를 이용하여 유저의 현재 위치와 숙소 사이의 거리 및 이동시간 계산 (현재 기능 구현에 우선하여 중복되는 컨트롤러와 클래스는 삭제할 예정입니다)

DistanceSearchController를 삭제하고 HomeController에서 해당 기능을 실행하도록 할 예정입니다.
중복되는 AroundSpotDto와 DistanceInfoResponse를 통합할 예정입니다.

궁금한점

조인컬럼 어노테이션(user_id)를 지우면 왜 컬럼명이 user_user_id가 되는지 궁금합니다

    @ManyToOne(fetch = FetchType.LAZY)
    @JoinColumn(name="user_id")
    private User user;

// ->아래로 바꿨을시에 이슈발생

    @ManyToOne(fetch = FetchType.LAZY)
    private User user;

반면 room은 해당 어노테이션을 해주지 않아도 이슈가 없었습니다.

  @ManyToOne(fetch = FetchType.LAZY)
  @JoinColumn(name="room_id")
    private Room room;

// -> 아래로 변경해도 이슈없음

   @ManyToOne(fetch = FetchType.LAZY)
    private Room room;

Airbnb의 Entity생성 및 매핑을 함

엔티티 설계 및 테스트 DB(RDS)에 더미 데이터 추가

chore: 초기 데이터 입력

Room의 ID를 입력받으면 해당 Room의 상세정보를 반환한다.

repository와 service 패키지 생성

- Address는 ios와 회의 결과 주소가 아닌 위도와 경도로 받아서 의미에 부합하도록 클래스 이름을 Location으로 변경 - Host와 Room의 DTO 생성 - Image 리스트의 경우 copyOf를 써서 불변객체를 반환 - 불필요한 import 삭제

[BE] 상세조회시 DTO를 반환하도록 변경

- gradle.yml 작성 - application-develop.yml의 datasource를 secret key로 관리하도록 수정

- deploy.sh에서 도커허브 레포지토리 변경

- 개발 메인 브랜치인 BE-develop에서 배포용 브랜치 BE-deploy로 수정

- 잘못 올린 파일 삭제

- IOS 기획서에 첫번째 페이지에서 히어로 이미지 영역의 내용 및 이미지 주소를 담당하는 클래스

- 홈화면에서 가까운 여행지 둘러보기를 담당하는 DTO

- Location은 각 행정지의 중심좌표를 의미

- HomeData는 MainEventDto, AroundSpotDto List, ThemeSpotDto List를 포함하고 있음

- api 구분을 주기위해서 Mapping에 room을 넣어줌

[BE] 가까운 여행지 기능을 위한 행정지역 클래스 생성

- 카카오 네비게이션 API를 사용

[BE] 카카오 api를 통해 현재 위치와의 거리와 이동 시간 계산

- 내장 라이브러리를 통해 시간 단위를 초에서 분으로 변경 - 외부 라이브러리를 통해 거리 단위를 미터에서 킬로미터로 변경

[BE] 라이브러리를 통한 시간 및 거리 단위 변환

- 단위 변환을 내장, 외장 라이브러리를 사용하여 해당 역할을 담당하던 DistanceInfo 클래스를 삭제하였습니다.

- district를 enum이 아닌 엔티티로 관리하기위해 수정 - DistanceInfoResponse를 AroundSpotDto로 대체

- HomeController에서 HomeService, DistrictSearchService를 통해 각 Dto를 반환받아 HomeData를 생성하는것으로 수정하였습니다.

wheejuni

수고하셨습니다!
리뷰 남긴 내용 참고해주세요~

wheejuni · 2022-06-09T22:45:21Z

BE/src/main/java/com/team11/airbnb/domain/District.java

+@NoArgsConstructor
+public class District {
+
+    @Id @GeneratedValue(strategy = GenerationType.IDENTITY)


한 줄에 하나의 애노테이션만 씁시다!!

wheejuni · 2022-06-09T22:46:32Z

BE/src/main/java/com/team11/airbnb/domain/Room.java

+
+    @Id @GeneratedValue(strategy = GenerationType.IDENTITY)
+    @JsonIgnore
+    private Long id;


보니까 User 에는 칼럼명을 user_id 로 주셨네요. id 필드에 해당하는 칼럼명을요.
칼럼명을 따라가다 보니까 저쪽에선 user_user_id 가 됐고 여긴 그냥 room_id 가 된 겁니다. ㅎㅎ

wheejuni · 2022-06-09T22:46:50Z

BE/src/main/java/com/team11/airbnb/domain/Room.java

+
+    @Id @GeneratedValue(strategy = GenerationType.IDENTITY)
+    @JsonIgnore
+    private Long id;


https://thorben-janssen.com/naming-strategies-in-hibernate-5/ 이 글도 읽어보셔여

wheejuni · 2022-06-09T22:50:27Z

BE/src/main/java/com/team11/airbnb/domain/User.java

+    private String name;
+
+    @OneToMany(mappedBy = "user")
+    private List<WishList> wishLists = new ArrayList<>();


순차성이 보장되어야 할 필요가 없다면(어차피 쿼리해서 꺼내오는 과정에서 많이 손상되기도 하고요)
Set 을 사용하시는 것도 좋습니다.

wheejuni · 2022-06-09T22:50:41Z

BE/src/main/java/com/team11/airbnb/web/controller/HealthController.java

+    @GetMapping("/health")
+    public String healthCheck(){
+        return "health ok";


actuator 사용은 어떨까요?

wheejuni · 2022-06-09T22:51:20Z

BE/src/main/java/com/team11/airbnb/web/service/DistrictSearchService.java

+
+    private Map<String, String> calculateDistanceAndTime(Position position, Location location)
+        throws JsonProcessingException {
+        RestTemplate restTemplate = new RestTemplate();


항상 정해진 호스트에 호출하는 것이죠?
그렇다면 빈으로 등록해서 주입받아 사용해주세요.

wheejuni · 2022-06-09T22:51:42Z

BE/src/main/java/com/team11/airbnb/web/service/HomeService.java

+        List<ThemeSpotDto> themeSpotDtoList = new ArrayList<>();
+        for (ThemeSpot themeSpot : themeSpotList) {
+            ThemeSpotDto themeSpotDto = new ThemeSpotDto(themeSpot.getImagePath(),
+                themeSpot.getTitle());
+            themeSpotDtoList.add(themeSpotDto);
+        }
+        return themeSpotDtoList;


스트림 한줄로 개선할 수 있을 듯 합니다

- TheamSpot 엔티티를 Dto로 변환하는 과정을 Stream을 이용하여 간소화하였습니다.

- 기존 HealthController를 통해 서버 상태를 확인하던 것을 Actuator로 바꾸었습니다.

workflow 수정

Minzino and others added 30 commits May 28, 2022 21:03

feat: Entity 설계

4fa868d

Airbnb의 Entity생성 및 매핑을 함

chore: 초기 데이터 입력

eb8b8ae

엔티티 설계 및 테스트 DB(RDS)에 더미 데이터 추가

Merge pull request #37 from wnsxor1993/feature-entity

dfd4694

chore: 초기 데이터 입력

Feat: Room 상세 조회기능 구현

5857eff

Room의 ID를 입력받으면 해당 Room의 상세정보를 반환한다.

Chore: GitHub Actions 세팅

af97364

chore: 패키지 분리

5bf5331

repository와 service 패키지 생성

refactor: WishList FetchType LAZY 적용 및 컬럼이름 삭제

6c166ab

Merge pull request #43 from wnsxor1993/feature-search

c3eeafc

[BE] 상세조회시 DTO를 반환하도록 변경

chore: github actions 설정

a96c81a

- gradle.yml 작성 - application-develop.yml의 datasource를 secret key로 관리하도록 수정

fix: 배포용 스크립트 파일 수정

19c051f

- deploy.sh에서 도커허브 레포지토리 변경

fix: GitHUb Actions 트리커 브랜치 수정

38845b7

- 개발 메인 브랜치인 BE-develop에서 배포용 브랜치 BE-deploy로 수정

fix: application.yml 파일 수정

2e70228

- 잘못 올린 파일 삭제

feat: MainEvent(히어로 이미지 영역) 엔티티 생성

059b7b7

- IOS 기획서에 첫번째 페이지에서 히어로 이미지 영역의 내용 및 이미지 주소를 담당하는 클래스

feat: ArroungSpotDto 생성

27cd4a6

- 홈화면에서 가까운 여행지 둘러보기를 담당하는 DTO

feat: 홈화면 테마 여행지(ThemeSpot) 담당 엔티티와 DTO 생성

a51fe14

fix: 변수 RoomType을 Enum 타입으로 변경

115554c

fix: 불필요한 변수 삭제

e5e2e5d

feat: 행정구역 인기여행지를 받아오기 위한 클래스 생성

7cdcd50

- Location은 각 행정지의 중심좌표를 의미

feat: 메인 홈 화면 Controller, Service, Dto 생성

f7020ff

- HomeData는 MainEventDto, AroundSpotDto List, ThemeSpotDto List를 포함하고 있음

fix: Location 생성자를 롬복으로 적용

ab00bfb

feat: ThemeRepository, MainEventRepository 생성

a308437

fix: RoomController의 RequestMapping 수정

50ee00f

- api 구분을 주기위해서 Mapping에 room을 넣어줌

Merge pull request #55 from wnsxor1993/feature-district

2debc26

[BE] 가까운 여행지 기능을 위한 행정지역 클래스 생성

fix: 행정구역의 구분을 주기위해 Dto에 id를 추가

efd722f

feat: Open API를 사용해서 현재위치를 기준으로 여행지와의 거리를 반환

f2529fd

- 카카오 네비게이션 API를 사용

Merge pull request #64 from wnsxor1993/feature-openapi

52533b6

[BE] 카카오 api를 통해 현재 위치와의 거리와 이동 시간 계산

fix: 라이브러리를 통한 시간 및 거리 단위 변환

55d8840

- 내장 라이브러리를 통해 시간 단위를 초에서 분으로 변경 - 외부 라이브러리를 통해 거리 단위를 미터에서 킬로미터로 변경

Merge pull request #67 from wnsxor1993/feature-openapi

9a1480a

[BE] 라이브러리를 통한 시간 및 거리 단위 변환

Update README.md

4c43731

PhilSoGooood added the review-BE Improvements or additions to documentation label Jun 9, 2022

PhilSoGooood requested a review from wheejuni June 9, 2022 03:07

PhilSoGooood assigned Minzino and PhilSoGooood Jun 9, 2022

PhilSoGooood and others added 7 commits June 9, 2022 12:09

refactor: 불필요한 클래스 제거

ca38301

- 단위 변환을 내장, 외장 라이브러리를 사용하여 해당 역할을 담당하던 DistanceInfo 클래스를 삭제하였습니다.

refactor: 불필요한 ENUM클래스와 DTO클래스 삭제

ecf0e0f

- district를 enum이 아닌 엔티티로 관리하기위해 수정 - DistanceInfoResponse를 AroundSpotDto로 대체

refactor: 홈화면에서 현재 위치와 여행지와의 거리 및 이동시간을 반환하도록 수정

4921ea2

- HomeController에서 HomeService, DistrictSearchService를 통해 각 Dto를 반환받아 HomeData를 생성하는것으로 수정하였습니다.

feat: 방 조회 기능 구현

fdd438a

test: 배포 테스트

ce23eeb

test: 배포 테스트

72d7c45

test: 배포 테스트

625a35e

wheejuni requested changes Jun 9, 2022

View reviewed changes

PhilSoGooood added 3 commits June 10, 2022 14:04

refactor: 코드 리뷰 반영(반복문 -> stream)

58f5c58

- TheamSpot 엔티티를 Dto로 변환하는 과정을 Stream을 이용하여 간소화하였습니다.

refactor: 코드 리뷰 반영(actuator 사용)

39e2d95

- 기존 HealthController를 통해 서버 상태를 확인하던 것을 Actuator로 바꾸었습니다.

chore: 환경 설정 변경

71c9750

workflow 수정

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[TEAM-11][BE][민지노&&필] 3주차 PR 입니다 #283

[TEAM-11][BE][민지노&&필] 3주차 PR 입니다 #283

PhilSoGooood commented Jun 9, 2022 •

edited

Loading

wheejuni left a comment

wheejuni Jun 9, 2022

wheejuni Jun 9, 2022

wheejuni Jun 9, 2022

wheejuni Jun 9, 2022

wheejuni Jun 9, 2022

wheejuni Jun 9, 2022

wheejuni Jun 9, 2022

[TEAM-11][BE][민지노&&필] 3주차 PR 입니다 #283

Are you sure you want to change the base?

[TEAM-11][BE][민지노&&필] 3주차 PR 입니다 #283

Conversation

PhilSoGooood commented Jun 9, 2022 • edited Loading

API 명세서

현재 상황

궁금한점

wheejuni left a comment

Choose a reason for hiding this comment

wheejuni Jun 9, 2022

Choose a reason for hiding this comment

wheejuni Jun 9, 2022

Choose a reason for hiding this comment

wheejuni Jun 9, 2022

Choose a reason for hiding this comment

wheejuni Jun 9, 2022

Choose a reason for hiding this comment

wheejuni Jun 9, 2022

Choose a reason for hiding this comment

wheejuni Jun 9, 2022

Choose a reason for hiding this comment

wheejuni Jun 9, 2022

Choose a reason for hiding this comment

PhilSoGooood commented Jun 9, 2022 •

edited

Loading