C 中的命令数组

Question

我正在尝试创建一个名为 History 的 char** ，它包含如下命令数组：

do thing1
做事2
做事3
do thing4

这些命令由用户一次输入一个，我希望它们在出现时添加到这个历史数组中。在程序结束时，我想按照写入的顺序将命令打印回用户。

这是我的代码：

#include <stdio.h>
#include <string.h>
#include <stdlib.h>

int BUFFER = 50;
int MAX_ARGS = 100;
int HISTORY_SIZE = 50;

int bufferSize;

//Read in line from user 
char* getLine(void){

    char* line = NULL;
    size_t len = 0;
    size_t lineSize = 0;
    lineSize = getline(&line, &len, stdin);
    return line;
}

int main(){

    //declare variables
    char *line, *lineCopy;
   
    //history structure
    char* history[100];
    int historyCount = 0;

    do{

        //main program loop

        //get command 
        printf("# ");
        line = getLine();

        //copy command
        strcpy(lineCopy, line);

        history[historyCount] = lineCopy;
        historyCount++
        

    }while(historyCount < 4); //while status

    for(int i = 0; i < historyCount; i++){
        printf("%d: %s\n", i, history[i]);
    }

    return 0;
}

这样，数组中的每个值都设置为输入的最后一个命令，输出是

do thing4
做事4
做事4
do thing4

我明白这只是因为我每次都将它指向 lineCopy 的内存地址，但我不知道如何修复它并得到

do thing1
做事2
做事3
做事4

Answer 1

1. 字符串复制到无效内存。

2. 由于每次调用包装器 getLine() 都会获得新的内存缓冲区，因此您可以立即使用该缓冲区，而无需重复。

    //copy command
    //strcpy(lineCopy, line);

    history[historyCount] = line;
    historyCount++

    //or just
    history[historyCount++] = getLine();

3. 使用完毕后，您需要释放所有存储的 history[] 行缓冲区。

4. 您没有在 getLine() 包装器中检查 getline() 的返回状态。

Answer 2

一个问题是您正在将行复制到 lineCopy，但您从未为 lineCopy 分配内存。

并且有一个由 100 个字符指针组成的数组，每个指针都指向 lineCopy，这实际上不是您想要的。因为 lineCopy 将始终指向您的情况中最新的字符串。这就是为什么历史记录中的每个命令本质上都是相同的。

Answer 3

你只需要复制该字符串

history[historyCount] = strdup(line);