構造体とポインタが絡んだC言語の関数について

Question

前提・実現したいこと

C言語に関する質問です。

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

void calc(int n, int d[], ANS *ans)関数の中で、変数ansに対して、

式には構造体または共用体型を使用する必要がありますが、型 "ANS *" が使用されています

というエラーが出ます。

該当のソースコード

#include <stdio.h>
#include <stdlib.h>
#include <time.h>

typedef struct {
    int min, max;
    double ave;
} ANS;

/*関数プロトタイプ宣言*/
int data[1000];
void calc(int n, int d[], ANS *ans);

/*メイン関数*/
int main(void) {
    int n = 999;
    int num;
    ANS ans;
    srand(time(NULL));
    num = gene(n, data);
    calc(n, data, &ans);
    printf("最小=%5d 最大=%5d 平均=%.2f¥n", ans.min, ans.max, ans.ave);
    return 0;
}

/*関数*/
void calc(int n, int d[], ANS *ans) {
    ans.max = 0;
    ans.min = 10000;
    int sum = 0;
    for (int i = 0; i < sizeof(d); i++) {
        if (ans.max > d[i]) {
            ans.max = d[i];
        }
        if (ans.min < d[i]) {
            ans.min = d[i];
        }
        sum += d[i];
    }
    ans.ave = (double)sum / 999;
}

###試したこと
void calc関数の一行目に

struct ANS *ans;

を追記するなどしました。

Answer 1

ANS *ans に対して，
ans.max とかいう記述がおかしい，という話．
ans->max とすべきところ．文法の問題なので復習されたい．

Answer 2

環境はどうなっているのでしょうか? コンパイラとか。

Visual studio 2019 でコンパイルすると、

error C2231: '.max': 左のオペランドが 'struct' へのポインターです。'->' を使用してください。

calc() の引数では、ANS *ansとなっているのに、ans.max としているのは間違いですね。
ans->max または、 (*ans).max とすべきですね。 (同様の問題、複数個所)

Comments

[pepperleaf]

最後に"(同様の箇所 複数)" と書いたつもりが消えてる。どうした? 修正画面には現れるのに。
(と書くと現れるか?)

Answer 3

calc内でansの型はANS*なので要素にアクセスするならans->maxとか(*ans).maxとしなければいけません。

ちなみに質問とは違いますが、sizeof(d)はいくらでしょう？

Comments

[pepperleaf]

残念 "*ans.max"は NG "(*ans).max" しないとダメですね。
"sizeof(d)" は気づかなかった。 ポインタサイズだったと思うけど、、と思って、修正、コンパイルと思ったが、 不足関数(gene)があるので、コンパイルできない。

[neconekocat]

*の方は書いた後に思い出して修正しました。

[kk_22]

すみません、院試の過去問で久々にC言語に触れてエラー処理できなくて詰まっていました。
Pythonのlen(d)みたいに、sizeof(d)で配列の長さを返すものだと思っていました。
大変助かりました。ありがとうございました。

[neconekocat]

補足するとsgzeofが配列の長さを返すのも正しい認識です。
引数として配列を渡した場合は単なるポインタとして扱われるため、要素数を取得できなくなるというものです。