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

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

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

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

プログラミング言語

プログラミング言語はパソコン上で実行することができるソースコードを記述する為に扱う言語の総称です。

Q&A

解決済

4回答

2172閲覧

モンテカルロ法における円周率計算について

tomo.223

総合スコア4

C

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

プログラミング言語

プログラミング言語はパソコン上で実行することができるソースコードを記述する為に扱う言語の総称です。

0グッド

0クリップ

投稿2020/05/12 01:25

前提・実現したいこと

C言語において、モンテカルロ法での円周率計算をしたいのですが、プログラムを実行すると動作停止を起こしてしまいます。
■■な機能を実装中に以下のエラーメッセージが発生しました。

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

~.exeは動作を停止しました。

C言語

#include <stdio.h> #include <stdlib.h> #include <math.h> #include <time.h> #define n 1000000 int main(){ FILE *fp_square, *fp_sector; int i, count = 0; double x[n], y[n], d, pi; fp_square = fopen("xy.txt", "w"); fp_sector = fopen("xy_in.txt", "w"); srand((unsigned int)time(NULL)); for(i = 0; i < n; ++i){ x[i] = (double)rand()/RAND_MAX; y[i] = (double)rand()/RAND_MAX; d = sqrt(pow(x[i], 2) + pow(y[i], 2)); fprintf(fp_square, "%lf\t%lf\n", x[i], y[i]); if(d <= 1){ count += 1; fprintf(fp_sector, "%lf\t%lf\n", x[i], y[i]); } } pi = 4.0*count/n; printf("The number of ... is %d\n", count); printf("pi (approx value): %lf\n", pi); fclose(fp_square); fclose(fp_sector); return 0; }

試したこと

nの数を少なくすると問題なく実行できるのですが、1000000などの大きい数字にすると停止します。

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

ここにより詳細な情報を記載してください。

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

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

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

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

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

stdio

2020/05/12 01:47

すみません、少し質問なのですが、止まるとは実行中にですか? それとも開始直後ですか? もし開始直後に止まっているようなら、このプログラムの場合x, yは配列にする必要がないと思います。
tomo.223

2020/05/12 01:50 編集

失礼しました。開始直後です。
guest

回答4

0

そのデカい配列、要らなくない?

投稿2020/05/12 02:05

episteme

総合スコア16614

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

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

0

ベストアンサー

double x[n], y[n], d, pi;double x, y, d, pi;に変えて、
プログラム中のx[i] y[i]x y に変えれば良いです。

投稿2020/05/12 02:55

otn

総合スコア84505

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

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

0

x[]とy[]をmain()の外で定義してみてください。
外部変数にするということです。

スタックをオーバーしているかも知れません。

投稿2020/05/12 01:48

nob.

総合スコア711

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

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

0

ローカルで用意した配列は(一般的な処理系の場合)スタックに取られますが、多くの場合スタックの容量は数MB程度です。doubleが200万個だとそれだけで16MBになるので、スタックでは容量が足りません。

  • staticとして宣言する、あるいは関数の外側で宣言して静的変数とする
  • mallocで動的に確保する

など、別なやり方で変数領域を用意する必要があります。

投稿2020/05/12 01:47

maisumakun

総合スコア145183

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

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

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.48%

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

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

質問する

関連した質問