3. Boost noncopyable funktioniert nicht

Boost noncopyable funktioniert nicht

  • P

    Hallo zusammen, Hallo pros !

    ich habe folgendes Problem: Boost::noncopyable scheint nicht zu greifen. Verwundert ? Hier der Beispiel-Code (getestet Windows, MSVS 2005):

    Eine static lib mit einer Datei (TestClasses.hpp) und folgendem Inhalt:

    #include <boost/noncopyable.hpp>

    class INonCopyNormalClass: boost::noncopyable
    {
    public:
    virtual INonCopyNormalClass* clone() = 0;
    };

    Eine DLL mit folgenden Dateien:
    DLLMain.cpp:
    #include "DLLmain.hpp"

    YetAnotherNonNormalCopyClass::YetAnotherNonNormalCopyClass()
    {
    }

    INonCopyNormalClass* YetAnotherNonNormalCopyClass::clone()
    {
    return new YetAnotherNonNormalCopyClass(*this);
    }

    DLLMain.hpp:
    #include "../TestLib/TestClasses.hpp"

    # if defined(UNCOPYABLETESTDLL_EXPORTS)
    # define DLL_API __declspec(dllexport)
    # else
    # define DLL_API
    # endif

    class DLL_API AnotherNonNormalCopyClass: public INonCopyNormalClass
    {
    };

    class YetAnotherNonNormalCopyClass: public AnotherNonNormalCopyClass
    {
    public:
    YetAnotherNonNormalCopyClass();

    virtual INonCopyNormalClass* clone();
    };

    Nun der Clou: alles kompiliert korrekt, sollte aber nicht möglich sein,
    da YetAnotherNonNormalCopyClass in der clone() fct nicht kopierbar sein sollte !
    Entfernt man nun das DLL_API (also das exportieren der Symbole), kommt es zum erwarteten Compiler-Fehler. Warum ?

    Grüße,
    Philipp

    PS: das Projekt ist natürlich sehr viel größer, es wurde auf das Kernproblem reduziert. Für alle, die es trotzdem wundert: Die Idee war, den Benutzer des Interfaces dazu zu zwingen, die clone() fct zu benutzen (slicing), daher soll es von boost::noncopyable erben.

    0
  • B

    Vermutung: Die Idee von noncopyable ist, dass man Kopierkonstruktor und Zuweisungsoperator privat macht und nicht implementiert, so dass das spätestens beim Linken auffällt. Dadurch, dass du deine Klasse als DLL exportierst, entfällt aber das Linken, so dass schonmal beim Bauen kein Fehler fliegen dürfte.
    Das Linken zur Laufzeit hat anscheinend nicht genau die gleiche Semantik wie das statische Linken, so dass es auch keinen Laufzeitfehler gibt.

    0
  • L

    Bashar schrieb:

    Die Idee von noncopyable ist, dass man Kopierkonstruktor und Zuweisungsoperator privat macht

    Dadurch, dass er privat ist, sollte der Aufruf desselben in der abgeleiteten Klasse doch auch schon verhindert werden?

    EDIT:
    Rest war quatsch.

    0
  • B

    Stimmt, Denkfehler.

    0
  • Z

    PM1302 schrieb:

    #include <boost/noncopyable.hpp>

class INonCopyNormalClass: boost::noncopyable 
{
public:
	virtual INonCopyNormalClass* clone() = 0;
};

    Nun private Vererbung von boost::noncopyable in deine Klasse hebt die Wirkung auf, welches noncopyable haben sollte. Private Vererbung stellt die Beziehung "ist Implementiert durch" da und hat mit der eigentlich Vererbung nicht viel gemeinsam ausser der Syntax.

    0
  • P

    Danke für die ersten Vermutungen, aber:
    a) durch wegnehmen von DLL_API zeigt die Vererbung von noncopyable den gewünschten Effekt, was in einem compiler fehler mündet, da der compiler den copy constructors der abgeleiteten Klassen nicht automatisch erzeugen kann

    b) ob public / private Vererbung von noncopyable ist egal. Der Effekt ist der gleiche: mit DLL_API funktioniert es (was es nicht sollte), ohne DLL_API nicht (wie es sein sollte).

    However, das Problem bleibt: Durch das exportieren "verschwindet die Wirkung" von noncopyable, was nicht sein darf !

    0
  • P

    BTW:

    "The public keyword in the inheritance syntax means that publicly accessible members inherited from the base class stay publicly accessible in the derived class".

    Da aber assigment operator/copy constructor bereits in der Basisklasse "private" sind, spielt es keine Rolle, ob private, protected oder pubic vererbt wird, oder irre ich mich jetzt ?

    0
  • M

    Bekommst du eigentlich die hier?

    http://msdn.microsoft.com/en-us/library/3tdb471s(VS.80).aspx

    0
  • P

    ja, C4275 bekomme ich. Aber ich verstehe den Zusammenhang nicht.

    BTW, wenn das der Grund wäre, könnte man boost::uncopyable ja gar nicht (bzw. das Konzept dahinter) immer sicher verwenden.

    Warum ist das nicht dokumentiert bzw. fällt das niemandem auf ?

    0
  • P

    Und es wird noch besser: (kleineres beispiel, eine DLL):
    DLLmain.hpp:
    #include "boost/noncopyable.hpp"

    # if defined(UNCOPYABLETESTDLL_EXPORTS)
    # define DLL_API __declspec(dllexport)
    # else
    # define DLL_API
    # endif

    class DLL_API AnotherNonNormalCopyClass: private boost::noncopyable
    {
    };

    class YetAnotherNonNormalCopyClass: public AnotherNonNormalCopyClass
    {
    public:
    virtual AnotherNonNormalCopyClass* clone();
    };
    DLLmain.cpp:
    #include "DLLmain.hpp"

    AnotherNonNormalCopyClass* YetAnotherNonNormalCopyClass::clone()
    {
    return new YetAnotherNonNormalCopyClass(*this);
    }

    Ich verstehe das nicht: das kompiliert/linkt !..... 😮 😮

    0
  • C
    Mod

    wie sieht es auf der Nutzerseite aus, also __declspec(dllimport) ?

    0
  • P

    Wie meinst Du das genau ? Ich exportiere ja noch keine Symbole....?

    0
  • L

    Doch, die Klasse.

    0
  • Z

    PM1302 schrieb:

    Und es wird noch besser: (kleineres beispiel, eine DLL):
    DLLmain.hpp:
    #include "boost/noncopyable.hpp"

    # if defined(UNCOPYABLETESTDLL_EXPORTS)
    # define DLL_API __declspec(dllexport)
    # else
    # define DLL_API
    # endif

    class DLL_API AnotherNonNormalCopyClass: private boost::noncopyable
    {
    };

    class YetAnotherNonNormalCopyClass: public AnotherNonNormalCopyClass
    {
    public:
    virtual AnotherNonNormalCopyClass* clone();
    };
    DLLmain.cpp:
    #include "DLLmain.hpp"

    AnotherNonNormalCopyClass* YetAnotherNonNormalCopyClass::clone()
    {
    return new YetAnotherNonNormalCopyClass(*this);
    }

    Ich verstehe das nicht: das kompiliert/linkt !..... 😮 😮

    Bei mir auf jeden fall nicht
    d:\workspace\dll_test\dll_test\main.h(11) : error C2248: 'boost::noncopyable_::noncopyable::noncopyable' : cannot access private member declared in class 'boost::noncopyable_::noncopyable'
    1> d:\develop\include\boost\noncopyable.hpp(27) : see declaration of 'boost::noncopyable_::noncopyable::noncopyable'
    1> d:\develop\include\boost\noncopyable.hpp(22) : see declaration of 'boost::noncopyable_::noncopyable'
    1> This diagnostic occurred in the compiler generated function 'AnotherNonNormalCopyClass::AnotherNonNormalCopyClass(const AnotherNonNormalCopyClass &)'

    0
  • L

    @Zeus: Hast Du UNCOPYABLETESTDLL_EXPORTS definiert?

    0
  • P

    Ok, welcher Compiler/Linker ? ServicePack ?

    0
  • P

    Ja, ok, die Klasse...sorry, mein Fehler

    0
  • Z

    LordJaxom schrieb:

    @Zeus: Hast Du UNCOPYABLETESTDLL_EXPORTS definiert?

    Oh irgendwas musste ich ja vergessen mit UNCOPYABLETESTDLL_EXPORTS gibst nur 1 Warnung <.<

    0
  • M

    Folgender Code in einem DLL-Projekt erzeugt einen internen Compilerfehler in MSVC 2005:

    class __declspec(dllexport) base
{
private:
    base( const base& );
};

class __declspec(dllexport) derived : public base 
{
public:
    void foo()
    {
        derived p( *this );
    }
};

    Übrigens auch ohne dllexport an der Basisklasse.

    0
  • P

    @MDK: Es geht aber nicht um die Compiler-Warning, sondern, dass die (beasichtigte und erwartete) Wirkung von boost::noncopyable aufgehoben wird und zwar so, dass es der Entwickler zur Compile/Link Zeit nichts bemerkt. Erst zur Laufzeit gibt es "komisches" Verhalten i.e. zeigt sich, dass clone() eben doch nicht funktioniert z.B. unitialisierte Member etc, aber keinen Crash

    0
Anmelden zum Antworten