C言語の初心者です。教えていただきたいです

Question

#include <stdio.h>

int main(void)
{
  int a[10000];
  int n, i, j, carry, tmp;

  a[0] = 1;
  n = 1;

  for (i=2; i<=1000; i++) {
    carry = 0;
    for (j=0; j<n; j++) {
      a[j] = a[j] * i + carry;
      carry = a[j] / a;
      a[j] = a[j] % 10000;

    }
    if (carry != 0) {
      a[n] = carry;
      n++;
    }
  }

  for (j=n-1; j>=0; j--) {
    printf("%04d", a[j]);
  }
  printf("\n");

  return 0;
}

Comments

[K_3578]

ソースコードはMarkDownのcode機能で提示してください。
参考質問:https://teratail.com/questions/238564

[episteme]

正しいコタエは得られるんですよね?

[rubato6809]

掛け算の演算子「*」が一つくらいはあってもよさそうなのに、無い。
穴埋め問題を答えなきゃいけなくなったので、我々に答えてもらおう、ってことですか？

[K_3578]

>穴埋め問題
なんか以前の質問もこの後どうすればみたいな書き方だからそうかもしれないっすね

[BeatStar]

課題丸投げするならいっそプログラミングやめましょう。

丸投げは言い換えると『自分は努力せずに他人に労力を強いること』です。

つまり『自分本位、自分勝手』です。

まずは自分なりにやってみましょう。

Answer 1

carry がどこでも増えてないので、
それなりに増やす必要があります。

Comments

[PingHermit]

３回目の修正点で、やっと本気出したのか !
正しいかどうかまでは見てないが(^^;

Answer 2

そうそう、内側の for ループの中を何行か埋めれば計算するようになるんですよ。でも、そのままだと carry = a[j] / a; の行がねえ。。。

これは「（10進数の4桁分に）掛け算した結果を、配列 a の（先頭アドレス）で割る」とでもいった、意味不明なコードになっています。それではコンパイラは許してくれず、コンパイル・エラーになっただろうと思います。

次の行 a[j] = a[j] % 10000; は間違ってないだけに惜しい。そこ、割る数さえ正しければ結果を出せたのに・・・というか、直し方がわかったので退会しちゃったのかもしれませんね。穴埋め問題だったからといって、逃げなくてもよかったんですよ。ここまで考えたんだから。

でも、そのコードだと tmp 変数が使われていません。
出題者は tmp = a[j] * i + carry; としてほしかったと思います。こうしたほうが良いという方に私も一票。