명령 패턴 디자인

이 이전의 명령 패턴 구현이 있습니다. 그것은 모든 DIOperation 구현을 통해 Context를 전달하는 것이지만, 학습과 학습 과정에서 멈추지 않는 나중에 실현되지 않는 것을 나중에 깨달았습니다. 또한 여기에있는 "방문"은 실제로 적합하지 않으며 혼동된다고 생각합니다.

나는 실제로 내 코드를 리팩토링하려고 생각하고 있습니다. 명령은 다른 것에 대해 아무것도 알지 못하고 현재 모두 동일한 키-값 쌍을 공유하기 때문입니다. 어떤 클래스가 어떤 키-값을 소유 하는지를 유지하는 것은 실제로 어렵고 때로는 변수를 복제합니다.

사용 사례의 예 : CommandB 에 CommandA에 의해 설정된 UserName 이 필요 하다고 가정 해 봅시다 . CommandA가 UserNameForCommandB = John 키를 설정해야합니까 ? 아니면 공통 UserName = John 키-값 을 공유해야 합니까? 세 번째 명령에서 UserName을 사용하면 어떻게됩니까?

이 디자인을 어떻게 개선 할 수 있습니까? 감사!

class DIParameters {
public:
   /**
    * Parameter setter.
    */
    virtual void setParameter(std::string key, std::string value) = 0;
    /**
    * Parameter getter.
    */
    virtual std::string getParameter(std::string key) const = 0;

    virtual ~DIParameters() = 0;
};

class DIOperation {
public:
    /**
     * Visit before performing execution.
     */
    virtual void visitBefore(DIParameters& visitee) = 0;
    /**
     * Perform.
     */
    virtual int perform() = 0;
    /**
     * Visit after performing execution.
     */
    virtual void visitAfter(DIParameters& visitee) = 0;

    virtual ~DIOperation() = 0;
};

명령 매개 변수의 변경 가능성이 약간 걱정됩니다. 지속적으로 변경되는 매개 변수로 명령을 작성해야합니까?

접근 방식의 문제점 :

다른 스레드 / 명령 perform이 진행되는 동안 매개 변수를 변경 하도록 하시겠습니까?

당신이 원하는 마 visitBefore와 visitAfter같은의 Command개체를 다른 호출 할 DIParameter객체?

명령에 대해 모르는 사람이 명령에 매개 변수를 제공하도록 하시겠습니까?

현재 디자인에서이 중 어느 것도 금지되지 않습니다. 일반 키-값 매개 변수 개념에는 때때로 장점이 있지만, 일반 명령 클래스와 관련하여 마음에 들지는 않습니다.

결과의 예 :

Command같은 클래스 의 구체적인 실현을 고려하십시오 CreateUserCommand. 이제 새 사용자를 만들도록 요청하면 명령에 해당 사용자의 이름이 필요합니다. 클래스 CreateUserCommand와 DIParameters클래스를 알고 있다면 어떤 매개 변수를 설정해야합니까?

userName매개 변수를 설정할 수 있습니다 username. 또는 매개 변수를 대소 문자를 구분하지 않습니까? 난 정말 몰랐어 .. 잠깐만 .. 어쩌면 그냥 name?

보시다시피 일반적인 키-값 매핑에서 얻을 수있는 자유는 클래스를 구현하지 않은 사람으로 클래스를 사용하는 것이 부당하게 어렵다는 것을 의미합니다. 최소한 다른 사람이 해당 명령에서 지원하는 키를 알 수 있도록 명령에 상수를 제공해야합니다.

가능한 다른 디자인 접근법 :

• 변경할 수없는 매개 변수 : Parameter인스턴스를 변경할 수 없도록 설정하면 여러 명령에서 인스턴스를 자유롭게 재사용 할 수 있습니다.
• 특정 매개 변수 클래스 : UserParameter사용자와 관련된 명령에 필요한 매개 변수를 정확하게 포함 하는 클래스가 주어지면 이 API를 사용하는 것이 훨씬 간단합니다. 여전히 매개 변수를 상속받을 수는 있지만 명령 클래스가 임의의 매개 변수를 취하는 것은 더 이상 의미가 없습니다. 물론 이것은 API 사용자가 어떤 매개 변수가 정확히 필요한지를 알고 있음을 의미합니다 .
• 컨텍스트 당 하나의 명령 인스턴스 : visitBeforeand와 같은 것을 사용해야하는 명령이 필요 visitAfter하고 다른 매개 변수와 함께 재사용하는 경우 다른 매개 변수로 호출되는 문제에 노출 될 수 있습니다. 여러 메소드 호출에서 매개 변수가 동일해야하는 경우, 호출 사이의 다른 매개 변수에 대해 전환 할 수 없도록 명령에 매개 변수를 캡슐화해야합니다.

디자인 원칙의 장점은 조만간 서로 충돌한다는 것입니다.

설명 된 상황에서 각 명령이 정보를 가져 와서 정보를 넣을 수있는 컨텍스트 (특히 키 값 쌍 인 경우)를 선호한다고 생각합니다. 이것은 트레이드 오프를 기반으로합니다 : 나는 서로에 대한 일종의 입력이기 때문에 별도의 명령을 결합하고 싶지 않습니다. CommandB 내에서 UserName이 어떻게 설정되어 있는지 상관하지 않습니다. CommandA의 동일한 내용 : 정보를 설정했습니다. 다른 사람이 정보를 사용하고 있는지 알고 싶지 않습니다.

이것은 일종의 통과 컨텍스트를 의미하며, 이는 당신이 나쁘게 찾을 수 있습니다. 저에게있어 대안은 더 나쁩니다. 특히이 간단한 키-값 컨텍스트 (게터와 세터가있는 간단한 빈이 될 수 있고 "자유 형식"요소를 조금 제한하기 위해)가 솔루션을 간단하고 테스트 할 수있게한다면 각각 고유 한 비즈니스 로직이있는 분리 된 명령.

명령 인터페이스가 있다고 가정합니다.

class Command {
public:
    void execute() = 0;
};

그리고 주제 :

class Subject {
    std::string name;
public:
    void setName(const std::string& name) {this->name = name;}
}

필요한 것은 :

class NameObserver {
public:
    void update(const std::string& name) = 0;
};

class Subject {
    NameObserver& o;
    std::string name;
private:
    void setName(const std::string& name) {
        this->name = name;
        o.update(name);
    }
};

class CommandB: public Command, public NameObserver {
    std::string name;
public:
    void execute();
    void update(const std::string& name) {
        this->name = name;
        execute();
    }
};

NameObserver& oCommandB에 대한 참조로 설정 하십시오. 이제 CommandA가 변경 될 때마다 주제 이름 CommandB가 올바른 정보로 실행될 수 있습니다. 더 많은 명령에서 이름을 사용하는 경우std::list<NameObserver>

이것이 프로그래머 에게이 문제를 처리하는 올바른 방법인지는 모르겠지만 (이 경우 사과드립니다) 모든 답변을 확인한 후 (@ Frank 's 특히). 이런 식으로 코드를 리팩터링했습니다.

• DI 매개 변수가 삭제되었습니다. DIOperation의 입력 (불변)으로 개별 (일반) 객체를 갖게됩니다. 예:

RelatedObjectTriplet {클래스
은밀한:
    std :: string const m_sPrimaryObjectId;
    std :: string const m_sSecondaryObjectId;
    std :: string const m_sRelationObjectId;

    RelatedObjectTriplet & operator = (RelatedObjectTriplet 기타);

공공의:
    RelatedObjectTriplet (std :: string const & sPrimaryObjectId,
                         std :: string const & sSecondaryObjectId,
                         std :: string const & sRelationObjectId);

    RelatedObjectTriplet (RelatedObjectTriplet const & other);


    std :: string const & getPrimaryObjectId () const;
    std :: string const & getSecondaryObjectId () const;
    std :: string const & getRelationObjectId () const;

    ~ RelatedObjectTriplet ();
};

• 다음과 같이 정의 된 새 DIOperation 클래스 (예)
  

템플릿 <class T = void> 
DIOperation 클래스
공공의:
    virtual int perform () = 0;

    가상 T getResult () = 0;

    가상 ~ DIOperation () = 0;
};

CreateRelation 클래스 : public DIOperation <RelatedObjectTriplet> {
은밀한:
    정적 std :: string const TYPE;

    // Params (불변)
    RelatedObjectTriplet const m_sParams;

    // 숨겨진
    CreateRelation & operator = (CreateRelation const & 소스);
    CreateRelation (CreateRelation const & source);

    // 내부의
    std :: string m_sNewRelationId;

공공의:
    CreateRelation (RelatedObjectTriplet const & params);

    int perform ();

    RelatedObjectTriplet getResult ();

    ~ CreateRelation ();
};

• 다음과 같이 사용할 수 있습니다 :

RelatedObjectTriplet triplet ( "33333", "55555", "77777");
CreateRelation createRel (triplet);
createRel.perform ();
const RelatedObjectTriplet res = createRel.getResult ();

도움을 주셔서 감사합니다. 여기서 실수하지 않았기를 바랍니다. :)