指针数组内容在传递给 C 中的函数时被擦除

Question

所以我从文本文件加载浮点数行并将它们存储在指针数组中，然后将它们保存回文本文件并添加对其大小的引用。文本文件中的值数量不同，因此数组必须是动态的。我像这样在 main 中定义我的指针数组。

size_t size = (int)100 * sizeof(float);
float * val = malloc(size);

然后我将指针数组传递给加载文本文件并将值保存到其中的函数，就像这样。

//Read file into array.
int readFile(float *val, int size) {

char buf[20] = { 0 };
val[0] = 0;
double temp = 0;
int i = 1;
FILE *file;
file = fopen("C:\\Users\\MoldOffice\\Dropbox\\VS\\finalproj\\ecgproject\\dataStream", "r");
if (!file) {
    printf("Coulding find file.\n");
    exit(1);
}

while (fgets(buf, 20, file) != NULL) {
    temp = atof(buf);
    if (temp != 0) {
        // Increment i to find the size of the useful data.
        val[i] = temp;
        //printf("%d",val[i]);
        i++;
        if (i == size / sizeof(float)) {
            size += 100*sizeof(float);
            double* val_temp = realloc(val, size);
            if (val_temp == NULL) {
                printf("Realloc failed.\n");
            }
            else {
                val = val_temp;
            }
        }
    }
}
//Test that the array is readable.
for (int i = 0; i < 5; i++) printf("val[%d]=%f\n", i, val[i]);

return(i);
fclose(file);

这很好用，当我在 main 中打印指针数组的内容时，它可以工作。然后我将相同的指针数组传递给另一个函数，该函数将数组保存在一个新的文本文件中，连同第一行的大小，问题是当我第二次传递指针数组时，内容已经改变（主要是0 和一些随机数）。我完全不知道为什么会这样。有什么想法吗？写入文件的函数在这里：

// Write file into array.
void writeFile(float *val,int size) {

printf("%d",sizeof(val));
FILE *file;
int sampleNum;
char buf[10];
file = fopen("sampleNum.txt", "r");
if (file == NULL) { sampleNum = 0; }
else { fscanf(file, "%d", &sampleNum); printf("%d",sampleNum);}
char fileString[10];
sprintf(fileString,"sample%d\0", sampleNum);
file = fopen(fileString, "w");

//Test that the array is readable.
for (int i = 0; i < 5; i++) printf("val[%d]=%f\n", i, val[i]);

//Print the array to a text file and save.
fprintf(file, "%d\n", size);
for (int i = 1; i < size; i++) {
    fprintf(file, "%f\n", val[i]); 
    printf("%f\n", val[i]); }
fclose(file);
}

main 的其余部分可以在这里找到：

int main() {

size_t size = (int)100 * sizeof(float);
float * val = malloc(size);

// Read the data into an array.
int arraySize = readFile(val, size);

//Test that the array is readable.
for (int i = 0; i < 5; i++) printf("val[%d]=%f\n", i, val[i]);

// Save the array to a text file, with the size of the array as the first element.
writeFile(val,arraySize);

}

Answer 1

        double* val_temp = realloc(val, size);
        if (val_temp == NULL) {
            printf("Realloc failed.\n");
        }
        else {
            val = val_temp;

此函数的调用者无法知道您已将数组移动到不同的位置。它仍然有旧的、现在无效的指针。

size 也有类似的问题。来电者怎么知道你改变了它？

你选择的职责分工很差。如果调用者负责分配缓冲区，那么这个函数应该要求调用者扩大它。如果这个函数负责分配缓冲区，它应该分配它。将管理一块内存分配的责任分开通常是一个非常糟糕的主意，这说明了其中一个原因。

也许传入一个指向结构的指针，该结构包含指向缓冲区的指针及其大小？这会行得通，但仍然表明职责分工很差。

也许让这个函数分配缓冲区并返回一个结构，其中包含指向它的指针和其中的元素数量？

如果您真的想这样做，请考虑向函数传递一个指向结构的指针，该结构包括指向数组的指针、数组的大小以及指向调整数组大小的函数的指针。调用者当然可以将此指针设置为指向realloc 函数（尽管它可能是一个更改结构的指针和大小成员的函数更好）。

你也可以使用这样的代码：

struct float_buffer
{
    float* buffer;
    int size;
};

struct float_buffer allocate_float_buffer(int size)
{
    struct float_buffer buf;
    buf.buffer = malloc (size * sizeof(float));
    buf.size = size;
    return buf;
}

bool resize_float_buffer(struct float_buffer* buf, int size)
{
    float* tmp = realloc(buf->buffer, size * sizeof(float));
    if (tmp == NULL)
        return false;
    buf->buffer = tmp;
    buf->size = size;
    return true;
}

然后给函数传递一个struct float_buffer *。