AYAAKIASA / movie

0 stars 0 forks source link

feedback #1

Open ggomma opened 6 months ago

ggomma commented 6 months ago

구현 사항 확인

필수

선택

피드백

총평

개선점

파일 분리

현재 html 파일 안에 css, js 코드가 모두 합쳐져 있습니다. 이 경우 서비스가 복잡해지면 코드를 이해하기 어려워집니다. 이를 방지하기 위해서는 css, js파일을 별도로 분리하여 html 파일에서 불러오는 것이 좋습니다.

Readme.md 파일 추가

코드를 다른 사람들과 공유할 때에는 코드에 대한 설명을 적어주는 것이 좋습니다.

추가 요소

아래 추가 개발을 권장하는 내용들입니다.

AYAAKIASA commented 6 months ago

알겠습니다. 감사합니다.

2024년 4월 29일 (월) 오후 12:44, ggomma @.***>님이 작성:

구현 사항 확인 필수

  • jQuery 라이브러리 사용없이 순수 바닐라 자바스크립트 사용하기
  • TMDB 오픈 API를 이용하여 인기영화 데이터 가져오기
  • 영화정보 카드 리스트 UI 구현
    • 카드 형태로 데이터 보여주기
    • 제목, 내용 요약, 포스터, 평점 정보 포함
    • 카드 클릭 시 영화 ID Alert 창 띄우기
  • 영화 검색 UI 구현
  • Javascript 문법 요소 이용
    • const와 let만을 이용한 변수 선언 필수
    • 화살표 함수 1개 이상 사용
    • 배열 메소드 2개 이상 사용
    • DOM 제어하기 메서드 2개 이상 사용

선택

  • CSS
    • flex 사용하기
    • grid 사용하기
  • 웹사이트 랜딩 또는 새로고침 후 검색 입력란에 커서 자동 위치시키기
  • 대소문자 관계없이 검색 가능하게 하기
  • 키보드 enter키를 입력해도 검색버튼 클릭한 것과 동일하게 검색 실행시키기

피드백 총평

  • 어렵다고 하셨지만 실제로는 매우 잘 구현하셨습니다.
  • 코드를 잘 작성하였습니다.
  • 현재 학습중인 내용에 대한 이해도가 매우 높은 것으로 보입니다.
  • search 기능을 구현하기 위해 API의 query 기능을 찾아 잘 적용하였습니다.
  • 필요한 코드만 깔끔하게 잘 작성하였습니다.

개선점 파일 분리

현재 html 파일 안에 css, js 코드가 모두 합쳐져 있습니다. 이 경우 서비스가 복잡해지면 코드를 이해하기 어려워집니다. 이를 방지하기 위해서는 css, js파일을 별도로 분리하여 html 파일에서 불러오는 것이 좋습니다. 추가 요소

아래 추가 개발을 권장하는 내용들입니다.

  • 영화 정보 카드에 평점이 노출되게 해주세요.
  • 배열 메소드를 최소 2개 이상 사용해보세요.
  • 웹사이트 랜딩 또는 새로고침 후 검색 입력란에 커서가 자동으로 위치되게 해보세요.
  • css, js를 html 파일 하나에 모두 작성하는 것보다 css, js, html 파일을 분리해보세요.

— Reply to this email directly, view it on GitHub https://github.com/AYAAKIASA/movie/issues/1, or unsubscribe https://github.com/notifications/unsubscribe-auth/BH2OIL2EAUGRXKYCOZXY54LY7W6Y5AVCNFSM6AAAAABG5SENKWVHI2DSMVQWIX3LMV43ASLTON2WKOZSGI3DQMBVHE3DMOA . You are receiving this because you are subscribed to this thread.Message ID: @.***>