replace-loop-with-pipeline

Replace Loop with Pipeline

GOAL

명령형 루프를 Stream API/Collection Pipeline으로 변환하여:

• what vs how: Pipeline은 "무엇을" 하는지, 루프는 "어떻게" 하는지 표현
• 필터링/변환/집계의 의도가 메서드 체인으로 드러남
• 중간 변수/플래그 변수 제거

CONSTRAINTS

• 계열: Tidy — 후보 보고 후 승인 없이 적용 (../../references/refactoring-procedure.md §0·§3-A)
• 동작 변경 금지: 구조 개선만 수행 (기능 변경 없음)
• 테스트 수정 금지: 구조 변경이 테스트를 실패시키면 되돌리기
• 명시적 git add: git add -A 금지, 변경된 파일만 명시

적용 패턴

패턴 1: 필터링 + 변환

// Before
List<String> result = new ArrayList<>();
for (Order order : orders) {
    if (order.isActive()) {
        result.add(order.getCustomerName());
    }
}

// After
List<String> result = orders.stream()
    .filter(Order::isActive)
    .map(Order::getCustomerName)
    .toList();

패턴 2: 집계

// Before
int total = 0;
for (LineItem item : items) {
    if (item.getQuantity() > 0) {
        total += item.getPrice() * item.getQuantity();
    }
}

// After
int total = items.stream()
    .filter(item -> item.getQuantity() > 0)
    .mapToInt(item -> item.getPrice() * item.getQuantity())
    .sum();

패턴 2-1: 집계 — 객체 타입 (BigDecimal 등)

primitive가 아닌 타입은 sum()이 없으므로 reduce(항등원, 누적 연산)을 쓴다. 금액(BigDecimal) 합산이 대표 사례:

// Before
BigDecimal total = BigDecimal.ZERO;
for (Line line : lines) {
    total = total.add(line.amount());
}

// After
BigDecimal total = lines.stream()
    .map(Line::amount)
    .reduce(BigDecimal.ZERO, BigDecimal::add);

패턴 3: 검색 (첫 번째 매칭)

// Before
Employee found = null;
for (Employee e : employees) {
    if (e.getDepartment().equals("Engineering")) {
        found = e;
        break;
    }
}

// After
Optional<Employee> found = employees.stream()
    .filter(e -> e.getDepartment().equals("Engineering"))
    .findFirst();

패턴 4: 존재 여부 확인

// Before
boolean hasOverdue = false;
for (Invoice invoice : invoices) {
    if (invoice.isOverdue()) {
        hasOverdue = true;
        break;
    }
}

// After
boolean hasOverdue = invoices.stream()
    .anyMatch(Invoice::isOverdue);