[3주차] 킹주영 월드컵 #4
Conversation
|
오늘도 할할놀놀을 몸소 실천 중인 웹파트원 ! 화이팅 :) |
| }; | ||
|
|
||
| const getRoundText = () => { | ||
| if (round <= 4) return round === 4 ? '준결승' : '결승'; |
There was a problem hiding this comment.
1강이랑 3강이 없는게 이 세상 사람이면 다 아는 사실이긴 한데 그래도 혹시 모르니까 switch case문으로 의미를 분명하게 해주시는 건 어떤가요?
2강이 결승인걸 모르는 사람이 세상에 있을 수 있는 것이니까요..
There was a problem hiding this comment.
저도 처음에 round === 4 인걸 결승 조건으로 했는데, 혹시 나중에 게임 라운드 수가 많아지거나 적어졌을 때 바꿔줘야 하는게 걸리더라구요! 그래서 나중에 확장성을 고려한다면, 라운드 수가 많든 적든 공통적으로 적용시킬 수 있는 걸 파이널 조건으로 해주면 좋을 것 같습니당! 저도 이부분이 매우매우 고민되었어요 ㅠ
| const choice = (direction: Direction) => { | ||
| if (!currentFighter) return; | ||
| if (round === fighterList.length) { | ||
| setWinnerList(() => [direction === 'LEFT' ? currentFighter.left : currentFighter.right]); |
There was a problem hiding this comment.
const currentWinner = direction === 'LEFT' ? currentFighter.left : currentFighter.right;요거 하나 선언하고 가면 좋을 것 같아용
아님 아예 getCurrentWinner함수를 선언해도 좋을 것 같아용
| if (currentFighter && fighterList.length === 0) { | ||
| goNextRound(); | ||
| } else { | ||
| setCurrentFighter({ left: fighterList[0], right: fighterList[1] }); |
There was a problem hiding this comment.
계속 배열 맨 앞에서 뽑는 것이죠?
배열 양끝 인덱스가 아닌 친구들은 굳이 접근할 필요가 없는 것 같네용?
스택 자료구조 써서 unshift같은 함수로 담는 것은 어때용?
| @@ -0,0 +1,8 @@ | |||
| { | |||
There was a problem hiding this comment.
prettier을 지정해준건가요? 오옹 좋습니다 👍
저도 나중에 시도해봐야겠어요!
| @@ -0,0 +1,15 @@ | |||
| import { Route, Routes } from 'react-router-dom'; | |||
| @@ -0,0 +1,12 @@ | |||
| import { default as 방청주영 } from '../assets/juyeong1.jpg'; | |||
There was a problem hiding this comment.
궁금한게 2가지 있습니다! :)
import juyeong1 from '../assets/juyeong1.jpg'; 이렇게 하지 않은 이유는 방청주영이라는 변수명으로 가져오기 위해서 인가요??
{ default as 방청주영 } 이건 ts 라서 이렇게 쓸수 있는 건가요?
| }; | ||
|
|
||
| const getRoundText = () => { | ||
| if (round <= 4) return round === 4 ? '준결승' : '결승'; |
There was a problem hiding this comment.
저도 처음에 round === 4 인걸 결승 조건으로 했는데, 혹시 나중에 게임 라운드 수가 많아지거나 적어졌을 때 바꿔줘야 하는게 걸리더라구요! 그래서 나중에 확장성을 고려한다면, 라운드 수가 많든 적든 공통적으로 적용시킬 수 있는 걸 파이널 조건으로 해주면 좋을 것 같습니당! 저도 이부분이 매우매우 고민되었어요 ㅠ
| } | ||
| }; | ||
|
|
||
| useEffect(() => { |
There was a problem hiding this comment.
useEffect()는 컴포넌트내 여러개 선언할 수 있군요!
useEffect() 하나만 써야하는줄 알고 굉장히 애를 먹었었는데 몰랐던 점 알아갑니다!
| ); | ||
| } | ||
|
|
||
| const StGame = styled.div` |
There was a problem hiding this comment.
styled-components를 Header, Img 등등.. 이런식으로 세분화하지 않고
으로 감싸서 & 로 해결하신 이유가 궁금합니다!!
✨ 구현 기능 명세
기본 과제 모두!
+ 신경 쓴 것
🎁 PR Point
벼락치기로 갈겼는데 특히 스타일드컴포넌트의 가독성이 엉망입니다,,
😭 어려웠던 점
상태 관리나 로직 짜는게 복잡했습니다! 꼬옥 리팩토링 해보고 싶네욘
😎 구현 결과물