型名が必要(C言語：分割コンパイル）

Question

分割コンパイルを始めたばかりです
初歩的なミスが出てきて、コンパイルできないので助けてください！

/* dft.h */
#include<stdio.h>
#include<math.h>
#include"dft.h"
#include"complex.h"

#define PI 2.0*asin(1.0)


void clear_array(double x[], int n)		 //要素数nの配列xに,すべて0を代入する(初期化)
{
	for(int i=0;i<n;i++){
		x[i] = 0.0;
	}
}

void square_wave(double x[], int n)		//要素数nの配列xに,任意の周波数の矩形波を書き込む.振幅は任意,例えば-1から1など
{
	const int f=5;		//周波数
	const double A=1.0;		//振幅
	int s=0;
	for(int i=0;i<n;i++){
		if((i%(2*f))<f){
			s=1;
		}else{
			s=0;
		}
		x[i]=A*cos(PI*s);
	}
}

void sin_waves(double x[], int n, double f1, double f2, double f3)
{
	for(int i=0;i<n;i++){
		x[i] = 2.0* sin(1.0*PI*i/n*f1) + 0.8*cos(2.0*PI*i/n*f2) + 0.7*sin(2*PI*i/n*f3);
	}
}

void dft(const double in[], Complex out[], const int N)
//
//  離散フーリエ変換(DFT)
//  元データは実数，変換後は複素数
//
//  in	:  元データ．これは変更しない．実数の配列．
//  out :  計算結果．こちらは複素数．
//  N   :  データ個数．inもoutも同じデータ数でなければならない
//
{
	int t;
	/*  出力用の配列を初期化 */
	for(t=0; t<N; t++) {
		out[t].re = 0.0;
		out[t].im = 0.0;
	}

	for(int f=0; f<N; f++){
		//  以下のループで 1個のF[omega]を計算する．
		double omega = 2 * PI * f;
		for(t=0; t<N; t++) {
			/*  積分の中身の計算 */
		out[f] = Cadd(out[f], Cmul( ToComplex(in[t], 0.0), Cexp( ToComplex(0.0, -omega * t/N) )));
		}
	}
}

 /* dft.h */
void clear_array(double x[], int n);
void square_wave(double x[], int n);
void sin_waves(double x[], int n, double f1, double f2, double f3);
void dft(const double in[], Complex out[], const int N);     //問題の４行目です。

この状態でmakefileのためのにobjファイルを作るための分割コンパイルを行ったのですが、コマンドプロンプトにおいて、以下のようなエラーが出ました。

C:\Users\TaKe\Desktop\情報処理\1年後期\授業8回目>bcc32 08-1.cpp complex.cpp dft.cpp
Borland C++ 5.5.1 for Win32 Copyright (c) 1993, 2000 Borland
08-1.cpp:
警告 W8004 08-1.cpp 9: 'fp' に代入した値は使われていない(関数 main()
)
complex.cpp:
dft.cpp:
エラー E2303 dft.h 4: 型名が必要
*** 1 errors in Compile ***

どこが間違っているのか教えてください。
分割コンパイルを行う場合、普段とは違う決まりなどがあるのでしょうか？
また、今回makeする為の.objを作るための分割コンパイルですが、.objを簡単に作る方法？などはありますか？

以上の3点についてよろしくお願いします！

Answer 1

**すみません、本題とはまったく関係ないのですけど、**本題のの回答は他の方に任せるとして、

#define PI 2.0*asin(1.0)

の部分は、かなり危険と思います。たまたま参照部分に問題となるパターンが無いだけで、式の書き方によっては（たとえば qpi = 4.0 / PI とか）、悲惨な事になりかねません。

#define PI (2.0*asin(1.0))

とした方が良いと思います。

Comments

[ISARIA]

ありがとうございます。
()、これからつけていくようにしていきます！

Answer 2

・Cは後方参照ができません。
・#includeはそこにファイルの中身をコピペするのとほぼ同じです。

#include"dft.h"
#include"complex.h"

この順番だとdft.hで#include"complex.h"していないので
Complexの定義が後になってしまいますね

だからエラーになります。

順番を逆にすればひとまず直りますが、include順番をいちいち気にするのはアホらしいので、
きちんとインクルードガードを付けて、
モジュールが使うモジュールはヘッダーでincludeするようにします。

今回の例で言えばdft.h内でcomplex.hをインクルードします。

Comments

[ISARIA]

ありがとうございます！

Answer 3

おそらくComplexが宣言されていないのでしょうね。
#include"dft.h"を#include"complex.hの後ろに移動すればよろしいのではないでしょうか。

Comments

[ISARIA]

ありがとうございます、includeの宣言の順番にも関係があるんですね。

Answer 4

既に他の方が回答されているようにdft.h内でinclude を定義します。

#ifndef _DFT_H_
#define _DFT_H_ 1
#include <complex.h>
 /* dft.h */
void clear_array(double x[], int n);
void square_wave(double x[], int n);
void sin_waves(double x[], int n, double f1, double f2, double f3);
void dft(const double in[], Complex out[], const int N);
#endif

分割コンパイルの場合、ヘッダーファイルにはこのような形で、定義します。
#pragma once が使える環境ならば、そちらでも良いです。

またincludeファイルを指定する括弧にも意味があります。
complex.hは標準ヘッダーファイルなので、二重引用符ではなく、<>で囲みましょう。