0

我有这种读取文件的方法。一个更具体的矩阵，其中前两个数字是行和列。但是，当我尝试使用 malloc 分配内存并使用行和列时，应用程序崩溃。

我正在使用的代码是这个：

#include stdio.h
#include stdlib.h
#include stdlib.h

float * readFile(char* nombre, int*renglones, int*columnas){
FILE *fp;
fp=fopen(nombre,"r");

fscanf(fp,"%d",&renglones);

printf("el numero de renglones es %d\n",renglones);

fscanf(fp,"%d",&columnas);

printf("number of rows %d\n",columnas);
float value;

fscanf(fp,"%f",&value);
printf("el numero de columnas es %f\n",value);
fscanf(fp,"%f",&value);
printf("el numero de columnas es %f\n",value);
printf("no llegue a malloc");
float * res = malloc(*renglones**columnas*sizeof(float)); //memory reservation and the line that breaks the program 
printf("after malloc");



fclose(fp);
return 0;

}

我知道包含在 <> 之间

最终代码是包含以下内容的代码：stdio.h 和 stdlib.h

  float * readFile(char* nombre, int*renglones, int*columnas){
FILE *fp;
fp=fopen(nombre,"r");
fscanf(fp,"%d",renglones);
printf("el numero de renglones es %d\n",renglones);
fscanf(fp,"%d",columnas);
printf("el numero de columnas es %d\n",columnas);
float value;
float * res = (float*)malloc(*renglones**columnas*sizeof(float)); //Reserva de memoria
printf("llegue a malloc\n");
int i;
for(i=0;i<*renglones**columnas;i++){
        fscanf(fp, "%f",&value);
        res[i]=value;
        printf("dato %f\n",value);
}
printf("%d",i);
fclose(fp);
return res;

}

谢谢！

4

1 回答 1

3

你的问题是你这样做：

...
fscanf(fp,"%d",&renglones);
...
fscanf(fp,"%d",&columnas);
...

结果，这 2 个数字（您从文件中读取的）成为指针renglones，columnas您可以进一步取消引用以计算malloc. 例如，如果您读取数字16和32，则renglones指向0x00000010并columnas指向0x00000020。但是，这些内存单元显然包含随机垃圾（从您的任务的角度来看），即它们可能包含任意巨大的数字，当它们相乘时会得到更大的数字，并且malloc根本无法分配那么多内存，最终导致碰撞。

相反，它应该是：

...
fscanf(fp,"%d",renglones);
...
fscanf(fp,"%d",columnas);
...

因为两者renglones和columnas已经是指针。这样，您将真正填充两个int变量，renglones并columnas获得预期的行为。

于 2013-05-11T00:27:19.763 回答