Skip to content

Conversation

@Kr-Verified
Copy link

No description provided.

@winshine0326
Copy link
Member

winshine0326 commented Jul 31, 2025

feat: 새로운 기능 추가
fix: 버그 수정
docs: 문서 작성 및 수정
style: 코드 포매팅 및 스타일 변경
refactor: 코드 리팩토링
chore: 기타 변경사항

다음부터는 이 형식에 맞춰서 커밋 제목 작성해주세요.
ex) feat :: coin 색상 랜덤 변동

커밋은 되도록이면 자주하는 것이 좋습니다.

@winshine0326
Copy link
Member

[2일차] {본인이름} 과제 제출합니다.

그리고 이 형식에 맞게 PR 제목 수정해주세요.
예시)
[2일차] 신은총 과제 제출합니다.

@winshine0326
Copy link
Member

App 컴포넌트를 제외하고 props를 주고받는 컴포넌트 2개 이상 사용

해당 요구사항을 맞추지 않았습니다. 요구사항 꼼꼼히 확인해주세요..

@winshine0326
Copy link
Member

winshine0326 commented Jul 31, 2025

  useEffect(() => {
    if (text.current) text.current.textContent = coin;
  }, [coin]);

해당 부분은 useEffect를 쓰기 위해 억지로 사용한 느낌이 듭니다.
이부분은 직접 DOM을 건드리기 때문에 추후 유지보수나 디버깅 시에도 불리할 수 있습니다.
따라서 coin 값에 의존하는 useEffect를 사용하는 것 보다

<p ref={text} style={{fontWeight: 'bolder'}}>0</p>

이부분을

<p style={{fontWeight: 'bolder'}}>{coin}</p>

이렇게 사용하는게 훨씬 깔끔할 것 같네요. 이렇게 사용하는게 성능 및 안정성에도 이점이 있습니다.

@Kr-Verified Kr-Verified changed the title 신은총-과제 완료 [2일차] 신은총 과제 제출합니다. Jul 31, 2025
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