- ベストアンサー
malloc関数の使い方について（初心者）
膨大なデータ数を扱うためにmalloc関数を用いて配列にデータを格納し、それを表示するプログラムを考えました。コンパイルはできるのですが、実行するとエラーが起こります。freeの開放の仕方が間違っているのでしょうか？よろしくお願いします。 /*ソース*/ #include <stdio.h> #include <stdlib.h> #define N 130000 int main(void) { double *p; int i; p = malloc(N); if(!p){ printf("割り当てエラー"); exit(1); } for(i=0;i<N;i++){ p[i]=i; printf("%f\n",p[i]); } free(p); }
- yuji221
- お礼率90% (9/10)
- C・C++・C#
- 回答数2
- ありがとう数3
- みんなの回答 （2）
- 専門家の回答
質問者が選んだベストアンサー
mallocに渡すのは、要素数ではなくサイズです。 なので、 p = malloc(N*sizeof(double)); としてみて下さい。 また、pの型がfloatやlong doubleやintなどに変わる可能性も考慮すると、 p = malloc(N*sizeof(*p)); とした方が良いと思います。 このsizeofは型情報をみているだけなので、 例えpがNULLでも問題は起きません。
その他の回答 (1)
- flowergoo
- ベストアンサー率38% (13/34)
取得するバイト数が足りないのでは malloc()の引数(N)は、確かバイ単位の指定だったと思います。 従って引数で指定するバイト数もdoubleの個数ではなく、 確保する変数の数 × doubleのバイト数 だけ必要になり、これが足りないためにエラーが出ている可能性があります。
お礼
その通りでした。親切で適切なご解答ありがとうございました。
関連するQ&A
- 関数とポインタ
整数Mを素因数分解して、その素因数を*primeに入れておきたいんですが、mainの方で呼び出すと違う値になってしまいます。。 同じ値になるようにしたいのですが…どなたかお願いします(>_<) #include<stdio.h> #include<stdlib.h> #define MAX 256 int prime_fac(int M, int* prime) { int i, n; i = 1; n = 2; while(M != 1){ while(M % n == 0){ *(prime+i) = n; printf("%d\n", *(prime+i)); M = M / n; i++; } n++; } return i; /*素因数の個数(重複あり)*/ } int main(){ int M; int *prime; prime = (int *)malloc(sizeof(int) * MAX); printf("M: "); scanf("%d", &M); prime_fac(M, prime); printf("%d\n", *prime); printf("%d\n", *prime+1); free(prime); return 0; }
- ベストアンサー
- C・C++・C#
- malloc メモリリークについて
#include<stdio.h> #include<stdlib.h> void alloconly(void) { const int sz=10; static int n=0; void *p; p=(void*)malloc(sz); if(p==NULL){ printf("動的確保不可 %d\n",n+1); printf("確保メモリサイズ %d byte\n",sz*n); exit(-1); } ++n; } int main(void) { for(;;){ alloconly(); } }と言うプログラムがある本に載っており、回答として 動的確保不可 *******(数字） 確保メモリサイズ *********byte(数字）となっていますが私の環境（Reshat Linux)では 強制終了します で終了してしまいます。原因が判らず困っています。ご解答願います。
- ベストアンサー
- C・C++・C#
- mallocについて
vine linux 3.0上でgccを使い下記のようなプログラム(読みにくいプログラムで申し訳ありません)を作成してみました。無事コンパイルも通りメモリも確保出来ました。 しかし、このプログラムを２つ起動しそれぞれでメモリを確保するとおかしな事が起きるのです。どういう症状かと言うとプログラム1もプログラム2も同じ先頭アドレスを確保しているのです。 例) プログラム1 keep memory size : 11 address : 0x8049700 プログラム2 keep memory size : 11 address : 0x8049700 なぜこのような事が起こるのでしょうか? ご教授宜しくお願いします。 #include <stdio.h> #include <stdlib.h> main(int argc, char *argv[]) { int size; unsigned int *p; char buf; size = atoi(argv[1]); printf("keep memory size : %d\n", size); p = (unsigned int *)malloc(size); printf("address : %p\n", p); scanf("%c", &buf); free(p); }
- ベストアンサー
- C・C++・C#
- C言語のポインタ
あまり意識せずにポインタを使っているせいか，次のプログラムではまってしまいました． #include<stdio.h> #include<stdlib.h> int main(void) { int *p, q; p = (int *)malloc(sizeof(int)); q = (int *)malloc(sizeof(int)); *p = 2; printf("%d\n", *p); return 0; } コンパイルエラーで実行ファイルが出力されません． このプログラムで変数qはなぜポインタじゃないのでしょうか？ 次にtypedefでptr_intという型を定義したプログラムは， 上のようなエラーが出力されず，期待とおりの結果になりました． #include<stdio.h> #include<stdlib.h> typedef int* ptr_int; int main(void) { ptr_int p, q; p = (int *)malloc(sizeof(int)); q = (int *)malloc(sizeof(int)); *p = 2; *q = 3; printf("%d\n", *p); printf("%d\n", *q); return 0; } typedefすることでなぜエラーを回避することができるのでしょうか？ よろしくおねがいします．
- ベストアンサー
- C・C++・C#
- 配列の動的確保を関数化
main()内でいくつかの配列を動的確保するとごちゃごちゃするので、 関数化してみました。しかし、下のプログラムではうまい事いきませ んでした。 #include<stdio.h> #include<stdlib.h> void array1d(int *box,int n) { int i; box=(int *)malloc(n * sizeof(int)); for(i=0; i<n; i++) { box[i]=0; } } main() { int *pol,i,n; printf("N pol\nN = "); scanf("%d",&n); array1d(pol,n); for(i=0; i<n; i++) { printf("[%d]=%d\n",i,pol[i]); } } 実行結果は [0]=-14646387 [1]=-1819410433 [2]=-224 array1d内では0を格納していますが、このようになりました。 どのようにすれば ０になるでしょうか？ 回答よろしくお願いします。
- ベストアンサー
- C・C++・C#
- ポインタ
#include<stdio.h> #include<stdlib.h> struct number{ int value; struct number *next; }; int main(void){ FILE *f; struct number *p; int n=0,i; f=fopen("number.txt","r"); p=malloc(sizeof(struct number)); if(p==NULL){ printf("error"); return 1; } while(fscanf(f,"%d",&(p+n)->value)!=EOF){ (p+n)->next=malloc(sizeof(struct number)); if(p+n==NULL){ printf("error"); return 1; } p+n=(p+n)->next; n++; } for(i=0;i<n;i++) printf("%d\n",(p+i)->value); free(p); fclose(f); return 0; } のp+n=(p+n)->next;の部分でバグが出てしまうのですがアロー演算子と通常の演算子を同時に使うことができないなどのような規則があるのでしょうか、それとも俺がバカなだけなのでしょうか？ご指摘お願いします。
- 締切済み
- C・C++・C#
- mallocで引数に変数を入れると警告がでます
#include<stdio.h> int main(void){ int n,i; int* array; puts("n=?"); scanf("%d",&n); array = (int*)malloc(sizeof(int) * n); for(i=0;i<n;i++) *(array+i) = i; for(i=0;i<n;i++) printf("%d ",*(array+i)); puts(""); return 0; } C言語で、上述のプログラムをコンパイルしたところ、 9: warning: incompatible implicit declaration of built-in function ‘malloc’ と警告がでます。 ./a.outで普通に実行できるのですが、 警告がでるということは、もっと正しいやり方があるということでしょうか？
- ベストアンサー
- C・C++・C#
- C言語 動的なメモリの確保 コンパイル時エラーも警告もないのに実行できない
配列A[3]を{2, 4, 6}と初期化させ、malloc関数を使いメモリを確保しそこへ先程の配列Aの要素を記憶させ、ポインタ*Nを使い確保したメモリの要素を表示するプログラムです。 ********************************************* #include <stdio.h> #include <stdlib.h> int main(void) { int A[3] = {2, 4, 6}; int i; int *N; N = (int *) malloc (3); for(i = 0; i < 3; i++){ N[i]= A[i]; } for(i = 0; i < 3; i++){ printf("%d", N[i]); } free(N); return 0; } ********************************************* VCを使いF10のデバッグテストでは正常に動くのですが、コマンドラインからではエラー報告画面がでます。なぜなのでしょうか？
- ベストアンサー
- C・C++・C#
- VC++ 再帰関数
■キーボードより読み込んだ文字列の長さを求めるプログラムを再帰関数を使って作る #include <stdlib.h> int unk_r(char*p) { if(*p == 0x00) return(0); else return(1+unk_r(p+1)); } int main() { char *i; i = (char *)malloc(20); scanf("%s",i); printf("%d\n",unk_r(i)); } このプログラムについて、行、単語ひとつひとつ細かいところまで具体的に解説お願いします。どういう働きをしているのか等。 例えば #include <stdlib.h> はmallocを使うのに必要などなど。 よろしくお願いします。
- ベストアンサー
- C・C++・C#
- mallocがうまく動かない
コマンドライン引数で指定された文字列を逆順に返すプログラムを作るため 下記のようなプログラムを組みました。 ところが変数strの大きさがargv[1]より大きくなってしまいます。 どうすればよいのでしょうか。 #include <stdio.h> #include <stdlib.h> char *mstrrev (char *s); int main(int argc, char *argv[]){ char *str; str = mstrrev(*(argv+1)); printf("%s",str); free(str); } char *mstrrev (char *s){ int length,i; char *str; for(length=0;*(s+length)!='\0';length++); str = (char *)malloc(sizeof(char)*length); for(i=0;i<length;i++){ str[i] = s[length-1-i]; } return str; }
- ベストアンサー
- C・C++・C#
お礼
素早いご解答懇切丁寧にありがとうございました。とても参考になりました。