ヘッダーファイル内での変数の宣言について

Question

前提・実現したいこと

C言語を勉強しているのですが、勉強をするにあたって苦しんで覚えるC言語というサイトを参考にしています。今はヘッダーファイルを扱う章を進めていて、そこで、サイトには動かないと書いてあったコードが動いてしまい不思議に思い質問しました。

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

こちらには

例えば、次のようにヘッダーファイル内で変数を宣言すると、

宣言が重複している、という意味のエラーが表示され、コンパイル出来ません。

/* sum.h */
int sum(int min,int max);
int Public;

と書いてあるのですが、自分の環境ではsum.hにこのまま書いても動いてしまいました。

該当のソースコード

/* main.c */

#include <stdio.h>
#include "sum.h"

int main(void)
{
    int value = 0;
    value = sum(1, 2);

    printf("%d\n", value);

    return 0;
}

/* sum.c */

int sum(int min, int max)
{
    int num;
    num = (min + max) * (max - min + 1) / 2;
    return num;
}

/* sum.h */
int sum(int min, int max);
int public;

補足情報

gccコンパイラを使用
% gcc main.c sum.c　でコンパイルしましたが、苦しんで覚えるC言語のサイトにgcc用のコマンドが載っていなかったので間違っていたら教えていただけると幸いです。

Answer 1

a.c, b.c というファイルのどちらもがsum.hをインクルードしている場合、リンクエラーがでます

ですから、sum.hを読み込むcファイルが1つしかない場合はエラーが出ません

Answer 2

すでに他の回答がありますように、sum.cでsum.hをインクルードしてないからですが、C言語では

int public;
```が複数のソースに書かれていても、処理系（コンパイラー）によってはエラーにならないものもあります。例えば、`gcc`や`VC++`はエラーにならないはずです。

`C言語`では

int public;
```は“仮の”定義です。複数の仮の定義があった場合、処理系によってはエラーになりますが、gccやVC++のように一つのインスタンスにまとめるものもあります。

とはいえ複数の定義があると混乱のものなので、定義は一カ所にして、他はexternを付けて宣言にした方がいいですね。

Answer 3

インクルードガードの問題かね...
もしくはこちらに書かれている事があてになると思います。

普通は両方行うので、片方ずつデバックを行うのがいいと思います。

Answer 4

sum.c のほうでもそのヘッダファイルをインクルードしましょう
そうすればエラーになります

---

なぜエラーになるのかを考えてみましょう
まあ、そもそもヘッダファイルで変数の定義は行っても、変数宣言は行うべきではありませんね

Comments

[退会済みユーザー]

回答ありがとうございます。
定義が２回行われるからでしょうか？

[y_waiwai]

複数の場所で同じ名前の変数が作成されるため、リンク時にエラーになります
まあ、これがグローバル変数（外部にも公開される変数）なのでこういうエラーが出るってことですね。

＃staticを付けた変数宣言だと、どういうことが起こるのか考えてみたらいいかもしれません

[退会済みユーザー]

なるほど、参考になります。

他の.cファイルで値を変更すれば中身も変わるのではないかと思いました。

[y_waiwai]

staticをつけた変数宣言では、その変数名は宣言されたファイルの中だけ有効となります

ということは、ヘッダファイルにstatic付きで変数宣言を行うと、そのヘッダをインクルードしたソース・ファイルの数だけその変数が生成されるってことが起こります
まあ、エラーも出ず、異常な動作もしない代わりに、ソース・ファイルの数だけ、無駄な変数領域が消費されるということになります

とあるソース・ファイルが数十あるプロジェクトで、やけにメモリ消費が多いな、と思ってたら、とあるヘッダファイルにstatic変数が大量に宣言されてたってことがありましたね。