Переменная C ++ используется без инициализации?

Question

Я получаю эту ошибку:

Переменная «thename» используется без инициализации

Вот мой код:

char *thename;
cm++;
sprintf(thename, "tutmap%d.map", cm);

Answer 1

Вы используете переменную без инициализации, и выполнение этого кода будет неопределенным поведением.

Возможно, вы имели в виду:

char thename[42];
cm++;
sprintf(thename, "tutmap%d.map", cm);

Если у вас есть snprintf, вот функция, которая документируетчто вы гарантируете («подтвердите»), что буфер достаточно большой, но затем также проверяет длину буфера и прерывает работу, если вы допустили ошибку:

template<int N>
void fixed_sprintf(char (&array)[N], char const *format, ...) {
  va_list args;
  va_start(args, format);
  int used = vsnprintf(array, N, format, args);
  va_end(args);
  if (used == N - 1) {
    throw whatever_exception_type_you_like("buffer too small");
    // or even:
    abort();
  }
}

«Фиксированный» означает «фиксированный размер»,не "противоположность сломанной".:)

Answer 2

Вы объявили указатель на строку C, но для нее не выделено никакой памяти.

Если вы хотите, чтобы память выделялась стеком, используйте

char thename[buffer_length];

Если вы предпочитаете выделение кучи, используйте

char *thename = malloc(buffer_length);

Я знаю, что вопрос помечен C ++, но ваш код для меня больше похож на C. Следовательно использование malloc.

Answer 3

Вместо записи:

char *thename;
cm++;
sprintf(thename, "tutmap%d.map", cm);

Запись:

cm++;
std::stringstream stream;     
stream << "tutmap" << cm << ".map";
std::string const name = stream.str();
// and if you really need char* then:
char const* thename = name.c_str();

Answer 4

Это довольно легко понять:

char *thename;

не инициализирует thename, а затем:

sprintf(thename, "tutmap%d.map", cm);

использует это. В этом случае с печальными результатами.

Answer 5

thename - указатель, который не был инициализирован. Измените thename на массив char, который будет соответствовать размеру полной строки, сгенерированной sprintf.

Answer 6

char *thename = new char[21]; //21 is used for maximum string length of 20
cm++;
sprintf(thename, "tutmap%d.map", cm);