C++：为什么我不能用 sprintf 打印 const char*？

Question

我在这里缺少什么？快把我逼疯了！

我有一个返回 const char*

的函数

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

现在在代码的另一部分我正在这样做：

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

但 str 保持不变。

如果我这样做：

.....
.....
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;
}

在您从函数Notation() 返回后，这里的s 将不会出现。如果您不关心线程安全，您可以将s 设为静态。

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

Answer 2

在这两种情况下，它都会调用未定义的行为，因为Notation() 返回一个本地数组，该数组在返回时被销毁。你不走运它在一种情况下有效，让你觉得它是正确的。

解决方法是使用std::string作为：

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(); 
}

然后：

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

std::ostringstream（和std::string）的好处（和美丽）是你不必事先知道输出的大小，这意味着你不必使用诸如@987654328之类的幻数@ 在数组声明中char s[10]。从这个意义上说，这些类是安全的。

Answer 3

Notation 中的char s[10] 被放置在堆栈上，因此在退出Notation 函数后它会被销毁。此类变量称为automatic。您需要使用new 将字符串保存在堆中：

char *s = new char[10];

但是你必须手动释放这个内存：

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

如果您真的使用 C++，请使用内置的 string 类，如建议的 Nawaz。如果您以某种方式限制为原始指针，则在 Notation 之外分配缓冲区并将其作为 destination 参数传递，就像在 sprintf 或 strcat 中一样。