1からNまでの二乗和を求める

Question

１からNまでの二乗和を求めるプログラムを作れ、という課題をやっているのですが、完成したと思いコンパイルし実行してみると、=0 という答えが返ってきました。

コンパイルした時、警告：関数は値を返すべき(関数 tasu(int))と表示されていました。
以下がそのソースコードです。
どこが違っているかと、その解説も出来たらお願いします！

#include<stdio.h>

int jijo(int a)
{
return a*a;
}

int tasu(int N)
{
int s=0;
for(int i=0;i<=N;i++)
{
s=s+jijo(i);
return s;
}
}

int main(void)
{
printf("１からＮまでの二乗和を求める。\n");

int n=0;
printf("N=");
scanf("%d",&n);

printf("1^2+2^2+……+N^2=%d",tasu(n));

}

---

Answer 1

int tasu(int N)
{
    int s=0;
    for(int i=0;i<=N;i++)
    {
        s=s+jijo(i);
        // return s;//ここで関数を抜けてしまうので
    }
    return s; //ここに移動する。
}

ちなみに今回のように
「動作がなんかおかしい」のを何とかする（デバッグ）方法で、
原始的な方法に、「printfを各所に仕込む」というのがあります。

たとえば今回の場合、

//(他にも怪しいと思うところにprintf()を仕込んでいる)
int tasu(int N)
{
    int s=0;
    for(int i=0;i<=N;i++)
    {
        s=s+jijo(i);
        printf("足した。\n");
        return s;
    }
}

とか書いておけば
「N=2以上で実行したのに「足した」が一回しかでねぇぞ」
「あれforループが一回しか回ってないな」
と、起きている問題がより具体的になります。

Comments

[ISARIA]

ありがとうございます！
これからは、バンバンprintf使って間違ってるとこ探していきます！

[ozwk]

まあ普通はデバッガ使うんですけどね。
簡単に言うとプログラムの処理を追うツールです。
「この行に来たら一時停止する」とかできます。

Answer 2

tasu関数の中のreturnの場所が間違っています。これではループせず、すぐにmain関数に帰っていってしまいます。ですのでreturnをfor文の外に出してあげましょう。

Comments

[ISARIA]

ありがとうございます！

Answer 3

#include<stdio.h>

int jijo(int a) {
  return a * a;
}

int tasu(int n) {
  int s = 0;
  for (int i = 0; i <= n; i++) {
      s += jijo(i);
  }
  return s;
}

int main(void) {
  printf("1 から N までの二乗和を求める。\n");

  int n = 0;
  printf("N=");
  scanf("%d",&n);

  // プログラムでループ
  printf("%d\n", tasu(n));
  // 数列の和の公式を利用
  printf("%d\n", (n + 1) * n * (2 * n + 1) / 6);
}

プログラムでループさせて足し算して行く方法だけでなく、
数列の和の公式で求めた値も表示させています。