Skip to content

🟩 μ½”λ“œ 리뷰법 #18

@InSeong-So

Description

@InSeong-So
  1. 주석과 파일
  • ν•„μš”ν•œκ°€?
  • ν•„μš”ν•˜μ§€ μ•Šλ‹€λ©΄ ν”Όλ“œλ°±μ„ μ£Όκ³ , ν•„μš”ν•˜λ‹€λ©΄ 의미λ₯Ό λ¬Όμ–΄λ³Ό 것.
  1. 넀이밍
  • 일관성이 μžˆμ–΄μ•Ό ν•œλ‹€. 동사, λͺ…사, ν˜•μš©μ‚¬.
  • μ–΄λ–€ 것은 동사+λͺ…μ‚¬λ‘œ λ˜μ–΄ μžˆλŠ”λ° μ–΄λ–€ 것은 동사+ν˜•μš©μ‚¬λ‘œ λ˜μ–΄ μžˆλ‹€.
  • λ³€μˆ˜/ν•¨μˆ˜λͺ…κ³Ό 같은 κΈ°λŠ₯을 ν•˜λŠ”μ§€ ν™•μΈν•œλ‹€.
    const getFalseArray = (size) => {
      // false λ°°μ—΄
      return Array.from({length:size}, () => false);
      return Array(size).fill(false);
    }
  1. 곡백, μ„Έλ―Έμ½œλ‘ , EOL(EndOfLine) λ“±
  • λΆˆν•„μš”ν•œ 곡백이 μžˆλŠ”κ°€?
  • μ„Έλ―Έμ½œλ‘ μ„ μ‚¬μš©ν•˜κ±°λ‚˜ μ‚¬μš©ν•˜λŠ” 일관성이 κΉ¨μ§€μ§„ μ•Šμ•˜λŠ”κ°€?
  • EOL이 ν†΅μΌλ˜μ—ˆλŠ”κ°€?
  1. μ€‘λ³΅λ˜λŠ” μ½”λ“œκ°€ μ‘΄μž¬ν•˜λŠ”κ°€?
  • λ™μΌν•œ κΈ°λŠ₯을 ν•˜λŠ” ν•¨μˆ˜κ°€ μ‘΄μž¬ν•˜λŠ”κ°€?
  • μ§€μ†μ μœΌλ‘œ λ˜‘κ°™μ€ λ‚΄μš©μ΄ λ°˜λ³΅λ˜λŠ”κ°€?
  • console.log()κ°€ λΆˆν•„μš”ν•˜κ²Œ μ‚¬μš©λ˜μ§„ μ•Šμ•˜λŠ”κ°€?
  • μ‚¬μš©ν•˜μ§€ μ•ŠλŠ” λ³€μˆ˜κ°€ μ‘΄μž¬ν•˜λŠ”κ°€?

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions