Каково время жизни результата std::string::c_str()?

Результат c_str() становится недействительным, если std::string уничтожается или если вызывается неконстантная функция-член строки. Таким образом, обычно вы захотите сделать копию, если вам нужно сохранить ее.

В случае вашего примера кажется, что результаты c_str() используются безопасно, потому что строки не изменяются в этой области. (Однако мы не знаем, что use_foo() или ~Foo() могут делать с этими значениями; если они копируют строки в другое место, то они должны сделать настоящую копию, а не просто скопировать указатели char.)

Технически ваш код в порядке.

НО вы написали таким образом, что его легко сломать тому, кто не знает код. Единственное безопасное использование c_str() — это когда вы передаете его в качестве параметра функции. В противном случае вы откроете для себя проблемы с обслуживанием.

Пример 1:

{
  std::string server = "my_server";
  std::string name   = "my_name";

  Foo foo;
  foo.server = server.c_str();
  foo.name = name.c_str();

  //
  // Imagine this is a long function
  // Now a maintainer can easily come along and see name and server
  // and would never expect that these values need to be maintained as
  // const values so why not re-use them

  name += "Martin";
  // Oops now its broken.

  // We use foo
  use_foo(foo);

  // Foo is about to be destroyed, before name and server
}

Итак, для обслуживания сделайте это очевидным:

Лучшее решение:

{
  // Now they can't be changed.
  std::string const server = "my_server";
  std::string const name   = "my_name";

  Foo foo;
  foo.server = server.c_str();
  foo.name = name.c_str();

  use_foo(foo);    
}

Но если у вас есть константные строки, они вам на самом деле не нужны:

{
  char const* server = "my_server";
  char const* name   = "my_name";

  Foo foo;
  foo.server = server;
  foo.name   = name;

  use_foo(foo);
}

OK. По какой-то причине вы хотите, чтобы они были строками:
Почему бы не использовать их только в вызове:

{
  std::string server = "my_server";
  std::string name = "my_name";

  // guaranteed not to be modified now!!!     
  use_foo(Foo(server.c_str(), name.c_str());
}

Он действителен до тех пор, пока с соответствующим объектом string не произойдет одно из следующих событий:

• объект уничтожен
• объект изменен

С вашим кодом все в порядке, если только вы не измените эти объекты string после того, как c_str() будут скопированы в foo, но до вызова use_foo().

Возвращаемое значение c_str() допустимо только до следующего вызова непостоянной функции-члена для той же строки.

const char*, возвращенный из c_str(), действителен только до следующего неконстантного вызова объекта std::string. В этом случае у вас все в порядке, потому что ваш std::string все еще находится в области действия в течение времени жизни Foo, и вы не выполняете никаких других операций, которые могли бы изменить строку при использовании foo.

Пока строка не уничтожена и не изменена, использование c_str() допустимо. Если строка изменена с использованием ранее возвращенного c_str(), это определяется реализацией.

Для полноты картины приведем ссылку и цитату с сайта cppreference.com:

Указатель, полученный от c_str(), может быть признан недействительным:

• Передача неконстантной ссылки на строку любой стандартной библиотечной функции или
• Вызов неконстантных функций-членов для string, за исключением operator[], at(), front(), back(), begin(), rbegin(), end() и rend().