return ohne delete[]

Reply to return ohne delete[] on Sun, 13 Dec 2009 01:52:24 GMT

weweqeq — Sun, 13 Dec 2009 01:52:24 GMT

hey,

wie war das nochmal?

wenn ich speicher mit new[] angefordert habe, diesen aber nicht freigeben kann, wegen einem return passiert was ?

Reply to return ohne delete[] on Sun, 13 Dec 2009 01:57:45 GMT

theliquidwave — Sun, 13 Dec 2009 01:57:45 GMT

Der Speicher bleibt allokiert und frisst unnötigen RAM. Ein Memoryleak ist das. Der sollte nicht bewusst herbeigeführt werden. Wenn dies des öfteren Auftritt, kann es sein, dass du den Speicher damit zumüllst. Lösch es lieber ordentlich

Gruß

Reply to return ohne delete[] on Sun, 13 Dec 2009 02:07:30 GMT

weweqeq — Sun, 13 Dec 2009 02:07:30 GMT

grml danke

Reply to return ohne delete[] on Sun, 13 Dec 2009 15:32:03 GMT

Nexus — Sun, 13 Dec 2009 15:32:03 GMT

weweqeq schrieb:

wenn ich speicher mit new[] angefordert habe, diesen aber nicht freigeben kann, wegen einem return passiert was ?

Wieso hindert dich ein return am Freigeben?

Moderne Konzepte wie RAII machen dynamische Speicherverwaltung übrigens sehr komfortabel, damit sind kaum noch explizite Freigaben notwendig.

Reply to return ohne delete[] on Sun, 13 Dec 2009 15:50:48 GMT

weweqeq — Sun, 13 Dec 2009 15:50:48 GMT

Weil ich in einer Funktion Speicher mittels new[] anfordere mit dem ich dann den Inhalt einer .txt einlese und diesen zurückgebe und ich da zu keinem return komme.

// read a file into memory
#include 
#include 
using namespace std;

int main () {
  int length;
  char * buffer;

  ifstream is;
  is.open ("test.txt", ios::binary );

  // get length of file:
  is.seekg (0, ios::end);
  length = is.tellg();
  is.seekg (0, ios::beg);

  // allocate memory:
  buffer = new char [length];

  // read data as a block:
  is.read (buffer,length);

  is.close();

  cout.write (buffer,length);

  return 0;
}

Quelle: fstream C++ Reference

Den Code hab ich halt in einer Funktion gekapselt.

Reply to return ohne delete[] on Sun, 13 Dec 2009 15:53:39 GMT

thesameoldstory — Sun, 13 Dec 2009 15:53:39 GMT

nimm std::vector

Reply to return ohne delete[] on Sun, 13 Dec 2009 15:54:38 GMT

Nanyuki — Sun, 13 Dec 2009 15:54:38 GMT

Deswegen solltest du hier ja std::vector oder zumindest smart pointer einsetzen.

Reply to return ohne delete[] on Sun, 13 Dec 2009 20:26:30 GMT

Kahino — Sun, 13 Dec 2009 20:26:30 GMT

Du musst delete[] nicht in der Selben Funktion aufrufen in der du new[] verwendet hast.
Wenn du den Text zurückgibst rufst du halt irgendwo delete[] auf sobald du den Text nicht mehr brauchst.
Fals diese Funktion von der du redest eine Klassenmethode ist kannst du dies auch in den Destruktor machen lassen.

Alternativ kannst du auch std::string verwenden und hängst die Neue Zeile halt immer wieder hinten dran.