文字列をchar *に変換する*

Question

std :: stringをchar *に変換する関数を記述しようとしています。 私が書かれている最初のものは、このでした：

char* $ (string str) 
{ 
    char* cstr; 
    const unsigned int length=str.size(); 
    cstr=new char[1000]; 
    for(int i=0;i<length;i++) 
     cstr[i]=str[i]; 
    cstr[length]=0; 
    return cstr; 
} 

しかし、問題は、メモリリークが発生した：のは、私はこれを行うと仮定してみましょう：

char* cstr; 
string str1("hello"),str2("hello2"); 
cstr=$(str1); 
cstr=$(str2); 

このcase.Theでメモリリークが最初にあります割り当てられた文字列は到達可能ではありませんが、参照は失われています。

char* $ (string str) 
{ 
    static char cstr[1000]; 
    const unsigned int length=str.size(); 
    for(int i=0;i<length;i++) 
     cstr[i]=str[i]; 
    cstr[length]=0; 
    return cstr; 
} 

しかし、問題は今、静的char型のフィールドがアクセス可能であることである： は、だから私は、静的な使用して同じにする

char* cstr; 
string str("hello"); 
cstr=$(str); 
$(str)[5]='!'; 

これはpossibileで、第六文字が変更され、これもCされますcstrが指す文字列が変更されます。

const char* $ (string str) 
{ 
    static char cstr[1000]; 
    const unsigned int length=str.size(); 
    for(int i=0;i<length;i++) 
     cstr[i]=str[i]; 
    cstr[length]=0; 
    return cstr; 
} 

問題はchar型のポインタがconstのchar型のポインタに対応していないということですので、私はこれを行うことはできません：

string str("hello"); 
char* cstr; 
cstr=$(str); 

をしかし、私は唯一使用できるのconstを使用して

const charポインタ 私は、戻り値が右のオペランドとしてしか置かれるが、代入の左のオペランドには配置されない関数を持つことです。これはどのように行うことができますか？

私はこれを実行しようとしました：

char* toCharArray(string& str) 
{ 
    std::unique_ptr<char>p(new char[1000]); 
    char* temp=p.get(); 
    for(unsigned int i=0;i<str.size();i++) 
    { 
     *temp=str[i]; 
     temp++; 
    } 
    return p.get(); 
} 

しかし、問題はまだそこにある、私はこれと私はstatic.Sinceにこのようなコードを使用して掲載、他のソリューションとの違いを見ていない：

char* cstr; 
string str("hello"); 
cstr=toCharArray(str); 
toCharArray(str)[0]='o'; 
cout << cstr; 

文字列を変更します（ "oello"を出力します）。 問題はまだ解決されません。

Answer 1

http://www.cplusplus.com/reference/string/string/c_str/

、具体的な例を見て、：

char * cstr, *p; 

string str ("Please split this phrase into tokens"); 

cstr = new char [str.size()+1]; 
strcpy (cstr, str.c_str()); 

Answer 2

約str.c_str()？コピーが必要な場合はstrdup(str.c_str())となります。

Answer 3

char *を新規作成し、std::stringの内容をコピーする必要があります。

strcpyを使用できます。

Answer 4

あなたがメモリリークに驚いている理由私は表示されません。何かのコピーを作っているなら、あなたはどこかに置く必要があります。

すべてのコピーを同じ場所に置くことができます（オーバーフローの危険性があり、クライアントに適切なコピーを取るか、予期しない破損の危険性があります）、または割り当てを行います（彼らはまだフリーズする必要がありますが、少なくともコピーする必要はありません）、またはバッファとサイズを関数に渡すようにします。

真剣に、関数名として$？

Answer 5

割り当てられた配列をstd::unique_ptr<char[]>またはstd::vector<char>のいずれかに戻すことでメモリリークを防ぐことができます。それらが再割り当てされるか範囲外になると、メモリは両方とも解放されます。

char*の内容を指すポインタは、それぞれptr.get()または&vec[0]となります。

ところで、長さが分かっているので、実際には配列の長さはlength+1でなく、1000でなければなりません。固定サイズのバッファは、オーバーランが発生するのを待っています。また、$は、関数の移植可能な名前ではありません。