사내 세미나 - Getter와 Setter를 함부로 사용하면 안되는 이유;;

들어가기 앞서

Getter와 Setter의 사용을 금지하라

Getter, Setter의 사용을 금지하라

Getter와 Setter란?

Getter와 Setter를 통해 객체 안의 데이터를 보호할 수 있다

class Human {
    public String name;
}

Human human = new Human();
human.name = "John";

lombok을 사용하면 많은 변수들의 getter setter 생성도 간편!

@Getter
@Setter
class Human {
    private String name;
    private int age;
}

Human human = new Human();
human.setName("John");
human.setAge(20);

데이터의 유효성을 검증할 수 있다

class Human {
    private String name;
    private int age;

    public void setAge(int age) {
        if (age < 0) {
            throw new IllegalArgumentException("나이는 0보다 작을 수 없습니다.");
        }
        this.age = age;
    }
}

Getter와 Setter, 이렇게 좋은데 왜 쓰지 말아야 하는걸까? 🤷‍♀️

들어가기 앞서..

Setter를 쓰지 말아야 할 이유

Lombok의 Setter는 모든 프로퍼티에 대해 setter를 생성한다

@Setter
class Human {
    private String name;
    private String juminNumber;
}

Setter 함수로는 값을 수정하는 이유를 드러내지 못한다

@Setter
class Member {
    private int memberType;
}

Member octoping = new Member();
octoping.setMemberType(1);

Setter 함수를 사용하면 도메인 로직이 분산된다

class MemberService {
    // ...

    public void resign(long id) {
        Member member = memberRepository.findById(id);

        if(member.getMemberRole() == MemberRole.ADMIN) {
            throw new IllegalStateException("관리자는 탈퇴할 수 없습니다.");
        }

        member.setMemberStatus(MemberStatus.RESIGNED);
        member.setResignedAt(LocalDateTime.now());

        memberRepository.save(member);
    }
}

Getter를 쓰지 말아야 할 이유

Getter 함수는 객체지향성을 해친다

public List<Human> findBirthdayMan(List<Human> humanList) {
    LocalDateTime now = LocalDateTime.now();

    return humanList.stream()
            .filter(human ->
                human.getBirthday().getMonth() == now.getMonth() &&
                human.getBirthday().getDayOfMonth() == now.getDayOfMonth()
            )
            .toList();
}

너 생일이 몇 월이야?

Getter 함수를 사용하면 변경에 취약해진다

@Getter
class HealthMan {
    private int 삼대중량;
    private String clothes;
}

class UnderArmourPolice {
    public void check(HealthMan 헬린이) {
        if(헬린이.get삼대중량() < 500 && healthMan.getClothes().equals("언더아머")) {
            throw new IllegalStateException("감히 3대 500미만이 언더아머를 입어?");
        }
    }
}

@Getter
class HealthMan {
    private int squat;
    private int benchPress;
    private int deadLift;

    private String clothes;
}

Getter 함수를 사용하더라도 값을 변경할 수 있다

class Human {
    List<Human> children = new ArrayList<>();

    public List<Human> getChildren() {
        return children;
    }
}

Human octoping = new Human();

octoping.getChildren().add(new Human());
octoping.getChildren().add(new Human());

octoping.getChildren().size(); // 2

그러면 어떻게 하라고요;; 😡

Tell, Don't Ask

"물어보지 말고 그냥 시켜라"

비즈니스 로직을 짤 때 Getter로 값을 가져와서 구현하지 말라

class HealthMan {
    // this..
    private int 삼대중량;

    // or this..
    private int squat;
    private int benchPress;
    private int deadLift;

    public int calculateBigThree() {
        // this
        return this.삼대중량;

        // or this
        return squat + benchPress + deadLift;
    }
}

Setter를 사용하지 말고, 상태를 바꾸라고 메시지를 보내라

class MemberService {
    // ...

    public void resign(long id) {
        Member member = memberRepository.findById(id);

        if(member.getMemberRole() == MemberRole.ADMIN) {
            throw new IllegalStateException("관리자는 탈퇴할 수 없습니다.");
        }

        member.setMemberStatus(MemberStatus.RESIGNED);
        member.setResignedAt(LocalDateTime.now());

        memberRepository.save(member);
    }
}

class Member {
    // ...

    public void resign() {
        if(this.memberRole == MemberRole.ADMIN) {
            throw new IllegalStateException("관리자는 탈퇴할 수 없습니다.");
        }

        this.memberStatus = MemberStatus.RESIGNED;
        this.resignedAt = LocalDateTime.now();
    }
}

class MemberService {
    // ...

    public void resign(long id) {
        Member member = memberRepository.findById(id);

        member.resign();

        memberRepository.save(member);
    }
}

이 모든 건 캡슐화

마무리하며