나는이 두 가지 비슷한 클래스를 가지고있다. 코드를 인수 분해하기 위해 어떤 설계를 사용합니까?

Question

저는 꽤 복잡한 그래프와 같은 데이터 구조를 가지고 있습니다. 명확성을 들어, 그것을 간단하게하자 : 당신이 볼 수 있듯이

는, 우리의 그래프는 다른 그래프처럼되지 않습니다 :이 날 두 종류의 존재, 두 AbstractEdge에서 상속. 이 디자인은 변경할 수 없습니다. 지금, 나는이의 라인을 따라 두 개의 클래스를 설계 할 수 있다고 가정 : 실제로

class OrientedEdge1 
{ 
    Edge1 * edge; 
    bool orientation; 

    void foo() { edge->foo(); } 
    void bar1() { edge->bar1(); } 
} 

class OrientedEdge2 
{ 
    Edge2 * edge; 
    bool orientation; 

    void foo() { edge->foo(); } 
    void bar2() { edge->bar2(); } 
} 

을 OrientedEdge1 :: foo는()와 OrientedEdge2 :: foo는()는 더 이상 단지 하나의 메소드를 호출하는 것보다,하지만 아이디어는 그것들이 동일하고, AbstractEdge로부터 상속받은 메소드만을 호출한다는 것입니다.

코드를 코드화하기 위해 어떤 설계를 사용합니까? 나는 세 가지 방법을 생각하고있다 :

1. 무료 기능

foo_impl(AbstractEdge * edge) { edge->foo(); } 

class OrientedEdge1 
{ 
    Edge1 * edge; 
    bool orientation; 

    void foo() { foo_impl(edge); } 
    void bar1() { edge->bar1(); } 
} 

class OrientedEdge2 
{ 
    Edge2 * edge; 
    bool orientation; 

    void foo() { foo_impl(edge); } 
    void bar2() { edge->bar2(); } 
} 

프로

사용 :

• 아주 간단한 솔루션, 모든 인수 분해하지 않는 것보다 훨씬 더합니다.

단점 :

• 모든 방법을 실시 할 수 있습니다 무료 기능.

• 선언 코드는 여전히 복제됩니다.

2. 상속을

class AbstractOrientedEdge 
{ 
    AbstractEdge * edge; 
    bool orientation; 

    void foo() { edge->foo(); } 
} 

class OrientedEdge1: public AbstractOrientedEdge 
{ 
    Egde1 * edge1() { return static_cast<Egde1*>(edge); } 

    void bar1() { edge1()->bar1(); } 
} 

class OrientedEdge2: public AbstractOrientedEdge 
{ 
    Egde2 * edge2() { return static_cast<Egde2*>(edge); } 

    void bar2() { edge2()->bar2(); } 
} 

프로를 사용 :

• 더 인수 분해.

• 나는이 두 클래스를 다형 적으로 사용할 계획이 없지만, 상속으로 인해 관련이 있다는 사실을 알고있는 사람은 나중에 유용하게 사용할 수 있습니다.

단점 :

• 이 OrientedEdge1 :: 가장자리는 항상 *는 Egde1를 가리키는 것을 적용하는 생성자/세터에서 치료를해야합니다.

• 어떻게 든 static_cast는 잘못되었습니다.

3.사용 템플릿

template <class EdgeT> 
class OrientedEdge 
{ 
    EdgeT * edge; 
    bool orientation; 

    void foo() { edge->foo(); } 
} 

class OrientedEdge1: public OrientedEdge<Edge1> 
{ 
    void bar1() { edge->bar1(); } 
} 

class OrientedEdge2: public OrientedEdge<Edge2> 
{ 
    void bar2() { edge->bar2(); } 
} 

프로

:

• 대부분의 인수 분해.

• 두 클래스 모두에 저장된 포인터의 캐스팅이 필요없는 올바른 유형을 가지고 있습니다.

단점 :

• AbstractEdge.h을 포함하도록 강제 헤더에서 공유 코드의 실행을 계속해야합니다 (이전 방법과 앞으로 선언과 피할 수).

질문 : 어떤 방법을 사용 하시겠습니까? 다른 해결책이나 제안이 있습니까?

3

Answer 1

1 3. 중복 보일러, 내가 원하는 목적지까지 한 움직임 구현을 제거합니다.