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

[ 7주차 과제 ] 카드게임 리팩토링 #10

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

Conversation

eastlaw80
Copy link

✨ 구현 기능 명세

1️⃣ jsx → tsx 로 모두 변환
2️⃣ recoil로 카드 게임에서 필요한 상태를 관리해


🌼 PR Point

게임 기능중 props로 state를 주고 받았던 nowScore와 curLevel을 recoil을 이용하여 props로 주고 받지 않게 관리.
그외에 모든 변수 타입 지정 및 props 주고 받을 때 Interface 설정 함.


🥺 소요 시간, 어려웠던 점

  • 8h
  • 처음에 변환하자마자 맞게된 오류에서 3시간을 넘게 씀,,
    스크린샷 2023-06-09 오후 5.38.39.png
  • 결국 단톡방에 물어본 결과 tsconfig에서 jsx를 react로 넣어준뒤에 모든 파일에 react import하는 식으로 문제 해결했다. 근데 왜 react-jsx가 안먹히는 지는 아직도 의문
  • 또한 아아래 와 같이 조건부로 받아오는 녀석들은 타입이 제대로 지정이 안되는 것인지, 작동은 잘 되지만 코드에선 빨간줄과 함께 할당 할 수 없다는 이야기가 나온다.. 이부분은 아예 데이터를 주고 받는 것 자체를 바꿔야 할 것 같은데 잘 모르겠다...
    스크린샷 2023-06-10 오후 11 55 38

🌈 구현 결과물

2023-06-10.11.57.38.mov

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant