c++ - 将整数转换为 C++ 中的字符指针，然后将其附加到另一个字符指针

Question

我有以下代码，用于将整数(得分)转换为字符，然后将其附加到玩家的名字 (player1)。之后显示。它是一个更大项目的一部分:

    #include <iostream>
#include <string.h>
using namespace std;

char* convertIntTochar(int number)
{
    char t[3];
    t[0] = 0;
    t[1] = 0;
    t[2] = '\0';

    int i = 0;
    for(; number != 0; i++)
    {
        t[i] = ((number%10) + 48);
        number/=10;
    }

    if(i == 2)
    {
        char temp = t[0];
        t[0] = t[1];
        t[1] = temp; 
    }
    else
        t[i] = '\0';
    char *ans = t;
    return ans;
}

int main()
{
    char str11[] = "Player1: ";
    char *str1 = str11;
    char *str2 = convertIntTochar(11);
    strcat(str1 , str2);

    while(*str1)
    {
        cout<<*(str1++);
    }

    return 0;
}

它编译正确，但是当我运行它时，它显示以下错误:

*** stack smashing detected ***: ./a.out terminated
======= Backtrace: =========
/lib/tls/i686/cmov/libc.so.6(__fortify_fail+0x50)[0x9b3390]
/lib/tls/i686/cmov/libc.so.6(+0xe233a)[0x9b333a]
./a.out[0x80487ff]
/lib/tls/i686/cmov/libc.so.6(__libc_start_main+0xe6)[0x8e7bd6]
./a.out[0x8048621]
======= Memory map: ========
00110000-00134000 r-xp 00000000 08:06 2887608    /lib/tls/i686/cmov/libm-2.11.1.so
00134000-00135000 r--p 00023000 08:06 2887608    /lib/tls/i686/cmov/libm-2.11.1.so
00135000-00136000 rw-p 00024000 08:06 2887608    /lib/tls/i686/cmov/libm-2.11.1.so
004b9000-004d4000 r-xp 00000000 08:06 2887597    /lib/ld-2.11.1.so
004d4000-004d5000 r--p 0001a000 08:06 2887597    /lib/ld-2.11.1.so
004d5000-004d6000 rw-p 0001b000 08:06 2887597    /lib/ld-2.11.1.so
0077d000-00866000 r-xp 00000000 08:06 2756275    /usr/lib/libstdc++.so.6.0.13
00866000-00867000 ---p 000e9000 08:06 2756275    /usr/lib/libstdc++.so.6.0.13
00867000-0086b000 r--p 000e9000 08:06 2756275    /usr/lib/libstdc++.so.6.0.13
0086b000-0086c000 rw-p 000ed000 08:06 2756275    /usr/lib/libstdc++.so.6.0.13
0086c000-00873000 rw-p 00000000 00:00 0 
008d1000-00a24000 r-xp 00000000 08:06 2887604    /lib/tls/i686/cmov/libc-2.11.1.so
00a24000-00a25000 ---p 00153000 08:06 2887604    /lib/tls/i686/cmov/libc-2.11.1.so
00a25000-00a27000 r--p 00153000 08:06 2887604    /lib/tls/i686/cmov/libc-2.11.1.so
00a27000-00a28000 rw-p 00155000 08:06 2887604    /lib/tls/i686/cmov/libc-2.11.1.so
00a28000-00a2b000 rw-p 00000000 00:00 0 
00a3b000-00a58000 r-xp 00000000 08:06 2883667    /lib/libgcc_s.so.1
00a58000-00a59000 r--p 0001c000 08:06 2883667    /lib/libgcc_s.so.1
00a59000-00a5a000 rw-p 0001d000 08:06 2883667    /lib/libgcc_s.so.1
00b74000-00b75000 r-xp 00000000 00:00 0          [vdso]
08048000-08049000 r-xp 00000000 08:06 4719693    /home/dhruv/Desktop/a.out
08049000-0804a000 r--p 00000000 08:06 4719693    /home/dhruv/Desktop/a.out
0804a000-0804b000 rw-p 00001000 08:06 4719693    /home/dhruv/Desktop/a.out
08b67000-08b88000 rw-p 00000000 00:00 0          [heap]
b77f7000-b77f9000 rw-p 00000000 00:00 0 
b780d000-b7810000 rw-p 00000000 00:00 0 
bfd2a000-bfd3f000 rw-p 00000000 00:00 0          [stack]
Player1: "�ӿ�XMAborted

这是什么原因？怎么改正。我已经将空终止字符放入 convertIntTochar 函数中。

Answer 1

这里有很多问题...

1. convertIntTochar 仅适用于 2 位数字。没有检查。
2. char t[3] 在 convertIntTochar 中定义的是一个局部变量，你不能返回一个指向它的指针并在 convertIntTochar 之外使用这个指针。
3. strcat(str1 , str2); 追加到一个已经满的数组 (str11)，因此您可以覆盖堆栈。

就换成std::strings，会更简单