Std::vector для строки с пользовательским разделителем
Я хотел бы скопировать содержимое vector
в один длинный string
с пользовательским разделителем. До сих пор я пробовал:
// .h
string getLabeledPointsString(const string delimiter=",");
// .cpp
string Gesture::getLabeledPointsString(const string delimiter) {
vector<int> x = getLabeledPoints();
stringstream s;
copy(x.begin(),x.end(), ostream_iterator<int>(s,delimiter));
return s.str();
}
но я получаю
no matching function for call to ‘std::ostream_iterator<int, char, std::char_traits<char> >::ostream_iterator(std::stringstream&, const std::string&)’
Я пробовал с charT*
, но я получаю
error iso c++ forbids declaration of charT with no type
Затем я попытался использовать char
и ostream_iterator<int>(s,&delimiter)
но я получаю странные символы в строке.
Может ли кто-нибудь помочь мне понять, что ожидает компилятор здесь?
Ответы
Ответ 1
Используйте delimiter.c_str()
как разделитель:
copy(x.begin(),x.end(), ostream_iterator<int>(s,delimiter.c_str()));
Таким образом, вы получаете const char*
, указывающий на строку, что означает ostream_operator
от вашего std::string
.
Ответ 2
Другой способ сделать это:
#include <iostream>
#include <string>
#include <vector>
#include <sstream>
using namespace std;
template <typename T>
string join(const T& v, const string& delim) {
ostringstream s;
for (const auto& i : v) {
if (&i != &v[0]) {
s << delim;
}
s << i;
}
return s.str();
}
int main() {
cout << join(vector<int>({1, 2, 3, 4, 5}), ",") << endl;
}
(С++ 11 на основе диапазона для цикла и "авто" )
Ответ 3
С++ 11:
vector<string> x = {"1", "2", "3"};
string s = std::accumulate(std::begin(x), std::end(x), string(),
[](string &ss, string &s)
{
return ss.empty() ? s : ss + "," + s;
});
Ответ 4
std::string Gesture::getLabeledPointsString(const std::string delimiter) {
return boost::join(getLabeledPoints(), delimiter);
}
Я не убежден в том, что в этот момент введю getLabeledPointsString
;)
Ответ 5
Это расширение для двух ответов, уже приведенных выше, поскольку производительность во время исполнения показалась темой в комментариях. Я бы добавил его в качестве комментариев, но у меня пока нет этой привилегии.
Я тестировал 2 реализации для производительности во время выполнения с помощью Visual Studio 2015:
Использование stringstream:
std::stringstream result;
auto it = vec.begin();
result << (unsigned short)*it++;
for (; it != vec.end(); it++) {
result << delimiter;
result << (unsigned short)*it;
}
return result.str();
Использование accumulate:
std::string result = std::accumulate(std::next(vec.begin()), vec.end(),
std::to_string(vec[0]),
[&delimiter](std::string& a, uint8_t b) {
return a + delimiter+ std::to_string(b);
});
return result;
Производительность развертывания релиза была близка к парам тонкостей.
Выполнение накопления было немного быстрее (20-50 мс, ~ 10-30% от общей продолжительности (~ 180 мс) на 1000 итераций по 256-элементному вектору). Однако реализация accumulate
выполнялась только быстрее, когда параметр a
для лямбда-функции передавался по ссылке. Передача параметра a
по значению привела к аналогичной разнице во времени во время реализации stringstream
. Реализация accumulate
также улучшила некоторые, когда возвратная строка возвращалась напрямую, а не была назначена локальной переменной, которая была немедленно возвращена. YMMV с другими компиляторами С++.
Отладочная сборка была в 5-10 раз медленнее с использованием accumulate
, поэтому я думаю, что дополнительное создание строки, отмеченное в нескольких комментариях выше, разрешено оптимизатором.
Я смотрел конкретную реализацию с использованием значений vector
uint8_t
. Полный тестовый код:
#include <vector>
#include <iostream>
#include <sstream>
#include <numeric>
#include <chrono>
using namespace std;
typedef vector<uint8_t> uint8_vec_t;
string concat_stream(const uint8_vec_t& vec, string& delim = string(" "));
string concat_accumulate(const uint8_vec_t& vec, string& delim = string(" "));
string concat_stream(const uint8_vec_t& vec, string& delimiter)
{
stringstream result;
auto it = vec.begin();
result << (unsigned short)*it++;
for (; it != vec.end(); it++) {
result << delimiter;
result << (unsigned short)*it;
}
return result.str();
}
string concat_accumulate(const uint8_vec_t& vec, string& delimiter)
{
return accumulate(next(vec.begin()), vec.end(),
to_string(vec[0]),
[&delimiter](string& a, uint8_t b) {
return a + delimiter + to_string(b);
});
}
int main()
{
const int elements(256);
const int iterations(1000);
uint8_vec_t test(elements);
iota(test.begin(), test.end(), 0);
int i;
auto stream_start = chrono::steady_clock::now();
string join_with_stream;
for (i = 0; i < iterations; ++i) {
join_with_stream = concat_stream(test);
}
auto stream_end = chrono::steady_clock::now();
auto acc_start = chrono::steady_clock::now();
string join_with_acc;
for (i = 0; i < iterations; ++i) {
join_with_acc = concat_accumulate(test);
}
auto acc_end = chrono::steady_clock::now();
cout << "Stream Results:" << endl;
cout << " elements: " << elements << endl;
cout << " iterations: " << iterations << endl;
cout << " runtime: " << chrono::duration<double, milli>(stream_end - stream_start).count() << " ms" << endl;
cout << " result: " << join_with_stream << endl;
cout << "Accumulate Results:" << endl;
cout << " elements: " << elements << endl;
cout << " iterations: " << iterations << endl;
cout << " runtime: " << chrono::duration<double, milli>(acc_end - acc_start).count() << " ms" << endl;
cout << " result:" << join_with_acc << endl;
return 0;
}
Ответ 6
int array[ 6 ] = { 1, 2, 3, 4, 5, 6 };
std::vector< int > a( array, array + 6 );
stringstream dataString;
ostream_iterator<int> output_iterator(dataString, ";"); // here ";" is delimiter
std::copy(a.begin(), a.end(), output_iterator);
cout<<dataString.str()<<endl;
output = 1; 2; 3; 4; 5; 6;
Ответ 7
string join(const vector<string> & v, const string & delimiter = ",") {
string out;
if (auto i = v.begin(), e = v.end(); i != e) {
out += *i++;
for (; i != e; ++i) out.append(delimiter).append(*i);
}
return out;
}
Несколько точек:
- вам не нужно дополнительное условие, чтобы избежать лишнего ограничителя.
- убедитесь, что вы не сработали, когда вектор пуст.
- не создавайте кучу временных данных (например, не делайте этого: x = x + d + y)
Ответ 8
более быстрый вариант:
vector<string> x = {"1", "2", "3"};
string res;
res.reserve(16);
std::accumulate(std::begin(x), std::end(x), 0,
[&res](int &, string &s)
{
if (!res.empty())
{
res.append(",");
}
res.append(s);
return 0;
});
он не создает промежуточные строки, а просто выделяет память один раз для всего результата строки и добавляет каждый элемент в конец & res