Skip to content

Conversation

@Yudongok
Copy link

📌 몇 주차 워크북인가요?

  • 예: Week01

✨ 이번 주에 작업한 내용

  • 구현/수정한 기능 요약
    • 예: 회원가입 API 구현
    • 예: UI 레이아웃 정리

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

  • 예: 비밀번호 유효성 검사 로직이 적절한지 확인 부탁드립니다.
  • 예: Controller 단의 코드 구조 피드백 원합니다.

✅ 체크리스트

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

@eldeoddt
Copy link

eldeoddt commented Nov 11, 2025

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

<MainActivity>

👍

  • 공통 클릭 이벤트를 함수로 분리
  • 감정별 TextView 리스트로 관리, 전체 색상 초기화 로직

💙 개선 제안

  • Color.parseColor("red") 대신 colors.xml에 색상 정의 후 리소스 참조 권장
  • 감정별 데이터(이름, 메시지, 색상)를 Enum 또는 데이터 클래스 형태로 관리하면 확장성 향상

<레이아웃>

👍

  • LinearLayout으로 감정별 아이콘 및 텍스트 세트 구성 명확
  • ConstraintLayout으로 상위 구조 잘 정리됨

💙 개선 제안

  • TextView의 문자열 리터럴은 strings.xml로 분리 필요
  • drawable 리소스 이름에 접두사 통일(emotion_happy, emotion_angry) 시 관리 용이함

<전반>

👍

  • 전체적으로 코드 구조 XML 정렬이 깔끔하고 명확
  • 클릭 이벤트, 토스트, 색상 변경 로직 모두 구현 완료

💙 개선 제안

  • UI/로직 분리 고려해서 MainActivity 내부의 클릭 처리 로직을 별도 클래스 or 함수로 관리하면 구조적 완성도 높아짐

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