在单独的函数中创建一个动态数组并从调用函数中对其进行操作答案

【问题标题】：Create a dynamic array in separate function and manipulate it from calling function在单独的函数中创建一个动态数组并从调用函数中对其进行操作
【发布时间】：2017-03-27 00:19:03
【问题描述】：

我正在使用一个打开文件的函数，将该文件的内容读入动态数组的函数，关闭文件的函数。

到目前为止，除了当我返回调用位置（主）时动态数组超出范围之外，我能够执行上述所有操作。我想在主函数甚至单独的函数中将其他数据存储在数组中。完成将数据添加到动态数组后，我会将其内容写回源文件，用新数据覆盖它，然后关闭该文件。目的是将数据附加到原始文件的顶部。我在 main 中无法访问或修改函数 char *LoadFileData(FILE *fp, char* charPtr); 做错了什么？

感谢您对此的帮助。

    FILE *fSource;       // create source file pointer instance
    char mode[] = "a+";  // default file open mode
    char inStr[80];      // string to get input from user
    char *tempFileData;  // dynamic string to hold the existing text file

// Open the source file
    strcpy(mode, "r");   // change the file opnen mode to read
    FileOpen(&fSource, mode);

// Load the source file into a dynamic array
    LoadFileData(fSource, tempFileData);  // this is where I fail that I can tell.

    printf("%s", tempFileData); // print the contents of the (array) source file //(for testing right now)
    FileClose(&fSource);  // close the source file

char *LoadFileData(FILE *fp, char* charPtr)
  {
    int i = 0;
    char ch = '\0';
    charPtr = new char; // create dynamic array to hold the file contents
    if(charPtr == NULL)
    {
        printf("Memory can't be allocated\n");
        exit(0);
    }
// loop to read the file contents into the array
   while(ch != EOF)
    {
        ch = fgetc(fp);  // read source file one char at a time
        charPtr[i++] = ch;
    }
    printf("%s", charPtr); // so far so good.
    return charPtr;
  }

【问题讨论】：

charPtr = new char; 不会为您分配动态数组。请改用new char[length]。
您的数组没有超出范围，您没有将其指针返回到您的调用函数。您的 charPtr 参数可以作为 char*& charPtr 传递。或者按照其他人的建议分配返回值。

标签： c++ c function dynamic-arrays file-pointer

【解决方案1】：

不要传入您从不使用其值的char *，而是将函数的返回值分配给tempFileData。

所以把函数改成这样：

char *LoadFileData(FILE *fp)
{
    char* charPtr;
    ...

然后这样称呼它：

tempFileData = LoadFileData(fSource);

【讨论】：

【解决方案2】：

其中一个问题是以下几行的组合：

charPtr = new char; // create dynamic array to hold the file contents

    charPtr[i++] = ch;

您只为一个char 分配内存，但继续使用它，就好像它可以容纳很多字符一样。

你需要：

查找文件中存在的字符数。
为所有字符分配内存（+1，如果你需要空终止数组）。
将文件内容读取到分配的内存中。

【讨论】：

【解决方案3】：

返回string怎么样？

string LoadFileData(FILE *fp, char* charPtr)

【讨论】：

【解决方案4】：

根据每个人的反馈，这是有效的修改。谢谢！

char* LoadFileData(FILE *fp) 
{
    off_t size; // Type off_t represents file offset value. 
    int i = 0;
    char ch = '\0';
    char *charPtr; // dynamic arrary pointer that will hold the file contents

    fseek(fp, 0L, SEEK_END); // seek to the end of the file
    size = ftell(fp);        // read the file size.
    rewind(fp);              // set the file pointer back to the beginning

    // create a dynamic array to the size of the file
    charPtr = new char[size + 1]; 

    if (charPtr == NULL) {
        printf("Memory can't be allocated\n");
        // exit(0);
    }

    while (ch != EOF) {
        ch = fgetc(fp); // read source file one char at a time
        if (ch < 0) { // do not copy it if it is an invalid char
        }
        else {
            charPtr[i++] = ch;
            // load the char into the next ellement of the array
            // i++;
        }// end else
    } // end while

    return charPtr; 
}

【讨论】：