ポインタを利用して配列にアクセスする関数

Question

ポインタがまだあまり理解できていないので、かなり酷いコードになっています。

前提・実現したいこと

引数としてdouble型の配列名を与えると，配列のすべての要素の和を返す関数を定義し，main関数から呼び出して動作を確認するプログラムを作成したい。

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

double *pの部分に"variable 'p' set but not used [-Wunused-but-set-variable]"という警告が表示されてしまう。

該当のソースコード

#include <stdio.h>

#define N 1000

double suma(double *p, double size);

int
main(void)
{
  double s;
  double a[N] = {1, 2, 3};
  double *p;
  p = a;

  a[N - 1] = -10;
  s = suma(a, sizeof a / sizeof a[0]);
  printf("%lf\n", s);

  return 0;
}

double
suma(double *p, double size)
{
  int i;
  double s;
  for (i = 0; i < N; i++) {
    s += *p++;
  }
  return s;
}

試したこと

-4.000000と表示される。

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

要素の和は-4であっていると思うのですが、そもそも引数としてdouble型の配列を与える仕方がこれであっているのかわからない。

Comments

[LouiS0616]

main関数のポインタ変数pは必要ですか？

Answer 1

その警告は、変数pに値が代入されてるけど、それが使われていない、というものです。
バグの可能性あるけど、それでいいの？と文字通り警告を出してくれています。

その値を使う必要がなければ、余計なお世話でその警告を無視してればいいです

＃まあ、警告も出さないようにする、というのは理想的ですんで、この場合は変数ｐの定義と代入文を削除すればその警告はなくなりますけど、それもよしあしですね

---

んで、明らかなバグを指摘しときます。

s += *p++;

ｓが初期化されてませんね。
C言語の場合、ローカル変数はでたらめな値を持ちますんで初期化しなければでたらめな値の結果となります
こっちの方も警告が出てると思いますがどうでしょう

Answer 2

配列はポインタで扱えますので, 合っていると思います.
ですが, suma はなぜ size を使っていないのでしょう. N を使っていては, suma の動作が関数外の定義に影響されることになり, 分かり難いバグの原因になる可能性があります.
また, 呼び出し側での sizeof a / sizeof a[0] が幾つなのかは確認されましたでしょうか.

Answer 3

書き換えてみたプログラムと、その実行結果をしめします。
コードの差の意味を考えてみてください。
dd.c

#include <stdio.h>

#define N 1000

double suma(double *p, int size);

int main(void) {
  double a[N] = {1, 2, 3};
  a[N - 1] = -10;

  double sum = suma(a, N);
  printf("%lf\n", sum);

  return 0;
}

double suma(double *p, int size) {
  double sum = 0.0;
  for (int i = 0; i < size; i++) {
    sum += *p++;
  }
  return sum;
}

実行例