Ich habe eine Funktion geschrieben die mir einer CRgn Klasse als Wert zurückgibt.

CRgn BorderRgn(CRgn &rgn)
{
CRgn MeineRegion;

...

return MeineRegion;
}

aber es lässt sich nicht kompilieren,
der Compiler meldet ...

error C2558: class 'CRgn' : Kein Kopierkonstruktor verfuegbar

Was soll ich tun ?

Vielen Dank für Eure Hilfe.

Gruß Ronny

???

Ist das so korrekt ?

und wie muss dann die Funktion deklariert sein ?

Danke

Wie auch immer. Wenn du einen Pointer zurückgeben willst, musst du das Objekt auf dem Heap anlegen, also:

CRgn *BorderRgn(CRgn &rgn)
{
  CRgn *MeineRegion = new CRgn();

  //... (MeineRegion ist jetzt ein Pointer, also alle member über -> ansprechen)

  return MeineRegion;
}

Allerdings musst du dann nachher höllisch aufpassen, dass du den Speicher wieder freigibst, außen rum müsste das also so aussehen:

CRgn param, *border_reg;
//... param createn usw.
border_reg = BorderRgn(param);
//... damit arbeiten
delete border_reg;

Sinnvoller wäre wohl:

CRgn &BorderRgn(CRgn &rgn, CRgn &MeineRegion) 
{
  //...
  return MeineRegion;
}

Das sieht zwar sehr nach C aus, hat aber den Vorteil, dass du keinen Speicher leckst. Der Aufruf sähe dann so aus:

CRgn param, border_reg;
//... param createn usw.
BorderRgn(param, border_reg);
//... damit arbeiten
//nachher ist kein delete nötig

Wenn du dir beide Möglichkeiten offen halten willst:

CRgn *BorderRgn(CRgn &rg, CRgn *MeineRegion = NULL) 
{
  if(MeineRegion = NULL) 
  {
    MeineRegion = new CRgn();
    //Createn usw.
  }
  //...
  return MeineRegion;
}

Dann sähe der Aufruf im zweiten Fall (wenn du delete vermeiden willst) allerdings so aus:

BorderRgn(param, &border_reg); //Adresse übergeben

weil in diesem Fall eine Referenz nicht ausreicht.

edit:
ein Objekt auf dem Heap anzulegen braucht man nicht (unbedingt), ein lokales Objekt kann man als static deklarieren.

HRGN GetRgn()
{
  CRgn rgn;
  ...
  return (HRGN) rgn.Detach(); 
}

CRgn rgn = GetRgn();

@Shlo:
return &CRgn(...) ist aus zwei Gründen schlecht:

a) Du gibst die Adresse eines temporären Objekts zurück. Die kann zwar (mit COPY-Semanti) zugewiesen werden, aber da brauch man ja wieder einen Copy-CTor, und ist generell schlechtes Design für eine Funktionssignatur.
b) Der Destructor von CGdiObject (von dem CRgn abgeleitet ist) ruft DeleteObject() auf - deine gekapselte HRGN ist also schon lange über den Jordan.

Und noch ein Dritter - die Adresse eines static-Members zurückzugeben geht nur solange gut, wie GetRgn auch nur mit einer Region arbeitet.

m.e. sind MFC's GDI-Objekt - Wrapperklassen eh' total verhunztund bereiten mehr Kopfschmerzen als vergnügen.

Meine Funktion sieht jetzt so aus..

HRGN BorderRgn(CRgn *SourceRgn)
{
   CRgn DestRgn;

   // ...

   return (HRGN) DestRgn.Destach();
}

soweit so gut.

nun möchte ich diese Region mit einer schon vorhandenen Region kombinieren...

CRgn MeineRegion, ZielRegion;
MeineRegion.CreateRectRgn(50, 50, 100, 100);

ZielRegion.CombineRgn(&MeineRegion, BorderRgn(&MeineRegion), RGN_OR);

Aber der Compiler streigt und sagt ...

error C2664: 'CombineRgn' : Konvertierung des Parameters 2 von 'struct HRGN__ *' in 'class CRgn *' nicht moeglich

Kann mir jemmand weiterhelfen

Danke

Genau wegen diesem gefummel bevorzuge ich gleich mit HRGN's zu arbeiten..

Warum stürtzt wenn ich diese Funktion aufrufe immer mein Programm ab ?
Hab schon alles ausprobiert aber es stürtzt immer ab.

es passiert weil ich die SourceRgn mehrmals verwende!
Ich habe Sie in einer for-Schleife und setze sie jeweils auf ein neues Rechteck.

Als ich alles mit HRGN's gemacht hab, hat es funktioniert.
Nur jetzt mit CRgn funktioniert es nicht mehr.

Weißt du wie ich diesen Fehler beheben kann ?