Der Kampf mit mehrfach weitervererbten Klassen

[Bebu]

Ich habe wieder mal ein Problem...
Initialisierungsliste von SqliteWrapper:

Code: Alles auswählen

Dedupe::Dataholding::SqliteWrapper::
SqliteWrapper( std::string Path ) : dbs_path( Path ),
				                   db_handle( NULL ),
                                                    ReturningTable( )

Initialisierungsliste von Dataholding:

Code: Alles auswählen

Dataholding::Dataholding( Dedupe::FilePath Path )
: Dataholding::SqliteWrapper( Path.native() )

Dataholding erbt von SqliteWrapper:

Code: Alles auswählen

class Dataholding : protected Dedupe::Dataholding::SqliteWrapper

Jetzt versuche ich aus allen Modulen eine Klasse Kernel zu verschmelzen, was mit allen Modulen funktioniert, bis auf Dataholding:

Code: Alles auswählen

class Kernel : protected Dedupe::FileSearch::SearchFiles,
                   protected Dedupe::Hash::Hash64,
                   protected Dedupe::Dataholding::Dataholding

Initialisierungsliste:

Code: Alles auswählen

Dedupe::Core::Kernel::Kernel( Dedupe::FilePath DatabaseLocation,
                                                      std::ostream &Messages )
: Dedupe::Dataholding::Dataholding( DatabaseLocation ),
   MessageOut( Messages )

Das ganze kompiliert Fehlerlos, aber ich bekomme ein "undefined reference to Dedupe::Dataholding::SqliteWrapper::SqliteWrapper( Dedupe::Dataholding::SqliteWrapper const&" zurück. Dataholding kann innerhalb einer Memberfunktion instanziert werden, also liegt es nicht daran, das er die Bibliothek nicht dazulinken kann. Wo ist mein Denkfehler?

Edit: Habe gerade noch den aktuellen Stand hochgeladen, habe allerdings die betreffenden Zeilen auskommentiert, damit sich das ganze auch bauen lässt.

[cloidnerux]

"undefined reference to Dedupe::Dataholding::SqliteWrapper::SqliteWrapper( Dedupe::Dataholding::SqliteWrapper const&"

Klingt so, als ob du dem Konstruktor die Falschen werte gibst und der Linker nun versucht einen Konstruktor zu finden, der dieses Argument annimmt.
So weit ich das gesehen habe, nimmt der SqlWrapper nur strings oder paths an?

[Xin]

Das ganze kompiliert Fehlerlos, aber ich bekomme ein "undefined reference to Dedupe::Dataholding::SqliteWrapper::SqliteWrapper( Dedupe::Dataholding::SqliteWrapper const&" zurück. Dataholding kann innerhalb einer Memberfunktion instanziert werden, also liegt es nicht daran, das er die Bibliothek nicht dazulinken kann. Wo ist mein Denkfehler?

Hier wird der Copy-Construktor für den Sqlite-Wrapper gerufen. C++ versucht also ein Sqlite-Wrapper-Objekt zu kopieren. Vermutlich fehlt irgendwo ein Referenz-Operator (&).

Was Du da soweit geschrieben hast, sieht okay aus. Ich vermute daher das Problem an einer ganz anderen Stelle.

Erstelle einen Copy-Construktor, lass ihn darin abschmieren ( *((int*)0) = 0 => "Segmentation Fault" ) und guck mal mit gdb, von wo Du in den Copy-Konstruktor überhaupt reingekommen bist.

Ja, ein Break-Point wäre auch okay ;->

[Kerli]

Es könnte sich auch um eine unbeabsichtigte Kopie bzw. temporäres Objekt handeln.

Ja, ein Break-Point wäre auch okay ;->

Oder noch besser den Copy-Constructor private machen bzw. in C++11 überhaupt entfernen:

Code: Alles auswählen

SqliteWrapper(SqliteWrapper const&) = delete;

[Xin]

Es könnte sich auch um eine unbeabsichtigte Kopie bzw. temporäres Objekt handeln.

Ja, ein Break-Point wäre auch okay ;->

Oder noch besser den Copy-Constructor private machen

Das nutzt nichts, falls das Problem in einer Member-Funktion ist.

bzw. in C++11 überhaupt entfernen:

Code: Alles auswählen

SqliteWrapper(SqliteWrapper const&) = delete;

Wieder was gelernt. ^^

[Kerli]

Das nutzt nichts, falls das Problem in einer Member-Funktion ist.

Nachdem der Fehler aber erst beim Ableiten aufgetreten ist vermute ich dass, das Problem wohl irgendeine implizite Typumwandlung ist und nicht in einer Memberfunktion.
Der Konstruktor sollte natürlich nicht implementiert werden. So gibt es bei einem Zugriff von außen eine genaue Fehlermeldung inkl. Zeile und bei einem Zugriff aus der Klasse selbst immerhin noch den Linkerfehler.

[Bebu]

Dank euren Hinweise habe ich den Fehlerteufel entdeckt:

Code: Alles auswählen

std::for_each(
      AddValues.begin(),
      AddValues.end(),
      boost::bind(
        &Dedupe::Core::Kernel::AppendFile, &Kernel, _1, Recursive));

Ich hatte vor Kernel den Referenzoperator vergessen Die Zeile sitzt in der Kommandozeilenklasse und kann so ohnehin nicht bleiben, sonst schieße ich mir beim Thema Multithreading selbst ins Knie. Aber im Moment beschäftige ich mich damit noch nicht...
So manchen Fehler kann man wirklich nicht so leicht finden...

[Xin]

Hier wird der Copy-Construktor für den Sqlite-Wrapper gerufen. C++ versucht also ein Sqlite-Wrapper-Objekt zu kopieren. Vermutlich fehlt irgendwo ein Referenz-Operator (&).

Dank euren Hinweise habe ich den Fehlerteufel entdeckt:
Ich hatte vor Kernel den Referenzoperator vergessen

menschen_tanz-5719.gif

(auf die Schnelle habe ich kein GIF gefunden, was cool tanzt, darum nehme ich den epileptischen Anfall hier ersatzweise)

[Bebu]

Hier wird der Copy-Construktor für den Sqlite-Wrapper gerufen. C++ versucht also ein Sqlite-Wrapper-Objekt zu kopieren. Vermutlich fehlt irgendwo ein Referenz-Operator (&).

Dank euren Hinweise habe ich den Fehlerteufel entdeckt:
Ich hatte vor Kernel den Referenzoperator vergessen

menschen_tanz-5719.gif

(auf die Schnelle habe ich kein GIF gefunden, was cool tanzt, darum nehme ich den epileptischen Anfall hier ersatzweise)

Schön, wenn sich ein Mensch so einfach erfreuen lässt