C＋＋ クラスのゲッター、セッターの使用方法

Question

###前提・実現したいこと
C++でゲッター・セッターを下記のように記入しましたが、コンパイルエラーが発生し解決できません。推測するに、thisポインタとstrcpyの使い方が分かっていないので、合わせて説明して頂けると幸いです。この前までJavaを扱っていたので、下記のようなソースコードを作成しました。

###発生している問題・エラーメッセージ
Main.cpp:37:16: error: array type 'char [80]' is not assignable
this->name = name;
~~~~~~~~~~ ^
Main.cpp:41:12: error: cannot initialize return object of type 'char' with an lvalue of type 'char [80]'
return this->name;
^~~~~~~~~~
2 errors generated.

###ソースコード
ヘッダーファイル ： Worker.h
class Worker{
int number;
char name[80];
double salary;
public:
Worker();
~Worker();
Worker(int number, char* name, double salary);
void ShowData();
void setNumber(int number);
int getNumber();
void setName(char* name);
char getName();
void setSalary(double salary);
double getSalary();

};

メイン文：
#include <iostream>
#include <string.h>
#include "Worker.h"

using namespace std;

Worker::Worker(){
number = 0;
strcpy(this->name, name);
salary = 0;
}

Worker::Worker(int number, char* name, double salary){
this->number = number;
strcpy(this->name, name);
this->salary = salary;
}

void Worker::ShowData(){
cout << "number = " << number << "\n";
cout << "name = " << name << "\n";
cout << "number = " << salary << "\n";
}

void Worker::setNumber(int number){
this->number = number;
}

int Worker::getNumber(){
return this->number;
}

void Worker::setName(char* name){
this->name = name;
}

char Worker::getName(){
return this->name;
}

void Worker::setSalary(double salary){
this->salary = salary;
}

double Worker::getSalary(){
return this-> salary;
}

int main(void){
Worker w1;

return 0;

}

###補足情報(言語/FW/ツール等のバージョンなど)
paiza.io

Answer 1

こんにちは。

どうもC言語の「文字列」の概念を理解されていないようです。
もし、C言語の文字列を学びたいのであれば、このサイトが分かりやすいです。

エラーをなくすには、下記修正でいけると思います。（未検証）

class Worker{ 
（中略）
    char* getName(); 
（中略）
};

（中略）

void  Worker::setName(char* name){ 
    strcpy(this->name, name); 
}

char* Worker::getName(){ 
    return this->name; 
}

私の力量では短く説明できないのて、上記サイトを参照下さい。

---

Javaから入られたのでしたら、文字列はstd::string型を使うとJavaに近い使い方ができます。
ただ、Javaのように簡単に使えますが、安易に使うとコピーが多発します。コピーの多発を防ぐ方法もありますが、道のりが長いです。

ポインタを理解すればC言語文字列で高速化できます。しかし、メモリ管理が非常に面倒です。バグも出やすいので、あまりお薦めできませんが、学習期間は短めで済みます。

std::stringを使えばメモリ管理も手間がかかりませんし、より便利な機能が多数あります。しかし、安易に使うとコピーが多発します。とは言え通常はJavaやC#より高速に動作するのでこれも有りと思います。
C++の参照を理解するとそこそこコピーを防げます。ポインタ＋α程度の理解でいけますのでこの辺で手を打つ人も多いだろうと思います。
更に、右辺値参照まで理解できればほとんど全ての場合で無駄なコピーを防げます。学習難易度はかなり高いので、理解できるようになってから使っても良いだろうと思います。

Comments

[saito.kaz]

ありがとうございます。
解決しました。

Answer 2

配列（あえて文字列とは言わず配列と表現します）のコピーは = では書けません。

配列の扱い方をここで講義しても良いのですが、Java経験があるということであればむしろ文字列を使った方が良いでしょう。つまりstd::stringクラスです。

nameフィールド、SetName関数の引数、GetName関数の戻り値をすべて std::string 型にしてください。<string>のインクルードが必要になります。それでこのクラスのコンパイルは通ります。

Comments

[saito.kaz]

ありがとうございます。
学びにつながりました。

Answer 3

おそらく、ここのコンストラクタで、nameが与えられてないのが原因かと・・・

Worker::Worker(){ 
    number = 0; 
    strcpy(this->name, name); 
    salary = 0; 
}

せっかく、こちらのコンストラクタがあるので・・・・・

Worker::Worker(int number, char* name, double salary){ 
    this->number = number; 
    strcpy(this->name, name); 
    this->salary = salary; 
}

このように、コンストラクタからコンストラクタを呼び出せばよいのでは。

Worker::Worker:Worker(0, null, 0){ 
}

すいません、未検証ですが・・・

Comments

[saito.kaz]

下記のように変更しましたが、エラーが発生しております。

-------------------------------エラー-------------------------------------------
Main.cpp:10:23: warning: multi-character character constant [-Wmultichar]
strcpy(this->name,'Notyet');
^
Main.cpp:10:23: warning: character constant too long for its type
Main.cpp:10:5: error: no matching function for call to 'strcpy'
strcpy(this->name,'Notyet');
^~~~~~
/usr/include/string.h:129:14: note: candidate function not viable: no known conversion from 'int' to 'const char *restrict' for 2nd argument
extern char *strcpy (char *__restrict __dest, const char *__restrict __src)
^
Main.cpp:37:16: error: array type 'char [80]' is not assignable
this->name = name;
~~~~~~~~~~ ^
2 warnings and 2 errors generated.

--------------------------------------ヘッダーファイル --------------------------------
#include <iostream>
#include <string.h>
#include "Worker.h"

using namespace std;

Worker::Worker(){
number = 0;
strcpy(this->name,'Notyet');
salary = 0;
}

Worker::Worker(int number, char* name, double salary){
this->number = number;
strcpy(this->name, name);
this->salary = salary;
}

void Worker::ShowData(){
cout << "number = " << number << "\n";
cout << "name = " << name << "\n";
cout << "number = " << salary << "\n";
}

void Worker::setNumber(int number){
this->number = number;
}

int Worker::getNumber(){
return this->number;
}

void Worker::setName(char* name){
this->name = name;
}

void Worker::getName(){
cout << this->name << "\n";
}

void Worker::setSalary(double salary){
this->salary = salary;
}

double Worker::getSalary(){
return this-> salary;
}

int main(void){
Worker w1;

return 0;
}