C++のstringのコンストラクタの使い方

Question

C++のstringコンストラクタを
http://www.cppll.jp/cppreference/cppstring_details.html#Constructors
を見ながら使おうとしてよくわからないことになりました。

FILE * stream
char * hoge;
std::string str

fgets(hoge,BUFF,stream);
str=str(hoge);

とすると、
‘(std::string {aka std::basic_string<char>}) (char*&)’ への呼び出しが適合しない
というエラーが出てしまいます。

引数が"fugafuga"っていう直接入力じゃないとダメなのでしょうか？
もしそうであれば、char * 型変数をstringにするにはどうすればいいのでしょうか？

当方の環境は、Cygwinで g++ -std=gnu++14でコンパイルしています。
よろしくお願いいたします。

Answer 1

最後の行

str = std::string(hoge);

ではないでしょうか。

Comments

[chijan]

ご回答ありがとうございます。
syntaxエラーを指摘されるエラーがでなくて気づきませんでした。以後気をつけます。

Answer 2

std::stringコンストラクタの正しい使い方は、ｍoscmoc-1970さん回答の通りです。他にもstd::string型変数へC文字列(char*型)を代入する方法はいろいろあります。

str = std::string(hoge);
// または
str = hoge;
// または
str.assign(hoge);

このコードでは、その前の「ストリームから文字列を読み込む処理」に大きな問題があります。下記引用コードは、どう頑張っても正常に動作しません。（たまたま動いたように見える可能性はありますが、大抵はプログラムクラッシュすると思います）

char * hoge;
fgets(hoge,BUFF,stream);

変数streamの型が明示されていないため推測となりますが、もしFILE*型ならば変数hogeを配列型char hoge[BUFF];として宣言する必要があります。<del>そうではなくC++のstd::istream系列であれば、根本的な誤りがあります。</del>

---

おまけ：簡単ですがエラーメッセージの解説・理由も付けておきます。

str=str(hoge); とすると
‘(std::string {aka std::basic_string<char>}) (char*&)’ への呼び出しが適合しない

誤って書いたstr=str(hoge);では、まず=の右辺str(hoge)が評価されます。C++言語の文法ではこの右辺を「変数strを関数オブジェクトとみなし、その引数にhogeを渡して呼び出す」部分式と解釈します。しかし実際には、変数strは関数オブジェクトでは無くstd::string型、正式名はstd::basic_string<char>型です。エラーメッセージの「呼び出しが適合しない」というのは、このような原因で出力されています。

Comments

[chijan]

ご回答ありがとうございます。
ほかの方法もあったのですね、勉強になります。

ご推察のとおり、FILE*です。fgetsの３つ目の引数なのでいいだろうと思って略しましたごめんなさい。

[chijan]

おまけについて。
なるほどです、解読はできる書き方だったためにsyntaxを突かれるエラーは出ず、呼び出しが適合しないということになったのですか……。

Answer 3

あらかじめstrを用意するのではなく

fgets(hoge,BUFF,stream);
std::string str(hoge);

で解決しました。お騒がせしました。