stringstreamをchar配列へ変換する方法

Question

stringstreamからchar配列へ変換するより簡単な方法が存在すれば教えていただきたいです。
現在、以下のように変換をしています。

int main() {
    std::stringstream ss;
    ss << "hello" << "," << "world";
    char char_array[256]{};
    for (int i = 0; i < ss.str().length(); ++i) char_array[i] = ss.str().at(i);
    std::cout << char_array << std::endl;
}

問題なく変換はできているのですが、ループさせるのが綺麗な方法ではない気がしています。別の方法がないか知りたいです。

Answer 1

getというメンバ関数があるので、それが使えます。getは最後に'\0'を追加します。

int main() {
    std::stringstream ss;
    ss << "hello" << "," << "world";
    char char_array[256];
    ss.get(char_array, 256);
    std::cout << char_array << std::endl;

Comments

[__ook]

ありがとうございます！メンバ関数があるのですね。勉強になりました。

Answer 2

#include <iostream>   // cout, endl
#include <sstream>    // stringstrem
#include <algorithm>  // copy

int main()
{
    std::stringstream ss;
    ss << "hello" << "," << "world";
    char char_array[256];
	const char *s = ss.str().c_str();
	std::copy(s, s + ss.str().size() + 1, char_array);
    std::cout << char_array << std::endl;
}

char char_array[256]{}; のように 0 で初期化しているのであれば、
copy の + 1 は不要です。

copy(s, s + ss.str().size() + 1, char_array); の代わりに、
copy_n(s, ss.str().size() + 1, char_array); でもかまいません。

追記

stringのc_str()が駄目なのではなく、stringstreamのstr()が返すstringは一時オブジェクトなため式の終わりまでしか寿命が無いのです。

SHOMIさん、ご指摘ありがとうございます。その通りですね。
次のように書けば大丈夫でしょう。

#include <iostream>   // cout, endl
#include <sstream>    // stringstrem
#include <algorithm>  // copy_n

int main()
{
    std::stringstream ss;
    ss << "hello" << "," << "world";
    char char_array[256]{};
	std::string s = ss.str();
	std::copy_n(s.begin(), s.size(), char_array);
    std::cout << char_array << std::endl;
}

最初、char_array が十分なサイズであれば、ss >> char_array; でも良いかな
と考えたのですが、ss にスペースが挿入されていた場合、>> では、そこで途切れ
るため、ss.str() を使うことにしました。

Beared-Ockhamさんの回答にあるように
ss.get(char_array, sizeof char_arrya); でよかったんですね。
勉強になります。

Comments

[SHOMI]

>const char *s = ss.str().c_str();
stringstreamのstr()が返すのは一時オブジェクトなので、c_str()でポインタを取って後からアクセスするのは駄目ですよ。

[__ook]

stringのc_str()ももしかしてだめなのでしょうか？
std::string s = "hello, world";
const char* p = s.c_str();
std::cout << p << std::endl;
のように書くのは…

[SHOMI]

いいえ。pを使用するまでにsが破棄されたり、sの内容が変更されたりしていなければ問題ありません。
stringのc_str()が駄目なのではなく、stringstreamのstr()が返すstringは一時オブジェクトなため式の終わりまでしか寿命が無いのです。

[__ook]

なるほど。stringのc_str()はstring内の文字列ポインタを渡しており、stringstreamのstr()はstringのオブジェクトを返しているのですね。ありがとうございます。

[SHOMI]

kazuma-sさんの追記のように一旦string変数に代入してそちらのc_str()を使用するか、
const参照(const std::string& s = ss.str();)や右辺値参照(std::string&& s = ss.str();)で一時オブジェクトを延命してそちらのc_str()を使用する、もしくは
std::copy_n(ss.str().c_str(), ss.str().size() + 1, char_array);
と一文で書いた場合は問題ありません。

>SHIOM
細かいですがSHOMIです…

[kazuma-s]

名前を間違えてすみません。修正しました。

Answer 3

strcpy と言う関数があります

＃まあ、stringにできるなら文字配列にできますわな

Comments

[__ook]

strcpy(char_array, ss.str().c_str());
のようにするのがベストでしょうか。
C++の関数にはないということでしょうか