C言語 配列で関数演算後の値を使いたいです。

Question

前提

ソースコードでは1つのファイル、1つの関数からの呼び出しですが、

配列はヘッダファイルに記述したいです。
配列は複数の関数で使いたいです。

以上のことを踏まえてどのような実装が綺麗か教えていただきたいです。

実現したいこと

関数の演算結果を配列の値に用いたいです。どのような実装をすれば綺麗にまとまるか知りたいです。
配列のclacNum_20_25　の箇所にはcalcNum(20,25)の値を入れたいです。

今の状態だと配列を使う関数で毎回 clacNum_20_25 の値を求めなければいけません。新しい関数を作成して、この配列を使う時に気づけない可能性があるのでこれは避けたいです。

該当のソースコード

#include <stdio.h>

#define ARRAY_X 2
#define ARRAY_Y 3

#define clacNum_20_25 0
#define clacNum_30_35 0
#define clacNum_40_45 0

int calcNum(int a , int b);

struct NUM {
    int a;
    int b;
};

struct NUM num[ARRAY_X][ARRAY_Y] = {
    {
        {35 , 40},
        {45 , 50},
        {55 , 60}
    },
    {
        {clacNum_20_25,50},//clacNum(20,25)
        {clacNum_30_35,60},//clacNum(20,25)
        {clacNum_40_45,70} //clacNum(20,25)
    }
};

int main (void){
    num[1][0].a = calcNum(20,25);
    num[1][1].a = calcNum(30,35);
    num[1][2].a = calcNum(40,45);

    int i,j;
    for(i = 0; i < ARRAY_X; i++){
        for(j = 0; j < ARRAY_Y; j++){
            printf("%d\t%d\n",num[i][j].a,num[i][j].b);
        }
    }

}

int calcNum(int a , int b){
    int c;
    c = a + b;
    return (c);
}

試したこと

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Comments

[jimbe]

配列、というのは struct NUM num[ARRAY_X][ARRAY_Y] のことでしょうか。
例としてかもしれませんが、各値は固定値に見えます。なぜ予め計算した結果を配列に入れないのでしょうか。

[pzzz_6]

配列、というのは struct NUM num[ARRAY_X][ARRAY_Y] のことでしょうか。
　そうです。
例としてかもしれませんが、各値は固定値に見えます。なぜ予め計算した結果を配列に入れないのでしょうか。
　例を簡単にしすぎていますが、本来は複雑な計算をする関数の値を配列に代入したいです。

質問の本質としては、関数の計算結果を配列の値で使いたい場合、どのような実装がよいか。ということになります。
載せているソースコードでは必要ないですが、よろしければ教えていただければと思います。

[maisumakun]

> 例を簡単にしすぎていますが、本来は複雑な計算をする関数の値を配列に代入したいです。

それを初期化で回してしまう、では何がよくないのでしょうか？

[dodox86]

@質問者 pzzz_6さん

ご質問に直接関係は無いですが、他者の文の引用をするときは行頭に">"、"＞"を付けると読んだ者に混乱が無くてよいです。これはEメールなどでも使われる一般的なマナーです。

例えば

> 載せているソースコードでは必要ないですが、よろしければ教えていただければと思います。

>> 配列、というのは struct NUM num[ARRAY_X][ARRAY_Y] のことでしょうか。
> 　そうです。

のように使います。

[pzzz_6]

>> それを初期化で回してしまう、では何がよくないのでしょうか？
> 私の知識不足で発想がありませんでした。ほかの方からの回答もあり解決しました。ありがとうございます。

dodox86さん、丁寧に教えていただきありがとうございます。

Answer 1

ライブラリを作る際等は、 include ファイル内にプリプロセッサ命令を書いて extern の有無を切り替えたりすることがありました。
num の表示を関数・別ファイル化する形にしてみました。
(NUM_MAIN の有無で extern/初期化を切り替えています。initNum も include 内で宣言していますが、main 以外で初期化する場合を想定しているだけです。)

q_lcdf8djc0wqo24.c

#include <stdio.h>

#define NUM_MAIN
#include "q_lcdf8djc0wqo24.h"

void printNum();

int main (void){
  initNum();

  printNum();
}

int calcNum(int a , int b){
  int c;
  c = a + b;
  return (c);
}

void initNum() {
  num[1][0].a = calcNum(20,25);
  num[1][1].a = calcNum(30,35);
  num[1][2].a = calcNum(40,45);
}

q_lcdf8djc0wqo24.h

#ifndef Q_LCDF8DJC0WQO24_H_
#define Q_LCDF8DJC0WQO24_H_

#define ARRAY_X 2
#define ARRAY_Y 3

struct NUM {
  int a;
  int b;
};

#ifndef NUM_MAIN
extern
#endif //NUM_MAIN
struct NUM num[ARRAY_X][ARRAY_Y]
#ifdef NUM_MAIN
= {
  {
    {35 , 40},
    {45 , 50},
    {55 , 60}
  },
  {
    {/*clacNum_20_25*/0, 50},
    {/*clacNum_30_35*/0, 60},
    {/*clacNum_40_45*/0, 70}
  }
}
#endif //NUM_MAIN
;

void initNum();

#endif /* Q_LCDF8DJC0WQO24_H_ */

q_lcdf8djc0wqo24_sub.c

#include <stdio.h>

#include "q_lcdf8djc0wqo24.h"

void printNum() {
  for(int i = 0; i < ARRAY_X; i++){
    for(int j = 0; j < ARRAY_Y; j++){
      printf("%d\t%d\n",num[i][j].a,num[i][j].b);
    }
  }
}

実行結果

35	40
45	50
55	60
45	50
65	60
85	70

Comments

[pzzz_6]

ありがとうございます。
具体的にコードを示していただけたので、大変勉強になりました。
参考にさせていただきます。

[pzzz_6]

コンパイルするとエラーが出ました（私のやり方が悪いのかもしれません）
undefined reference to `printNum'

q_lcdf8djc0wqo24_sub.cをq_lcdf8djc0wqo24.cでincludeしたら改善したのですが、対処法としてこれは合っているのでしょうか？

[jimbe]

*.c を *.c に include することは、余程なことで無ければやらないことと思います。

undefined ということはコードをコンパイルした後のリンクで sub を入れていないのではないでしょうか。

[pzzz_6]

その通りでした。ありがとうございます。

Answer 2

ヘッダファイルには、

#define ARRAY_X 2
#define ARRAY_Y 3

struct NUM {
    int a;
    int b;
};

extern struct NUM num[ARRAY_X][ARRAY_Y];

と記述しておけばいいです

Comments

[pzzz_6]

ありがとうございます。
externを使うのですね。
この通りにしてみると配列への代入のための計算は一度で済みました。勉強になりました。

この構造体配列を複数の関数で使い、その呼び出しの順番が決まっていない場合についてです。
この場合の実装は、main関数でそれらの関数が呼び出される前に値の計算と代入を行えば良いという認識であっていますでしょうか？

[y_waiwai]

そのとおりです。
値を代入しないと、初期値が出てくるだけです

[pzzz_6]

勉強になりました。ありがとうございます。