ich habe 3 Klassen: Fsm, State, ImportUnprocessedState.

ImportUnprocessedState ist eine verebte Klasse von State.
Mein Konstruktor von Fsm sieh so aus:

Fsm(State* state);

Definition:

Fsm::Fsm(State* state)
{
m_state = state; ////State* m_state;

}

Ich möchte jetzt ein Object der Fsm Klasse erstellen:

ImportUnprocessedState importState = new ImportUnprocessedState();
Fsm importFsm = new Fsm(&importState);

Ich habe aber diese fehler
C2440 Initialisierung ImportUnprocessedState* kann nicht in ImportUnprocessedState konvertiert werden
und dasselbe für Fsm

Was mache ich falsch?

Fsm importFsm(new ImportUnprocessedState());

Ich übergebe einen Pointer auf die Klasse. Ich habe viele Linker errors.

ok wieso kann ich sowas nicht schreiben:

Fsm importFsm(new ImportUnprocessedState());

Ich übergebe einen Pointer auf die Klasse. Ich habe viele Linker errors.

Fehler treten auf, wenn das Programm als ganzes fehlerhaft ist, und nicht notwendigerweise an der Stelle, die du gerade betrachtest. Manchmal ist auch nicht das, was dasteht, das Problem, sondern das, was fehlt. Das Glaskugellesen überlasse ich anderen.

Fsm importFsm( new ImportUnprocessedState());

bei dem Konstruktor Fsm(State * state), wo ImportUnprocessedState : State?

und denk bitte daran, den Speicher wieder freizugeben. Evtl. wäre ein shared pointer das Mittel deiner Wahl...

aber ist das von der Syntax richtig so zu schreiben:

Nein, ist es nicht. C++ ist nicht Java.

koschka schrieb:

aber ist das von der Syntax richtig so zu schreiben:

Nein, ist es nicht. C++ ist nicht Java.

von der Syntax ist das korrekt, es wird so kompilieren. Ob es guter Stil ist, steht auf einem ganz anderen Blatt.

Fsm importFsm(new ImportUnprocessedState());

auf jedenfall verletzen.

Soweit ich im ersten Beitrag lese, verwendest du viel zuviel new:

ImportUnprocessedState importState = new ImportUnprocessedState();
Fsm importFsm = new Fsm(&importState);

Der Hinweis das new einen Zeiger zurückgibt, hat dich wohl das machen lassen:

ImportUnprocessedState* importState = new ImportUnprocessedState();
Fsm* importFsm = new Fsm(&importState);

Viel besser wäre aber wahrscheinlich das gewesen:

ImportUnprocessedState importState();
Fsm importFsm(importState);

Selbst falls du tatsächlich Zeiger brauchst, wäre das wohl immer noch besser (smart pointer verwenden), weil es sich auch um die Speicherverwaltung kümmert:

unique_ptr<ImportUnprocessedState> importState(new ImportUnprocessedState());
uniquite_ptr<Fsm> importFsm(new Fsm(importState));

Woher deine Linker-Fehler kommen kann man ohne Glaskugel wohl nicht sagen - hilfreich wäre wohl, wenn du erstmal die Fehler hier reinschreibst, und eventuell relevanten code.

ImportUnprocessedState importState();

Das ist eine Funktionsdeklaration. Mach die Klammern weg.

Sup schrieb:

uniquite_ptr<Fsm> importFsm(new Fsm(importState));

Sollte auch unique_ptr heißen.