팀 소개 페이지 코드리뷰

마라탕천재 ㅣ 2024. 7. 24. 13:47

👉오늘 리뷰한 코드 보러가기

 

ourTeamPage/personalPage.html at develop · Sparta-IEEE/ourTeamPage

Team introduction web page. Contribute to Sparta-IEEE/ourTeamPage development by creating an account on GitHub.

github.com

 

1. 스크립트의 위치

HTML 문서의 body 끝나기 직전에 위치시키는 관행적이다.

<head>에 배치한 경우

  • 페이지 렌더링 전에 실행해야 하는 중요한 스크립트의 경우 이 곳에 배치함.
  • 이 경우 'defer' 또는 'async' 속성을 사용하여 페이지 로딩을 차단하지 않도록 하는 것이 좋음.

</body> 직전에 배치한 경우

  • 페이지 로딩 속도
    • HTML 콘텐츠가 먼저 로드되고 렌더링되므로 사용자가 더 빨리 페이지 내용을 볼 수 있음.
    • 스크립트 다운로드와 실행이 페이지 렌더링을 차단하지 않음.
  • DOM 접근
    • 스크립트가 실행될 때 이미 모든 DOM 요소가 로드되어 있어, JavaScript에서 요소에 쉽게 접근 가능함.
  • 의존성
    • 외부 라이브러리나 다른 스크립트에 의존하는 경우 먼저 로드되도록 할 수 있음.

 

2. 예외처리 부재

예외처리를 하는 것이 코드의 안정성, 디버깅, 그리고 사용자 경험 측면에서 도움이 될 것 같다.

  • 대부분의 비동기 함수(getMemberData, loadProjectCards)에 try-catch 블록이 없다.
  • Firebase 작업에 대한 예외 처리: Firebase 관련 작업(CRUD)에 대한 구체적인 에러 처리가 부족함.
  • 사용자 입력에 대한 유효성 검사가 부족하다. 보안성 강화를 위해 추가하면 좋을 것 같다.

 

3.  깃허브 업로드 시 보안강화

퍼블릭 리포지토리임에도 불구하고 파이어베이스의 키 값이 그대로 노출되어있음.

git-secret를 이용해 볼 것! 이라는 튜터님의 조언...