extern 変数の使い方

Question

前提・実現したいこと

ヘッダーファイルに外部変数を置きたいのですが
externの使い方がいまいちわかっていません。
externはどうやって使うのでしょうか？

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

1>------ ビルド開始: プロジェクト: 練習extern, 構成: Debug Win32 ------
1>コンパイルしています...
1>stdafx.cpp
1>コンパイルしています...
1>練習extern.cpp
1>マニフェストをリソースにコンパイルしています...
1>Microsoft (R) Windows (R) Resource Compiler Version 6.1.6723.1
1>Copyright (C) Microsoft Corporation. All rights reserved.
1>リンクしています...
1>練習extern.obj : error LNK2005: "int * size" (?size@@3PAHA) は既に stdafx.obj で定義されています。
1>C:\Users\user\Documents\Visual Studio 2008\Projects\練習extern\Debug\練習extern.exe : fatal error LNK1169: 1 つ以上の複数回定義されているシンボルが見つかりました。
1>ビルドログは "file://c:\Users\user\Documents\Visual Studio 2008\Projects\練習extern\練習extern\Debug\BuildLog.htm" に保存されました。
1>練習extern - エラー 2、警告 0
========== ビルド: 0 正常終了、1 失敗、0 更新不要、0 スキップ ==========

該当のソースコード

// 練習extern.cpp : コンソール アプリケーションのエントリ ポイントを定義します。
//

#include "stdafx.h"


int _tmain(int argc, _TCHAR* argv[])
{
	printf( "size:%d\n", size[0]);

	return 0;
}

// stdafx.h : 標準のシステム インクルード ファイルのインクルード ファイル、または
// 参照回数が多く、かつあまり変更されない、プロジェクト専用のインクルード ファイル
// を記述します。
//

#pragma once

#include "targetver.h"

#include <stdio.h>
#include <tchar.h>



// TODO: プログラムに必要な追加ヘッダーをここで参照してください。

#ifdef	EXT
extern	int	size[1] = {32};
#else
		int size[1] = {32};
#endif
#define	EXT

---

変更後

// 練習extern.cpp : コンソール アプリケーションのエントリ ポイントを定義します。
//

#include "stdafx.h"
#define	EXT
#include "hoge.h"


int _tmain(int argc, _TCHAR* argv[])
{
	printf( "size:%d\n", size[0]);

	hogehoge();

	return 0;
}

#include "stdafx.h"
#include "hoge.h"


void	hogehoge()
{
	printf( "size:%d\n", size[0] );
}

// stdafx.h : 標準のシステム インクルード ファイルのインクルード ファイル、または
// 参照回数が多く、かつあまり変更されない、プロジェクト専用のインクルード ファイル
// を記述します。
//

#pragma once

#include "targetver.h"

#include <stdio.h>
#include <tchar.h>



// TODO: プログラムに必要な追加ヘッダーをここで参照してください。

void hogehoge();

#ifdef	EXT
int	size[1] = {32};
#else
extern int size[1];
#endif

Answer 1

ヘッダファイルで変数宣言を行ったり、externの切り替えをおこなうってのはバグの温床になります
そんなところで手を抜くのはやめておいたほうがいいかと。

まあ、動けばそんでいいってことならいいでしょうけど

Comments

[itagagaki]

> ヘッダファイルで変数宣言を行ったり

定義、のことですよね

[y_waiwai]

ああ、そうですね。補足ありがとうございます

自分だけがそのコードを扱うってならまあ、いいんでしょうけど、
他人のコードのメンテって時にこれが出てくると、そいつを締めたくなりますｗ

[yukkuri_55]

ご指摘ありがとうございます。

Answer 2

int size[1] = {32};は、「データを定義」します。
「データを定義」するとは、そのデータの現物を用意して、それ用の記憶域を割り当てることを意味します。

たとえば上記の定義をグローバル変数の定義としてヘッダファイルに書いてしまったとします。
そして、そのヘッダを a.c と b.c がそれぞれインクルードするとします。

インクルードというのは、そこに別のファイルの内容を取り込むだけですから、
a.c と b.c は、どちらも上記の定義を行ってしまうことになります。

a.c と b.c のコンパイルはできますが、この2つをリンクするとエラーとなります。
a.c と b.c の両方にグローバル変数sizeが存在し、重複してしまうからです。

ですので、グローバル変数の定義は、全体の中でどこか１カ所でなければなりません。
で、定義したデータを参照するためには「データの宣言」が必要になります。
それが extern int size[1]; です。
このexternは、「ストレージクラス指定子」と言って、「このデータは別のソースファイルで定義されているよ」という意味です。
externとして宣言された変数自体に記憶域は割り当てられません。
ただ参照するためだけの宣言になります。

提示されているヘッダファイルのコードでは、プリプロセッサマクロEXTが定義されているかによって、sizeの「定義」と「宣言」を切り替えているようです。
そのヘッダファイルをインクルードするCファイルのうち１つだけがプリプロセッサマクロEXTを定義することで、sizeを1つだけ定義するという策略だと思います。よく見るコードですが、この策略には賛否があると思います。

ところで

extern    int    size[1] = {32};

これはよろしくありません。宣言のつもりなのに初期化子を指定しています。
上記のとおり、宣言はデータの現物を用意するものではありませんので、これは誤りです。
単に

extern int size[1];

とすべきです。

Comments

[yukkuri_55]

丁寧な説明ありがとうございます。

Answer 3

ヘッダファイルに#define EXTを書いてしまうのでは、せっかく上で#ifdefした意味がなくなります。

どこか1つのソースファイルでのみ有効化してください。あと、stdafx.hは変化しないシステムヘッダーを入れるのが前提のファイルなので、自作の定義を入れるには適切な場所ではありません。

// variables.c

#define    EXT
#include "hoge.h"

Comments

[yukkuri_55]

maisumakunさんのやるやり方で
やってみようかと思います。
修正したのを編集にてあげておきます。
ありがとうございます。

Answer 4

--main.cpp--
#define ETN
#include "global.h"
printf( "size:%d\n", size );

--global.h--
#ifdef ETN
       int size = 32;
#else
extern int size;
#endif

--other.cpp--
#include "global.h"
printf( "size:%d\n", size );

Answer 5

関数だとヘッダで宣言し、C-codeのどれか一つに定義するでしょ? 変数でも同じです。

// sub.h
extern int global_value; 

// sub.c
#include "sub.h"
int global_value;