• 목록

✅ 실무에서 PHP 코드 리팩토링 패턴 5가지

코드가 돌아가는 것만으로는 충분하지 않습니다. 시간이 지날수록 복잡도는 증가하고, 유지보수가 어려워지기 때문에 **지속적인 리팩토링(Refactoring)**이 필수입니다. 이 글에서는 PHP에서 실무에 자주 쓰이는 5가지 리팩토링 패턴을 소개합니다.

1️⃣ 긴 함수 분해 (Extract Method)

Before:

[code=php]
function registerUser(array $data) {
    // 유효성 검사
    if (!filter_var($data['email'], FILTER_VALIDATE_EMAIL)) {
        throw new Exception('이메일 오류');
    }

    // DB 저장
    $pdo = new PDO(...);
    $stmt = $pdo->prepare('INSERT INTO users ...');
    $stmt->execute([$data['email'], $data['name']]);
}
[/code]

After:

[code=php]
function registerUser(array $data) {
    validateEmail($data['email']);
    saveUserToDB($data);
}

function validateEmail(string $email) {
    if (!filter_var($email, FILTER_VALIDATE_EMAIL)) {
        throw new Exception('이메일 오류');
    }
}

function saveUserToDB(array $data) {
    $pdo = new PDO(...);
    $stmt = $pdo->prepare('INSERT INTO users ...');
    $stmt->execute([$data['email'], $data['name']]);
}
[/code]

• ➕ 테스트 용이성 증가

• ➕ 재사용 가능성 증가

2️⃣ 조건문 다형성으로 치환 (Replace Conditional with Polymorphism)

Before:

[code=php]
function calculateDiscount(string $type): int {
    if ($type === 'vip') return 30;
    if ($type === 'gold') return 20;
    return 0;
}
[/code]

After:

[code=php]
interface DiscountPolicy {
    public function getRate(): int;
}

class VipDiscount implements DiscountPolicy {
    public function getRate(): int {
        return 30;
    }
}

class GoldDiscount implements DiscountPolicy {
    public function getRate(): int {
        return 20;
    }
}
[/code]

[code=php]
function calculateDiscount(DiscountPolicy $policy): int {
    return $policy->getRate();
}
[/code]

• ➕ 조건문 제거

• ➕ 확장성 및 책임 분리

3️⃣ 매직 문자열 제거 (Replace Magic String/Number with Constant)

Before:

[code=php]
if ($user['role'] === 'admin') { ... }
[/code]

After:

[code=php]
class UserRole {
    public const ADMIN = 'admin';
    public const USER = 'user';
}

if ($user['role'] === UserRole::ADMIN) { ... }
[/code]

• ➕ 오타 방지

• ➕ IDE 자동완성

• ➕ 변경 용이

4️⃣ 중복 제거 (Consolidate Duplicate Conditional Fragments)

Before:

[code=php]
if ($user->isAdmin()) {
    log('관리자 로그인');
    sendWelcomeMail($user);
} else {
    log('일반 로그인');
    sendWelcomeMail($user);
}
[/code]

After:

[code=php]
log($user->isAdmin() ? '관리자 로그인' : '일반 로그인');
sendWelcomeMail($user);
[/code]

• ➕ 중복 제거

• ➕ 유지보수 쉬움

5️⃣ 서비스 객체 분리 (Extract Service Class)

Before:

[code=php]
class UserController {
    public function register(Request $request) {
        // 로직이 컨트롤러에 몰림
        ...
    }
}
[/code]

After:

[code=php]
class UserService {
    public function register(string $email, string $name): void {
        ...
    }
}
[/code]

[code=php]
class UserController {
    public function register(Request $request, UserService $service) {
        $service->register($request->get('email'), $request->get('name'));
    }
}
[/code]

• ➕ 역할 분리

• ➕ 테스트, 재사용 용이

✅ 요약

지속적인 리팩토링은 버그를 줄이고, 미래의 자신을 도와주는 가장 확실한 투자입니다.