strcpyでエラーがでます。

Question

エジをクリックするととばされるサイト
(https://docs.microsoft.com/en-us/cpp/error-messages/compiler-warnings/compiler-warning-level-3-c4996?redirectedfrom=MSDN&view=vs-2019)
参考書を見ながら下記のコードを入力したのですが、strcpy(pName,pN);の部分でエラーがでます。
エラーメッセージが全て英語だったので、Google翻訳ですがリクエストが承認されませんと言われます。

#include<iostream>
#include<string>
using namespace std;

class Car
{
private:
    int num;
    double gas;
    char* pName;
public:
    Car(char* pN, int n, double g);
    //デストラクタ
    ~Car();
    void show();
};

Car::Car(char* pN, int n, double g)
{
    //コンストラクタ内で動的にメモリ確保
    //strlen(pN) pNの中身の文字列のNULLより前の文字数を数える
    pName = new char[strlen(pN) + 1];
    //strcpy(pName, pN) pNameにpNの中身の文字列をコピーする
    strcpy(pName, pN);
    num = n;
    gas = g;
    cout << pName << "を作成しました。" << endl;
}

Car::~Car()
{
    cout << pName << "を破棄します。" << endl;
    delete[] pName;
}

void Car::show()
{
    cout<<"車のナンバーは"<<num<<"です。"<<endl;
    cout<<"ガソリン量は"<<gas<<"です。"<<endl;
    cout<<"名前は"<<pName<<"です。"<<endl;
}


int main()
{
    Car car1("mycar", 1234, 25.5);
    car1.show();

    return 0;
}
```![エラーメッセージ](5fefd4c0f55f8efdb55c6179a453a1b8.png)

Comments

[thkana]

翻訳などしなくてよいのでエラーメッセージは原文ママ示して下さい。

情報は、よくわかっている内容でもない限り(それなら質問しないでしょう)、余計な加工や解釈、まとめなどをしないでそのまま提示したほうがよいことがほとんどです。

[yu_de_egg]

ご指摘ありがとうございます。
エラーメッセージをクリックするととばされるサイトのURLを貼り付けました。

[thkana]

なぜエラーメッセージそのものでなくサイトのURLなのですか?
エラーメッセージにはエラーになった場所などの情報も含まれていたのではないかと思いますが、その情報を敢えて削るのはなぜですか?
「そのまま」です。「そのまま」。

自分が頑張ったところを見せたいのなら、元々の情報に追加する形にして下さい。

[yu_de_egg]

エラーメッセージの画像を貼り付けました。
これで大丈夫でしょうか？

[yominet]

次回からは、開発に使用しているソフト（バージョンも）も提示しましょう
#言語は進化・変化しつづけているため、エラー・警告の内容が開発ソフト・バージョンによって変わります

[yu_de_egg]

情報が足らず申し訳ありません。
次からはバージョンも記載します。

Answer 1

本題の方は十分な情報が集まったと思います。
ので、多少邪道なやり方を。

プログラミングに関する情報はネット上多く転がっています。なので、出たメッセージを(またまた)「そのまま」検索にかけるとそのものズバリだったりかなり参考になったりする情報が引っかかることも多いです。
'strcpy': This function or variable may be unsafe.
を「そのまま」検索してみて下さい。どうでしょうか。

Comments

[yu_de_egg]

ご教授ありがとうございます。
プログラマには問題を自分で調べて解決することも大事ですよね。
次からは、質問する前に自ら調べてみます。

Answer 2

VisualStudioではstrcpyは推奨されておりません。

「strcpy_sに書き換えてください」と言われております。なお、引数として配列長を指定する必要があります。

Answer 3

C++ では文字列リテラルは const char* 型として扱われるので、コンストラクタの第一引数は char* ではなく、const char* にする必要があります。

文字列リテラル - cppreference.com

その点を直せば、動くと思います。

- Car(char *pN, int n, double g)
+ Car(const char *pN, int n, double g)

追記: C++11 だと思っていたのですが、C++03 では、文字列リテラルを char* に変換するのが許可されているので、この点は変更しなくても動きますね。

追記

Warning の内容はより安全な関数 strcpy_s() を使ってくださいという内容です。
strcpy との違いはコピーする長さを指定する必要がある点です。

- strcpy(pName, pN);
+ strcpy_s(pName, strlen(pN) + 1, pN);

修正後のコード

#include <iostream>
#include <string>
using namespace std;

class Car
{
private:
    int num;
    double gas;
    char *pName;
    
public:
    Car(const char *pN, int n, double g);
    //デストラクタ
    ~Car();
    void show();
};

Car::Car(const char *pN, int n, double g)
{
    //コンストラクタ内で動的にメモリ確保
    //strlen(pN) pNの中身の文字列のNULLより前の文字数を数える
    pName = new char[strlen(pN) + 1];
    //strcpy(pName, pN) pNameにpNの中身の文字列をコピーする
    strcpy_s(pName, strlen(pN) + 1, pN);
    num = n;
    gas = g;
    cout << pName << "を作成しました。" << endl;
}

Car::~Car()
{
    cout << pName << "を破棄します。" << endl;
    delete[] pName;
}

void Car::show()
{
    cout << "車のナンバーは" << num << "です。" << endl;
    cout << "ガソリン量は" << gas << "です。" << endl;
    cout << "名前は" << pName << "です。" << endl;
}

int main()
{
    Car car1("mycar", 1234, 25.5);
    car1.show();
    
    return 0;
}

Comments

[HogeAnimalLover]

VisualStudioのエラーメッセージを見る限りは、「strcpy_sに書き換える」ことが正しいと思います。

[tiitoi]

コメントありがとうございます。
回答した段階ではエラーの内容が記載されていなかったため、こちらでコンパイルした際の問題点を記載したのですが、追記を見ると、ご指摘いただいた点が問題のようです。修正しました。

[yominet]

tiitoiさんが回答した時点ではエラーメッセージの提示はなかったので、そこは仕方ないことかと

[thkana]

明らかになっていないことを明らかにしないまま推測して間違えたのは当然に起こりうることではありますが、「仕方がないこと」なのでしょうか?

#明らかになっていない質問に回答しないとすると相当数の質問が回答不能になる、という現実もありますけどね。

[tiitoi]

To: thkana さん

コメントありがとうございます。
teratail の質問で修正依頼なしで回答に必要な情報が揃っている場合というのは、あまりありません。その場合、追記、修正依頼しつつも、情報がなかなか出てこない場合は、ある程度推測できる場合は回答するようにしています。
もちろん、その場合、質問者さんの意図と異なる回答となっている場合があるので、質問欄編集やコメントで追加で情報が得られた場合は回答は修正するようにしています。

[yu_de_egg]

皆さんのご教授感謝します。ありがとうございました。
自分の開示した情報が少なすぎて、皆さんにはご迷惑をおかけしました。申し訳ありません。
次回からの質問からは、細かいところまで情報を開示しようと思います。
今後ともよろしくお願いいたします。

[thkana]

私も最初はconstだろうと思っていましたし、とりあえずそれで回答しつつ質問者の真意を探る、というのも１つのやり方だというのは理解します。ただ、情報不足な質問が出て、それにとりあえず回答するのが普通な現状を「仕方ない」と言ってしまうのはなにか違う気がしたのでコメントしています。(「仕方ない」と言ったのはtiitoiさんじゃないですね) 本当に「仕方ない」のかもしれませんが...

[thkana]

yu_de_eggさん
いえいえ、ちゃんと対応して頂いたので大丈夫です。
回答するには情報が足りない、といわれると怒り出す人も多かったりするなか、大変気持ちよくやり取りが出来たと思っています。こちらこそありがとうございました。

[tiitoi]

To: thkana さん

すいません。今回は確かに早合点して回答してしまいました。
今後はもう少し必要な情報が揃うのを待ってから回答しようと思います。