[10기 soonysoo] TodoList with Ajax #100
Conversation
dalcon10028
left a comment
dalcon10028
left a comment
There was a problem hiding this comment.
안녕하세요!
코드리뷰를 하게된 이연권입니다.
옵저버 패턴에 대해 아직 공부하지 않아 모르지만 알고있는 부분에서 최대한 리뷰해보겠습니다!
|
|
||
| export const userAPI = { | ||
| async getAllUser(){ | ||
| return await fetch(BASE_URL).then(data => data.json()); |
There was a problem hiding this comment.
then-catch대신 async-await를 사용 하신 것 같습니다 !
then 사용 대신 변수에 담아서 반환하셔도 될 것 같아요 !
try {
const res = await fetch(BASE_URL);
return res.json();
} catch (error) {
// Error
}There was a problem hiding this comment.
그렇네요! async-await적용을 전체를 다 하지못했네요~
수정하겠습니당!
| @@ -0,0 +1,28 @@ | |||
| const HTTP_REQUEST ={ | |||
| update(){ | ||
|
|
||
| } | ||
| } No newline at end of file |
There was a problem hiding this comment.
사실 없어도 되지만 observer의 형태라는 의미를 명확히 전달해주기 위해 사용했습니다ㅎㅎ
| async onDeleteAllTodo(){ | ||
| const selectedId = this.selectedUserState.get()._id; | ||
| const response = await todoAPI.deleteAllTodoItem(selectedId); | ||
| if(response.ok){ |
There was a problem hiding this comment.
응답코드 체크는 만드신 api파일에서 확인해주시면 더 간결해 보일 것 같습니다!
hyoungnam
left a comment
hyoungnam
left a comment
There was a problem hiding this comment.
조건처리도 깔끔하고 template을 잘 활용하셔서 잘 보고 배워갑니다:)
|
|
||
| mounted(){ | ||
| const toggleBtns = $$('.toggle'); | ||
| toggleBtns.forEach(Btn => Btn.addEventListener('click',this.onToggleTodo.bind(this))); |
There was a problem hiding this comment.
모든 btn에 이벤트를 달기보다 이벤트위임을 이용하시는것도 좋을 듯합니다 [https://ko.javascript.info/event-delegation)]
There was a problem hiding this comment.
이벤트 위임을 공부하는게 점점 늦어지네요! ㅠㅠ
공부해서 적용해볼게요~
| super(); | ||
| this.state = selectedUserIdState; | ||
| } | ||
| template(){ |
| templete(){ | ||
| const filter = this.filterState.get(); | ||
| const todo = this.selectedUserState.get().todoList; | ||
| const count = this.counTotalTodo(filter, todo); |
| _edit.forEach((li) => { | ||
| li.classList.remove('editing'); | ||
| }); | ||
| e.target.parentNode.parentNode.classList.add('editing'); |
There was a problem hiding this comment.
parentNode를 사용하기보다 직접 $('')을 찾아서 바꿔주는게 이해하기 좋을꺼 같아요
| } | ||
|
|
||
| getRanking(priority){ | ||
| if(priority==PRIORITY.FIRST){ |
3 participants
안녕하세요 🤗
이번 미션은 Observer 패턴을 이용해서 구현해보았습니다!
observer패턴을 스터디하고 이해하고 구현하는 과정이 어려웠습니다.. 😞
리팩토링은 차후에 진행하여 개선시키겠습니다 👩
###🎯 요구사항
###🎯🎯 심화 요구사항
🕵️♂️ 제약사항