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

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

ただいまの
回答率

90.51%

  • C++

    4436questions

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

  • オブジェクト指向

    328questions

    オブジェクト指向プログラミング(Object-oriented programming;OOP)は「オブジェクト」を使用するプログラミングの概念です。オブジェクト指向プログラムは、カプセル化(情報隠蔽)とポリモーフィズム(多態性)で構成されています。

C++ のクラスの使用とstaticとグローバル変数の使い方

解決済

回答 1

投稿

  • 評価
  • クリップ 0
  • VIEW 2,445

TAKAYUKI_MIWA

score 58

前提・実現したいこと

現在、C++でクラスを定義しゲッターとセッターを作成しました。
またCompanyNameという、ヘッダーファイルで定義したstatic変数を付けたのですが、エラーが発生し、実行できません。ポインタの使い方がおかしいと思うのですが、サンプルプログラムを自分なりにアレンジして作成したので、このプログラムの中でポインタを使う理由もあまり分かりません。またなぜ、C++ではStringを使わず、文字列を毎回をポインタにして使用しているのでしょうか。

発生している問題・エラーメッセージ

Main.cpp:9:15: error: redefinition of 'ComapanyName' with a different type: 'char *' vs 'char *[80]'
char* Worker::ComapanyName = 'ABC';
              ^
./Worker.h:3:18: note: previous definition is here
    static char* ComapanyName[80];
                 ^
Main.cpp:9:30: warning: multi-character character constant [-Wmultichar]
char* Worker::ComapanyName = 'ABC';
                             ^
Main.cpp:13:30: warning: multi-character character constant [-Wmultichar]
    strcpy(this->department, 'NotDefinedDepartment');
                             ^
Main.cpp:13:30: warning: character constant too long for its type
Main.cpp:13:5: error: no matching function for call to 'strcpy'
    strcpy(this->department, 'NotDefinedDepartment');
    ^~~~~~
/usr/include/string.h:129:14: note: candidate function not viable: no known conversion from 'char *[80]' to 'char *restrict' for 1st argument
extern char *strcpy (char *restrict dest, const char *restrict src)
             ^
Main.cpp:14:24: warning: multi-character character constant [-Wmultichar]
    strcpy(this->name, 'NotDefinedName');
                       ^
Main.cpp:14:24: warning: character constant too long for its type
Main.cpp:14:5: error: no matching function for call to 'strcpy'
    strcpy(this->name, 'NotDefinedName');
    ^~~~~~
/usr/include/string.h:129:14: note: candidate function not viable: no known conversion from 'char *[80]' to 'char *restrict' for 1st argument
extern char *strcpy (char *restrict dest, const char *restrict src)
             ^
Main.cpp:23:9: error: out-of-line definition of 'Worker' does not match any declaration in 'Worker'
Worker::Worker(int number, char* department,char* name, double salary){
        ^~~~~~
Main.cpp:25:5: error: no matching function for call to 'strcpy'
    strcpy(this->department, department);
    ^~~~~~
/usr/include/string.h:129:14: note: candidate function not viable: no known conversion from 'char *[80]' to 'char *restrict' for 1st argument
extern char *strcpy (char *restrict dest, const char *restrict src)
             ^
Main.cpp:26:5: error: no matching function for call to 'strcpy'
    strcpy(this->name, name);
    ^~~~~~
/usr/include/string.h:129:14: note: candidate function not viable: no known conversion from 'char *[80]' to 'char *restrict' for 1st argument
extern char *strcpy (char *restrict dest, const char *restrict src)
             ^
Main.cpp:34:43: error: expected ';' after expression
    cout << "department = " << *department
                                          ^
                                          ;
Main.cpp:48:5: error: no matching function for call to 'strcpy'
    strcpy(this->name, name); 
    ^~~~~~
/usr/include/string.h:129:14: note: candidate function not viable: no known conversion from 'char *[80]' to 'char *restrict' for 1st argument
extern char *strcpy (char *restrict dest, const char *restrict src)
             ^
Main.cpp:52:12: error: cannot initialize return object of type 'char *' with an lvalue of type 'char *[80]'
    return this->name;
           ^~~~~~~~~~
5 warnings and 9 errors generated.

ソースコード

ヘッダーファイル<Worker.h>
using namespace std;

int Worker::num=0;
char* Worker::ComapanyName = 'ABC';

Worker::Worker(){
    int number = num;
    strcpy(this->department, 'NotDefinedDepartment');
    strcpy(this->name, 'NotDefinedName');
    salary = 0;
    num++;
}

Worker::~Worker(){
    cout << "This is ~Worker();" << "\n";
}

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

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

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

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

void  Worker::setName(char* name){
    strcpy(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){

    return 0;
}

メイン <Main.cpp>

using namespace std;

int Worker::num=0;
char* Worker::ComapanyName = 'ABC';

Worker::Worker(){
    int number = num;
    strcpy(this->department, 'NotDefinedDepartment');
    strcpy(this->name, 'NotDefinedName');
    salary = 0;
    num++;
}

Worker::~Worker(){
    cout << "This is ~Worker();" << "\n";
}

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

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

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

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

void  Worker::setName(char* name){
    strcpy(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){

    return 0;
}

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

paiza.io

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

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

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

    クリップを取り消します

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

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

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

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

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

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

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

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

    質問の評価を下げる

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

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

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

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

checkベストアンサー

0

こんにちは。

またなぜ、C++ではStringを使わず、文字列を毎回をポインタにして使用しているのでしょうか。

std::stringを使っていない理由は不明ですが、そのサンプルの作者がstd::stringを知らなかったのかもしれません。C++をベターCとして使う人も少なくないですし。(かく言う私も一昨年までその一人でした。)


エラーの意味を少し解説してみます。

Main.cpp:9:15: error: redefinition of 'ComapanyName' with a different type: 'char *' vs 'char *[80]' 
char* Worker::ComapanyName = 'ABC'; 
              ^ 
./Worker.h:3:18: note: previous definition is here 
    static char* ComapanyName[80]; 
                 ^ 

main.cppの9行目とWorker.hの3行目で、CompanyNameの定義が異なります。
前者はchar型へのポインタ、後者は「char型へのポインタ」80個の配列となってます。

Main.cpp:9:30: warning: multi-character character constant [-Wmultichar] 
char* Worker::ComapanyName = 'ABC'; 
                             ^ 

''で括ることができるのは1文字だけです。結果はchar型のリテラルになります。
3文字括ってはいけません。たぶん、"ABC"と書きたかったのでは?
文字列リテラルは""で括って下さい。
文字リテラルを``で括ります。(文字列でなく文字なので、1文字だけ括れます。)
以下同様なエラーは省略。

Main.cpp:23:9: error: out-of-line definition of 'Worker' does not match any declaration in 'Worker' 
Worker::Worker(int number, char* department,char* name, double salary){ 
        ^~~~~~ 

同じコンストラクタがWorker.hでも定義されています。

Main.cpp:25:5: error: no matching function for call to 'strcpy' 
    strcpy(this->department, department); 
    ^~~~~~ 

Workerクラス内にdepartmentを定義していないのでは?
以下略。

C言語の基礎を勉強しましょう。C言語の基礎解説はQAサイトの役割ではないと思います。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2016/01/19 14:29

    ありがとうございます。

    キャンセル

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

  • C++

    4436questions

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

  • オブジェクト指向

    328questions

    オブジェクト指向プログラミング(Object-oriented programming;OOP)は「オブジェクト」を使用するプログラミングの概念です。オブジェクト指向プログラムは、カプセル化(情報隠蔽)とポリモーフィズム(多態性)で構成されています。