Skip to content

🟩 코드 리뷰법 #18

Description

@InSeong-So
  1. 주석과 파일
  • 필요한가?
  • 필요하지 않다면 피드백을 주고, 필요하다면 의미를 물어볼 것.
  1. 네이밍
  • 일관성이 있어야 한다. 동사, 명사, 형용사.
  • 어떤 것은 동사+명사로 되어 있는데 어떤 것은 동사+형용사로 되어 있다.
  • 변수/함수명과 같은 기능을 하는지 확인한다.
    constgetFalseArray=(size)=>{// false 배열returnArray.from({length:size},()=>false);returnArray(size).fill(false);}
  1. 공백, 세미콜론, EOL(EndOfLine) 등
  • 불필요한 공백이 있는가?
  • 세미콜론을 사용하거나 사용하는 일관성이 깨지진 않았는가?
  • EOL이 통일되었는가?
  1. 중복되는 코드가 존재하는가?
  • 동일한 기능을 하는 함수가 존재하는가?
  • 지속적으로 똑같은 내용이 반복되는가?
  • console.log()가 불필요하게 사용되진 않았는가?
  • 사용하지 않는 변수가 존재하는가?

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions