Wie vermeidet man Zeiger mit Google Mock?

Question

In unserem Projekt verwenden wir Google Mock, aber an mehreren Stellen machen wir Konstruktionen in unserem Produktionscode, nur um sicherzustellen, dass Klassen "mockbar" sind. Wir tun dies, weil wir die Vorteile von Google Mock nutzen möchten, aber andererseits möchten wir lieber einen optimalen Produktionscode haben. Der folgende Fall ist etwas, was wir oft machen und gerne loswerden möchten.

class A{ 
    public: 
    void doSomething(); //Does something with _someB 
    void setSomeB(B* mockedB); //This is only here for GMock 
    private: 
    B* _someB; //This should not be a pointer, it is a pointer only for GMock 
} 

Dies ist nur ein vereinfachtes Beispiel, wie Sie sehen können, habe ich die Details weggelassen. Im Grunde wollen wir loswerden, dass B ein Zeiger ist. Der Grund, warum wir es als Zeiger haben, ist, dass wir B in unserem Testcode ableiten können (mock it) und setzen Sie es mit diesem Setter.

Gibt es eine Möglichkeit, dies zu vermeiden? Können wir nicht zulassen, dass B in der Klasse berücksichtigt wird?

Dank

Answer 1

Wenn Sie wirklich _someB als Zeiger gehalten nicht wollen, wäre eine Option seine Erklärung in Präprozessorbefehle einzuwickeln:

#ifdef MOCK_TEST 
    B _someB; 
#else 
    MockB _someB; 
#endif 

Wenn Sie dies nicht tun, gibt es keine einfache Möglichkeit, gmock-Erwartungen und/oder Assertions auf _someB zu platzieren, da es keine Instanz eines Mock-Objekts sein wird. Abgesehen von der Hässlichkeit hat dies auch den Nachteil, dass die lib, in der Class A residiert, zweimal kompiliert wird, einmal mit MOCK_TEST definiert und einmal ohne. Es entfällt jedoch die Notwendigkeit eines Setter für _someB.

Auch wenn Sie _someB als Zeiger zu halten, vielleicht eine bessere Option als eine öffentliche Setter schafft, ist die Testklasse ein Freund Klasse von A ist der Test _someB direkt eingestellt und den Zugang zu ermöglichen, zu erklären.

Nebenbei, es ist oft considered poor form Variablen mit einem führenden Unterstrich zu benennen.