Wie return std::string aus der C ' s "getcwd" - Funktion

Wenn du dabei bleiben willst-standard getcwd ist nicht erforderlich, etwas zu tun, wenn Sie an Sie übergeben einen NULL; stattdessen sollten Sie reservieren auf dem stack einen Puffer, der ist "groß genug" für die meisten Anlässe (z.B. 255 Zeichen), aber seien Sie bereit für die Gelegenheit, in die getcwd fehlschlagen mit errno==ERANGE; in diesem Fall sollte man reservieren dinamically Sie einen größeren Puffer ein, und erhöhen Sie Ihre Größe, wenn nötig.

Etwas, wie das funktionieren könnte (Achtung: nicht getestet, nur geschrieben von scratch -, kann sicherlich noch verbessert):

string getcwd()
{
    const size_t chunkSize=255;
    const int maxChunks=10240; //2550 KiBs of current path are more than enough

    char stackBuffer[chunkSize]; //Stack buffer for the "normal" case
    if(getcwd(stackBuffer,sizeof(stackBuffer))!=NULL)
        return stackBuffer;
    if(errno!=ERANGE)
    {
        //It's not ERANGE, so we don't know how to handle it
        throw std::runtime_error("Cannot determine the current path.");
        //Of course you may choose a different error reporting method
    }
    //Ok, the stack buffer isn't long enough; fallback to heap allocation
    for(int chunks=2; chunks<maxChunks ; chunks++)
    {
        //With boost use scoped_ptr; in C++0x, use unique_ptr
        //If you want to be less C++ but more efficient you may want to use realloc
        std::auto_ptr<char> cwd(new char[chunkSize*chunks]); 
        if(getcwd(cwd.get(),chunkSize*chunks)!=NULL)
            return cwd.get();
        if(errno!=ERANGE)
        {
            //It's not ERANGE, so we don't know how to handle it
            throw std::runtime_error("Cannot determine the current path.");
            //Of course you may choose a different error reporting method
        }   
    }
    throw std::runtime_error("Cannot determine the current path; the path is apparently unreasonably long");
}

Übrigens, in deinem code gibt es eine sehr falsche: Sie versuchen, dellocate a_cwd (die vermutlich in die nicht-Standard-Erweiterung, zugeordnet wird, die mit malloc oder mit einigen anderen Speicher-Aufteilung-Funktion, da getcwd ist gedacht für C) mit delete: Sie absolut nicht tun sollten, dass, im Hinterkopf behalten, dass jede aufteilungsmethode hat seine Freigabe Gegenstück, und Sie müssen nicht nicht überein.

Du bist soll der ISO-C++ konforme version _getcwd denke ich. Es gibt keinen Punkt der Rückkehr eine const string, und Sie sollten free freigeben (zumindest laut MSDN):

string getcwd()
{
    char* a_cwd = _getcwd(NULL, 0);
    string s_cwd(a_cwd);
    free(a_cwd);
    return s_cwd;
}

Natürlich sollten Sie auch überprüfen, ob _getcwd() NULL zurück.

Müssen Sie prüfen, für a_cwd NULL. Dann funktioniert es auf Mac, Windows, Linux. Jedoch, es ist nicht POSIX-konform.

EDIT: perror nicht das Programm beenden, so sollten Sie verlassen, werfen Sie eine Ausnahme, oder etwas tun.

Als "string-Konstruktor" alles für Sie tun:

#include <stdio.h>  //defines FILENAME_MAX
#include <unistd.h> //for getcwd()

std::string GetCurrentWorkingDir()
{
    std::string cwd("\0",FILENAME_MAX+1);
    return getcwd(&cwd[0],cwd.capacity());
}