char *Wert_Ermitteln(string search, char *datei)
{
    string buf,wert;
    ifstream ID(datei);
    if(!ID.is_open())
    {
        MessageBox(NULL,"Die angegebene Datei könnte nicht geöffnet werden","Fehler",0);
        return NULL;
    }
    for (int i=1;!ID.eof();i++)
    {
        getline(ID, buf);
        if(-1!= buf.find(search,0))
        {
           buf.replace(0,(search.length()),"");
           int S = buf.find(";",0);
           buf.replace(S,1,"");
           wert=buf;
           break;
        }
    }
    ID.close();
    char *ary = new char[wert.length()+1];
    strcpy (ary, wert.c_str());
    return ary;
    delete[] ary;
}

Wie ihr seht ist mein return vor delete was heißt das der Speicherplatz nicht mehr freigegeben wird. Wäre Top wenn jemand eine Idee dazu hat.

mfg

wenn du ne Methode mit return beendest, wird der Code natürlich nicht mehr
ausgeführt.
Also ich würde an deiner Stelle ein std::string zurückgeben, dann wird
der Speicher ja automatisch freigegeben, wenn das Scope verlassen wird.
Ansonsten wird dir nichts anderes übrig bleiben als dich nach dem Methoden-Aufruf
jedesmal um das delete zu kümmern.

Gruß.
CSpille

Dieses Post enthält vertrauliche und/oder rechtlich geschützte Informationen. Wenn Sie nicht der richtige Adressat sind oder dieses Post irrtürmlich erhalten haben, informieren Sie bitte sofort den Absender und vernichten Sie dieses Post. Das unerlaubte Kopieren sowie die unbefugte Weitergabe dieses Posts ist nicht gestattet.

char *ary;

char *Wert_Ermitteln1(string search, char *datei)
{
    string buf,wert;
    ifstream ID(datei);
    if(!ID.is_open())
    {
        MessageBox(NULL,"Die angegebene Datei könnte nicht geöffnet werden","Fehler",0);
        return NULL;
    }
    for (int i=1;!ID.eof();i++)
    {
        getline(ID, buf);
        if(-1!= buf.find(search,0))
        {
           buf.replace(0,(search.length()),"");
           int S = buf.find(";",0);
           buf.replace(S,1,"");
           wert=buf;
           break;
        }
    }
    ID.close();
    char *ary = new char[wert.length()+1];
    strcpy (ary, wert.c_str());
    return ary;
    delete[] ary;
}

char *Wert_Ermitteln(string search, char *datei)
{
    char * wert=Wert_Ermitteln1(search, datei);
    delete[] ary;
    return wert;
}

Nein es muss char* sein.

Warum?

Das delete nach dem return bringt immer noch nichts und ein delete auf einen Pointer den du zurückgeben willst ist falsch.

Dieses Post enthält vertrauliche und/oder rechtlich geschützte Informationen. Wenn Sie nicht der richtige Adressat sind oder dieses Post irrtürmlich erhalten haben, informieren Sie bitte sofort den Absender und vernichten Sie dieses Post. Das unerlaubte Kopieren sowie die unbefugte Weitergabe dieses Posts ist nicht gestattet.

Dieses Post enthält vertrauliche und/oder rechtlich geschützte Informationen. Wenn Sie nicht der richtige Adressat sind oder dieses Post irrtürmlich erhalten haben, informieren Sie bitte sofort den Absender und vernichten Sie dieses Post. Das unerlaubte Kopieren sowie die unbefugte Weitergabe dieses Posts ist nicht gestattet.

if (std::string::npos != buf.find(search, 0))

=> std::size_t ist nicht negativ! usw. ... dein Code ist eine Ansammlung von Fehlern ...

Nein es muss char* sein...

Wenn Du in einer Funktion unbedingt dynamisch ein char-Array anlegen willst, das Du ausserhalb brauchst, bleibt Dir nicht Anderes übrig, als das Löschen auch außerhalb anzustoßen. Schließlich gibt's bei der Entscheidung, ob der Speicher noch gebraucht wird, keinen Kompromiss.
Das kannst Du quick&dirty erledigen

void nutzendeFunktion() {
   char * arr = Wert_Ermitteln1(search, datei);
   WINAPI_Funktion_die_charArray_braucht(arr);
   delete[] arr;
}

... oder mit allem möglichen Tricks wie

class Aufraeumer {
   std::vector<char*> arrays;
public:
   void add(char* neuesArray) { arrays.push_back(neuesArray); }
   ~Aufraeumer() { 
      std::vector<char*>::const_iterator it = arrays.begin();
      while(it != arrays.begin()) {
         delete[] *it;
         ++it;
      }
   }
};

// ACHTUNG: Nicht Threadsafe !!!
static Aufraeumer aufraeumer;

char *Wert_Ermitteln1(string search, char *datei)
{
// ...  Restkram
    char *ary = new char[wert.length()+1];
   aufraeumer.add(ary);
   return ary;
}

... oder smartpointer (falls die Schnittstelle das hergibt) ... oder sonstwie.

Aber letztlich bleibt's: Wird's außerhalb gebraucht, muss es außerhalb gelöscht werden.

Gruß,

Simon2.

Wegen des tollen Spamfilters gibts den Code hier
http://rafb.net/p/M2uxBY76.html

was für eine Problematik denn sonst?

std::string s = get_string();

MessageBox(..., s.c_str(), ...);

Und nur, wenn wirklich ein nicht-konstantes Char-Array benötigt wird, sollte man dieses bereitstellen und dafür kann man ja dann ideal vector<char>(x) benutzen.

Warum immer noch von Hand mit char-arrays hantiert wird, kann ich einfach nicht verstehen...

Warum immer noch von Hand mit char-arrays hantiert wird, kann ich einfach nicht verstehen...

Weil diese Personen kein C++ können.

Vielleicht sollte man dann noch ein Pseudo-C++ Forum hier erzeugen, in der die ganzen Threads dann verschoben werden können

Nö, man sollte dem Threadsteller klipp und klar sagen (wie inzwischen ja auch geschehen ist), dass er definitiv kein char* braucht sondern mit string bestens bedient ist.

Sorry, Konrad, hast du wohl meine Ironie nicht verstanden?

*g* doch, schon. Aber ich wollte nicht, dass da noch jemand auf dumme Gedanken kommt. (Nö, die Gefahr bestand nicht. Trotzdem …)

Ich selber hatte ja den Hinweis auf string::c_str() gebracht.

Was er schon von Anfang an hatte.

Th schrieb:

Ich selber hatte ja den Hinweis auf string::c_str() gebracht.

Was er schon von Anfang an hatte.

… aber an der falschen Stelle.