class ClassA
{
public:
  ClassA()
  {
    pClassB = new ClassB;
  }
  ~ClassA() {}

  ClassB* getPtr() const
  {
    return pClassB;
  }
private:
  ClassB *pClassB;
};

Ok Memory Leak gefunden!!! Ganz einfach.
Nö ist es nicht.
Im Code wird die Klasse 'ClassA' benutzt um 'ClassB' zu erzeugen.
Diese Klasse wird dann über die Funktion getPtr() const in eine andere Klasse Transferiert wo diese dann zerstört wird.

Wir hatten dann eine kleine Diskusion über das const hinter der Funktion.
Kollege hat die Meinung das C++ dieses ja Zulässt und am eigentlichen Pointer ja nichts geändert wird. Also durchaus const Korrekt ist.

Meine Frage ist: Ist das in C++ so das hier der Pointer keine Const Problem verursacht obwohl der ganze Code mehr als nur Überdenkenswert ist.

BeiDemThemaNicht kein Name

Das Design ist soweit abstrahiert absolut schlecht. Factory ist in Ordnung, aber nicht das...

Ich weiß jetzt auch nicht, was Du hören willst. Vielleicht stellst Du Deine Frage mal präzieser. Was verstehst Du unter "const problem"?

> nichts "bitweise" verändert wird <<

Danke, das war das was ich suchte.

@krümelkacker
Das ist die einzige Funktion der Klasse. Es gibt noch mehr solcher Klassen.

> Sie verstößt gegen jeden guten Geschmack.
Mmm da hätte ich noch ganz andere "Ausdrücke" für.

Der ganze Code darum sieht noch viel Schlimmer aus.

BeiDemThemaNicht

Wenn die getPtr Methode nur auf konstante Referenzen aufgerufen werden soll, dann gehört da das const hin, damit die Klasse const Korrekt ist.

Etwas anders formuliert bedeutet das: eine Klasse ist const-korrekt wenn sie const-korrekt ist.
Finde ich gut

eine Klasse ist const-korrekt wenn sie const-korrekt ist.

Nina Hagen würde wohl damit antworten:

class Foo
{
public:
    Foo & getInternals() const
    {
        return *const_cast<Foo*>(this);
    }
};

class Foo
{
    int a, * b;
public:
    Foo() : a(0), b(&a) {}
    void set_a(int a) const
    {
	*b = a;
    }
};

Allerdings ist das schon mehr als nur dreckig. Dann doch lieber den Cast oder sich überlegen, ob man nicht etwas falsch macht.

Wenn einem der Cast stört, folgendes ist auch valide:

Dann doch lieber einfach mutable

brotbernd schrieb:

Wenn die getPtr Methode nur auf konstante Referenzen aufgerufen werden soll, dann gehört da das const hin, damit die Klasse const Korrekt ist.

Etwas anders formuliert bedeutet das: eine Klasse ist const-korrekt wenn sie const-korrekt ist.
Finde ich gut

Oder etwas allgemeiner: Etwas ist richtig, wenn es richtig ist.