Как эффективно скопировать std::string в вектор

Если вам действительно нужен вектор (например, потому что ваша функция C изменяет содержимое строки), то следующее должно дать вам то, что вы хотите, в одной строке:

std::vector<char> v(s.c_str(), s.c_str() + s.length() + 1);

Поскольку c_str() возвращает строку с завершающим нулем, вы можете просто скопировать ее целиком в вектор.

Однако на самом деле я не уверен, насколько оптимизирован этот конструктор. Я действительно знаю, что std::copy оптимизирован настолько, насколько это возможно, поэтому, возможно (измерьте!) следующее быстрее:

std::vector<char> v(s.length() + 1);
std::copy(s.c_str(), s.c_str() + s.length() + 1, v.begin());

Если функция C не изменяет строку, просто передайте c_str() напрямую и отбросьте константность. Это безопасно, пока функция C читает только строку.

В большинстве случаев вам не нужен вектор char, так как std::string в значительной степени является контейнером char. std::string также имеют функции begin и end. И у него также есть функция c_str(), которая возвращает c-строку, которую вы можете передать любой функции, которая ожидает const char*, например:

void f(const char* str); //c-function

std::string s="some string";
f(s.c_str());

Итак, зачем вам std::vector<char>?

На мой взгляд, vector<char> очень-очень редкая потребность, но если она мне когда-нибудь понадобится, я, наверное, напишу это:

std::vector<char> v(s.begin(), s.end());

И для меня v.push_back('\0') не имеет особого смысла. Нет такого требования к вектору, чтобы последний элемент был '\0', если value_type равно char.

Хорошо, как вы сказали, std::string::c_str() возвращает const char*, а c-функция нуждается в неконстантном char* , тогда вы можете использовать std::vector, потому что хотите воспользоваться RAII, который реализует вектор:

void g(char* s); //c-function

std::vector<char> v(s.begin(), s.end());
s.push_back('\0');

g(&v[0]);

что мне кажется хорошо. Но RAII — это все, что вам нужно, тогда у вас есть и другой вариант:

{
  std::vector<char> memory(s.size()+1);
  char *str = &memory[0]; //gets the memory!
  std::strcpy(str, s.c_str());

  g(str);
  //....

} //<--- memory is destroyed here.

Используйте std::strcpy, std::memcpy или std::copy в зависимости от того, что быстрее, так как я не могу сказать, какой из них обязательно быстр, без профилирования.

Я не думаю, что std::strcpy(&v[0],s.c_str()); - хороший выбор. Я думаю, что c_str() разрешено перераспределять.

Если вам каким-то образом "нужен" \0 для работы с C-API, то полагайтесь на string::c_str(), который предоставит его вам по запросу. Не думайте, что вам нужно поместить его в вектор-символ, большинство вещей, которые вы можете сделать с самой строкой, как с вектором.

Обновление:

Если вы убедитесь, что ваш вектор инициализируется с помощью 0s, вы можете обойти вызов c_str, используя strncopy:

std::vector<char> v(s.length()+1, 0);  // added '0'
std::strncpy(&v[0],&s[0],s.length());  // no c_str()