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

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

ただいまの
回答率

90.34%

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

解決済

回答 3

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 3,894

TAKAYUKI_MIWA

score 71

前提・実現したいこと

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

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

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 3

checkベストアンサー

0

こんにちは。

どうも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++の参照を理解するとそこそこコピーを防げます。ポインタ+α程度の理解でいけますのでこの辺で手を打つ人も多いだろうと思います。
更に、右辺値参照まで理解できればほとんど全ての場合で無駄なコピーを防げます。学習難易度はかなり高いので、理解できるようになってから使っても良いだろうと思います。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/01/19 13:14

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

    キャンセル

0

おそらく、ここのコンストラクタで、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){ 
}


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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/01/19 11:52

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

    -------------------------------エラー-------------------------------------------
    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;
    }

    キャンセル

0

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

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

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

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2016/01/19 13:14

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

    キャンセル

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

  • ただいまの回答率 90.34%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

同じタグがついた質問を見る