[클린코드 스터디] 3장. 함수

3장. 함수

🎯 함수는 작고 명확하게

• 함수는 작아야 한다. 그리고 더 작아야 한다.
• 함수가 짧을수록, 들여쓰기 레벨이 얕을수록, 읽기 쉬운 구조가 된다.
• 한눈에 코드를 읽고 이해할 수 있어야 하며, 한 가지 작업만 수행해야 한다.

// 길고 복잡한 함수 예시
public void processUser(User user) {
    if (user.isActive()) {
        sendEmail(user);
    }
    updateLastLogin(user);
    saveToDatabase(user);
}

// 작고 명확하게 분리한 예시
public void processUser(User user) {
    if (isActiveUser(user)) {
        notifyUser(user);
    }
    recordLogin(user);
    persistUser(user);
}

private boolean isActiveUser(User user) {
    return user.isActive();
}

private void notifyUser(User user) {
    sendEmail(user);
}

private void recordLogin(User user) {
    updateLastLogin(user);
}

private void persistUser(User user) {
    saveToDatabase(user);
}

 

🎯 한 함수는 하나의 작업만 해야 한다

• 하나의 함수가 여러 작업을 담당하면, 수정이 필요할 때 여러 이유로 변경이 발생한다.
• 함수는 오직 하나의 책임만을 가져야 하며, 이름도 그 책임을 분명히 표현해야 한다.

 

🎯 위에서 아래로 읽히는 코드

• 코드는 이야기처럼 위에서 아래로 자연스럽게 읽혀야 한다.
• 각 함수는 바로 아래에 등장하는 세부 동작을 설명해야 한다.

public void generateReport() {
    fetchReportData();
    formatReport();
    printReport();
}

private void fetchReportData() { /* ... */ }
private void formatReport() { /* ... */ }
private void printReport() { /* ... */ }

 

🎯 switch 문은 최소화하고 숨겨라

• switch 문은 본질적으로 여러 작업을 포함하기 때문에 함수가 커질 위험이 있다.
• 다형성을 사용해 switch 문을 객체 생성 단계로 숨기는 것이 바람직하다.

// 좋지 않은 switch 문
public double calculatePay(Employee e) {
    switch (e.type) {
        case SALARIED: return calculateSalariedPay(e);
        case HOURLY: return calculateHourlyPay(e);
        default: throw new InvalidEmployeeType();
    }
}

// 다형성으로 개선
abstract class Employee {
    public abstract double calculatePay();
}

class SalariedEmployee extends Employee {
    public double calculatePay() { /* ... */ }
}

class HourlyEmployee extends Employee {
    public double calculatePay() { /* ... */ }
}

 

🎯 좋은 함수 이름 짓기

• 함수 이름은 그 자체로 무엇을 하는지 명확하게 설명해야 한다.
• 함수 이름만 보고도 기능을 예측할 수 있어야 한다.

예시:

• 나쁜 이름: doStuff(), handleData()
• 좋은 이름: saveUserProfile(), validateOrderRequest()

 

🎯 인수는 최대한 줄여라

• 함수 인수는 0~2개가 이상적이다.
• 인수가 많을 경우, 관련 있는 데이터끼리 묶어 객체로 전달하는 방법이 좋다.

// 인수가 많은 경우
public void registerUser(String name, String email, String password, String phone) {}

// 객체로 묶어 개선
public void registerUser(UserRegistrationInfo info) {}

 

🎯 부수 효과를 피하라

• 함수는 예상치 못한 상태 변화를 일으키면 안 된다.
• 특히, 입력값을 수정하거나 전역 변수를 변경하는 행동은 주의해야 한다.

잘못된 예시:

public void updateUser(User user) {
    user.setLastLogin(LocalDate.now());
    database.save(user);
    // 부수 효과로 다른 모듈의 상태 변경
    SessionManager.resetSession(user);
}

개선된 예시:

public void updateUser(User user) {
    user.setLastLogin(LocalDate.now());
    database.save(user);
}

public void resetSessionIfNeeded(User user) {
    SessionManager.resetSession(user);
}

 

🎯 명령과 조회를 분리하라

• 함수를 호출할 때 "읽기"와 "쓰기" 역할이 혼합되어 있으면 혼란을 초래한다.
• 데이터 조회 함수는 상태를 변경하지 않아야 하고, 명령 함수는 값을 반환하지 않아야 한다.

// 잘못된 예시
public boolean setName(String name) {
    this.name = name;
    return true;
}

// 개선된 예시
public void setName(String name) {
    this.name = name;
}

public String getName() {
    return this.name;
}

 

 

---

✨ 정리하며

3장을 읽으면서 가장 크게 느낀 것은, 좋은 함수는 '짧고 명확해야 한다'는 기본 원칙이었다.
또한, 하나의 함수가 여러 책임을 지는 순간 코드는 급격히 복잡해진다는 점을 깨달았다.

 

특히 인상 깊었던 것은 "위에서 아래로 읽히는 흐름" 이다. 하나의 큰 그림을 먼저 보여주고,
그 밑에 구체적인 내용을 단계적으로 풀어내는 방식은 유지보수성과 가독성을 크게 높여준다.

 

앞으로는 함수를 작성할 때, 이 함수가 정말 하나의 일만 하는지, 이 함수가 충분히 작은지,
함수 이름만으로 의도를 알 수 있는지, 항상 스스로 점검해가며 작성해야겠다.