Today 요약

1. 멘토링
2. 멘토님 피드백 답변하기

1. What I did?

1.1 멘토링

오늘은 프로젝트를 가지고 해결되지 않은 피드백을 토대로 이야기를 주고 받았다. 논의할 부분이 많았기에 모든 피드백을 해결하지는 못했다. 하지만 그래도 어정쩡하게 끝내고 새로운 프로젝트를 들어가는 것보다는 이게 훨씬 낫다. 아마도 남은 멘토링 기간 동안은 현재 프로젝트를 마무리하는 걸로 진행될 것 같다.

• 오늘 피드백 중에 알게된 것들
  • label태그의 높이보다 입력된 한글이 높이가 더 클때 이상한 UI 버튼이 노출될 수 있음.. (이 경우 영어는 기본적으로 글자가 한글보다 작아서 테스트하기 힘듦, 아마 영어도 글자가 크다면 같은 현상이 일어날 거라고 예측됨)
  • useError에서는 UI에 표시되는 error를 관리해라.

이외에도 피드백 받은 게 더 많지만 나머지 피드백은 녹화해놓은 영상을 복기하면서 내일 정리해야 할 것 같다.

1.2 멘토님 피드백 답변하기

내 코드에 대한 기준 명확하게 하기

• 기존에 TodoApp안에서 TodoApi와 AuthApi를 정의해서 사용중이었는데, 두 개를 정의해서 사용하는 부분에서 통일성이 없었다.
• TodoApi의 경우 외부 코드를 참고해서 가져와 작성하다보니까 일반적인 느낌이 많이 들었던 것 같고, AuthApi의 경우 내가 컴포넌트를 분리하면서 정의했던 부분이기 때문에 다소 어색했던 부분이 많았던 것 같다.

기타 피드백

• 기존에 변수에 담아서 다시 호출했던 함수, 즉 변경의 여지를 줄 수 있고 가독성도 떨어지는 코드를 즉시 실행 함수로 변경해서 해결 (이 경우에는 멘토님이 직접 해결방법을 제시했는데, 이런 경우에 나는 어떻게 더 좋게 할 수 있을까라는 방법을 찾아보는 것도 좋을 것 같다. 하지만 이 부분에 대해서는 나 역시 당장은 즉시 실행 함수로 해결하는 방법이 가장 깔끔하다고 생각한다.)

  // 기존 코드
  const fetchTodos = async () => {
    const todos = await readTodo()
    setTodos(todos)
  }
  
  fetchTodos()

  // 해결 코드
  ;(async () => {
    const todos = await readTodo()
    setTodos(todos)
  })()