Ｃ言語の関数を読み込みません。

質問

今、ゲームのようなプログラムを作っていて、下はそのダメージ計算量を計算するプログラムを書いたつもりでいたのですが、どうやら「compatibility」と作った関数が計算されていないようです。

int compatibility(int a[2][3], int s, int t, struct monster monster[])
{
int m;
int add;

printf("a2\n");
if(
(monster[a[s][0]-1].tricktype == 1 && (monster[a[t][0]-1].type == 3 || monster[a[t][0]-1].type == 4)) ||
(monster[a[s][0]-1].tricktype == 2 && monster[a[t][0]-1].type == 1) ||
(monster[a[s][0]-1].tricktype == 3 && (monster[a[t][0]-1].type == 2 || monster[a[t][0]-1].type == 4)) ||
(monster[a[s][0]-1].tricktype == 4 && (monster[a[t][0]-1].type == 2 || monster[a[t][0]-1].type == 5)) ||
(monster[a[s][0]-1].tricktype == 5 && (monster[a[t][0]-1].type == 1 || monster[a[t][0]-1].type == 3))
/* 効果抜群 */
){
m = (int)rand() % 2;

if(m == 0) add = 1;
if(m == 1) add = 2;
}else{
if(
(monster[a[s][0]-1].tricktype == 1 && (monster[a[t][0]-1].type == 2 || monster[a[t][0]-1].type == 5)) ||
(monster[a[s][0]-1].tricktype == 2 && monster[a[t][0]-1].type == 3) ||
(monster[a[s][0]-1].tricktype == 3 && (monster[a[t][0]-1].type == 1 || monster[a[t][0]-1].type == 5)) ||
(monster[a[s][0]-1].tricktype == 4 && (monster[a[t][0]-1].type == 1 || monster[a[t][0]-1].type == 3)) ||
(monster[a[s][0]-1].tricktype == 5 && monster[a[t][0]-1].type == 4 )
/* 効果いまひとつ */
){
m = (int)rand() % 2;

if(m == 0) add = 1;
if(m == 1) add = 1/2;
}else{
m = (int)rand() % 10;

if(m == 0){
add = 2;
}else{
if(m == 15){
add = 1/2;
}else{
add = 1;
}
}
}
}

return add;
}

int damage(int i, int a[2][3], struct monster monster[], int b[2])
{
int add;
double damage;
int s, t, u;

s = (i+1) % 2;
t = i % 2;
u = (int)rand() % 32;

add = compatibility(a, s, t, monster);
・
・
・
｝

printf("a2\n");を関数内に用いて、実際にcompatibility関数を読み込んでいるのか確かめたのですが、コンパイラーにそう表示されることなく、関数を読み込んでいないのではと判断したしだいです。
長文すいません、お手数ですがどこをどのように変更すればいいのか、教えていただけるとありがたいです。
よろしくお願いします。
ちなみにa[2][3]はint型の配列、monsterはint型のtypeやtrickeffectを収入したもののつもりです。

回答No.1

chie65536（@chie65535） · ベストアンサー

＞printf("a2\n");を関数内に用いて、実際にcompatibility関数を読み込んでいるのか確かめたのですが、コンパイラーにそう表示されることなく、関数を読み込んでいないのではと判断したしだいです。

であれば「damage()関数そのものが呼び出されて無い」と思います。

int damage(int i, int a[2][3], struct monster monster[], int b[2])
{
int add;
double damage;
int s, t, u;

の直後に

printf("d1\n");

と入れて「damage()関数が本当に呼び出されているのかどうか？」を確かめましょう。

「d1」と言う表示が行なわれず、damage()関数が呼ばれていないようなら、更に「damage()関数を呼び出している上位の関数が、ちゃんと呼び出されているかどうか？」を、同じような方法で確かめ、更にその上位関数も同じように確かめましょう。

そうすれば「この関数まではきちんと呼び出されているが、ここに来た時点で、この関数が呼ばれてない」ってのが確認できる筈です。

お礼

ご指摘、ありがとうございます！
damage関数の中身を変えたところ、ちゃんと機能いたしました！

回答No.3

chie65536（@chie65535）

追記。

蛇足ですが、compatibilityとか、damageとか「一般的な名詞のみの関数名」は、使用しない方が良いです。

もし「同名の関数が、標準ライブラリに存在したり、拡張ライブラリに存在したりする」と自作関数ではなく、既存の同名のライブラリが呼ばれてしまい、意図した結果にならなくなる可能性があります。

こういう「独自の関数」を作る場合は「Player_Damage()」とか「Enemy_Damage()」とか「絶対に他では使われてないだろう関数名」にしましょう。

お礼

ありがとうございます！
以後、プログラミングの練習をする際は気をつけるようにします＾＾

回答No.2

8shi8

質問内容がちょっと間違っていると思いますので確認します。
コンパイラーにprintf("a2\n");の結果が表示されることはありません。
コンパイラーはCのコードをコンピュータが実行可能な形に変更してくれるだけです。
なので、コンパイルした結果として実行時にprintf("a2\n");の結果である"a2"がコンソールに表示されないという事でまちがいないでしょうか？
だとするとdamage関数がcompatibility関数を呼び出していて、ここに書かれている部分コードの通りだとするとdamage関数が呼び出されていないという事になります。
チェックをするにはdamage関数のs = (i+1) % 2;の行の前に同様にprintf文を追加してみてはいかがでしょうか？
プログラムの実行結果で、追加したprintfの出力結果がコンソールに表示されないようであればdamage関数を呼び出している箇所をチェックしてみてください。

お礼

ご指摘、ありがとうございます！
おっしゃるとおり、damage関数内を見直したら無事動きました！