C言語の領域確保について

質問

C言語を始めたばかりのものです。　配列を計算させようと下のようなプログラムを作り、一応動作はするのですが、配列の領域確保がいまいちわかりません。配列はcsvから値をとっています。確保する領域は、代入する値の数でいいので、buff[8×64]、とかarv[4×32]でいいと思い、それより少し大きな値くらいで実行すると失敗してしまいます。
　自分の理解の仕方が間違っているとは思っているのですが、(char *)malloc(M*N*sizeof(char))でやっても失敗しました。どうすればよいかお教え願います。ちなみに、csvの値は0.000852354710804709のように桁が半端ないです。
　またfgetsのライン数の値も正しくするにはどうすればよいのでしょうか。できたらmallocを使えたらよいと思っています。

#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <math.h>
#define N 8　//8行
#define M 64　//64列

int main()
{
char *fname = "test.csv";//8行64列
char *ffname = "tk.csv";//4行32列
FILE *IN1, *IN2, *OUT1, *OUT2;
char buff[100000];//←問題の部分
char *pbuff;
char arv[400];//←問題の部分
char *parv;
double a[N][M+30], b[N][M], c[N][M], w[4][32];
int i, j, x;
double m, n;
int count=0;

if((IN1 = fopen(fname, "r")) == NULL){
printf("can't open the %s", fname);
return 0;
}
if((IN2 = fopen(ffname, "r")) == NULL){
printf("can't open the %s", ffname);
return 0;
}

i=j=0;//配列の取得
while(fgets(buff, 100000, IN1) != NULL){
pbuff = buff;
while((pbuff = strtok(pbuff, ",\n")) != NULL){
a[i][j++] = strtod(pbuff, NULL);
pbuff = NULL;
}
i++;
j=0;
}

i=j=0;//配列の取得
while(fgets(arv, 400, IN2) != NULL){
parv = arv;
while((parv = strtok(parv, ",\n")) != NULL){
w[i][j++] = strtod(parv, NULL);
parv = NULL;
}
i++;
j=0;
}
　　　　　　　　　　　　：
　　　　　　　　　　　　：
　　　　　　　　　　　　：

このプログラムでも計算はできるのでいいのですが、最終的には500行2048列の計算をしないといけないので、いっそう数字が膨大となるために質問させていただきました。Visual Stdio 2010 c/c++ 使ってます。

回答No.2

chie65535 · ベストアンサー

buffもarvも「ファイルの中で、最も長い行の文字数＋α」で良い。

例えば、最長の行が「先頭から改行まで、トータルで5329文字、改行も入れると5330文字」と言うのであれば、＋αして、5400バイトあれば問題無い。

＞最終的には500行2048列の計算をしないといけないので

2048列で、１つの列にある「数値」は「最大桁数」が決まっている筈。

例えば、最大桁数が50桁なら、１行の最大文字数は「50×2048＋2047＋1＝104448」と計算できる（「2047」は「,」の分、「1」は改行の分）

安全の為に「＋α」して、104500バイトのbuffを用意すれば良い。

arvの最大必要バイト数も、同じように計算が可能。

＞できたらmallocを使えたらよいと思っています。

char buff[100000];//←問題の部分

を

char *buff;
buff=malloc(100000);
if (buff == NULL) { abort(); return; }

に変えるだけだ。

もし、計算で求めた「１行の最大バイト数＋α」が「104500」になったのなら、100000を104500に変えれば良い。

お礼

なるほど，桁数も考慮して確保する領域を決めるのですね．csvに関する疑問が多く困っていたのですが，これでかなり解消されました．丁寧に説明していただきありがとうございます！

回答No.1

Tacosan

とりあえず, プログラムには動作を説明するコメントを入れてください. もちろん自分だけで使うプログラムならコメントを書こうと書くまいと私の知ったことではありませんが, 他人に見せるプログラムにはある程度の処理のまとまりごとに「そこで何をしているのか (あるいは何をしたいのか)」をコメントで入れてください. 特にここでは他人に「見てもらう」プログラムなのだから, そのくらいの手間は惜しまないでください.

あと, 「失敗する」というのが何を意味するのかが分かりません. あなたのいう「失敗」とは, 具体的にはどういうことなのですか?

「fgetsのライン数の値も正しくするにはどうすればよいのでしょうか」は意味不明. 「fgets のライン数の値」ってなんですか?

お礼

説明不足で失礼しました．失敗に関しては，VisualStdioごと停止していました．またfgetsは1ライン(行)で文字列を受け取るバイト数のことですが，chie65535さんのおかげで解決しました．回答ありがとうございます．