質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.48%
C

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

Q&A

2回答

1055閲覧

モジュールを用いて 1!から53! までの計算をしたい

nayuta314

総合スコア12

C

C言語は、1972年にAT&Tベル研究所の、デニス・リッチーが主体となって作成したプログラミング言語です。 B言語の後継言語として開発されたことからC言語と命名。そのため、表記法などはB言語やALGOLに近いとされています。 Cの拡張版であるC++言語とともに、現在世界中でもっとも普及されているプログラミング言語です。

0グッド

0クリップ

投稿2017/05/17 07:55

###前提・実現したいこと
1!〜53!を計算してそれらを右揃えで出力するプログラムをC言語で作成したいのですが、無限ループのエラーが発生してしまいます。
原因がわからないので、どうしてエラーが発生するのかと、その解決方法を教えていただきたいです。
エラーメッセージは以下のとおりです。
###発生している問題・エラーメッセージ

エラーメッセージ 000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000004274883284060025564298013753389399649690343788366813724672000000000000 000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000004274883284060025564298013753389399649690343788366813724672000000000000

###該当のソースコード

c

1#include<stdio.h> 2 3void set(int process[],int size,int init_value); 4void kaijou(int process[],int); 5void print(int process[],int); 6 7int main(void){ 8 int process[20],k; 9 set(process,70,1); 10 kaijou(process,53); 11 print(process,53); 12} 13 14void set(int process[],int size,int init_value){ 15 int i; 16 17 for(i=1;i < size;i++){ 18 process[i]=0; 19 } 20 process[0]=init_value; 21} 22 23void kaijou(int process[],int n){ 24 int i,s,a,count,k; 25 count=0; 26 27 for(i=1;i<=n;i++){ 28 a=0; 29 for(s=0;s<=count;s++){ 30 process[s]=process[s]*i+a; 31 a=0; 32 if((process[s]/10000)!=0){ 33 a = process[s]/10000; 34 process[s]=process[s]%10000; 35 } 36 } 37 if(a!=0){ 38 process[s]=a; 39 count++; 40 } 41 } 42} 43 44void print(int process[], int n){ 45 int i,s,count; 46 47 for(i=1;i<=n;i++) { 48 for(s=0;s<=count;s++){ 49 if(s==count){ 50 printf("%d!=%*d",i,70-4*count,process[count]); 51 }else{ 52 for(s=count-1;s>=0;s--) 53 printf("%04d",process[s]); 54 } 55 printf("\n"); 56 } 57 } 58} 59

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答2

0

指摘のみ。

C

1int process[20],k; 2set(process,70,1);

だと

C

1void set(int process[],int size,int init_value){ 2 int i; 3 4 for(i=1;i < size;i++){ 5 process[i]=0;

process配列の範囲外をアクセスしてしまいます。(そのあと何が起きても不思議ではない)

投稿2017/05/17 08:09

can110

総合スコア38262

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

0

can110さんの指摘する点もバグですが…

バグはprint関数の中に2点あります。1つ目は初期化されていないローカル変数countを利用していること、2つ目はfor文で、同じsという変数をループカウンタに利用していることです。
修正方法は自分で考えてみてください(簡単なのでトレーニングと思ってやってみてください、ヒントは書きました)。

そもそも、iで回すfor文は必要?
なぜ二重ループ?

追記: 我流で書いてみた。電車の中からスマホでぱぱっと書いただけなのでテストしてませんorz

C

1#include <stdio.h> 2#define countof(arr) (sizeof(arr)/sizeof(arr[0])) 3#define BASE 10000 4#define LOGBASE 4 5 6void set(int* digits, int size, int value); 7void mul(int* digits, int size, int n); 8void fact(int* digits, int size, int n); 9void printdigits(int* digits, int size); 10 11int main(void) 12{ 13 int digits[20]; 14 int n = 53; 15 fact(digits, countof(digits), n); 16 printf("%d!=", n); 17 printdigits(digits, countof(digits)); 18 printf("\n"); 19} 20 21void set(int* digits, int size, int value) 22{ 23 int i; 24 process[0] = value; 25 for(i=1; i < size; i++) process[i] = 0; 26} 27 28void mul(int* digits, int size, int n) 29{ 30 int i, m = 0; 31 for(i = 0; i < size; i++){ 32 m += digits[i] * n; 33 digits[i] = m % BASE; 34 m /= BASE; 35 } 36} 37 38void fact(int* digits, int n) 39{ 40 int i; 41 set(digits, size, 1); 42 for(i = 1; i <= n; i++) mul(digits, size, i); 43} 44 45void print(int* digits, int size) 46{ 47 int i; 48 for(i = size - 1; i >= 0; i--) printf("%0*d", LOGBASE, digits[i]); 49} 50 51

投稿2017/05/17 12:07

編集2017/05/18 10:57
majiponi

総合スコア1720

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

nayuta314

2017/05/18 07:52 編集

print関数の部分を言われたとおりに直してみたのですが、うまくいかなかったです。下記が直した部分です。 void print(int process[], int n){ int s,t,count; count=0; for(s=0;s<=count;s++){ count=count+1; if(s==count){ printf("%2d!=%*d",count,70-4*count,process[count]); }else{ for(t=count-1;t>=0;t--) printf("%04d",process[t]); } printf("\n"); } }
majiponi

2017/05/18 09:55

そりゃforループの中でcountを弄っているもの、上手くいきませんよ。forの第2文で、sizeがcount以下と言いながら、countを毎回1づつ増やしているので、sizeとcountは常に一致してしまいます。 countは、何を意図した変数ですか?
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだベストアンサーが選ばれていません

会員登録して回答してみよう

アカウントをお持ちの方は

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問