[2주차/썽아] 워크북 제출합니다.

[sungahbak]

✅ 워크북 체크리스트

• 모든 핵심 키워드 정리를 마쳤나요?
• 핵심 키워드에 대해 완벽히 이해하셨나요?
• 이론 학습 이후 직접 실습을 해보는 시간을 가졌나요?
• 미션을 수행하셨나요?
• 미션을 기록하셨나요?

✅ 컨벤션 체크리스트

• 디렉토리 구조 컨벤션을 잘 지켰나요?
• pr 제목을 컨벤션에 맞게 작성하였나요?
• pr에 해당되는 이슈를 연결하였나요?(중요)
• 적절한 라벨을 설정하였나요?
• 파트장에게 code review를 요청하기 위해 reviewer를 등록하였나요?
• 닉네임/main 브랜치의 최신 상태를 반영하고 있는지 확인했나요?(매우 중요!)

📌 주안점

[jiyoung02]

LGTM! 수고하셨습니다!!
아래는 더 나은 코드를 위해 개선하면 좋을 부분이에요.

1. 텍스트 단위는 dp가 아닌 sp 사용해주세요
   텍스트 크기는 화면 해상도뿐 아니라 사용자 접근성 설정(글자 크기 확대/축소)에도 대응해야 합니다. 따라서 dp 대신 sp를 사용하는것을 권장합니다.

2. 탭 레이아웃에 ViewPager 연결해주세요
   탭 레이아웃을 미리 적용한 건 아주 좋습니다!! 다만 탭 아래를 ScrollView로 두면 화면 전환이 불편해집니다. 보통은 ViewPager2와 연결해서 스와이프 전환이 가능하도록 구현하는 게 표준적인 구조이니 적용해주세요:)

3. 메인 액티비티에서 바텀네비에 관한 코드가 없어요!
   현재 메인 액티비티에서 바텀네비에 관한 코드가 없어서 화면이 전혀 표시되지 않고 있습니다ㅠㅠ 다음 주차까지 반드시 바텀네비 코드를 추가해주세요
   또, 미션 제출 전에는 꼭 빌드해서 의도한 대로 화면이 동작하는지 확인하는 습관을 들여주세요!!

4. songActivity.xml에서 레이아웃이 겹치는 영역이 존재해요.
   충분히 큰 화면이면 문제가 없지만, 작은 화면 기기에서는 비율이 깨질 수 있습니다. ConstraintLayout 종속성과 적절한 margin을 활용해 겹치지 않도록 조정해 주세요.

5. 색상은 하드코딩된 값을 사용하지말고, 별도의 스타일 파일로 분리한 후 사용해주세요
   XML에 직접 색상값을 넣으면 가독성이 떨어지고 유지보수도 어렵습니다. colors.xml이나 스타일 리소스에 색상을 정의한 뒤 사용하는 방법이 권장됩니다.