質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.39%
C++

C++はC言語をもとにしてつくられた最もよく使われるマルチパラダイムプログラミング言語の1つです。オブジェクト指向、ジェネリック、命令型など広く対応しており、多目的に使用されています。

Q&A

解決済

2回答

60630閲覧

C++において,文字列リテラルから 'char*型' への変換が非推奨? エラーメッセージ

yakkuru

総合スコア9

C++

C++はC言語をもとにしてつくられた最もよく使われるマルチパラダイムプログラミング言語の1つです。オブジェクト指向、ジェネリック、命令型など広く対応しており、多目的に使用されています。

0グッド

1クリップ

投稿2015/05/10 14:30

こんにちは,初めて投稿をさせてもらいます。

参考書を読んでいて,あるソースコードをコンパイルしたときに,以下のメッセージが出ました。

warning: conversion from string literal to 'char *' is deprecated [-Wc++11-compat-deprecated-writable-strings]

char *subname[] = {"Math", "English", "Science"};
^

lang

1#include<iostream> 2#include<string.h> 3 4using namespace std; 5 6class Student{ 7public: 8 void SetID(int num){ 9 id = num; 10 } 11 void SetName(char *str){ 12 strcpy(name, str); 13 } 14 int GetID(){ 15 return id; 16 } 17 char *GetName() const{ 18 return (char *)name; 19 } 20 21private: 22 int id; 23 char name[30]; 24}; 25 26enum Subjects{Math, English, Science}; 27 28class Exam{ 29public: 30 void SetInfo(int id, char *name, Subjects s, int num); 31 int GetPoint() const{ 32 return point; 33 } 34 void GetResult(char *buf) const; 35 36 Student student; 37private: 38 Subjects subject; 39 int point; 40}; 41 42void Exam::SetInfo(int id, char *name, Subjects s, int num){ 43 student.SetID(id); 44 student.SetName(name); 45 subject = s; 46 point = num; 47} 48 49void Exam::GetResult(char *buf) const{ 50 char *subname[] = {"Math", "English", "Science"}; 51 sprintf(buf, "%s:%d", subname[subject], point); 52} 53 54void PrintResult(const Exam &Exam){ 55 cout << Exam.student.GetName() << endl; 56 char buf[30]; 57 Exam.GetResult(buf); 58 cout << buf << endl; 59} 60 61int main(){ 62 Exam Exam[3]; 63 Exam[0].SetInfo(1, "Ken", Math, 60); 64 PrintResult(Exam[0]); 65 return 0; 66}

「文字列リテラルから 'char*型' への変換は非推奨です。」ということなのですが,
0. このソース内の,どこで変換を実施しようとしているのか。
0. 実行出来るようにはどのように変えればいいのか。
0. 非推奨ということは,無理矢理実行することも可能なのか。
を教えていただきたいです。よろしくお願いします。

ソースコードの引用元 (株)アンク著「C++の絵本」 (一部改変)

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

1.このソース内の,どこで変換を実施しようとしているのか。

C++では文字列リテラル(ソースコードに"〜"で直接書いた文字列)は、変更できないconst char配列として扱われます。

lang

1char *subname[] = {"Math", "English", "Science"};

上記コードでは配列subnameの3要素の初期化において、const char *からchar *への変換が必要であり、コンパイラはこの変換処理について警告しています。

lang

1// char *subname[] = {"Math", "English", "Science"}; と等価な処理 2const char e0[5] = "Math"; 3const char e1[8] = "English"; 4const char e2[8] = "Science"; 5char *subname[3]; 6subname[0] = e0; // const char*をchar*に代入しようとしている! 7subname[1] = e1; // (同上) 8subname[2] = e2; // (同上)

2.実行出来るようにはどのように変えればいいのか。

配列subnameの型を、char *の配列からconst char*の配列に変更すればOKです。

lang

1const char *subname[] = {"Math", "English", "Science"};

3.非推奨ということは,無理矢理実行することも可能なのか。

今回のコードに限れば、全く問題なく実行することができます。文字列リテラルは変更できない(=変更してはいけない)const char配列ですが、その文字列の一部(それぞれの文字)を書き換えようとしない限りは安全です。

lang

1void Exam::GetResult(char *buf) const{ 2 char *subname[] = {"Math", "English", "Science"}; 3 sprintf(buf, "%s:%d", subname[subject], point); 4 // 各要素=文字列リテラルを読み出すだけで、書き換えは行わない。 5}

本来C++言語では、const char*からchar *への変換はコンパイルエラーとして扱われるます。しかし、C言語では「文字列リテラル(const char配列)からchar *への自動変換を特別に許可」しており、C言語から派生したC++言語でも歴史的経緯として、特例でこの自動変換を許容する(ただし警告付き)コンパイラが多いです。

投稿2015/05/11 12:31

yohhoy

総合スコア6191

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

yakkuru

2015/05/12 10:26

回答ありがとうございます。 C言語とC++の仕様が異なるために起こしたエラーや,コンパイルが警告を出した場所等,非常に分かりやすい解説でした。 また分からないことがあればよろしくお願いします。
guest

0

ベストアンサー

1.このソース内の,どこで変換を実施しようとしているのか。
文字列リテラルは、これは正確な言い方ではないかもしれませんが、const char型です。char型に代入することが変換に該当します。

2.実行出来るようにはどのように変えればいいのか。
const char*subname[]に変更して下さい。
より適切な方法としてはLPCTSTR等のマクロ利用があります。
機会があれば調べてみてください。(こちらは割愛します)

3.非推奨ということは,無理矢理実行することも可能なのか。
コンパイラに依存することなので、正確に回答しかねます。たぶんコンパイラオプションを弄ればいけるとは思いますが、推奨しません。

解説
文字列リテラルは内容の書き換えが不可能な文字列です。つまり、文字列リテラルのポインタを扱う場合は const char* 型を使用する必要があります。
const修飾子がついていないために警告がでているのだと思います。

これで意味がわからない場合、以下の違いについて考えてみてください。

char[] = {"Math"} だと警告はでませんが、char* = "Math" だと同じ警告がでると思います。

char[] = {"Math"} では長さ5のローカルの文字列変数に "Math" の値を詰め込んでいます。つまり、文字列リテラル(定数)の中身を一旦ローカル変数にコピーしています。この文字列が編集されても定数は変更されません。

一方、charだとconst無しのポインタに定数自体のポインタを代入している形になります。定数はプログラムのリソース部分にあたります。定数自体を編集する(=この状態でcharの参照先の文字列を編集する)場合の動作は未定義=禁止事項です。このため、const修飾子を着けて文字列リテラルを保護することが推奨されます。

投稿2015/05/10 15:43

編集2015/05/12 01:25
haru666

総合スコア1593

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

yohhoy

2015/05/11 12:34

`const *char`は文法上誤りで、正しくは`const char*`ですね。
haru666

2015/05/12 01:26

typoを修正しました。ありがとうございます。
yakkuru

2015/05/12 10:22

回答ありがとうございます。 文字列リテラルが内容書き換え不可能な文字列,ということを知りませんでした。 新たな解決策として,マクロ利用も教えていただきありがとうございます。 また分からないことがあればよろしくお願いします。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.39%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問