ヘッダーファイルからの取り込みについて

質問

ヘッダーファイルから二次元配列を取り込みたいのですが、上手く取り込めません。うまく取り込む方法を詳しく教えて下さい。お願いします。

回答No.2

Fooky

確かにＣで２次元配列を扱うのは少し難しいんですが、それ以前に色々と問題
があるコードですね。

１．関数file()の戻り値がvoidになっている
　これは２次元配列を返す関数のつもりで書かれているんではないんでしょう
か？それならば、void（戻り値なし）ではなく、２次元配列に相当するものを
返すように宣言しなければなりません。関数の戻り値の指定(void)とreturnの
中に書く変数の型は当然ながら一致していないといけません。

２．それじゃ、２次元配列をどうやって返すのか？
　ここが難しいところなんですが、Ｃでは配列を一つのモノ（オブジェクト）
として扱うという考え方はしません。ＲＡＭの中で配列が格納されているメモ
リの場所（アドレス）を受け渡しします。この、アドレスを入れる変数をポイ
ンタと言います。したがって、file()関数は、１０×１０配列が格納されてい
るアドレスを指すポインタを返す関数にしないといけません。こんな説明を書
き出すと切りがないので、答え（と言ってもまだまだ間違えています）を書く
と、

char (*file())[10]
{
char nijigen[10][10] = {
{'#','#','#',...,'#'},
....,
{'#','#','#',...,'#'}};

return(nijigen);
}

配列を指すポインタ変数とそれを返す関数の書き方は、

char (*変数名)[10]：「10個のchar型が格納されているメモリを指すポインタ」
　　　　　　　　　　型の変数。
char (*関数名(関数の引数))[10]：「10個のchar…ポインタ」型を返す関数

となります。

３．file()の戻り値を受けるポインタ変数の宣言
　nijigenの宣言は合ってましたが、char nijigen2[10][10];は間違いです。
こう書くと、Ｃは、新たに１０×１０の配列をメモリ上に用意します。今、や
りたいことはそれではないはずです。file()の中で用意された１０×１０配列
を扱いたいんですから。

nijigen[10][10]が正解なのは、nijigen[10][10]のところでは実際に
メモリ上に配列を用意するから正解なのです。

正解は、
char (*nijigen2)[10];
nijigen2 = file();

４．表示が転置されてる？
　printf("%c", nijigen2[a][b]);
となってますが、これだと縦がa+1番目、横がb+1番目の要素が
表示されますが良いんですか？

５．局所変数を戻り値にしている
　これは大問題です。file()の中の変数は、file()を出るときに全て解体され
ます。ということは、file()の中でメモリ上に作った10x10配列は、file()か
らreturnするときに消えてなくなります。したがって、ここは、nijigenを大
域変数として定義するのが良いでしょう。

～file.h～
...
static char nijigen[][10] = {
{'#', ..., '#'},
...};

６．ヘッダファイルの使い方がおかしい？
　ヘッダファイルには普通、関数の中身（定義）は書きません。もう少し規模
の大きいプログラムになると即問題が発生します。ヘッダファイルを複数の.c
ファイルから読み込んだときに、同じ名前の関数が２箇所で重複して定義され
ていることになります。変数も同様です。上のstaticというのは、この変数は
今の.cファイルの中だけの話しですよ、ということを示すことによって、
file.hが複数の.cファイルから読まれても大丈夫なようにしています。（その
代わり、読まれた回数だけメモリを消費します）

ふぅ。何にせよ、本を一冊買ってポインタの辺りを勉強することをお勧めしま
す。

私がこのコードを書くと下のように成ります。当然ながらベストとは言いませ
ん。私の好みということです。２次元配列をつくるための「10個のchar…ポイ
ンタ」はできるだけ使わない方が、char (*nijigen2)[10];みたいなややこし
い宣言を使わなくて済みます。

～file.h～

static const int RowNum = 10;
static const int ColNum = 10;

static const char nijigen[RowNum][ColNum] = {
{...},
...
};

～file.c～

int
main(int argc, char **argv)
{
int r, c;
const char *pointer = nijigen[0];

for( r = 0 ; r < RowNum ; r++ ){
for( c = 0 ; c < ColNum ; c++, pointer++ )
printf("%c", *pointer);
printf("\n");
}
}