데이터 클래스는 경우에 따라 유효합니다. DTO는 Anna Lear가 언급 한 좋은 예입니다. 그러나 일반적으로 메소드가 아직 발아되지 않은 클래스의 씨앗으로 간주해야합니다. 오래된 코드에서 많은 코드를 사용하는 경우 강력한 코드 냄새로 취급하십시오. 그것들은 종종 OO 프로그래밍으로 전환 한 적이없고 절차 적 프로그래밍의 표시 인 오래된 C / C ++ 프로그래머들에 의해 종종 사용됩니다. 게터와 세터에 항상 의존하는 경우 (또는 개인이 아닌 개인이 직접 액세스하는 경우)는 알기 전에 문제를 일으킬 수 있습니다. 의 정보가 필요한 외부 메소드의 예를 고려하십시오 Bottle.

다음 Bottle은 데이터 클래스입니다.

void selectShippingContainer(Bottle bottle) {
    if (bottle.getDiameter() > MAX_DIMENSION || bottle.getHeight() > MAX_DIMENSION ||
            bottle.getCapType() == Cap.FANCY_CAP ) {
        shippingContainer = WOODEN_CRATE;
    } else {
        shippingContainer = CARDBOARD_BOX;
    }
}

여기에 우리는 Bottle몇 가지 행동을 주었습니다 .)

void selectShippingContainer(Bottle bottle) {
    if (bottle.isBiggerThan(MAX_DIMENSION) || bottle.isFragile()) {
        shippingContainer = WOODEN_CRATE;
    } else {
        shippingContainer = CARDBOARD_BOX;
    }
}

첫 번째 방법은 Tell-Don’t-Ask 원칙에 위배되며, Bottle바보 를 유지함으로써 병에 대한 암묵적인 지식 (예 : 하나의 fragle ( Cap)을 Bottle클래스 외부의 논리에 빠지게 함)을 알 수 있습니다. 습관적으로 게터에게 의지 할 때 이런 종류의 ‘누설’을 막으려면 발가락에 있어야합니다.

두 번째 방법은 Bottle작업을 수행하는 데 필요한 사항 만 묻고 , Bottle깨지기 쉬운 지 또는 주어진 크기보다 큰지를 결정합니다. 결과적으로 메소드와 병 구현 사이의 연결이 훨씬 느슨해집니다. 유쾌한 부작용은이 방법이 더 깨끗하고 표현력이 뛰어나다는 것입니다.

이러한 필드와 함께 클래스에 상주해야 할 논리를 쓰지 않고 객체의 많은 필드를 사용하는 경우는 거의 없습니다. 해당 논리가 무엇인지 파악한 다음 논리를 해당 위치로 이동하십시오.

답변

public class Bottle {
    public int height;
    public int diameter;
    public Cap capType;

    public Bottle(int height, int diameter, Cap capType) {
        this.height = height;
        this.diameter = diameter;
        this.capType = capType;
    }
}

public class Bottle {
    private int height;
    private int diameter;
    private Cap capType;

    public Bottle(int height, int diameter, Cap capType) {
        this.height = height;
        this.diameter = diameter;
        this.capType = capType;
    }

    public int getHeight() {
        return height;
    }

    public void setHeight(int height) {
        this.height = height;
    }

    public int getDiameter() {
        return diameter;
    }

    public void setDiameter(int diameter) {
        this.diameter = diameter;
    }

    public Cap getCapType() {
        return capType;
    }

    public void setCapType(Cap capType) {
        this.capType = capType;
    }
}