-
Notifications
You must be signed in to change notification settings - Fork 5
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
[2주차] 이성진 학습 PR 제출합니다. #13
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
불변성과 관련해서는 closer
에 대해 먼저 학습하고 useState
가 어떻게 구현되어 있는지 공부해보시면 좋을 것 같습니다.
리액트를 공부하기 전에 리액트의 어떤 장점 때문에 리액트를 많이 사용하는지, 그렇다면 리액트의 단점은 무엇인지, 해당 단점들을 어떻게 커버할 수 있을 지 생각해보면 좋을 것 같아요. 수고하셨습니다.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
리뷰 답변달기 및 수정부탁드립니다
week02/이성진/todo/src/App.jsx
Outdated
|
||
function App() { | ||
const [inputValue,setInputValue] = useState('') | ||
const [todolist,setTodolist]=useState([]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
전체적으로 일관성있게 카멜케이스 적용하면 좋을 것 같아요.
4. **JSX는 반드시 하나의 부모태그가 감싸는 형식이어야 한다**. | ||
- React 내에서 Virtual DOM이 내부의 요소들의 변화를 감지할 때, 효율적인 비교를 위해 DOM 트리를 구성해야 한다. 따라서 코드를 작성할 때 **<div>** 로 감싸주어야 한다. | ||
|
||
5. **모든 프로퍼티의 표기는 CamelCase로 작성한다.** |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
다른 네이밍 규칙에는 어떤 것이 있을까요??
컴포넌트 명명 규칙은 무엇일까요 ?
✅ 학습 내용 요약
🤔 고민했던 부분
🔊 도움이 필요한 부분