Skip to content

healtheloper/fe-kakaopage

 
 

Folders and files

NameName
Last commit message
Last commit date

Latest commit

 

History

9 Commits
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 

Repository files navigation

fe-kakaopage

3주차 - 2번째 PR

🧐 어떤 생각으로?

  • Contents Box 렌더 마무리
  • Carousel 버그 수정
  • 슬라이더 버튼 추가
  • v2(카카오페이지 api response를 긁어온) api 로 데이터 구조 바꿔보기
  • 만든 Carousel 활용해서 추천이벤트, 서브배너에서 사용하기

✅ PR 리뷰에서 개선한 점

  • Dony 와 Inu 의 라이브 리뷰를 보고서 폴더구조를 대등한 역할을 하는 것들로 묶어야겠다는 생각이 들어 제 나름의 생각으로 폴더구조를 짜보았는데 제가 파일을 찾을땐 확실히 더 편해졌다는 느낌을 받았습니다, 그런데 이게 다른사람의 눈에 어떨지가 궁금해졌습니다.

  • 되도록 매직넘버를 사용하지 않도록 변수명에 특정 숫자를 넣어서 사용하는 식으로 사용해보았습니다.

  • 긴 콜백함수는 줄여보도록 노력했습니다.

궁금한점?

  • prototype 패턴으로 메서드를 만드는 것과 생성자패턴의 함수로 만드는 것을 말씀해주셨는데 객체를 여러개 생성하는게 아니면 함수로 구현해도 되긴 하다 라는 것이 그렇게 해야한다 는 아닌 것으로 이해를 했는데요, 저는 코드의 통일성을 위해 prototype 패턴으로 메서드를 만들어 사용하는게 더 좋다고 생각이 들었습니다. 이 생각을 계속 가져가도 될까요?

  • 구조분해할당을 하기위한 요소가 10개 이상이 넘어가도, 사용하는게 맞을지 라는 생각이 들었습니다.

const { title, image, rank, rating, age_grade, badge, read_count, waitfree } =
  webtoon;

Releases

No releases published

Packages

No packages published

Languages

  • JavaScript 84.5%
  • CSS 14.9%
  • HTML 0.6%