Почему этот код вылетает при запуске?

Question

[Даниил Демидко]

Я хочу спокойно без лишних велосипедов конвертировать строки в числа и обратно.
Для этого я пишу оболочки для стандартной atof(char*) из stdlib.h и нестандартной функции, расширения gcc, itoa(int, char*, int) из того же stdlib.h
С первой функцией проблем не возникло, а вот со второй пришлось поломать голову.

Вот рабочий пример на C++:

#include <iostream>
#include <cstdlib>

int GetInt( const int  &number )
{
    char *result;
    return itoa(number, result, 10);
}

int main()
{
    std::cout << GetInt( 475); // Все прекрасно работает
}

Но вот подвох - так как функция нестандартная, она просто не будет подключаться если компилировать код c -c++11 или -c++14 .
Возиться с макросами неохота, да и нехорошо, поэтому я просто добавляю в проект файл Numbers.c:

// Numbers.c
#include <stdlib.h>

int GetInt(const int number)
{
    char *result;
    return itoa(number, result, 10);
}

Так как теперь это сишный код, то проблемы с включением itoa теперь быть не должно, и проект можно компилировать с -c++14

Правим main.cpp:

#include <iostream>

extern "C" int GetInt(const int);

int main()
{
    std::cout << GetInt( 475); 
    // Компилируется, но при запуске сразу вылетает
}

Почему все вылетает?

Answer 1

[Ivan Bogachev]

Я хочу спокойно без лишних велосипедов конвертировать строки в числа и обратно.

Так и конвертируйте, кто вам мешает?

#include <iostream>
#include <string>

int main()
{
    std::string number = "123456";

    int value = std::stoi(number);
    std::cout << value; // 123456 (int)

    std::string number2 = std::to_string(value);
    std::cout << number2; // 123456 (string)
}

Comments

[Даниил Демидко]

Я бы рад, но у меня почему то не работает:
||=== Build: Debug in CPPExpr (compiler: GNU GCC Compiler) ===|
In function 'int main()':|
main.cpp|8|error: 'stoi' is not a member of 'std'|
main.cpp|11|error: 'to_string' is not a member of 'std'|
||=== Build failed: 2 error(s), 0 warning(s) (0 minute(s), 0 second(s)) ===|

[Даниил Демидко]

флаг C++14 включен

[Ivan Bogachev]

Даниил Демидко: Проверьте флаги еще раз, у gcc версии 4.8.5 (к примеру) для C++14 нужно использовать -std=c++1y, а для C++11 можно просто -std=c++11. А ваши ошибки возникают при -std=c++98

Answer 2

[MiiNiPaa]

Ну, вы пишете результат конверсии в память, на которую указывает указатель result. А он указывает... куда?

И чем itos/to_string не угодили? Зачем велосипеды?

EDIT:

так как функция нестандартная, она просто не будет подключаться если компилировать код c -c++11 или -c++14 .

Простите, что?

Comments

[Даниил Демидко]

Там в stdlib.h есть такое:

#if !defined (__STRICT_ANSI__)

/*
* NOTE: Officially the three following functions are obsolete. The Win32 API
* functions SetErrorMode, Beep and Sleep are their replacements.
*/

_CRTIMP char* __cdecl __MINGW_NOTHROW _itoa (int, char*, int);
_CRTIMP char* __cdecl __MINGW_NOTHROW _ltoa (long, char*, int);
_CRTIMP char* __cdecl __MINGW_NOTHROW _ultoa(unsigned long, char*, int);
_CRTIMP wchar_t* __cdecl __MINGW_NOTHROW _itow (int, wchar_t*, int);
_CRTIMP wchar_t* __cdecl __MINGW_NOTHROW _ltow (long, wchar_t*, int);
_CRTIMP wchar_t* __cdecl __MINGW_NOTHROW _ultow (unsigned long, wchar_t*, int);

#ifdef __MSVCRT__
_CRTIMP __int64 __cdecl __MINGW_NOTHROW _atoi64(const char *);
_CRTIMP char* __cdecl __MINGW_NOTHROW _i64toa(__int64, char *, int);
_CRTIMP char* __cdecl __MINGW_NOTHROW _ui64toa(unsigned __int64, char *, int);
_CRTIMP __int64 __cdecl __MINGW_NOTHROW _wtoi64(const wchar_t *);
_CRTIMP wchar_t* __cdecl __MINGW_NOTHROW _i64tow(__int64, wchar_t *, int);
_CRTIMP wchar_t* __cdecl __MINGW_NOTHROW _ui64tow(unsigned __int64, wchar_t *, int);

[Даниил Демидко]

Вот поэтому и подумал, что нестандартая

[MiiNiPaa]

Даниил Демидко: Расширения компилятора можно подключать в независимости от текущего стандарта. Хотя это хорошая идея от них отказываться. Так чем вам С++11 stoi/to_string и компания не угодили?

[Даниил Демидко]

MiiNiPaa: Не работают. У меня MinGW

[MiiNiPaa]

Даниил Демидко: Обновите MinGW (Например на сборку mingw-w64). У меня всё работает.

Answer 3

[Петр]

А память под результат кто выделять будет?

char *result; // не инициализированна
return itoa(number, result, 10);

Для 32 битного числа рекомендуется как минимум 33 байта. И так как вы возращаете результат из функции, то нельзя будет выделять эту память на стеке и поэтому придется далее позаботиться об освобождении выделенной памяти.

Comments

[MiiNiPaa]

Значение 32битной переменной с максимальной длминой — −2147483648. 11 символов. Даже если добавить разделители (+3) и нулевой символ (+1) получается 15 байт. Куда ещё 17?

[Петр]

MiiNiPaa: Согласен. 33 это для бинарного представления.