목록개발 - 기록/Code Review (3)
개발여행의 블로그
styled-components에 대한 활용 react에만 집중해서 과제를 하다보니 styled-components는 단순히 컴포넌트의 스타일을 지정하는 정도로만 사용했다. 역시나 멘토님께서 해당 부분에 대해 코멘트를 남겨주셨다. 더 찾아보고 적용해보았어야 했는데라는 생각이 들었다. 글로벌 스타일 지정(createGlobalStyle), theme, styled-component의 확장 등등 활용법이 굉장히 많다는 것을 배웠다. 1) 전역 스타일 지정 구현된 코드에는 HTML 파일에 style sheet를 지정해서 전역 스타일을 지정해주었는데, styled component를 사용하면 아래와 같이 사용할 수 있다. import { createGlobalStyle } from 'styled-componen..
1번째 과제 코드 리뷰에서는 기본적인 것들에 대해 피드백을 받았다. 완전 기본 중에 기본인데 convention을 제대로 확인하지 않고 과제를 제출하여 많이 반성했다. (너무 기본적인거라 민망했다.😅😅) 개선해야 할 사항 if와 else if의 앞뒤는 한 칸 띄우기 // bad if(){} // good if () {} 주석 주석을 일일이 달 필요는 없다. 변수 네이밍, 함수 네이밍과 로직이 잘 짜여있다면 더욱더 주석을 달 필요가 없다. 만약 주석을 달아야 될 것 같다면 지금 구현한 코드가 가동성있게 작성한 것인지 고민해볼 필요가 있다. 주석은 가능한 최소한으로 줄이는 게 좋다. 객체 선언에 세미콜론 붙이기 (생각치 못한 곳에서 세미콜론을 빼먹었었다...) 코드의 일관성 string 값을 적을 때 '(s..
unnecessary white space 3번째 매개변수 전달 후 불필요한 공백이 있는 것으로 확인. 과제 제출하기 전 확인! // before (choiceItem ,) createChildElement($choiceContainer, 'div', CSS_CLASSNAME.choiceItem , textContent, i); // after (choiceItem,) createChildElement($choiceContainer, 'div', CSS_CLASSNAME.choiceItem, textContent, i); function naming endGame이라는 함수명보다 내부의 로직을 더욱 명확히 설명할 수 있는 네이밍으로 변경하기. // before function endGame(correct..