반응형

리팩터링 46

리팩터링 기법. 파생 변수를 질의 함수로 바꾸기(Replace Derived Variable with Query)

파생 변수를 질의 함수로 바꾸기(Replace Derived Variable with Query)는 파생 변수를 질의 함수로 대체하는 리팩터링 기법입니다. 파생 변수란, 다른 변수들의 값을 조합하여 계산된 값으로, 코드의 가독성을 높이기 위해 사용되는 변수입니다. 파생 변수는 일반적으로 한 번 설정되면 그 값을 유지합니다. 하지만 이러한 파생 변수를 사용하면, 값이 변경되었을 때 각각의 참조지점에서 파생 변수의 값을 수정해주어야 합니다. 이는 오류의 위험성이 높아지며, 유지보수가 어려워집니다. 따라서 파생 변수를 질의 함수로 바꾸기는 파생 변수를 대체하여, 값을 계산하는 메서드를 직접 호출하여 사용하는 것입니다. 이렇게 하면, 파생 변수를 사용하는 코드를 모두 변경하지 않고도 값을 바로 얻을 수 있으며,..

리팩터링 기법. 세터 제거하기 (Remove Setting Method)

세터 제거하기(Remove Setting Method)는 객체의 내부 상태를 직접적으로 변경하는 setter 메서드를 제거하여 불변성을 유지하도록 하는 리팩터링 기법입니다. Setter 메서드는 객체의 내부 상태를 외부에서 변경할 수 있게 해주는데, 이러한 메서드는 객체를 불안정하게 만들어서 디버깅이 어렵고, 예상치 못한 버그가 발생할 수 있는 원인이 됩니다. 세터 제거하기는 해당 객체의 상태를 변경하는 setter 메서드를 모두 제거하고, 대신에 객체 생성자를 이용하여 객체를 초기화하거나, 메서드 호출을 통해 객체 상태를 변경하도록 하는 것입니다. 이렇게 하면 객체의 내부 상태를 직접적으로 변경하는 일이 없기 때문에 불변성을 유지할 수 있습니다. 세터 제거하기는 객체의 내부 상태를 불변성을 유지하도록 ..

리팩터링 기법. 질의 함수와 변경 함수 분리하기(Separate Query from Modifier)

질의 함수와 변경 함수 분리하기(Separate Query from Modifier)는 함수가 값을 반환하면서 동시에 인자를 수정하는 것을 분리하는 리팩터링 기법입니다. 이를 통해 가변 데이터 문제를 해결할 수 있습니다. 보통 함수는 인자를 받아서 일부 처리를 수행하고, 결과를 반환하는 것이 일반적입니다. 하지만 때로는 함수가 값을 반환하면서 동시에 인자를 수정하는 경우가 있습니다. 이 경우 함수를 호출할 때마다 다른 결과를 얻을 수 있기 때문에 예측하기 어렵고, 디버깅도 어렵습니다. 질의 함수와 변경 함수 분리하기를 적용하면 이러한 문제를 해결할 수 있습니다. 함수를 두 개로 분리하여 값을 반환하는 함수와 값을 수정하는 함수로 나누는 것입니다. 이렇게 하면 값을 수정하는 함수를 호출할 때는 인자만 변경..

냄새 6. 가변 데이터 (Mutable Data)

가변 데이터(Mutable data)는 값이 변경될 수 있는 데이터를 의미합니다. 이는 데이터가 동시에 여러 곳에서 변경될 가능성이 높아져 코드를 이해하고 유지보수하기 어렵게 만들며, 오류의 가능성도 높아집니다. 이러한 문제를 해결하기 위해서는 불변성(Immutability)을 지향해야 합니다. 즉, 값을 변경하는 대신 새로운 값을 반환하도록 구현하면 됩니다. 이를 통해 코드 유지 보수성이 향상되며, 동시성 이슈 등 다른 문제들을 예방할 수 있습니다. 가변 데이터를 캡슐화하여 객체로 만들고, 값을 사용할 때는 불변 객체를 반환하여 데이터 변경에 대한 책임을 객체 내부에 캡슐화하는 것입니다. 그리고 전역 변수의 사용을 최소화하고, const를 적극 활용하여 불변성을 보장하는 것이 좋습니다. 불변 데이터를 ..

리팩터링 기법. 변수 캡슐화하기 (Encapsulate Variable)

변수 캡슐화하기(Encapsulate Variable)는 이러한 전역 데이터 문제를 해결하기 위한 방법 중 하나입니다. 전역 데이터를 사용하지 않고 클래스 내부 변수로 변경함으로써 클래스 내에서 데이터를 캡슐화하고 관리할 수 있습니다. 변수 캡슐화하기의 장점은 다음과 같습니다. 데이터 보호: 변수에 직접 접근하는 것을 막아, 데이터를 보호할 수 있습니다. 일관성 유지: 변수에 접근하는 메소드를 통일하여 일관성을 유지할 수 있습니다. 유지보수성 개선: 객체 내부 구현을 변경해도 클라이언트 코드가 영향을 받지 않습니다. 예를 들어, 다음은 전역 변수를 사용하여 이름과 나이를 저장하는 간단한 프로그램입니다. #include #include std::string name; int age; int main() {..

냄새 5. 전역 데이터 (Global Data)

전역 데이터는 프로그램 전체에서 공유되는 데이터를 의미합니다. 전역 변수, 전역 상수, 싱글턴 인스턴스, 레지스트리 항목 등이 포함될 수 있습니다. 전역 데이터는 여러 모듈에서 자유롭게 접근할 수 있기 때문에 모듈 간 결합도가 높아지고, 코드의 복잡도가 증가하며, 디버깅이 어려워질 수 있습니다. - 전역 데이터는 아무곳에서나 변경될 수 있다. - 전역 데이터로 문제가 발생 시 어떤 코드로 인해 값이 바뀐 것인지 알기 어렵다. - 클래스 맴버 변수도 비슷한 문제가 발생 할 수 있다. 또한 전역 데이터가 변경될 경우 해당 데이터를 사용하는 모든 코드를 수정해야 하므로 유지보수성도 저하될 수 있습니다. 전역 데이터를 사용하지 않고 지역 변수나 매개변수로 데이터를 전달하거나 객체지향 설계 원칙 중 하나인 의존성..

리팩터링 기법. 여러 함수를 클래스로 묶기 (Combine Functions into Class)

"Combine Functions into Class" 리팩터링 기법은 "긴 매개 변수 목록" 문제를 해결하는 데 도움이 되는 기법 중 하나입니다. 이 기법은 여러 개의 함수를 하나의 클래스로 묶는 것입니다. 이를 통해 관련된 기능을 하나의 단위로 묶어서 더 간결하고 유지보수가 용이한 코드를 작성할 수 있습니다. 예를 들어, 다음과 같은 두 개의 함수가 있다고 가정해 봅시다. void process_order(std::string customer_name, std::string product_name, int quantity, double price); void cancel_order(std::string customer_name, std::string product_name); 위 함수들은 주문을 처리..

리팩터링. 플래그 인수 제거하기 (Remove Flag Argument)

"플래그 인수 제거하기 (Remove Flag Argument)" 리팩토링 기법은 함수의 매개변수로 전달되는 플래그(flag) 인자를 제거하는 기법입니다. 보통 함수에 여러 가지 동작을 수행하는 데 필요한 여러 개의 플래그 인자를 전달하게 되면, 함수를 호출할 때 플래그 값만 보고 어떤 동작을 수행하는지 이해하기 어려울 수 있습니다. 또한 플래그 인자가 많아질수록 함수의 시그니처(signature)가 복잡해지고, 함수를 호출할 때 인자를 전달하기 어렵게 됩니다. 이를 해결하기 위해 "Remove Flag Argument" 리팩토링 기법을 사용하면 함수 내부에서 플래그 값에 따라 분기하는 대신, 각각의 플래그 값에 대응하는 별도의 함수를 만들어서 호출합니다. 이를 통해 함수의 시그니처가 단순해지고, 함수 ..

리팩터링. Use Default Values

Use Default Values는 긴 매개 변수 목록 문제를 해결하는 방법 중 하나로, 객체 생성 시 일부 매개변수에 대해 기본값을 지정하는 방식입니다. 객체 생성에 필요한 매개변수 중 일부는 일반적으로 고정값을 가지는 경우가 많습니다. 예를 들어, 생성되는 객체의 색상이나 크기 등은 사용자가 매번 변경하지 않는 경우가 많기 때문입니다. 이 경우, 이러한 매개변수에 대해 미리 기본값을 지정해두고, 객체 생성 시 해당 매개변수의 값을 지정하지 않으면 자동으로 기본값을 사용하도록 합니다. 이렇게 Use Default Values 방식을 사용하면 객체 생성 시 필수적으로 매개변수를 모두 전달할 필요가 없어지기 때문에 코드가 간결해지고 가독성이 높아집니다. 또한, 기본값을 사용하도록 미리 설정해두면, 객체 생..

반응형