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

[sungahbak]

✅ 워크북 체크리스트

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

✅ 컨벤션 체크리스트

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

📌 주안점

[jiyoung02]

LGTM! 수고하셨습니다!!
화면 깔끔하게 너무 잘 구현해주셨어요. 몇가지 개선사항이 있어서 말씀드리겠습니다.

1. 화난우표 글자 색 변경 확인
   마지막 화난우표를 선택한 이후 다른 우표를 클릭했을 때, 글자 색이 검정색으로 돌아오지 않고있습니다.
   직접 원인을 찾아보고 수정해서 트러블 슈팅일지를 작성해주세요!!

2. LinearLayout보다 ConstraintLayout을 사용하면 더 좋을 것 같습니다!
   LinearLayout은 기본적으로 한 방향(가로/세로)만 배치할 수 있고, 교차 배치가 필요하면 RelativeLayout이나 중첩이 필요합니다.
   ConstraintLayout은 좌우/상하/중앙 정렬, 기준선 맞춤 등 복잡한 제약 조건을 한 번에 처리할 수 있어 훨씬 유연합니다.
   이외에도 중첩레이아웃 최소화, 반응형 UI 구현등 여러 장점이 있어, 공식문서에서도 권장하는 방법이니 지금부터 연습하시면 좋을것 같습니다!

3. xml파일과 코틀린 파일의 이름을 통일시켜주세요
   현재 MainActivity에서 Week1Bindging을 호출해서 사용하고 있는데, 간단한 프로젝트에서는 괜찮지만, 프로젝트 규모가 커지면 혼동이 발생할 확률이 높습니다. 협업의 편의성과 유지보수를 위해 두 파일의 이름을 통일해주는 것이 좋습니다.

4. 안드로이드스튜디오에서 제공하는 기본 상단바를 제거해주세요.
   안드로이드에서 기본으로 제공되는 상단바(ActionBar)는 디자인 컨셉과 맞지 않으니 제거하는 걸 권장합니다. (화면상으로도 뒤로가기 버튼부분을 가리고 있어요 ㅠㅠ )
   Theme.MaterialComponents.DayNight.NoActionBar 같은 테마를 적용해서 쉽게 제거할 수 있으니 참고해주세요.

5. xml네이밍 컨벤션에 대해서 공부해주세요!
   xml네이밍도 공식규칙은 아니지만, 어느정도 공식화 되어있는 컨벤션이 존재합니다.
   다음 미션 전에 학습하고 적용해보면 좋겠습니다!