Ｃ言語でソースファイル単位で、関数を公開/非公開にする方法について

Question

前提

機能の複雑化に伴い、ソースファイルを分けるのですが、
複数人数でリポジトリを使って開発を行っているため、
「特定のソースからしか呼ばせたくない関数」の記述にどのような方法があるか調べております。

該当のソースコード

static int counter = 0;
void countup( void )
{
    counter += 1;
}

extern void countup( void );
static void ExecFuncA1( void )
{
    countup();
}

FuncA.c を 「FuncA0.c」「FuncA1.c」に分けました。
(当該コードより簡略化して記載しております。)
元々 static void countup( void ) だったものから static を外しました。

複数人数で開発している環境では
他の方のソースファイルで、extarnすれば、countup関数の呼び出しが可能です。
これを何らかの方法で、「呼び出せない」または「ビルドが通らない」ようにする方法を検討しております。

アドバイスを頂きたいこと

今、私の頭に浮かんでいる対応方法としては、
案① FuncA0.c FuncA1.c はライブラリ化する。
案② static_assert を用いて ヘッダに公開する(抑止程度ですが…)

#ifndef UNPUBLISHED_FUNCTIONS_H
#define UNPUBLISHED_FUNCTIONS_H
void countup( void );
static_assert( FUNC_A1_ONLY == true, ”FuncA1 only!!“ );
#endif

#define FUNC_A1_ONLY    ( true )
#include "UnpublishedFunctions.h"
void ExecFuncA1( void )
{
    countup();
}

案①は、目的はある程度達成できますが、ビルドを２種類用意する必要があること。
案②は、やろうと思えば、どのソースからも呼び出すことができること。(抑止力程度…)
と、
その他になにか方法がないか、アドバイスを頂ければと思います。
宜しくお願い致します。

Comments

[int32_t]

> 元々 static void countup( void ) だったものから static を外しました。

なぜ外したのでしょうか。

[GuielNo4]

質問ありがとうございます。
ビルド環境によるかもしれませんが、
FuncA0.c と FuncA1.c の関数は別物と認識されないよう外しました。
一応確認のため、外すと、
A) FuncA0.c → 参照されていません。的なワーニング
B) FuncA1.c → 定義がありません。ビルドエラー
となりました。

[GuielNo4]

申し訳ありません。
FuncA1.c のサンプルコードを訂正させて頂きました。
「void ExecFuncA1( void )」→「static void ExecFuncA1( void )」
解釈が異なってしまう記述で申し訳ありません。引き続き宜しくお願い致します。

Answer 1

「分ける」という作業のやり方を間違っている，という話に見えます．

「特定のソースからしか呼ばせたくない関数」

を，元々はそのソース内に static 関数として書いていた，という話なのでしょうから，そこはそのままにしておくべきでは．

---

どうしても「ファイル」という単位で分けたいというのであれば，

//※このファイルはコンパイル対象としない
static void countup( void ){ ... }  //static関数の実装記述

#include "FuncA0.c_"

とかしておけばまぁ…（こんなことやる意味あるのか？ってのは別として）

Comments

[GuielNo4]

アドバイスありがとうございます。
ヘッダではないものインクルードする的な発想がありませんでした。
今後、別の何かに利用したいと思います。
回答ありがとうございました。

[fana]

（後半の話はわりとネタ気味なので変に利用できるとか考えない方が）

重要なのは前半です．
「（そこは）分けない」のが正解ちゃうんかと．

意味的に内部リンケージでなければならないものを翻訳単位の外部に追い出し→それで生じた齟齬を何かしらの小手先の技でごまかそう，
みたいな作業を実施しようと考えていること自体がそもそも間違っている．

Answer 2

標準のC言語の範囲内で完全に使用を防ぐには、countup() を FuncA1.c に移動して static を付ける以外の手段はないと思います。

案①について : FuncA*.c を1つの動的ライブラリ(*.dll *.so *.dylib)にすると、ライブラリから公開するシンボルを選べるので実現できます。静的ライブラリでは宣言を書けば使えるので完全に防ぐことはできません。

案②に似たような話ですが、FuncA*.c を1つのコンポーネントとして考えて、それらをコンパイルするときにはビルドルールで IN_FUNC_A などのマクロを定義するようにします。IN_FUNC_A が定義されているときだけ countup() をヘッダで宣言すれば意図は伝わるんじゃないかと思います。

私が仕事で関わったプロジェクトだと、どのディレクトリからどのファイルを #include できるかできないかのルールが定義してあって、コードレビューの前処理としてそのルールのチェックが行われていました。この場合、countup() を宣言するヘッダは FuncA*.c があるディレクトリ以外からの利用を禁止すればいいですね。

C++ だと passkey pattern などで呼び出し元クラスを制限できます。

Comments

[GuielNo4]

アドバイスありがとうございます。
やはりバグを出さないために検討しているので、
頂いております回答より、
「ビルドルール」「コードレビュー」で対応することも
検討すべきだと思いました。
回答ありがとうございました。

Answer 3

「特定のソースからしか呼ばせたくない関数」の記述にどのような方法があるか調べております。

無理です。あきらめよう

Comments

[GuielNo4]

アドバイスありがとうございます。
ないかもしれないものを、すっと探し続けるよりは
先達に「ない」とアドバイス貰ったほうがいいタイミングだったかもしれません。
回答ありがとうございました。

[y_waiwai]

まあ、せいぜいライブラリにして、ソースを公開しない、必要なところ以外には仕様を公開しない、とすることでしょうね。