[FE - ConnecTo] DAY50 TIL👩🏻‍💻

22.09.21 수업 시간 중 공부한 내용들을 정리하였습니다 😊
피드백은 언제나 환영입니다! 🍊

1번

• 화살표 함수로 되어있던 것을 그냥 변수에 바로 담음 → 굳이 한 번 더 호출해서 사용할 필요가 없기 때문 → throttle 위치 변경으로 다시 추가
• 변수 이름이 긴가?

controlScrollButtonVisibility

• throttle → 기존 함수에 걸어뒀었는데, 이벤트 핸들러 안에서 걸어두는 것이 맞지 않나..? 재사용성 측면에서도..!

---

2번

• update count를 counter 밖에서 사용할 일이 없기 때문에, counter 내부로 위치를 옮겨줌
• HTML에 종속적이지 않으려면, 따로 초기화를 해야겠지만, 그런 경우가 아니라고 생각하기 때문에 HTML의 값을 가져와서 사용하기로 함.

let count = $countBox.textContent;

• 이벤트 리스너는 즉시 실행함수 내부에 있어야 하는가? 외부에 있어야 하는가? 재사용을 하지 않는데 굳이 왜 밖에 있어야 하는가..???
  

---

3번

• undefined보다 ‘’가 더 가시적으로 의미가 와닿아서 변경함.

if (targetText === undefined) return;

if (targetText === '') return;

• checkPalindrome(targetText) 을 한 번만 사용하기 때문에, 변수로 따로 선언하지 않고, 바로 사용하는 것으로 변경함.

checkPalindrome(targetText) ? '' : 'not'} a palindrome

• 이벤트 핸들러 함수를, showResult로 따로 분리하였습니다. → 왜? 더 가시적으로 코드를 파악할 수 있기 때문이라고 생각함.
  

---

4번

• darkmode 를 변수에 담아서 사용하자 → 너무 많이 재사용되고 있기 때문임
• saveTheme → setTheme 으로 이름을 변경해서, getTheme와 세트를 맞춰준다
• 이벤트 핸들러 함수를 toggleTheme으로 분리해서 가독성 높이기
• 이벤트 핸들러와 리스너의 공간을 분리한다? 만다? → 분리한다
  코드 순서를 어떤 기준으로 분리하느냐의 문제
  
• 함수 기능에 맞게 함수명을 수정함.
• 코드 순서를 조정하여, 중복 코드를 제거하였습니다.

---

5번

• state로 상태관리를 하다가 하지 않는 것으로 변경
• 불필요한 변수 선언을 제거하였음
• 이벤트 핸들러 안의 기능을 분리하였습니다.