c++ ソースファイルとインクルードすると原因のわからないエラーでる

Question

初心者なのですがヘッダーではなくソースファイルをインクルードしてみたのですが原因のわからないエラー出てしまい困っています。
そもそもソースファイルをインクルードすること自体悪いのではと思っています。
原因を教えてくれますでしょうか？

１エラー LNK2005 "void __cdecl test(void)" (?test@@YAXXZ) は既に Main.obj で定義されています。
２エラー LNK1169 1 つ以上の複数回定義されているシンボルが見つかりました

#include <iostream>
#include "conio.h"

//#include "Header.h"
#include "Source2.cpp"

using namespace std;


int main()
{

	test();
//	cout << eint;
	//source_f();
	//cout << eint;


	int _r = _getch();
	return 0;
}

#ifndef source2___
#define source2___
#include <iostream>

void test()
{
	std::cout << "テスト";
}

#endif

Comments

[cateye]

〉原因のわからないエラー出てしまい・・・エラーメッセージを貼って下さい。
あと、環境（OS,コンパイラなど）はなんですか？

Answer 1

ソースファイルをインクルードしてみた

なぜ。。。

1 つ以上の複数回定義されているシンボルが見つかりました

C/C++のincludeは自動コピペ機能です。
ですので、main関数のあるファイルとSource2.cppの両方にtest関数の実体があることになります。

対策
どちらでも対処できます...が、前者を選ぶ理由は無いでしょう。

• Source2.cppをコンパイル対象から外す
• おとなしくヘッダファイルを使う

Comments

[cateye]

あ！なるほど・・・コンパイル対象とincludeと2個あるってことですね？　配列とか構造体の初期化データは別にすることがありますが、拡張子変えてますね。・・・c++ *.cppでコンパイルできるしｗ

[退会済みユーザー]

そもそもやってはダメな行為ということいいのでしょうか？

[LouiS0616]

@_Beginner__ さん
絶対ダメというわけでは無いですが、普通は想定されないものですし、そもそもメリットがありません。

[退会済みユーザー]

ではエラーにならないための方法を教えてくれますでしょうか？
気になってしまっているので。

[LouiS0616]

void test(); と書いたヘッダファイルを用意して(インクルードガードもお忘れなく)、二つのcppファイルそれぞれからインクルードすれば良いです。

Answer 2

ソースコードに相いれない定義を複数書いても、includeしたことで相いれない定義が並んだとしても、ダメなのは同じです。

includeしたものも含めて、全部を集め、その中で重複しているものを見つけましょう。

重複している定義を削っていって、重複がなくなれば、重複定義のエラーは自然と解消されます。

＝＝＝

Answer 3

同じプロジェクトで以前に何か作りましたか？・・・以前のデータが有ると新しいプロジェクトに影響が出ます。新しくプロジェクトを起こして下さい。・・・たぶん治る
あと、プリコンパイルヘッダは使わないほうがいいです。

Comments

[cateye]

WindwsやVSには疎いです、間違ったことを言っていたらご容赦下さい。

[退会済みユーザー]

いえ、何も作成していません。初心者なのでエディタの設定は特にいじるほどの知識がいので触ってないのですがなぜでしょうか？一応プロジェクトを作り直しましたが同じエラーになりました。