Primitive Obsession 제거 — 검증·연산이 따라다니는 primitive 필드(금액+통화, 이메일 문자열 등)를 도메인 개념을 담은 Value Object로 치환. "값 객체 도입", "primitive obsession", "Money 클래스로", "/discover-value-object" 요청 시 사용. 단, 함께 전달되는 파라미터 묶음을 객체로 바꾸는 것은 /introduce-parameter-object, 컬렉션을 감싸는 것은 /first-class-collection이 적합. /discover-value-object [commit-ref]로 호출.
インストールする前に、エージェントに与えられる指示の中身を確認できます。
Primitive Obsession을 제거하여 도메인 개념을 명시적으로 표현하는 Value Object 발견.
../../references/refactoring-procedure.md §0·§3-B)Discover Value Object 리팩토링 단계:
Primitive Obsession 패턴 식별
Value Object 클래스 생성
관련 로직 이동
사용처 변경
// Before: Primitive Obsession
public class Order {
private int amount; // primitive 타입
private String currency; // primitive 타입
public Order(int amount, String currency) {
// 검증 로직 산재
if (amount < 0) {
throw new IllegalArgumentException("Amount cannot be negative");
}
if (currency == null || currency.isEmpty()) {
throw new IllegalArgumentException("Currency is required");
}
this.amount = amount;
this.currency = currency;
}
// 비즈니스 로직 산재
public boolean isExpensive() {
return amount > 1000;
}
public String formattedPrice() {
return currency + " " + amount;
}
public double convertToUSD(double exchangeRate) {
if ("USD".equals(currency)) {
return amount;
}
return amount * exchangeRate;
}
}
// After: Value Object 추출
record Money(int amount, String currency) {
// 생성자에서 검증
public Money {
if (amount < 0) {
throw new IllegalArgumentException("Amount cannot be negative");
}
if (currency == null || currency.isEmpty()) {
throw new IllegalArgumentException("Currency is required");
}
}
// 비즈니스 로직을 Value Object로 이동
public boolean isOver(int threshold) {
return amount > threshold;
}
public String formatted() {
return currency + " " + amount;
}
public Money convertTo(String targetCurrency, double exchangeRate) {
if (currency.equals(targetCurrency)) {
return this;
}
return new Money((int) (amount * exchangeRate), targetCurrency);
}
public Money add(Money other) {
if (!currency.equals(other.currency)) {
throw new IllegalArgumentException("Cannot add different currencies");
}
return new Money(amount + other.amount, currency);
}
}
public class Order {
private Money price; // Value Object
public Order(Money price) {
this.price = price;
}
// 비즈니스 로직이 간결해짐
public boolean isExpensive() {
return price.isOver(1000);
}
public String formattedPrice() {
return price.formatted();
}
}
Discover Value Object를 적용해야 하는 경우:
BigDecimal, LocalDate, UUID, JDBC ResultSet, 외부 API 응답 DTO 등)이 도메인 개념을 그대로 대변
발견 후보:
BigDecimal → Money, LocalDate → BusinessDate, String UUID → OrderId)PaymentApiResponse → PaymentResult, HttpResponse<String> → AuthToken)공통 골격(대상 파일 수집 → 후보 제시(계열별 승인 규칙) → 적용 → 테스트 → 커밋/되돌리기, 브랜치·PR이
필요한 조건)은 이 스킬 디렉터리 기준 ../../references/refactoring-procedure.md가 정본이다.
아래는 이 기법에 고유한 부분만 규정한다.
대상 파일에서 다음 패턴을 찾는다:
BigDecimal, LocalDate, UUID 등)이 필드/파라미터로 직접 노출되고, 관련 계산/검증 로직이 호출부에 산재후보를 하나씩 제시하고 사용자 확인:
## 리팩토링 후보 1: Discover Value Object
**파일**: Order.java
**대상**: int amount, String currency
**현재 사용**:
- 필드: private int amount; private String currency;
- 검증: 생성자, setAmount() 등 3곳에서 검증 반복
- 로직: isExpensive(), formattedPrice(), convertToUSD() 등
**현재 코드**:
[관련 코드 블록]
**제안 변경**:
1. Money record 생성 (amount, currency)
2. 검증 로직을 Money 생성자로 이동
3. 비즈니스 로직을 Money 메서드로 이동
- isOver(threshold)
- formatted()
- convertTo(currency, rate)
- add(Money)
4. Order 필드를 Money 타입으로 변경
**적용할까요?** (yes / no / 수정 요청)
모든 후보 확인 후 최종 실행 목록을 보여주고 진행 여부 확인.
확정된 리팩토링을 하나씩 수행:
커밋 메시지 형식:
refactor: discover value object [객체명] in [클래스명]
사용자에게 보고:
공통 실패 조건(계열별 승인 규칙 위반, 테스트 실패 방치, 테스트 수정, 커밋 단위, git add -A, heredoc
한글 메시지)은 ../../references/refactoring-procedure.md에 있다. 아래는 이 기법에 고유한 것만.
まだレビューはありません。使ってみた感想をお寄せください。
概要と使いどころ
동일한 결과를 내는 여러 조건문(OR 나열·중첩 AND)을 하나로 통합하고 의미 있는 boolean 메서드로 추출. "조건문 합쳐", "같은 결과 반환하는 if 정리", "중첩 if 평탄화", "/consolidate-conditional" 요청 시 사용. 단, 여러 메서드에 흩어진 동일 조건을 호출자 쪽으로 올리는 것은 /lift-up-conditional, 복잡한 조건식·분기를 메서드로 쪼개는 것은 /decompose-conditional이 적합. /consolidate-conditional [commit-ref]로 호출.
日本語の概要は準備中です。原文の説明を表示しています。
기능의 external behavior를 Cucumber 인수 테스트(주 검증층)로 구축 — .feature 실행으로 문서↔코드 드리프트를 구조적으로 차단, Four Layer(Steps→Protocol Driver→SUT), 태그 기반 가역 제외, 기존 JUnit 인수 테스트 이관. "인수 테스트 도입", "Gherkin을 실행 가능하게", "cucumber 셋업" 요청 시 사용. /cucumber-acceptance로 호출.
日本語の概要は準備中です。原文の説明を表示しています。
복잡한 if/then/else의 조건식과 각 분기를 의미 있는 메서드로 추출하여 가독성 향상. "조건문 분해", "if 가독성", "복잡한 조건식에 이름 붙여", "/decompose-conditional" 요청 시 사용. 단, 같은 결과를 내는 조건문들을 하나로 합치는 것은 /consolidate-conditional, 타입별 분기를 클래스로 바꾸는 것은 /replace-conditional-with-poly가 적합. /decompose-conditional [commit-ref]로 호출.
日本語の概要は準備中です。原文の説明を表示しています。
컬렉션 getter가 내부 List/Set을 직접 노출하는 것을 방지 — unmodifiable 반환 + add/remove 메서드 제공. "컬렉션 캡슐화", "getter가 List 그대로 노출", "unmodifiable로", "/encapsulate-collection" 요청 시 사용. 단, 컬렉션과 관련 로직을 전용 클래스로 뽑는 것은 /first-class-collection이 적합. /encapsulate-collection [commit-ref]로 호출.
日本語の概要は準備中です。原文の説明を表示しています。
암묵적 의존성(전역 변수·클래스 필드·싱글턴 접근)을 명시적 파라미터로 전환하여 메서드 투명성 향상. "숨은 의존성 드러내", "필드 대신 파라미터로", "전역 참조 제거", "/explicit-parameters" 요청 시 사용. 단, 파라미터가 많아져 묶어야 하면 /introduce-parameter-object, I/O와 계산 분리는 /segregate-functional-core가 적합. /explicit-parameters [commit-ref]로 호출.
日本語の概要は準備中です。原文の説明を表示しています。
지역 변수가 서로 얽혀 Extract Method가 불가능한 거대 메서드를 별도 클래스(Method Object)로 추출. "긴 메서드를 클래스로", "지역 변수 때문에 추출 불가", "method object", "/extract-method-object" 요청 시 사용. 단, 지역 변수 얽힘 없이 잘라낼 수 있는 긴 메서드는 /intent-revealing-names(grouping→extract)가 적합. /extract-method-object [commit-ref]로 호출.
日本語の概要は準備中です。原文の説明を表示しています。