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 / 곽민준 #43

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

Conversation

mlnwns
Copy link
Collaborator

@mlnwns mlnwns commented May 8, 2024

1. 구현 모습

image

2. 해결 과정

useState를 사용해 두 가지 상태를 정해주었습니다. (화면에 표시될 단어들의 배열(words), 게임이 끝났는지 여부(isGameOver))
useEffect 훅을 사용하여 게임이 진행 중일 때 2초마다 fruits 배열에서 무작위로 단어를 선택하여 words 배열에 추가하고, 만약 words 배열의 길이가 10이 넘어가면 "게임 종료!"라는 메시지를 표시하도록 했습니다.


3. To 리뷰어에게

이번 미션에서는 CSS가 크게 들어가지 않는다고 판단해서, 스타일드 컴포넌트를 사용하지 않고 기본적인 CSS만을 사용해보았습니다. CSS코드가 길지 않아서 index.css에 몰아서 CSS 코드를 작성했는데, CSS 코드가 늘어난다면 index.css에서만 관리하는것은 관련 코드를 찾기도 어렵고, 코드가 너무 길어지는 문제가 있기에 CSS파일을 나누어서 사용해야겠다고 생각했습니다. 또한 index.css는 전역적으로 관리되는 코드만 다루는 곳으로 코드를 작성하고 다시 생각해보니 위치를 잘 못 잡았다는 생각이 드네요!
추가적으로 잘못된 부분이나 리팩토링하면 좋을 부분을 알려주신다면 감사하겠습니다! 바쁘실텐데 시간 내어 살펴봐주셔서 감사합니다 :)


@mlnwns mlnwns added the mission 미션 입니다! label May 8, 2024
@mlnwns mlnwns requested a review from GiPyoo May 8, 2024 01:33
@mlnwns mlnwns self-assigned this May 8, 2024
Copy link

vercel bot commented May 8, 2024

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

Name Status Preview Comments Updated (UTC)
donut-study ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 8, 2024 1:33am

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