Skip to content

Conversation

@codedbyminjae
Copy link

📌 몇 주차 워크북인가요?

  • Week01

✨ 이번 주에 작업한 내용

  • Figma에서 제공된 단일 감정 선택 UI 화면 구현
  • XML 기반의 ConstraintLayout 사용
  • 세 가지 방식으로 레이아웃 정렬 실습
    • 고정 dp로 정렬
    • Guideline을 활용한 정렬
    • layout_constraint 속성을 활용한 정렬
  • Vector Asset으로 SVG 이미지 등록 (happy, angry 등 감정 이미지)
  • ViewBinding 적용 및 MainActivity → EmotionActivity로 런처 교체

🙋 리뷰 요청/확인 받고 싶은 부분

  • 에뮬레이터 실행 시 앱이 자동 실행되도록 LauncherActivity를 지정해두었는데, 이 설정이 올바른지 혹은 수동 실행 방식이 더 적절한지도 궁금합니다.
  • 감정 이미지에 대해 Vector Asset으로 등록 시, 이름 설정 및 구조 관리 방식에 대해 피드백 주시면 감사하겠습니다.
  • 문자열(String)과 색상(Color) 리소스를 res/values에서 분리해 관리했는데, 이렇게 하는 방식이 올바른지, 더 나은 구조가 있는지 조언 부탁드립니다.

✅ 체크리스트

  • week01/ 폴더 안에 과제 정리 완료
  • PR 생성 시 base = 조직 내 본인 브랜치, compare = 내 Fork main 브랜치로 설정했는지 확인
  • PR 제목에 [Week01] 제이/김민재 미션 제출 규칙 맞게 작성

@eldeoddt
Copy link

1주차 피드백 완료 ✅
수고하셨습니다 🙂 👍

🙋 리뷰 요청 / 확인 받고 싶은 부분

  • LauncherActivity 지정
    학습시에는 상관없는데, 실무에서는 Splash/Main에서 Intent로 분기하는 구조 권장

  • Vector Asset 관리
    ic_emotion_* 식으로 prefix 통일
    감정 아이콘은 /drawable/emotion 폴더로 정리 추천

  • 리소스 분리 (String/Color)
    현재 분리 방식 좋아요 기능 단위(colors_emotion.xml 등)로 세분화 시 더 효율적임

<EmotionActivity>

👍

  • ViewBinding 적용 정확
  • 감정별 클릭 리스너 및 색상/문자열 리소스 활용 적절

💙 개선 제안

  • 중복 클릭 리스너 구조 단순화 (Map 등으로 묶어 처리하면 유지보수성 향상)
  • 색상 리소스명 prefix 통일 (emotion_happy 등) 추천

<MainActivity>

👍

  • ViewBinding 활용 및 기본 레이아웃 구조 명확

<리소스 관리>

👍

  • strings.xml, colors.xml 분리 관리 구조 적절
  • Vector Asset 등록 및 네이밍 명확

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants