C++ operator=(char* p)で警告が出る

Question

中学校でコンピュータ部に所属している者です。

JavaやC#の様なstring型をC++で実現しようとstrクラスを作っているのですが,
g++でコンパイルする度に警告がでます。
以下にコードを示します。

//C:\DEV\str.h

#include <cstring>

class str {
    public :
        ~str();
        str& operator=(char *p);
        //+= , + , 等価演算子・比較演算子は後で実装
    //private : //一時的にコメントアウト
        char *pstr = 0;
};

str::~str() { if(this->pstr != 0) { delete[] this->pstr; } }

str& str::operator=(char *p)
{
	unsigned int count = std::strlen(p);

	if(this->pstr != 0) { delete[] this->pstr; }
	this->pstr = new char[count];
	while(count != 0)
	{
		this->pstr[count] = *(p + count);
		count--;
	}
	this->pstr[0] = *p;

	return *this;
}

//C:\DEV\test.cpp
#include <cstdio>
#include "str.h"

int main()
{
    str s;
    s = "This is test.";
    std::puts(s.pstr);

    return 0;
}

コンパイル

c:
cd \DEV
g++ test.cpp

結果

test.cpp: In function 'int main()':
test.cpp:7:6: warning: ISO C++ forbits converting a string constant to 'char*' [-Wwrite-strings]
 s = "This is test.";
     ^^^^^^^^^^^^^^^

環境

OS:Windows10 Home 64bit
コンパイラ:MinGW-W64

Comments

[episteme]

[別件] ヘッダに(メンバ関数の)定義を置いちゃダメ!

Answer 1

str& str::operator=(char *p)

を

str& str::operator=(const char *p)

警告なので無視できますが...
いわゆるキャストが曖昧で、コンパイラからしかられているだけです。
本来C++言語では、const char*からchar *への変換はコンパイルエラーとして扱われるます。
しかし、C言語では「文字列リテラル(const char配列)からchar *への自動変換を特別に許可」されています。
C++言語でも歴史的経緯(C言語が親)として、互換性の意味合いで自動変換(キャスト)を許容するします。（警告付き）

Answer 2

operator=(char* p)

"This is test."　を引数にとるには
operator=( const char* p ) たるべき，という話かと．

Answer 3

文字列リテラルは書き換えできないので、

        str& operator=(char *p);

char* ではなく const char* にしましょう。

Comments

[退会済みユーザー]

const指定したら警告が出なくなりました!
ありがとうございます

[SaitoAtsushi]

余談ですが C++03 までは文字列リテラルを char* へ暗黙に型変換することは許されていました。 (C++03 の時点でも非推奨ではありましたので g++ は警告を出していましたが forbits converting ではなく deprecated conversion というメッセージでした。)
そういう事情もあって古い資料では文字列の const の有無に無頓着な場合があり、そのコードのままだと新しい仕様に合致しないということもあります。
C++ は基本的には大きく互換性を損なわないように改定されていますが、特に問題のある部分は非推奨にした上でさらに後の改定で廃止するということも少しはあるので、読む資料の選択においてはその資料がいつの時代に書かれたものか、いつの仕様を根拠としたものかというのは気にしておくとよいでしょう。

[fana]

型変換の可否がどうのいう話があろうがなかろうが
このoperatorはポインタが指す場所を変更しないのだから引数はconstにすべきところだと思う．

[SaitoAtsushi]

「すべき」という点ではその通りです。 ですがそうなっていない資料が存在するので気を付けろという意図です。

[退会済みユーザー]

str型を完成させたのですが・・・
もしパラメータをstr型とする関数funcを宣言したとして,
func("test");
みたいな呼び出しはエラーになりますよね?
これって解決できるんですか?

[SaitoAtsushi]

const char * を引数とするコンストラクタを定義してください。
関数の仮引数と実引数の関係は変数の初期化のメカニズムに対応します。