Skip to content

Conversation

@amorfati0310
Copy link

저번에 익명함수 커맨드 부분 말고는 ES6 클래스로 되어 있어서 많이 수정은 못했습니다

@crongro
Copy link
Contributor

crongro commented Apr 12, 2018

지금 구현한게 mutable과 immutable 이 섞여 있어요.
그리고 새로운 객체를 매번 만들어야 하는데, 그렇게 되면 문제가 객체지향적인 것을 걷어내는게 깔끔한것이라,
class개념을 배웠다가 무너뜨리는 상태입니다.

우선 클래스를 todo/todos 나눈 정도는 잘 한상태입니다.
여기까지 merge하고, 다시 immutable을 개별적으로 배우고 완성할 수 있도록 가이드 할게요.

@crongro crongro merged commit 6e8fa74 into code-squad:amorfati0310 Apr 12, 2018
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