C++：ランタイムエラー・メンバ変数について

###前提・実現したいこと
下記ソースコードを実行したところ、Runtimeエラーが発生して実行することができません。
staticの使い方が間違っているのでしょうか。
すいませんが、宜しくお願い致します。
###プログラム
ヘッダー
class Worker{
static int num;
static char CompanyName;*
int number;
char* department;
char* name;
double salary;
public:
Worker();
~Worker();
Worker(int number, char* department,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;

int Worker::num=0;
char* Worker::CompanyName = "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 = " << CompanyName << "\n";
cout << "department = " << department << "\n";
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){
Worker w1(1, "Adep", "Takayuki", 1233.0);
w1.ShowData();

return 0;

}

###エラー

ここにご自身が実行したソースコードを書いてください

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

行動規範の内容に同意します

回答2件

ベストアンサー

こんにちは。

Workerクラス内の下記2行が原因です。

char* department;
char* name;

これではchar型へのポインタを定義しただけです。
strcpy()しているので文字列をコピーしたいのだと思いますが、そのためには文字列を記憶できる領域（char型の配列）へのポインタとする必要があります。
そして、C/C++言語では例えば`int foo[10];'でint型の配列を定義した時、fooはfoo[10]先頭へのポインタになります。
従って、上記を例えば次のように書けば通ります。

char department[100];
char name[100];

char name[100];は最大99文字の文字列を保存できます。文字数は適切な値を設定するのが良いです。
バッファオーバーフローに注意下さい。

因みにリファレンスをみて下記のように書くと通常はruntimeエラーになります。

C++
1char *name;
2strcpy(name, "Cheironian");

C言語のFAQです。リファレンス通り書いたのに！！って感じた人が、たぶん何万人もいると思います。
strcpy()は第１引数が示すアドレスへ第２引数が示すアドレスからの文字列をコピーします。
char *name;と書いただけだとnameの値は不定ですね。なので、第一引数が示すアドレスは不定です。
たまたまそれが適切なアドレスにならない限り、不正なメモリへの書き込みとなるのです。

投稿2016/01/19 07:38

編集2016/01/19 07:50