エラーが出てしまい実行できません

Question

大谷紀子さん著書の人工蜂コロニー（ABCアルゴリズム）の項目のソースコードを書き写しコンパイルしているのですがエラーが大量に出ています．誤字脱字は直しました

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

g++ -o bee sample04.cpp
sample04.cpp:10:25: warning: conversion from string literal to 'char *' is deprecated [-Wc++11-compat-deprecated-writable-strings]
   fSet = new FlowerSet("sampledata.csv");
                        ^
sample04.cpp:16:17: warning: illegal character encoding in string literal [-Winvalid-source-encoding]
      printf("%d<89><F1><96>ځF<8D>ŗǕ]<89><BF><92>l%f\n", i, fSet->bestValue);
                ^~~~~~~~~~~~  ~~~~   ~~~~~~~~~~~~
sample04.cpp:16:7: error: use of undeclared identifier 'printf'
      printf("%d<89><F1><96>ځF<8D>ŗǕ]<89><BF><92>l%f\n", i, fSet->bestValue);
      ^
2 warnings and 1 error generated.

該当のソースコード

#include "FlowerSet.h"
#pragma once
#define filename data.txt
//コントラスタ
// filename: データセットのファイル名
FlowerSet::Flower(char *filename){
  int i,best;

  dataset=new Dataset(filename);
  flower=new Flower* [EBEE_NUM];
  best=0;

  for(i=0;i<EBEE_NUM;i++){
    flower[i]=new Flower(this);
    if(flower[best]->value>flower[i]->value){
	 best=i;
    }
  }
  bestPos=new double[dataset->exVarNum];
  for(i=0;i<dataset->exVarNum;i++){
    bestPos[i]=flower[best]->pos[i];
  }
  bestValue=flower[best]->value;
  newFlower=new Flower(this);
  trValue=new double[EBEE_NUM];
}

//デストラクタ
FlowerSet::~FlowerSet(){
  int i;

  for(i=0;i<EBEE_NUM;i++){
    delete flower[i];
  }
  delete [] flower;
  delete [] bestPos;
  delete newFlower;
  delete [] trValue;
  delete dataset;

  //収穫蜂フェーズを実行する
  void FlowerSet::employedBeePhase()
    {
    int i;
    Flower *tmp;

    for(i=0;i<EBEE_NUM;i++){
	 newFlower->change(i);
	 if(flower[i]-> value > newFlower -> value){
	   tmp=newFlower;
	   newFlower=flower[i];
	   flower[i]=tmp;
	 }
	 flower[i]->visitNum++;
    }
  }
  //追従蜂フェーズを実行する
  void FlowerSet::onlookerBeePhase()
    {
    int i,j;
    Flower *tmp;
    double max,min,denom,prob,r;

    for(j=0;j<OBEE_NUM;j++){
	 //評価値を変換する
	 max=DBL_MIN;
	 min=DBL_MAX;
	 for(i=0;i<EBEE_NUM;i++){
	   if(max<flower[i]->value){
		max=flower[i]->value;
	   }
	   if(min<flower[i]->value){
		min=flower[i]->value;
	   }
	 }
	 denom=0.0;
	 for(i=0;i<EBEE_NUM;i++){
	   trValue[i]=(max-flower[i]->value)/(max-min);
	   denom+=trValue[i];
	 }

	 //収穫蜂を選択する
	 r=RAND_01;
	 for(i=0;i<EBEE_NUM-1;i++){
	   prob=trValue[i]/denom;
	   if(r<=prob){
		break;
	   }
	   r -=prob;
	 }

	 //収穫蜂フェーズと同様に処理する
	 newFlower->change(i);
	 if(flower[i]->value>newFlower->value){
	   tmp=newFlower;
	   newFlower=flower[i];
	   flower[i]=tmp;
	 }
	 flower[i]->visitNum++;
    }
  }

  //偵察蜂フェーズを実行する
  void FlowerSet::scoutBeePhase()
    {
    int i;
    for (i=0;i<EBEE_NUM;i++){
	 if(VISIT_MAX<=flower[i]->visitNum){
	   flower[i]->renew();
	 }
    }
  }

  //最良食糧源を記録する
  void FlowerSet::saveBestPos()
    {
    int i,best;

    best= -1;
    for(i=0;i<EBEE_NUM;i++){
	 if(bestValue>flower[i]->value){
	   best=i;
	 }
    }
    if(best!=-1){
	 for(i=0;i<dataset->exVarNum;i++){
	   bestPos[i]=flower[best]->value;
	 }
	 bestValue=flower[best]->value;
    }
  }
  //結果を表示する
  void FlowerSet::printResult()
    {
    dataset->setCoef(bestPos);
    dataset->printEquation();
  }
  
  

//Flower.h
#ifndef INCLUDED_FLOWER
#define INCLUDED_FLOWER
#include "DataSet.h"
#include "FlowerSet.h"
#pragma once
class FlowerSet;

//定数の定義
#define REPEAT_NUM 1000 //繰り返し数
#define EBEE_NUM 100 //収穫蜂の数
#define OBEE_NUM 10 //追従蜂の数
#define VISIT_MAX 10 //蜜の採取可能回数
#define COEF_MIN -1 //標準偏回帰係数の最小値
#define COEF_MAX 1 //標準偏回帰係数の最大値

//0以上１以下の実数乱数
#define RAND_01 ((double)rand() / RAND_MAX)

class Flower{
public:
	   Flower(FlowerSet *argFSet);
	   ~Flower();
	   void change(int base); //近くの食糧源に変更する
	   void renew();//新しい食糧源に変更する．

	   FlowerSet *fSet; //属している食糧源集合
	   double *pos; //位置
	   double value;//評価値
	   int visitNum;//蜜の採取回数

private:
	   void evaluate();//評価値を算出する
};
#endif

//Flower.cpp
#include "Flower.h"
#pragma once

//コントラスた
//argFSet:属している食糧源集合

Flower::Flower(FlowerSet *argFSet){
  int i;

  fSet=argFSet;
  pos=new double[fSet->dataset->exVarNum];
  for(i=0;i<fSet->dataset->exVarNum;i++){
    pos[i]=COEF_MIN+(COEF_MAX-COEF_MIN)*RAND_01;
  }
  visitNum=0;
  evaluate();
}

//デストラクタ
Flower::~Flower(){
  delete[]pos;
}
//baseの近くの食糧源に変更する
//base:元にする食糧源の添字
void Flower::change(int base){
  int i,j;

  for(i=0;i<fSet->dataset->exVarNum;i++){
    pos[i]=fSet->flower[base]->pos[i];
  }
  i=rand() % fSet->dataset->exVarNum;
  j=(base+(rand() % (EBEE_NUM-1)+1)) % EBEE_NUM;
  pos[i]=pos[i]+(rand() / (RAND_MAX/2.0)-1)*(pos[i]-fSet->flower[j]->pos[i]);
  visitNum=0;
  evaluate();
}

//新しい食糧源に変更する
void Flower::renew(){
  int i;
  for(i=0;i<fSet->dataset->exVarNum;i++){
    pos[i]=COEF_MIN+(COEF_MAX - COEF_MIN)*RAND_01;
  }
  visitNum=0;
  evaluate();
}

//評価値を算出する
void Flower::evaluate(){
  int i,j;
  double diff;

  value=0.0;
  for(i=0;i<fSet->dataset->dataNum;i++){
    diff=fSet->dataset->resSData[i];
    for(j=0;j<fSet->dataset->exSData[i][j];j++){
	 diff -= pos[j]*fSet->dataset->exSData[i][j];
    }
    value += pow(diff,2.0);
  }
}

//FlowerSet.h
#ifndef INCLUDED_FLOWERSET
#define INCLUDE_FLOWERSET
#include "Flower.h"
#pragma once
class Flower;

class FlowerSet
{
public:
  FlowerSet(char *filename);
  ~FlowerSet();
  void employedBeePhase(); //収穫蜂フェーズを実行する
  void onlockerBeePhase();//追従蜂フェーズを実行する
  void scoutBeePhase();//偵察蜂フェーズを実行する．
  void saveBestPos();//最良食糧源を記録する．
  void printResult();//結果を表示する．

  Dataset *dataset;//データセット
  Flower **flower;//食糧源の集合のメンバ
  double *bestPos;//最良食糧源の位置
  double bestValue;//最良食糧源の評価値

private:
  Flower *newFlower; //新しい食糧源
  double *trValue; //格食糧源の変換後の評価値
};

#endif

試したこと

著書には載っていませんでしたがインクルードガードなどは行いました

補足情報（FW/ツールのバージョンなど）

環境は
macbook pro macOS BigSur ver.11.4
コンパイラ環境はEmacsです

Comments

[episteme]

ヘッダ: FlowerSet.h も見せておくれ

[RyoBaya]

#ifndef INCLUDED_FLOWERSET
#define INCLUDE_FLOWERSET
#include "Flower.h"
#pragma once
class Flower;

class FlowerSet
{
public:
FlowerSet(char *filename);
~FlowerSet();
void employedBeePhase(); //収穫蜂フェーズを実行する
void onlockerBeePhase();//追従蜂フェーズを実行する
void scoutBeePhase();//偵察蜂フェーズを実行する．
void saveBestPos();//最良食糧源を記録する．
void printResult();//結果を表示する．

Dataset *dataset;//データセット
Flower **flower;//食糧源の集合のメンバ
double *bestPos;//最良食糧源の位置
double bestValue;//最良食糧源の評価値

private:
Flower *newFlower; //新しい食糧源
double *trValue; //格食糧源の変換後の評価値
};

#endif

お願いします

[episteme]

ダメでしょコレ。
Flower.h に class Flower が定義されてるはず。にもかかわらず class Flower; してるから多重定義。

[episteme]

> 著書には載っていませんでしたがインクルードガードなどは行いました

#pragma once がインクルード・ガードだからやんなくていい。

あと
> //コントラスタ
> // filename: データセットのファイル名
> FlowerSet::Flower(char *filename){

いきなりマチガイ。コンストラクタなら FlowerSet::FlowerSet(...

[RyoBaya]

#ifndef INCLUDED_FLOWER
#define INCLUDED_FLOWER
#include "DataSet.h"
#include "FlowerSet.h"
#pragma once
class FlowerSet;

//定数の定義
#define REPEAT_NUM 1000 //繰り返し数
#define EBEE_NUM 100 //収穫蜂の数
#define OBEE_NUM 10 //追従蜂の数
#define VISIT_MAX 10 //蜜の採取可能回数
#define COEF_MIN -1 //標準偏回帰係数の最小値
#define COEF_MAX 1 //標準偏回帰係数の最大値

//0以上１以下の実数乱数
#define RAND_01 ((double)rand() / RAND_MAX)

class Flower{
public:
Flower(FlowerSet *argFSet);
~Flower();
void change(int base); //近くの食糧源に変更する
void renew();//新しい食糧源に変更する．

FlowerSet *fSet; //属している食糧源集合
double *pos; //位置
double value;//評価値
int visitNum;//蜜の採取回数

private:
void evaluate();//評価値を算出する
};
#endif

flower.hです．確かに20行目で定義されていると思います．どうすればいいでしょうか．

[episteme]

だからそいつは質問に追記して。質問は編集できるから。

> どうすればいいでしょうか．
FlowerSet.h 内の class Flower; が余分なのでは?

[RyoBaya]

今追記しました．

5行目のclass Flower;を消すということで大丈夫ですか？

[episteme]

やってみたらわかる。いちいち許可をもらわんでよろし。

[jbpb0]

> ソースコードを書き写し

https://www.ohmsha.co.jp/book/9784274222382/#tab3
からダウンロードしてます？

[jbpb0]

書籍に記載のコードを、こんなに載せちゃってもいいの？

[RyoBaya]

それは知りませんでした．
ありがとうございました．

g++ -o bee sample04.cpp
sample04.cpp:10:25: warning: conversion from string literal to 'char *' is deprecated [-Wc++11-compat-deprecated-writable-strings]
fSet = new FlowerSet("sampledata.csv");
^
sample04.cpp:16:17: warning: illegal character encoding in string literal [-Winvalid-source-encoding]
printf("%d<89><F1><96>ځF<8D>ŗǕ]<89><BF><92>l%f\n", i, fSet->bestValue);
^~~~~~~~~~~~ ~~~~ ~~~~~~~~~~~~
sample04.cpp:16:7: error: use of undeclared identifier 'printf'
printf("%d<89><F1><96>ځF<8D>ŗǕ]<89><BF><92>l%f\n", i, fSet->bestValue);
^
2 warnings and 1 error generated.

こんなエラーが発生しています．
元質問も変えます．

[jbpb0]

https://www.ohmsha.co.jp/book/9784274222382/#tab3
からダウンロードしたソースファイルはシフトJISのファイルなので、Macでコンパイルする場合は、もしかしたらUTF-8とかに変換しないとダメかもしれない (よく知らない)

参考
https://qiita.com/mm_sys/items/7b8b415a54d1cd0caa5e
https://social.msdn.microsoft.com/Forums/ja-JP/62164982-9795-4f5a-bd7c-85ec121f9126/mac

[episteme]

DataSet.cpp と sample04.cpp に #include <cstdio> を追加。

[RyoBaya]

clang: error: no input files
というエラーが出ます

[jbpb0]

> clang: error: no input files

何をしたのでしょうか？
ソースコードファイルに「#include...」を追記したり、ソースコードファイルの文字コード(シフトJISとか)を変換して保存し直したりしただけでは、上記のようにはならないはず

下記のようなことをしてませんか？
・ソースコードファイルが置いてある場所ではない場所がカレントディレクトリになってる状態でコンパイルしようとした
・コンパイル時に指定したソースコードファイルの名前が違う
・ソースコードファイルを指定せずにコンパイラのコマンドだけ実行した

[RyoBaya]

g++ -o sample04.cpp
このように実行しています

[episteme]

僕とこ(Win10/Visual C++)ではコンパイル/実行できました。
sampledata.csvがないのであっけなく停止するけど。

[episteme]

> g++ -o sample04.cpp

それじゃダメ。
g++ -o sample4 *.cpp

[RyoBaya]

コマンドを実行したところsample4という実行ファイルができました．
./sample4とターミナルで打ち込みましたがzsh: segmentation fault ./sample4が出てしまいました．
コマンドが違いますか？

[episteme]

"sampledata.csvが開けません"で終了しませんか? 僕とこと挙動が異なるな...

[jbpb0]

> zsh: segmentation fault ./sample4

当方のMacで、ダウンロードしたソースに

> DataSet.cpp と sample04.cpp に #include <cstdio> を追加。

を行い、文字コードの変換はしないで、

g++ -o sample4 *.cpp

を実行して、できた「sample4」を

./sample4

で実行したら、「sampledata.csv???J???܂???D」と表示されました
ソースコードの文字コードがシフトJISだからですね

そこで、

./sample4 | nkf -w

と、結果表示をUTF-8に変換したら、「sampledata.csvが開けません．」と表示されました

【追記】 ただし、当方のMacのxcodeは古いです (OSが古いので)
gcc -v
...
Apple LLVM version 10.0.1 (clang-1001.0.46.4)

[RyoBaya]

utf-8に変換した後，g++ -o sample4 *.cppを実行し./sample4を実行しました．
文字化けに関する警告は無くなりました．しかし，zsh: segmentation fault ./sample4も残っています．

[jbpb0]

当方のMacで、ダウンロードしたソースに

> DataSet.cpp と sample04.cpp に #include <cstdio> を追加。

を行い、全ファイルの文字コードを「nkf -w 」でUTF-8に変換し、

g++ -o sample4 *.cpp

を実行して、できた「sample4」を

./sample4

で実行したら、「sampledata.csvが開けません．」と表示されました
「segmentation fault」にはなりません

違いは何でしょうね？
たとえば、
https://qiita.com/monhan/items/ea7e642b91392bfce8a6
や
https://perogram.hateblo.jp/entry/visual_studio_code_mac
に載ってるような簡単なコードをコンパイルして実行したら、正常に実行できるのですよね？

> 以下の警告が残っていました

それはこちらでも出ます (ソースコードファイルの文字コードに関係なく)

参考
https://teratail.com/questions/9652

[jbpb0]

もし「sampledata.csv」を用意して実行していたら、そのファイルが無い状態で実行してみてください

[RyoBaya]

sampledata.csvが原因のようです．sampledata.csvを同じフォルダに入れておかなければ開けませんと出ます．しかし同じフォルダに入れておくとsegmentation faultが出ます

[RyoBaya]

質問に使用しているファイルのスクショを貼りました．これは.txtファイルですが実際は.csvを使ってます．

[RyoBaya]

現在発生している警告文です．
sample04.cpp:11:25: warning: conversion from string literal to 'char *' is deprecated [-Wc++11-compat-deprecated-writable-strings]
fSet = new FlowerSet("sampledata.csv");
^

[episteme]

↑ コンストラクタ引数にconstつけて FlowerSet::FlowerSet(const char* ... としろ、ってことやろね。

[jbpb0]

CSVの最後の行も改行してください
最後の行だけ改行してないのでは？

> 現在発生している警告文です．

いやだから、それは上のコメントに書いたように、「segmentation fault」とは関係ないですって
参考に挙げたteratailの過去質問の回答を読んでください

[RyoBaya]

元質問に.csvのスクショを追加しました．
作り直しましたがだめでした．

[jbpb0]

当方では、CSVの最後の行を改行してない状態では、「Segmentation fault」になりました
最後の行も改行したら、「Segmentation fault」にならず、計算されました

最後の行も改行する、というのは、テキストエディットでCSVファイルを開いたら、数字が記入されてる最後の行の一つ下の一番左にカーソルがある状態になる、ということです
質問に挙げた画像の場合なら、「5.36」の下の一番左にカーソルを持ってこれる、という状態です

ちなみに、CSVファイルの最後の改行はたくさんあっても大丈夫なので、数字が記入されてる最後の行の下に、何回か改行のみ追加してみてください

[RyoBaya]

できました！
お二方とも詳しくありがとうございました！

Answer 1

int32_tさんの回答の

質問内容が変わってしまったので追記:

の内容に加えて、「sampledata.csv」は最後の行も改行する必要があります
最後の行を改行せずに当方のMacで実行したら「Segmentation fault」になりました

Answer 2

#ifndef INCLUDED_FLOWERSET
#define INCLUDE_FLOWERSET

このインクルードガードの不一致を直せば redifinition は直るかもしれません。

flowerset.cpp:7:12: error: C++ requires a type specifier for all declarations
FlowerSet::Flower(char *filename){
flowerset.cpp:11:14: error: must use 'class' tag to refer to type 'Flower' in this scope
  flower=new Flower *[EBEE_NUM];

これは、「関数 Flower() に返り値の型が必要だぞ」と「関数 Flower() 内では Flower クラスを参照したければ class Flower にしろ」という意味ですが、そもそも関数名が間違っている気がします。FlowerSet() では?

---

質問内容が変わってしまったので追記:

• FlowerSet コンストラクタの引数の型を char* から const char* に変更してください。
• ソースコードの文字コードをUTF-8に変換してください。Emacsの場合、該当のファイルを読み込んで M-x set-buffer-file-coding-system で utf-8-unix を指定してから、保存します。
• sample04.cpp に #include <stdio.h> を足してください。

Comments

[RyoBaya]

>このインクルードガードの不一致を直せば redifinition は直るかもしれません。
episteme様のご指摘で#pragma onceがあればいいとのことでしたのでコメントアウトしてしまいました．

FlowerSetに変更しました．

Answer 3

error: redefinition of 'FlowerSet'
class FlowerSet
      ^
flowerset.cpp:7:12: error: C++ requires a type specifier for all declarations
FlowerSet::Flower(char *filename){
flowerset.cpp:11:14: error: must use 'class' tag to refer to type 'Flower' in this scope
  flower=new Flower *[EBEE_NUM];
             ^
             class 

「redefinition」なので、二重定義になっているようですね。
「flower=new Flower *[EBEE_NUM];」「class」は、「class」定義されていないということでしょうね。
ぱっと見、Flower というクラスはないように見えますし、FlowerSet には、コンストラクタがないように見えます。
本は 読んでいないので、わたしが 浅はかに 見てしまったのかもしれませんが・・・質問者さんは、「C++」に関しては、学習されていますか？ そうでないようでしたら、学習が先かと思います。

Comments

[RyoBaya]

ヘッダファイルがFlowerSet.h Flower.h Dataset.h
ソースファイルがFlowerSet.cpp Flower.cpp DataSet.cpp main.cpp
があります．c++の勉強は付け焼き刃です．cに関しては勉強しています．

[showkit]

episteme さんが おっしゃるように、全部 ソースを 提示しないと 原因がつかめないでしょうし、「Flower.h に class Flower が定義されてるはず。にもかかわらず class Flower; してるから多重定義」なんでしょうね。質問は 編集できるので、全部 code タグで 載せた方がいいかな・・・ ただし、著作権の問題にかかるようでしたら、掲載は できないですが。それと その本 のソース が 正しく コンパイル・リンク できて、実行できることは 保証されていますか？ 保証されていないのであれば、あきらめた方がいいです。

[RyoBaya]

codeタグで追記しました．