Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MISSION6 / 강하은 #47

Open
wants to merge 9 commits into
base: main
Choose a base branch
from

Conversation

kanghaeun
Copy link
Collaborator

1. 구현 모습

image


2. 해결 과정

단어들이 보이는 화면인 TypingArea와 값을 입력할 수 있는 TypingInput으로 컴포넌트를 분류하였습니다.
TypingArea에서 3초마다 callBackWordList를 호출하였으며 wordList가 10개가 초과될 시 game over가 되도록 로직을 짰습니다.
TypingInput에서는 값을 입력시 filter함수를 이용하여 해당 값을 배열에서 제거해 주었는데 중복 단어가 존재할 경우 동일한 값 중 하나만 삭제되도록 하는 로직도 추가해 주었습니다.

3. To 리뷰어에게

상태 변경 함수에 대해 요즘 고민이 많습니다. 이 코드에서처럼 상위컴포넌트에서 setWordList함수를 하위 컴포넌트에 직접 넘겨주어 상태를 변경하는 코드와 상위 컴포넌트에서 상태 변경 로직을 관리하고 그 함수를 하위컴포넌트에다 props로 보내주는 코드 중 어떤 것을 더 선호하시나요? 둘의 차이가 궁금합니다!
바쁘신 와중 도움 주셔서 감사합니다!

@kanghaeun kanghaeun added the mission 미션 입니다! label May 18, 2024
@kanghaeun kanghaeun self-assigned this May 18, 2024
Copy link

vercel bot commented May 18, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
donut-study ❌ Failed (Inspect) May 18, 2024 7:04pm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
mission 미션 입니다!
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant