-
Notifications
You must be signed in to change notification settings - Fork 10
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주차] 박지수 미션 제출합니다. #10
base: master
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.
전반적으로 깔끔한 코드에 배열을 다루는 메서드도 잘 이해하고 쓰신 것 같아요😊👍🔥
2주차 과제도 고생많으셨습니다!!
import { | ||
AppContainer, Main, TodoContainer, MainText, Line, TodosBody, | ||
TodoHeader, TodoList, TodoItem, TodoInput, | ||
Input, Button, RemoveButton | ||
} from './styles'; |
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.
style.js에서
export const S = {
AppContainer, Main, ...
};
과 같이 씀으로써 export도 일일이 쓰지 않고
import { | |
AppContainer, Main, TodoContainer, MainText, Line, TodosBody, | |
TodoHeader, TodoList, TodoItem, TodoInput, | |
Input, Button, RemoveButton | |
} from './styles'; | |
import { S } from './styles'; |
import문은 파일 자체를 지정하여 아래쪽 return 부에서 컴포넌트를 쓸 때 <S.MainText></S.MainText>
와 같이 쓰면 더 간단할 것 같아요!
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.
오.. 지원님이 말씀해주신 방법 정말 간결하고 좋네요
TodoHeader, TodoList, TodoItem, TodoInput, | ||
Input, Button, RemoveButton | ||
} from './styles'; | ||
import { GlobalStyle } from './styles'; |
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.
글로벌 스타일은 파일을 따로 빼도 좋을 것 같아요! 물론 컴포넌트들이 여기서는 따로 없지만 쪼개진다면 한눈에 전역 스타일이 있는지, 혹은 어느 스타일이 특수한지 쉽게 파악할 수 있으니까요 :)
// 할 일 삭제 | ||
const removeTodo = (index) => { | ||
const updatedTodos = todos.filter((_, i) => i !== index); | ||
setTodos(updatedTodos); | ||
}; |
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.
useCallback
함수로 감싸서 해당 App.js가 렌더링 될 때마다 불필요하게 리렌더링 되는걸 막으면 최적화에 더 좋을 것 같습니당
// 할 일 삭제 | |
const removeTodo = (index) => { | |
const updatedTodos = todos.filter((_, i) => i !== index); | |
setTodos(updatedTodos); | |
}; | |
// 할 일 삭제 | |
const removeTodo = useCallback((index) => { | |
const updatedTodos = todos.filter((_, i) => i !== index); | |
setTodos(updatedTodos); | |
}, []); |
// 엔터키로 할 일 추가 | ||
const handleKeyDown = (e) => { | ||
if (e.key === "Enter") { | ||
addTodo(); | ||
} | ||
}; |
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.
저는 인풋을 form 태그로 감싸서 button에 submit 타입을 지정해주고 onClick만 더 지정해주었는데 keyDown이라는 해당 방식도 알아갑니다!
}; | ||
|
||
// 완료된 할 일 개수 | ||
const completedTodosCount = todos.filter(todo => todo.complete).length; |
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.
filter이란 함수를 자유자재로 잘 쓰시는 것 같아요 👍👍
@@ -0,0 +1,134 @@ | |||
import styled from 'styled-components'; |
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.
css-in-js 코드를 따로 만든 스타일 파일로 빼서 App.js 코드를 줄이셨군요!
저도 따로 만든 스타일 파일이 있어서 이 방식을 잠깐 고민했는데, styled-component는 지난주 바닐라와 달리 컴포넌트
별로 스타일을 한 파일 안에서 쉽게 다룰 수 있어서 편리하다고 느꼈어요! 그래서 따로 만든 스타일 파일에는 공용 박스, 글자 등의 스타일만 넣어두었는데, 이 방식도 고려해보시면 좋을 것 같아요!
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.
저도 지원님 의견에 동감해요!
개인적으로 styled-component
를 사용했을 때 가장 큰 장점은 css-in-js
로 한 파일 내에서 스타일까지 한 번에 정의해서 개발 속도가 css-in-css
보다 훨씬 빨라지는 것이라고 생각하고 있어요. 🤩
css-in-js vs css-in-css
해당 문서 읽어보시면 css-in-js와 css-in-css의 차이에 대해 더 자세히 알 수 있어서 도움이 되실 것 같아요!
Better<p></p> | ||
than<p></p> | ||
Yesterday! |
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.
줄 바꿈을 p태그로 쓰신 점 새로워요😯 의미를 좀 더 명확하게 하려면 <br />
로 대신해도 좋을 것 같습니다!
<TodoInput> | ||
<Input | ||
type="text" | ||
value={newTodo} | ||
onChange={(e) => setNewTodo(e.target.value)} | ||
onKeyDown={handleKeyDown} | ||
placeholder="할 일 추가" | ||
/> | ||
<Button onClick={addTodo}>추가</Button> | ||
</TodoInput> |
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.
인풋에 onChange 속성을 주면서 value가 바뀔 때마다 해당 컴포넌트(여기서는 App.js)가 불필요하게 계속 리렌더링된다고 해요! 해당링크 참고해보시면 도움 많이 되실 것 같습니다 😊😊
(+저도 처음에 이 방식으로 최적화해야하나 하고 고쳐보았었는데 다시 보니 저는 인풋이 들어있는 컴포넌트가 분리되어 있어서 상위 컴포넌트에서 addTodo 함수를 정의하고 useCallback으로 감싸주었더니 리렌더링이 발생하지 않더라고요! 여러 방식이 가능할 것 같아요)
src/styles.js
Outdated
background-color: aliceblue; | ||
padding: 10px; | ||
border-radius: 5px; | ||
margin-bottom: 10px; | ||
display: flex; | ||
justify-content: space-between; | ||
`; |
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.
제 pr 참고자료에도 올려둔 링크인데, css 속성 순서와 관련해서 참고해보시면 좋을 것 같아 남깁니다🔥
https://uxkm.io/publishing/css/03-cssMiddleclass/10-css_attr_rule#gsc.tab=0
`; | ||
|
||
export const Input = styled.input` | ||
flex-grow: 1; |
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.
저는 이번에 flex-grow와 flex-shrink 속성을 제대로 알고 써보았는데, 필요한 최소한의 속성만 알맞게 잘 쓰시는 것 같아요👍
|
||
// 할 일 삭제 | ||
const removeTodo = (index) => { | ||
const updatedTodos = todos.filter((_, i) => i !== index); |
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.
_
를 사용하는 게 특이해서 찾아보니 해당 기호로 표현하는걸 언더스코어라고 하는군요.~
저같은 경우에 filter
나 map
함수와 같이 인자를 넘겨줄 수 있지만 안쓰는 경우에도 그냥 변수로 할당해줬는데 담부턴 언더스코어를 활용해볼 수 있을것같아요
<Input | ||
type="text" | ||
value={newTodo} | ||
onChange={(e) => setNewTodo(e.target.value)} | ||
onKeyDown={handleKeyDown} | ||
placeholder="할 일 추가" | ||
/> |
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.
해당 부분 민재님 코드리뷰하다가 useRef
라는 훅을 사용하면서 렌더링 최적화 하는 방식을 배웠습니다!
지수님도 관련 부분 찾아보면 도움이 될 것 같아요~
style={{ | ||
textDecoration: todo.complete ? "line-through" : "none" | ||
}} |
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.
인라인 스타일을 작성하셔서 스타일을 한 눈에 확인할 수 있다는 점이 좋네요
그렇지만 앞으로 프로젝트의 규모가 커지면 이런 인라인 스타일은 유지보수가 까다로울 수 있으니, 외부에 스타일 파일을 하나 만들어서 따로 관리하는 방식도 좋을 것 같습니다.
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.
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.
지수님! 이번 과제도 수고 많으셨어요 😀
상태 관리와 이벤트 처리 부분에서 깔끔한 코드가 돋보였던 것 같아요!
로컬 스토리지 활용해서 리팩토링하시면 UX적으로 크게 개선될 것 같아요!
다음 과제도 기대하겠습니다 🙌
@@ -0,0 +1,134 @@ | |||
import styled from 'styled-components'; |
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.
저도 지원님 의견에 동감해요!
개인적으로 styled-component
를 사용했을 때 가장 큰 장점은 css-in-js
로 한 파일 내에서 스타일까지 한 번에 정의해서 개발 속도가 css-in-css
보다 훨씬 빨라지는 것이라고 생각하고 있어요. 🤩
css-in-js vs css-in-css
해당 문서 읽어보시면 css-in-js와 css-in-css의 차이에 대해 더 자세히 알 수 있어서 도움이 되실 것 같아요!
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.
모든 컴포넌트를 App.js
안에 정의하고 계시네요!
VanillaJS
와 비교했을 때, React
의 이점은 컴포넌트 기반 아키텍처와 재사용성인 것 같아요!
React
와 styled-components
를 사용하는 핵심 목적은 UI를 작은, 재사용 가능한 단위로 나누어 관리하고, 상태와 로직을 각각의 책임에 맞게 분리하는 것이라고 생각해요 :)
모든 UI와 로직을 한 파일에 몰아넣는 방식은 이러한 장점을 훼손할 수 있어요..!
작은 프로젝트에서부터 컴포넌트 단위로 쪼개고 재사용하는 습관을 잡아놓으면 나중에 볼륨이 큰 프로젝트에서 빛을 발할 것 같아요 🥹
export const TodoInput = styled.div` | ||
display: flex; | ||
justify-content: space-between; | ||
border-radius: 10px; | ||
border-style: solid; | ||
border-width: 1px; | ||
margin-bottom: 10px; | ||
`; |
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.
focus
상태일 때 인풋박스 outline
은 없애주기로 해요 🌞
export const TodoInput = styled.div` | |
display: flex; | |
justify-content: space-between; | |
border-radius: 10px; | |
border-style: solid; | |
border-width: 1px; | |
margin-bottom: 10px; | |
`; | |
export const TodoInput = styled.div` | |
display: flex; | |
justify-content: space-between; | |
border-radius: 10px; | |
border-style: solid; | |
border-width: 1px; | |
margin-bottom: 10px; | |
&:focus { | |
outline: none; | |
} | |
`; |
padding: 8px 8px; | ||
background-color: skyblue; | ||
border: none; | ||
border-radius: 5px; |
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.
style={{ | ||
textDecoration: todo.complete ? "line-through" : "none" | ||
}} |
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.
- ̗̀ 결과물 ̖́-
https://react-todo-20th-final.vercel.app/
느낀점
바닐라에서 리액트로 넘어가면서 확실히 리액트로 상태 관리를 하기가 수월해서 더 편리함을 느낀 것 같습니다.
styledcomponent는 처음 사용해봤는데 코드가 훨신 깔끔하게 보여서 좋은 것 같습니다. 이전에 css를 사용할 때는 className으로 구분했는데, 확실히 div 태그속의 className이 아니라 클래스 자체 이름을 설정해서 보니 코드 가독성도 훨씬 좋아진 것 같네요. 앞으로도 코드 짤 때 styledComponent 사용할 것 같습니다.
이번주 KeyQuestion이 최적화에 대한 것이었는데, 사실 최적화에 대한 것은 생각해보지 못해서 아쉽습니다... 스터디 하면서 배우고 나중에는 개발 단계부터 최적화를 생각하는 여유를 가지고 싶습니당..
아직 코드 고쳐야 될 점이 많은데, 주말 안 수정하도록 하겠습니다🥹
Key Question
1) Virtual-DOM은 무엇이고, 이를 사용함으로서 얻는 이점은 무엇인가요?
실제 DOM을 조작하는 방식이 아닌, DOM을 모방한 가상의 DOM을 구성해 원래 DOM과 비교하여 달라진 부분을 리렌더링 하는 방식으로 작동하는 것을 말한다. 즉, DOM을 직접 조작하지 않고 변경사항을 하나의 가상 돔에 모았다가 DOM에 한 번에 보내는 기술이다. 리액트는 가상 돔 방식을 사용하고 있다. 왜 리액트는 실제 DOM을 조작하지 않고 가상 돔을 사용하는 것일까?
[재조정 과정]
[DOM을 사용한 이점]
2) React.memo(), useMemo(), useCallback() 함수로 진행할 수 있는 리액트 렌더링 최적화에 대해 설명해주세요. 다른 방식이 있다면 이에 대한 소개도 좋습니다.
React.memo()
언제 React.memo()를 쓰는게 좋을까?
useMemo()
언제 useMemo()를 쓰는게 좋을까?
useCallback()
언제 useCallback()를 쓰는게 좋을까?
3) React 컴포넌트 생명주기에 대해서 설명해주세요.
생명 주기 단계는 크게 세 가지로 구분할 수 있다.
컴포넌트가 처음 DOM에 렌더링되는 단계이다.
주로 초기 상태 설정, API 호출, DOM 조작, 이벤트 등록 등을 처리한다.
컴포넌트의 상태나 props가 변경되어 재렌더링 되는 단계이다. 이 시점에서 API 호출이나 DOM 조작을 할 수 있다. useEffect 와 같은 의존성 배열에 전달한 값이 변경될 때마다 이 함수가 실행된다.
컴포넌트가 DOM에서 제거되는 단계이다. useEffect에서 반환하는 함수는 컴포넌트가 언마운트되거나 업데이트되기 직전에 실행된다. 여기서 정리 작업을 수행할 수 있다.
생명주기와 관련된 훅은 useState, useEffect, useMemo, useCallback 등이 있다.