strcpy char указатель прервать

Question

strcpy char указатель прервать

У меня есть этот код:

     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.

0

c++malloc qt strcpy

Решение

Другие решения

Вы не проверяете, действительно ли у вас больше строк, чем у оригинала.

 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, происходит сбой программы.
проблема? Я очень уверен, что я достаточно большой.

Да! вам нужно проверить на NULL.

Если вы используете C ++ и Qt, почему не контейнеры, потоки?

2

Источник

Accepted Answer

Есть несколько проблем с кодом, я прокомментирую то, что я считаю, должно работать …:

 // 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

3