클래스의 메소드가 자체 getter 및 setter를 호출해야합니까?

내가 일하는 곳에서 다음과 같은 일을하는 많은 수업을 봅니다.

public class ClassThatCallsItsOwnGettersAndSetters {

    private String field;

    public String getField() {
        return field;
    }

    public void setField(String field) {
        this.field = field;
    }

    public void methodWithLogic() {
        setField("value");
        //do stuff
        String localField = getField();
        //do stuff with "localField"
    }
}

이것을 처음부터 작성했다면 methodWithLogic()대신 다음과 같이 작성했을 것입니다.

public class ClassThatUsesItsOwnFields {

    private String field;

    public String getField() {
        return field;
    }

    public void setField(String field) {
        this.field = field;
    }

    public void methodWithLogic() {
        field = "value";
        //do stuff            
        //do stuff with "field"
    }
}

클래스가 자체 getter 및 setter를 호출하면 코드를 읽기가 더 어려워집니다. 나에게 그것은 거의 우리의 경우에 결코 그렇지 않더라도 그 메소드 호출에서 복잡한 논리가 일어나고 있음 을 거의 암시 합니다. 익숙하지 않은 코드를 디버깅 할 때 버그가 해당 방법에서 부작용이 아니라고 말하는 사람은 누구입니까? 다시 말해, 코드를 이해하는 여정에서 많은 사이드 트립을 수행하게됩니다.

첫 번째 방법에는 이점이 있습니까? 첫 번째 방법이 실제로 더 낫습니까?

어느 쪽이 더 나은지 나쁜지는 말하지 않을 것입니다. 그러나 게터와 세터가 나중에 구현을 변경하고 무시하면 해당 논리를 건너 뛸 수 있습니다.

예를 들어, 나중에 일부 세터 필드에 "dirty"플래그를 추가하면 어떻게됩니까? 내부 코드에서 setter를 호출하면 다른 코드를 변경하지 않고 dirty 플래그를 설정합니다. 많은 상황에서 이것은 좋은 일입니다.

세터를 직접 호출하는 것은 그 자체로는 문제가되지 않습니다. 문제는 실제로 다음과 같습니다. 왜 우리 코드에 세터가 있습니까?

가변 클래스는 위험한 게임이며, 특히 클래스 자체가 자체 상태를 관리하는 경우에 효과적입니다. 그러한 세터 중 몇 명이 실제로 존재해야하는지 고려하십시오. 생성자에 몇 개를 설정 한 다음 클래스 자체로 완전히 캡슐화 할 수 있습니까?

필드를 외부에서 설정할 수 있어야하는 경우 논리가있는 방법이 있는지 스스로에게 물어보십시오. 수업 자체가 데이터 / 주 수업입니까? 이 수업에는 두 가지 이상의 책임이 있습니까?

나를 잘못 이해하지 마십시오. 이것이 좋은 경우가 있습니다. 나는 논리가있는 클래스의 setter를 완전히 제거해야한다고 말하지 않습니다. 그러나 이것들은 예외가 아니라 규칙입니다. 그리고 그러한 경우에는 직접 액세스 할 수 있어야합니다.

예, 클래스의 메소드는 getter 및 setter를 호출해야합니다. 게터와 세터를 작성하는 요점은 미래의 증거입니다. 모든 속성을 필드로 만들고 데이터를 클래스의 사용자에게 직접 노출 할 수 있습니다. getter와 setter를 빌드하는 이유는 복잡한 논리가 있기 때문에 꼭 필요한 것은 아니지만 나중에 인터페이스를 추가해야 인터페이스가 중단되지 않기 때문입니다.

한마디로 질문에 대답하기 위해 그렇습니다.

클래스에 자체 getter 및 setter를 호출하면 확장 성이 추가되고 향후 코드를위한 더 나은 기반이 제공됩니다.

다음과 같은 것이 있다고 가정하십시오.

public class Vehicle
{
    private int year;
    private String make;

    public Vehicle(int year, String make)
    {
        setYear(year);
        setMake(make);
    }

    public void setYear(int year)
    {
        this.year = year;
    }

    public void setMake(String make)
    {
        this.make = make;
    }
}

1 년 동안 setter를 호출하고 현재 make는 기능을 추가하지 않을 수 있지만 setter에 입력 유효성 검증과 같은 것을 추가하려면 어떻게해야합니까?

public class Vehicle
{
    private int year;
    private String make;

    public Vehicle(int year, String make)
    {
        setYear(year);
        setMake(make);
    }

    public void setYear(int year)
    {
        if(year > 0)
        {
            this.year = year;
        }
        else
        {
            System.out.println(year + " is not a valid year!");
        }
    }

    public void setMake(String make)
    {
        this.make = make;
    }
}

언급되지 않은 한 가지는 getter 및 setter (모든 메소드)가 Java에서 가상이라는 것입니다. 이렇게하면 코드에서 항상 사용할 수있는 다른 기능이 추가됩니다. 다른 사용자가 클래스를 확장하여 게터와 세터를 덮어 쓸 수 있습니다. 그러면 기본 클래스는 자체 클래스 대신 서브 클래스의 데이터를 사용합니다. 가상 함수를 명시 적으로 표시하는 언어에서는이를 수행 할 수있는 함수를 예측하고 선언 할 수 있으므로 훨씬 편리합니다. Java에서는 항상 알아야 할 사항입니다. 원하는 동작이라면 코드에서 사용하는 것이 좋습니다. 그렇지 않으면 그렇게 많이하지 않습니다.

공용 인터페이스에서 제공하는 것을 달성하려는 경우 getter / setter를 사용하십시오.

그러나 강의의 소유자 / 개발자로서 강의 내에서 코드의 개인 섹션에 액세스 할 수는 있지만 위험을 완화 할 책임도 있습니다.

따라서 내부 목록을 반복하는 게터가 있고 반복기를 늘리지 않고 현재 값을 얻으려고합니다. 이 경우 개인 변수를 사용하십시오.

public class MyClass
{
    private int i;
    private List<string> list;
    public string getNextString()
    {
        i++;
        return list[i];
    }

    private void getString()
    {
        // Do not increment
        string currentString = list[i];

        // Increment
        string nextString = getNextString();
    }
}

예. 게터와 세터는 상태를 나타내므로이 질문을 돌리십시오. 필요한 경우가 아니라면 동일한 방법으로 객체 상태를 변경하는 여러 가지 방법을 추적해야합니까?

더 잘 추적해야 할 것이 적을수록 불변의 객체를 다루기가 더 쉬워집니다.

공공 장소와 공공 게터 / 세터를 둘 다 막는 데는 아무런 어려움이 없습니다.하지만 무엇을 얻을 것입니까?

계기로 될 수있다 바람직 당신은 무슨 일이 생기면 것을 멀리 부끄러워하지 말아야 직접 하나 이상의 이유로 필드에 액세스 할 수 또는 필요. 그러나 정의 된 이점이있는 경우에만 그렇게해야합니다.

두 방법 모두 유스 케이스가 있습니다. 퍼블릭 세터는 필드 값 (및 / 또는 바운드 값)의 일관성을 유지하므로 메소드 논리가이 일관성 논리를 방해하지 않는 경우 세터를 사용해야합니다. "속성"을 설정 한 경우 setter를 사용하십시오. 반면에 일부 설정에 직접 액세스해야하는 상황이 있습니다 (예 : 무거운 세터를 사용한 대량 작업 또는 세터 개념이 너무 간단한 작업).

일관된 상태를 유지하는 것은 귀하의 책임입니다. Setter는 정의에 따라이 작업을 수행하지만 복잡한 경우는 다루지 않습니다.

나는 아니오라고 말할 것입니다.

게터 / 세터가 단지 게으른 초기화, 검사 없음 등을 얻고 설정 한 경우 변수를 사용하십시오. 나중에 getter / setter를 변경하면 변경되는 methodWithLogic()클래스 때문에 매우 잘 변경할 수 있으며 직접 할당 대신 getter / setter를 호출 할 수 있습니다. getter / setter에 대한이 호출을 문서화 할 수 있습니다 (나머지 클래스 코드가 변수를 직접 사용하는 경우 getter / setter를 호출하는 것이 이상 할 수 있음).

JVM은 getter / setter에 대한 빈번한 호출을 인라인합니다. 따라서 결코 성능 문제가 아닙니다. 변수 사용의 이점은 가독성과 IMHO입니다.

도움이 되었기를 바랍니다..