FE_PJT4-2 부스트코스 리뷰

• FE_PJT4-2 부스트코스 PASS 리뷰(운이 좋게도? 한 번에 PASS가 떴다.)
  • 코드 리뷰
  • 내가 질문했던 것(요약)

FE_PJT4-2 부스트코스 PASS 리뷰(운이 좋게도? 한 번에 PASS가 떴다.)

코드 리뷰

• 범용적으로 쓰이는 만든 함수들(addClass, removeClass)을 자바스크립트 파일이 아닌 Util 객체의 메소드로 만들어서 사용하자.
• 콤마(,)를 한 후 개행하지 말고 개행하고 다음 줄 맨 앞에 콤마를 하자. 그래야 라인이 길어져도 앞 줄에 콤마가 보이기 때문에 실수하지 않는다.
• 객체 리터럴로 선언해줄 때 프로토타입 코드들은 대문자로 하고 메소드들은 소문자로 하자.
• img의 alt 속성의 사용은 웹 접근성과 관련이 있다. 눈이 안 보이는 사용자는 스크린 리더를 통해 alt속성에 작성한 텍스트가 읽히기 때문. 필수로 있어야 한다.

내가 질문했던 것(요약)

• 리뷰어님의 답변 : 자바스크립트 native로 URL의 파라미터를 각각 받아오는 방법은 없다. 내 생각 : 가져오려면 직접 함수를 만들어서 가져오거나 아니면 JSP에서 javascript태그 안에서 ajax함수 생성 후 EL태그로 가져와야될 것 같다.
• CSS animation으로 클릭했을 때 돌아가는 슬라이드를 구현하려면 keyFrame에 0%, 100%를 정의한 후 해당 값을 스크립트로 제어하면 된다.
• 객체 내 메서들간 공유가 필요한 경우에 객체의 필드로 사용하고 객체 내 메소드 안에서만 사용된다면 this가 사용되는 것. 메소드안에 함수가 있어 this가 중첩된다면 해당 경우네는 this키워드가 아니라 메소드의 파라미터로 값을 받아 이용할 수 있다.
• 함수를 선언하는 위치와 객체 리터럴을 선언하는 위치는 상관없다. 다만 함수가 나왔다가 객체가 나왔다가 하는 방식으로 구현하면 코드리딩에 어려움이 있기 때문에 최대한 가독성이 좋도록 관련성있는 것들과 묶어놓자.