✨ implement recipe steps retrieval #83
Conversation
| return recipeService.readRecipes(request); | ||
| public List<MainRecipeResponse> readRecipes(@RequestParam int pageNumber, @RequestParam int pageSize) { | ||
| return recipeService.readRecipes(pageNumber, pageSize); | ||
| } |
There was a problem hiding this comment.
혹시 MainRecipeRequest를 살리고 싶으시면
@ModelAttribute 를 고려해봐도 좋을것 같아요!
There was a problem hiding this comment.
@ModelAttribute를 사용하면 param을 모두 기재할 필요없이 하나의 객체처럼 받아올 수도 있네요 👍🏻
상황에 따라 적용하는것도 고려해봐야겠어요.
|
|
||
| private int sequence; | ||
|
|
||
| public long recipeId() { |
There was a problem hiding this comment.
getRecipeId() 는 RecipeStep의 다른 필드의 getter와 구분하기 위한 네이밍 인가요?
There was a problem hiding this comment.
RecipeStepRepository에서 RecipeId로 조회하는 메서드를 작성하기 위해 사용되는 메서드입니다.
recipeId를 가져오는 메서드가 없으면 Repository에서 findAllByRecipeId()로 검색했을 때 에러가 발생했습니다. 에러를 해결하려면 메서드 명에 findAllByRecipe_Id() 와 같이 실제 매핑되는 컬럼명을 적어주어야했어요.
getRecipeId()처럼 일반 getter와 같은 형태로 메서드 명을 작성해도 동작하지 않아서 recipeId()를 명시적으로 표시해두었습니다.
There was a problem hiding this comment.
좋네요! 레벨2에서 slot_startAt 이런식으로 쓴적있는데 그때 알려주시지.. 왜 지금 알려주시죠
|
구현 API
/api/recipes/{id}/steps수정 API
/api/recipes참고사항
(이 부분은 별도 이슈로 관리되어야 했던 것 같은데 리팩토링 하다보니 같이 작업하게 됐습니다. 다음번에는 분리하겠습니다!)