| Цитата(Anikmar @ 18.5.2007, 19:39) | Посмотрел я код внимательнее и увидел там не один, а несколько багов
Скидываю подправленный вариант:
| Код |
int main(int argc, char* argv[])
{
char* filename; //имя файла
int i = 0; //счетчик
FILE* fd; //дескриптор файла
char str[STRLEN]; //строка, которая читается из файла
array = NULL;
filename = "input.txt"; //Имя файла
fd = fopen(filename, "rt"); //Открываем файл
while(!feof(fd))
{
fgets(str,STRLEN,fd); //читаем из файла по STRLEN байт
++i; //Наращиваем счетчик
array = ( char**)realloc(array,sizeof(char*)*i); //растягиваем массив на количество символов из файла
array[i-1] = ( char*)malloc(strlen(str)+1); //подготавливаем память для копирования строки из файла
strcpy(array[i-1],str);
}
puts("\nEND OF FILE READ\n"); //Для себя пометка, что чтение файла закончилось
int j = 0;
//цикл для проверки значений массива
for(j=0; j<i; j++){
printf("%s\n",array[j]);
}
getch();
}
|
Какие баги в вашем коде я увидел (и исправил):
1.
array = ( char**)realloc(array, sizeof(char*));
Данная функция, на сколько я помню, увеличивает массив не на указанную величину, а до указанной величины - т.е. надо задавать новый размер массива
2.
array[i] = ( char*)malloc(strlen(str)+1);
У нас первое значение i будет 1 - потеряется нулевой элемент - так как i инкрементируется до этого
3.
array[i] = str;
Так строки не копируется - этим вы копируете указатель на ваш статический массив
4.
for(j=0; i<=j; j++){
Посмотрите внимательно на условие цикла. Вы перепутали переменые. |
Громадное спасибо, код реально поправил положение дел, я кстати грешил на инкрементную переменную и связанные с ней моменты , даже делал в цикле проверку на наличие в array(i-1) данных - они там там были в период прохождения цикла по файлу array(i-2) - вызывали Segmentation fial, а по поводу переменных в последнем цикле - это даже от себя такой баги не ожидал, если честно долго смеялся сам над собой когда понял. Цитируемый код заработал сразу, но это по сути небольшая верхушка айсберга, приведенный мною пример - это код одной из функции программы, по-этому если не трудно, господа, подскажите каким образом, вернее сказать как более правильно будет организовать функцию которой передается в качестве аргумента массив и указатель на файл а возращаемым агрументом был бы массив, который содержит строки файла, без "\n".
Спасибо огромное всем кто ответил!
ЗЫ: Извиняюсь возможно за некоторое незнание некоторых моментов в С, о я последнюю строчку на С, написал в 99 году, по-этому сейчас спустя 8 лет достаточно сложно восстановить знания, прошу отнестись с пониманием  |
