0

私はこのコードを持っています:

     char **arr;
     char* line=NULL;
     int i=0;
     size_t len=0;
     ssize_t read1;

     fp=fopen("list.txt","r");
     if(fp==NULL)
         exit(EXIT_FAILURE);

     while((read1=getline(&line,&len,fp))!=-1)
         i++;
     fclose(fp);

     fp=fopen("list.txt","r");
     if(fp==NULL)
         exit(EXIT_FAILURE);

     arr=(char**)malloc(i*sizeof(char*)); // i is a variable i use to know the number of lines
     i=0;

     while((read1=getline(&line,&len,fp))!=-1)
     {
         line[strlen(line)]='\0';
         arr[i]=(char*)malloc(strlen(line)+1);
         strcpy(arr[i],line);
         i++;
     }

strcpyプログラムを試してみるとクラッシュしmallocます。問題はありますか? 私はそれiが十分に大きいと確信しています。そして、それはline最初です。char*NULL

編集: このプログラムが Qt にあることを忘れていました。

4

2 に答える 2

3

コードにはいくつかの問題があります。うまくいくと思われるものについてコメントします...：

 // I **assume** that these are the definitions for these variables 
 // based on your comments
 size_t len = 0;
 char *line = NULL;
 ssize_t read1;

 // I **assume** that i has a reasonable value here, but this is not good to assume, 
 // what if the file is a line longer tomorrow? I hope that you calculate the number 
 // of lines somehow, that would be "less bad"
 int i = 10; // 10 lines in the file, who knows ?!?
 char **arr;

 // don't bother casting...
 arr = malloc(i * sizeof(char*)); 
 i=0;

 while((read1 = getline(&line, &len, fp)) != -1) {

     // THIS LINE DOES NOTHING, so we can just remove it
     // line[strlen(line)]='\0';

     arr[i] = line; // since you asked getline to allocate a buffer for 
                    // you (line was NULL), you can just store the buffer directly
                    // it's YOURS
     i++;

     // THIS IS THE BIG ONE:
     // it is needed because otherwise the NEXT call to getline will 
     // reuse the same buffer, which may not be big enough
     line = NULL;
 }

また、後でクリーンアップするために、次のようなことを行う必要があります。

int j;
for(j = 0; j < i; ++j) {
    free(arr[j]);
}
free(arr);
arr = NULL; // not necessary, but good practice to avoid double frees and such
于 2013-01-24T19:19:00.300 に答える
2

元のiよりも実際に多くの行があるかどうかをテストしません

 arr=(char**)malloc(i_ori*sizeof(char*));//i_ori is a variable i use to know the number of lines
 i=0;

 while((read1=getline(&line,&len,fp))!=-1 && i<i_ori)

また、 malloc が NULL を返すかどうかをテストすることはありません!! https://stackoverflow.com/a/2280342/1458030を参照してください。

@Emil Grigore : strcpy を試すと、プログラムがクラッシュします。malloc の問題ですか? 私は十分に大きいと確信しています。

はい！NULL をテストする必要があります。

C++ と Qt を使用している場合、コンテナーやストリームを使用しないのはなぜですか?

于 2013-01-24T19:10:20.807 に答える