Speicher alokieren

Bitte such dir ein Buch oder Tutorial zum Thema C++.
Hier mangelt es offensichtlich an den absoluten Basics.

Hansi schrieb:
> Hallo Zusammen,
>
> würde gerne erfahren, wie ich Speicher in C++ allokieren kann?

machst du doch schon.

> Wie kann ich hier am einfachsten Speicher allokieren, also bevor ich
> diese Zeile ausführe:
>
> KlasseCMP aCMP= KlasseCMP (aShape1, aShape2);

machs nicht so kompliziert.

1

KlasseSHAPE XYZ(KlasseSHAPE aShape1, KlasseSHAPE aShape2)

2

{

3

  KlasseCMP aCMP( aShape1, aShape2 );

4

  aCMP.Build();

5

  return aCMP.Shape();

6

}

Bist du sicher, dass du die KlasseSHAPE Objekte alle per Kopie übegeben 
willst?

Hansi schrieb:
> Wieso sollte ich diese nicht als Kopie übergeben?

Ich weiß natürlich nicht, wie groß die Klassen sind.
Ich weiß auch nicht, was die CMP Klasse mit den ihm übergebenen Objekten 
macht.
Ich hab keine Ahnung, was mit der von der Funktion zurückgelieferten 
Kopie im Returnwert passiert.

Aber es ist ungewöhnlich, Objekte per Kopie zu übergeben.

> ich bekomme die Fehlermeldung std::bad_alloc at memory location ...

Dann musst du mal mehr zeigen.

Hansi schrieb:
> Hallo Zusammen,
>
> würde gerne erfahren, wie ich Speicher in C++ allokieren kann?

Mit new. Und per delete wieder freigeben nicht vergessen.

Hat den Vorteil, dass das Objekt auf dem Heap auch dann noch existiert, 
wenn der aktuelle Scope verlassen wird.
Hat den Nachteil, dass man sich damit viele schöne Speicherlecks bauen 
kann. Das wird in der Entwicklung oft und gerne so gemacht ;-)

Hansi schrieb:

> Ich habe das jetzt so modifiziert, es funktioniert aber immer noch
> nicht:
>
>

1

> KlasseCMP *aCMP ;

2

> aCMP = new KlasseCMP(aShape1, aShape2);

3

> aCMP->Build();

4

> return aCMP->Shape();

5

>

Natürlich nicht.
Das ist konzeptionell immer noch das gleiche wie

1

KlasseSHAPE XYZ(KlasseSHAPE aShape1, KlasseSHAPE aShape2)

2

{

3

  KlasseCMP aCMP( aShape1, aShape2 );

4

  aCMP.Build();

5

  return aCMP.Shape();

6

}

Ob du das Objekt lokal erzeugst, oder ob du es mittels new aus der Taufe 
hebst, ändert ja nichts am Objekt.
Das einzige was du getan hast ... du hast dir mit deinem new ein schönes 
Speicherloch (Memory Leak) eingefangen. Aber mehr hast du nicht getan 
bzw. verändert.

Du bellst den falschen Baum an.
Da musst du tiefer einsteigen: Wo genau passiert der Fehler? Wie sieht 
die Umgebung an dieser Stelle aus? Was war an dieser Stelle die Absicht 
im Code? Sind die Shape-Geometrien in Ordnung? etc.

Der Fehler mag sogar banal sein, aber ihm auf die Spur zu kommen ist es 
nicht. So wie in allen komplexen Systemen.

Karl Heinz Buchegger schrieb:

> Der Fehler mag sogar banal sein, aber ihm auf die Spur zu kommen ist es
> nicht. So wie in allen komplexen Systemen.

Und sinnvollerweise fängt man erst mal von vorne an.
d.h. hier

1

KlasseSHAPE aShape1 = readSTEP(argv[1]); 

2

KlasseSHAPE aShape2 = readSTEP(argv[2]);

Diese beiden Shapes lässt man sich mal wo ausgeben und sieht sich an, ob 
die korrekt eingelesen wurden.
Dazu lädt man dann natürlich nicht das CAD_Modell vom Eifelturm in das 
Programm mit 2einhalb Millionen Faces, sondern erst mal ganz einfache 
Körper: Würfel, Tetraeder.

Mir persönlich fehlt da schon mal die Fehlerbehandlung, denn in argv[1] 
und argv[2] kann ja auch kompletter Unsinn oder auch gar nichts 
drinstehen.

Oder etwas vermeintlich Sinnvolles, wie der Name einer zwar vorhandenen 
Datei, die aber riesengroß ist und das bei leider zu knappem (oder zu 
knapp konfiguriertem) Speicher auf dem Zielsystem.

Mark Brandis schrieb:
> Mir persönlich fehlt da schon mal die Fehlerbehandlung, denn in argv[1]
> und argv[2] kann ja auch kompletter Unsinn oder auch gar nichts
> drinstehen.


Darüber hatte ich auch schon einen Post fertig. Über das Nichtbehandeln 
von Fehlercode, die die ReadFile Methode vom myReader garantiert 
zurückgibt.

Ich hab dann allerdings gesehen, dass er sich mit

   cout<<"IGES Faces: "<<nIgesFaces<<" Transferred:"<<nTransFaces<<endl;

da die Konvertierungsstatistik ausgeben lässt und daher gehe ich mal 
davon aus, dass ihm das aufgefallen wäre, wenn da überall 0 steht.


> wie der Name einer zwar vorhandenen Datei, die aber riesengroß ist

Das ist leider oft der Fall, dass Neulinge in einem komplexen System 
gleich mal die komplexeste Geometrie reinstopfen, die sie finden können, 
anstelle erst mal mit kleinen überschaubaren Geometrien erste Versuche 
zu fahren und sich langsam an all die kleinen Fallen ranzutasten, die 
einem in der CAD Geometrie erwarten.

Karl Heinz Buchegger schrieb:
> und sich langsam an all die kleinen Fallen ranzutasten, die
> einem in der CAD Geometrie erwarten.

Dein Spezialgebiet? :-)