C++: Prawidłowa implementacja przekazywania std :: string do funkcji C, która chce zmienić ciąg znaków?

Question

Mam funkcję w bibliotece stron trzecich napisane w C: char* fix_filename_slashes(char* path). Ta funkcja oczekuje, że zmieniony łańcuch C zostanie przekazany do niej, aby mógł zmienić wszystkie ukośniki w ścieżce do poprawnego użycia w oparciu o system operacyjny. Wszystkie ciągi, których używam w mojej fasadzie, są zadeklarowane jako std::string s. Starałem się po prostu użyć foo.c_str() jak każdej innej funkcji, która oczekuje ciąg C nie ją zmienić i spodziewa się const char *, ale ta funkcja powoduje błąd: Error: Argument of type "const char *" is incompatible with parameter of type "char *"

jest wynikiem wymyśliłem:

char* tempf = const_cast<char*>(filename.c_str()); 
filename = std::string(fix_filename_slashes(tempf)); 
tempf = NULL; 

za "poprawne" lub czy istnieją inne (bardziej poprawne?) Sposoby wykonania tego zadania?

EDIT

Whups. Podobno funkcja zwraca KOPIĘ ciągu znaków. Nadal jest już kilka fajnych odpowiedzi.

Answer 1

Jeśli długość ciągu nie ulegnie zmianie, można użyć wskaźnika do pierwszego znaku ciągu. Jest to niezdefiniowane zachowanie w standardzie C++ 03, ale wszystkie znane implementacje działają poprawnie i są jawnie dozwolone w standardzie C++ 11.

fix_filename_slashes(&filename[0]); 

Jeśli rozmiar napisu może ulec zmianie, będziesz musiał wykonać trochę więcej pracy.

filename.resize(max_size, 0); 
append_filename_suffix(&filename[0]); 
filename.resize(strlen(filename.c_str())); 

Answer 2

przekształcić go w sekwencji nul znaków przechowywanych w std::vector: przykład

template <typename Character> 
std::vector<Character> to_vector(std::basic_string<Character> const& s) 
{ 
    std::vector<Character> v; 
    v.reserve(s.size() + 1); 
    v.insert(v.end(), s.begin(), s.end()); 
    v.push_back(0); 
    return v; 
} 

Zastosowanie:

std::string filename = get_filename(); 
std::vector<char> filename_cstr = to_vector(filename); 
filename = std::string(fix_filename_slashes(&filename_cstr[0])); 

Answer 3

Jeśli string stosowany oddzielny bufor do przechowywania ciąg c_str ten Wouldn Zmodyfikuj oryginalny ciąg.

Lepiej jest utworzyć bufor char na stosie lub sterty, skopiuj do niego znaki (zakończone znakiem null), wywołaj funkcję fix, a następnie przypisz bufor do łańcucha.

Answer 4

Ponieważ masz problemy, wystarczy spełnić wymagania funkcji C i skopiować ciąg znaków do tablicy znaków, a następnie po utworzeniu ciągu znaków z tablicy znaków lub wymusić przypisanie kopii do oryginału strunowy.

char* temp = new char[str.size() + 1] 
    // Force a copy of the result into another string 
    str = (const char*)fix_filename_slashes(strncpy(temp, str.c_str(), str.size() + 1)); 
    delete [] temp; 

Answer 5

Oto inne podejście, które wymaga niewielkiej konfiguracji, ale działa automatycznie po tym. Opiera się na tymczasowym obiekcie, który pobiera kopię oryginalnego łańcucha i kopiuje zmodyfikowany ciąg z powrotem do destruktora. Oczywiście całe to kopiowanie nie będzie zbyt wydajne, ale w większości przypadków sprawność nie będzie miała znaczenia.

class mutable_string 
{ 
public: 
    mutable_string(std::string & str, int maxlen = 0) : m_str(str) 
    { 
     m_buffer.resize(max(maxlen, str.length()) + 1); 
     memcpy(&m_buffer[0], str.c_str(), str.length()+1); 
    } 
    ~mutable_string() 
    { 
     m_str = m_buffer; 
    } 
    operator char*() 
    { 
     return &m_buffer[0]; 
    } 
private: 
    std::string &  m_str; 
    std::vector<char> m_buffer; 
}; 

fix_filename_slashes(mutable_string(filename));