Преобразование целого числа в символьный указатель в 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, это будет проще.

Answer 2

   char str11[] = "Player1: ";

Это проблема. Недостаточно места для объединения строк. Попробуйте это:

   char str11[100] = "Player1: ";

Еще лучше, используйте std::string вместо C-like char*. Наименьшие возможные изменения, которые исправляют проблемы со строками, таковы (поскольку существует using namespace std, тогда std:: часть в std::string может быть пропущена, но я просто предпочитаю оставить ее в):

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

std::string convertIntTochar(int number)
{
    ...
}

int main()
{
    std::string str1 = "Player1: ";
    std::string str2 = convertIntTochar(11);
    str1 += str2;

    cout << str1;

    // Or even more effective, just one line of code:
    cout << "Player1: " << convertIntTochar(11);

    return 0;
}

Answer 3

Используйте std :: strings и std :: ostringstreams, это намного проще.

#include <sstream>
#include <iostream>
std::ostringstream player_score_stream;
player_score_stream << "Player1: " << score_as_an_integer;
std::string player_score(player_score_stream.str());
std::cout << player_score;

и если вы хотите использовать C-строку только для чтения, используйте player_score.c_str (), которая возвращает const char*

Answer 4

Чтобы преобразовать int в char *, посмотрите, можете ли вы использовать itoa с вашим компилятором.
Если он не поддерживается, вы можете найти его реализацию, чтобы делать то, что вы хотите.
Это если вам нужносделать это с помощью C-строк