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

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

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

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

Q&A

解決済

3回答

65370閲覧

const char*とchar*の互換性

typeratus

総合スコア7

C++

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

0グッド

1クリップ

投稿2018/03/22 15:44

C++の絵本からの引用で、以下に示すコードをVisualStudio2017で動かそうとしています。
すると、main関数の
Exam[0].SetInfo(1, "しおり", Math, 60);
のところで
E0167:型"const char *" の引数は型 "char *" のパラメーターと互換性がありません
というエラーが発生しています。

対応として、具体的には
文字セットを変える、#define _CRT_SECURE_NO_WARNINGSを追加、charからstring仕様に書き直す
といったことを試しましたが、いずれもダメでした。

いろいろ調べてみましたが、それでも原因がわからないのでどなたかご教授願います。

#include "stdafx.h" #include <iostream> #include <string.h> using namespace std; //生徒クラス 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; //生徒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() { Exam Exam[3]; Exam[0].SetInfo(1, "しおり", Math, 60); Exam[1].SetInfo(1, "しおり", English, 75); Exam[2].SetInfo(1, "しおり", Science, 88); PrintResult(Exam[0]); PrintResult(Exam[1]); PrintResult(Exam[2]); cout << endl; cout << "平均:" << GetAvg(Exam, 3) << "点" << endl; return 0; }

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

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

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

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

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

guest

回答3

0

ベストアンサー

文字列リテラルを単なるcharポインタに放り込もうとしているのが問題です。
変更不能であることを明示するために、const指定した方が良いでしょう。

具体的には、Exam::SetInfoとStudent::SetNameのシグネチャを次のように変えてください。

  • void SetInfo(int id, const char *name, Subjects s, int num)
  • void SetName(const char *str)

投稿2018/03/22 15:55

LouiS0616

総合スコア35658

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

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

typeratus

2018/03/22 17:04

const指定でうまく動かすことができました。 ありがとうございます!
guest

0

こんにちは。

constを使い始めた当初にたいへんよくハマるつらい状況ですね。
対策は LouiS0616さんが書かれている通りです。

const char*型はchar型へのポインタですが、そのポイント先を修正するコードをコンパイル・エラーにするという宣言です。
そして、char*型は普通のchar型へのポインタですので、そのポイント先を修正することを許可します。ポイント先を書き換える必要がある時にはそうする必要があります。

さて、ご提示のソースではnameポインタはchar*型ですのでポイント先を変更可能と宣言しています。
そして、"しおり"はこの文字列先頭へのポインタになりますが、そのポイント先は定数ですから書き換えては行けませんからconst char*型となります。なのに、変更可能と宣言している引数に渡しています。
変更不可なものを変更可能なものに間違って渡すと、多くの場合、渡された先で変更されてしまいバグになります。
ですので、それを事前に回避するためにコンパイラがエラーにしています。

ところで、渡されたnameポインタは最後にstrcpy()でコピーされておりポイント先を変更していません。であれば、const char*で十分です。そのような時に変更を許可するのは愚かな行為です。ですから、LouiS0616さんが書かれている通りconst char*と宣言するのが望ましい対策です。

constについて詳しいことを、ちょっとイライラするconst。でも、実は頼もしい奴です。で解説したことがあります。もし、よろしければ参考にされて下さい。

投稿2018/03/22 16:23

Chironian

総合スコア23272

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

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

typeratus

2018/03/22 17:14 編集

constの役割や意味についてまだまだ理解が足りていないので、リンク先の解説も参考にさせていただきます。
guest

0

文字列として直接書いたもの (文字列リテラル) は、つまりこの場合は "しおり" の型は const char[] です。 これは暗黙の型変換によって const char* になります。 この const はポインタが指す先のオブジェクトを書き換えてはいけないということを表しています。

書き換えてはいけない文字列に対して、書き換えてもかまわないことを意味する char* で受取ろうとしたからエラーです。 受取る側、つまり仮引数も const char* にすると問題なく通るはずです。 この場合は Exam::SetInfoStudent::SetName の仮引数に const を付ければよいです。

古い C++ (いわゆる C++03) では文字列リテラルを char* に変換するのが許されていたのですが、後に (C++11 で) 廃止されたという経緯があります。 更に Visual Stuido はユーザへの便宜のためにかなり後まで変換を許すように実装していたらしいです。

投稿2018/03/22 16:05

SaitoAtsushi

総合スコア5437

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

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

typeratus

2018/03/22 17:08

そのようなバックボーンがあったのですね...とても参考になります。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問