Question

您好我编写了一个程序来执行文本文件中的命令。下面的代码用于首先逐行存储到char数组中。

所以我希望它能做类似

的事情

args[0]= The first line of text file  
args[1]= The second line of text file
... and so on

在我的代码中，所有数组都将被最后一个数组覆盖。我无法弄清楚原因。

任何人都可以帮我解决这个问题并告诉我为什么我的代码会这样做。另外我需要保留char * args []。我稍后会将它与execvp（）一起使用。

int main(int argc, const char * av[]) {    
    FILE *fp;    
    fp = fopen(av[1],"r");

    int n_lines=0;        
    char in[100],*args[16];        
    int size=sizeof(in);

     while(fgets(in, size, fp)!=NULL){        
        args[n_lines] = in;                
        printf("Args[0] is %s\n",args[0]);            
        n_lines++;
    }

     printf("Now Args[0] is %s\n",args[0]);
}

输出

zacks-MacBook-Pro:prac2 zack$ ./a.out test    
Args[0] is ./addone    
Args[0] is ./add    
Now Args[0] is ./add

Answer 1

int n_lines=0;        
char in[100],*args[16];        
int size=sizeof(in);

while(fgets(in, size, fp)!=NULL){        
    args[n_lines] = in;                
    printf("Args[0] is %s\n",args[0]);            
    n_lines++;
}

每次迭代都会覆盖in的值，您需要预留空间（如果可用，请使用malloc->strcpy或strdup）：

char in[100], *args[16];

while (fgets(in, sizeof in, fp) != NULL) {
    args[n_lines] = strdup(in);
    ...
    n_lines++;
}

或者使用2D数组（sizeof中需要调整fgets）：

char in[16][100];

while (fgets(in[n_lines], sizeof in[0], fp) != NULL) {
    ...
    n_lines++;
}

正如@MichaelWalz在评论中指出的那样：如果你的文件超过16行，你会遇到问题。

更改为

while (fgets(in[n_lines], sizeof in[0], fp) != NULL) {
    ...
    if (++n_lines == (sizeof in / sizeof in[0])) break;
}

使用char指针将文本文件中的逐行存储到char数组中

1 个答案: