Skip to content

Conversation

@shail1027
Copy link
Member

📌 몇 주차 워크북인가요?

  • 예: Week01

✨ 이번 주에 작업한 내용

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

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

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

✅ 체크리스트

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

@eldeoddt
Copy link

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

<MainActivity>

👍

  • 이미지 클릭 시 TextView 색상 변경 및 Toast 출력 로직 명확
  • 공통 클릭 이벤트 setClick() 함수로 중복 제거한 점 좋음

💙 개선 제안

  • Color.parseColor("red") 하드코딩보다 colors.xml에 정의된 리소스 사용 권장
  • 감정별 데이터(이름, 메시지, 색상)를 리스트나 Enum으로 관리하면 확장성 좋음
  • textViews.forEach { it.setTextColor(Color.BLACK) } 부분도 공통 색상 상수로 관리하면 가독성 개선

<레이아웃>

👍

  • LinearLayout을 활용해 각 감정 아이콘/텍스트 그룹화 구조 명확
  • 가로/세로 비율에 맞춘 Constraint 구성 시도

💙 개선 제안

  • 문자열 리터럴("오늘 하루는 어땠나요?") -> strings.xml로 분리 필요

<전반적>

👍

  • 전체 코드 전반적으로 초기 세팅 구조 깔끔

💙 개선 제안

  • 색상, 문자열, 이미지 이름에 prefix 통일 (emotion_happy, emotion_angry) 권장

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