Форум программистов [Powered by Invision Power Board]

Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате

Форум программистов > C/C++: Общие вопросы > Корректно ли так использовать c_str() метод

Автор: zhekha 23.1.2012, 19:27

Код



extern int major, minor;

const char* getVersion() {

    std::stringstream str;
    str << major << "." << minor;
    return str.str().c_str();
}

Чем это отличается от следующего?

Код


const char* getVersion() {

    return "1.2";
}

Утверждается, что в первом случае переданный указатель становится недействительным...

Автор: newbee 23.1.2012, 19:32

Некорректно, после выполнения функции строка будет удалена, а указатель, когда-то ей принадлежащий, станет битым.

Цитата(zhekha @ 23.1.2012, 20:27

)

Чем это отличается от следующего?

Во втором случае ты возвращаешь указатель на статическую область памяти программы, она будет действительна всегда на протяжении работы программы.

Сделай так:

Код


char const getVersion(){
 static char buf[42]={0};
 if(buf[0]==0){
  /*заполни буфер нужной версией*/
  }
 return buf;}

Автор: zhekha 23.1.2012, 20:12

Или, как вариант, возвращать string

Код


extern int major, minor;

std::string getVersion() const {

    std::stringstream str;
    str << major << "." << minor;
    return str.str();
}

Или это тоже некорректно?

Автор: Michrutka 24.1.2012, 10:32

zhekha,
Во втором случае ты возвращаешь объект std::string, следовательно, будет вызван копирующий конструктор в строчке

Код


std::string version = getVersion();

то есть при выходе из функции, вызовется копирующий конструктор, значение присвоится переменной version, а потом локальные переменные функции будут уничтожены.

Потому все корректно.

Автор: bsa 24.1.2012, 10:36

Цитата(Michrutka @ 24.1.2012, 11:32

)

то есть при выходе из функции, вызовется копирующий конструктор, значение присвоится переменной version, а потом локальные переменные функции будут уничтожены.

Более того, если компилировать под стандарт C++11, то вместо копирующего конструктора будет вызван перемещающий, работа которого сведется к копированию значений указателей (без перевыделения памяти).

Автор: zhekha 24.1.2012, 17:02

Всем спасибо, вопрос исчерпан.

Автор: bsa 24.1.2012, 17:16

Цитата(zhekha @ 24.1.2012, 18:02

)

Всем спасибо, вопрос исчерпан.

Ну так пометь тему решенной