C++ Conversion from String Literal to Char* is deprecatedとなります。。

Question

C++プログラミング初心者です。
成績を表示するクラスを作ったのですがコンパイル後、
Conversion from String Literal to Char* is deprecatedとなります。。
xcodeを使用して開発しているのですが、黄色い△の!というアイコンが出てしまいます。。
これをなくすにはどうすれば良いでしょうか？。。

以下コードです。

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

/* Studentクラス */
class Student {
public:
void SetID(int num) { id = num; }
void SetName(char *str) {
strcpy(name, str);
}
int GetID() {return id; }
char *GetName() const {
return (char *)name;
}
private:
int id; // Student ID
char name[30]; // 名前
};

enum Subjects{Math, English, Science};

class Exam {
public:
void SetInfo(int id, char *name, Subjects s, int num);
int GetPoint() const {return point; }
void GetResult(char *buf) const;

Student student;

private:
Subjects subject; //科目
int point;
};

void Exam::SetInfo(int id, char *name, Subjects s, int num)
{
student.SetID(id);
student.SetName(name);
subject = s;
point = num;
}

void Exam::GetResult(char *buf) const
{
const char *subname[] = { "数学", "英語", "理科" };
sprintf(buf, "%s:%d点", subname[subject], point);
}

void PrintResult(const Exam &Exam)
{
cout << Exam.student.GetName() << endl;
char buf[30];
Exam.GetResult(buf);
cout << buf << endl;
}

double GetAvg(const Exam *Exam, int num)
{
double sum = 0;
for (int i = 0; i < num; i++) {
sum += Exam[i].GetPoint();
}
return sum / num;
}

int main(int argc, const char * argv[])
{
Exam Exam[3];
Exam[0].SetInfo(1, "太郎", Math, 82);
Exam[1].SetInfo(1, "太郎", English, 84);
Exam[2].SetInfo(1, "太郎", Science, 83);
PrintResult(Exam[0]);
PrintResult(Exam[1]);
PrintResult(Exam[2]);
cout << endl;
cout << "平均:" << GetAvg(Exam, 3) << "点" << endl;
return 0;
}

あと、このコードを改善するとすればどう書き直しますでしょうか？
2つ質問してしまいましたが回答宜しくお願いします

Answer 1

Conversion from String Literal to Char* is deprecatedとなります

メッセージの通り、文字列リテラル("ABC"とか)からchar*型への変換(代入)は、C++では非推奨となっています。（C言語では怒られないのですが、C++ではダメ。）

char * s0 = "ABC";        // NG
const char * s1 = "ABC";  // OK

このコードを改善するとすればどう書き直しますでしょうか？

変更しない/させない文字列では、const char*型を利用すべきです。プログラマが意図せず文字列を変更しようとしたとき（バグ）、コンパイラが問題を見つけて指摘してくれます。

もしくは、C++標準ライブラリのstd::string型を用いる実装も考えられます。こちらのほうがC++らしいですかね。

Comments

[heihoo]

ありがとうございました。できました！
Cだとおこられませんでした。C++では非推奨となっていたのですねー