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

Carrot/#2 #10

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

Carrot/#2 #10

wants to merge 5 commits into from

Conversation

sansan20535
Copy link
Contributor

이 주의 과제 ❗️

  • 등록된 상품들에 대한 지역별 List GET
  • 한 상품에 '좋아요 추가' 기능

API 명세서

https://pie-porch-1c1.notion.site/2-7c540b18e17c4ce29705f31b2132983f?pvs=4

요구사항 분석 ❗️

지역별 List GET

@GetMapping("/list")
public ResponseEntity<SuccessResponse> findSellListByPlace(
@RequestParam("place") Place place
) {
return ResponseEntity.status(HttpStatus.FOUND)
.body(SuccessResponse.of(SuccessMesasge.SELLLIST_FIND_BY_PLACE_SUCCESS, sellService.findSellListByPlace(place)));
}

장소를 파라미터로 받았습니다.

public interface SellRepository extends JpaRepository<Sell, Long> {
List<Sell> findSellByPlace(Place place);
}

public List<SellFindDto> findSellListByPlace(Place place) {
List<Sell> sellList = sellRepository.findSellByPlace(place);
List<SellFindDto> sellFindDtoList = sellList.stream().map(SellFindDto::of).toList();
return sellFindDtoList;
}

findByPlace를 이용하여 입력받은 장소를 포함하는 게시글을 List로 반환하였습니다.

'좋아요 추가' 기능

@Id
@GeneratedValue(strategy = GenerationType.IDENTITY)
private Long id;
@ManyToOne(fetch = FetchType.LAZY)
private Customer customer;
@ManyToOne(fetch = FetchType.LAZY)
private Sell sell;

'좋아요' 엔티티에 좋아요를 누른 사용자의 아이디와 누른 판매 글의 아이디를 포함하도록 하였습니다.

구현 고민 사항 ❗️

  • 변수명에 type을 포함하는 것은 별로 좋지 못하다고 들었습니다...(ex : List numberList) 이런 경우 좋은 변수 명이 있는지가 궁금합니다.

- 고객 등록 API 추가
- 판매 정보에 판매자의 별명과 판매여부 추가
- 판매 정보 중 판매자의 별명은 판매자의 아이디를 이용하여 판매자의 정보를 찾고 별명 저장
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.

[CloneCodingAssignment/#2] 2차 클론 코딩 과제
1 participant