C++：sprintfでconst char *を印刷できないのはなぜですか？

Question

私はここで何が欠けていますか？それは私のナッツを運転している！

..... 
..... 
char str[50];  
sprintf(str, "%s", Notation()); 
..... 
..... 

しかしSTRは変わらない：

は、私がこれをやっているコードの別の部分で今のconstのchar *

const char* Notation() const 
{ 
    char s[10]; 
    int x=5; 
    sprintf(s, "%d", x); 
    return s; 
} 

を返す関数を持っています。

代わりに私がこれを行う場合：

..... 
..... 
char str[50]; 
str[0]=0; 
strcat(str, Notation()); 
..... 
..... 

STRが正しく設定されています。

私は予想通りはsprintfが動作しない理由を疑問に思って...

Answer 1

あなたがスタック上に割り当てられた配列を返すようにしようとしているし、その動作は未定義です。

ここ

const char* Notation() const 
{ 
    char s[10]; 
    int x=5; 
    sprintf(s, "%d", x); 
    return s; 
} 

sは、関数Notation()から戻ってきた後、周りのことを行っていません。スレッドの安全性に関心がなければ、sを静的にすることができます。両方の場合において

Answer 2

const char* Notation() const 
{ 
    static char s[10]; 
    .... 

Notation()が復帰に破壊されますローカル配列を返すように、それは、未定義の動作を呼び出します。あなたは不運です。それは1つのケースで働くので、はと感じています。

溶液としてstd::stringを使用することである。std::ostringstreamの

char str[50];  
sprintf(str, "%s", Notation().c_str()); 

利点（および美）：

として

std::string Notation() const 
{ 
    char s[10]; 
    int x=5; 
    sprintf(s, "%d", x); 
    return s; //it is okay now, s gets converted into std::string 
} 

またはC++ストリームを使用して：

std::string Notation() const 
{ 
    int x=5; 
    std::ostringstream oss; 
    oss << x; 
    return oss.str(); 
} 

、次いで（とstd::string）は、出力iのサイズを知る必要がないということですこれは、配列宣言char s[10]で10のようなマジックナンバーを使用する必要がないことを意味します。これらのクラスはその意味で安全です。

Answer 3

char s[10]Notationがスタックに置かれますので、Notation機能を終了した後に破壊されます。そのような変数はautomaticと呼ばれます。

char *s = new char[10]; 

しかし、あなたは手動でこのメモリを解放する必要があります：あなたは newを使ってヒープ内のあなたの文字列を保存する必要があり

char str[50]; 
const char *nt = Notation(); 
sprintf(str, "%s", nt); 
printf("%s", str); 
delete[] nt; 

をあなたが本当にその後、ビルトイン使用stringクラスNawazが提案のようにC++を使用している場合。何らかの理由で生ポインタに制限されている場合は、Notationの外側にバッファを割り当てて、またはstrcatのようにdestenderのパラメータを渡します。