developeryeon/project1

project1's Issues

코드리뷰 - 정보연님

보연님 고생많으셨습니다~
html css js 폴더로 가볍고 깔끔하게 잘 나눠주셨습니다. 이런 습관은 계속 유지해주시면 좋을 것 같아요.

project1/css/style.css

Line 85 in d7c3c52

#movies_card {

css grid 로 잘 활용해주셨네요!

project1/src/config.js

Line 1 in d7c3c52

const config = {

project1/src/script.js

Line 10 in d7c3c52

Authorization: `Bearer eyJhbGciOiJIUzI1NiJ9.eyJhdWQiOiIyNDI2MmNhZTU2NTg4OTIyZmJiYmZmYjQ3NzIyOTI2ZiIsInN1YiI6IjY1OGUyOWY5ZDc1YmQ2NDE0ZTcyOTczMiIsInNjb3BlcyI6WyJhcGlfcmVhZCJdLCJ2ZXJzaW9uIjoxfQ.R7gFfyBqP0L6j3F1MaKwImsNmvCin8Apwoh00iKAlOo`

config.js 에 config key 등등만 빼서 해주신 점은 너무 좋아요! 다음번에는 .gitignore 같은걸로 아예 깃에서도 안보이게끔 해보시면 apiKey 의 보안을 좀 더 신경써서 할 수도 있을 것 같습니다.

그리고 아래 코드도 보안에 관련된 옵션들이라서 config.js 로 옮겼어도 좋았을 것 같아요.

project1/src/script.js

Line 82 in d7c3c52

window.addEventListener("load", () => {

Load 이벤트에 사용하고 싶으신 함수를 잘 연결연결 해서 작성해주셨어요. 너무 잘하셨는데요?

project1/src/script.js

Line 89 in d7c3c52

searchBtn.addEventListener("click", e => {

이 부분은 동작에 오류가 있을 것 같다는 생각이 들어요. 물론 자바스크립트에서 바로 document Id 에 접근할 수 있지만 오해의 소지가 있을 수 있습니다!
searchBtn element 를 함수 위에서 불러와서 addEventLisnter 를 붙이는게 좋을 것 같습니다.

project1/src/script.js

Line 86 in d7c3c52

searchBox.focus();

위 피드백과 일맥상통한 부분인데, 이 부분도 명확히 searchBox 가 상단에 선언되어 있지 않아보여요. 변수가 어떤 변수를 참조하고 있는지 좀 더 잘 모일 수 있도록 상단에 바로 선언해도 좋고 만약 전역변수로 사용하시고 싶다면 사용하는 함수 가까이에 선언되어 있어도 좋을 것 같습니다!

project1/src/script.js

Line 39 in d7c3c52

let movieCard = document.getElementById("movies_card");

여기 라인에서는 잘 선언해서 사용해주시고 계신걸 보니 이런식으로 하단의 함수에서 혹은 전역에서 사용해주시면 좋을 것 같습니다 :)

질문사항들

404 not Found 나는 부분이 어떤건지 잘 이해가 안되어서! 이거 보시면 저 찾아오시면 같이 찾아봐요 :)

project1/src/script.js

Line 69 in d7c3c52

cardTitle.addEventListener("click", () => {

title 을 클릭했을때 overview 가 나오는 부분을 toggle 이라는 함수로 잘 적어주셨는데요?!
잘해주셨습니다 고생하셨어요 :)