2016-11-06 2 views
0

Здесь я пытаюсь написать свою собственную функцию realloc, и я застрял на копировании старого массива в новый массив. Это не проблема, если один файл содержит только 16 или даже меньше строк каждой строки. Проблема возникает, когда строки поверх исходной настройки 16. Это означает, что проблема относится к моей реализации функции realloc, я думаю. В моей программе я установил три шага для управления изменениями массива.реализация моей собственной функции realloc в c

Моя мысль:

Первый шаг: когда линия файла меньше или равно 16, дают значение new_array

Второй шаг: когда линия файла равен 17, должен таНос еще один пространство. скопируйте старый массив в новый и дайте текущей строке последнему пробелу

Третий шаг: аналогично второму шагу, но старый массив является предыдущим расширенным массивом.

На данный момент я изо всех сил пытаюсь понять, почему это не работает. И когда мой файл имеет ровно 17 строк, например:

1 
2 
3 
4 
5 
6 
7 
8 
9 
10 
11 
12 
13 
14 
15 
16 
17 

Результат

here start 
1 

2 

(null) 
(null) 
(null) 
(null) 
(null) 
(null) 
(null) 
(null) 
(null) 
(null) 
(null) 
(null) 
(null) 
(null) 
17 

The file 'foo' had 17 lines. 

utitlityFunction.c

#include "utilityFunction.h" 
#include <stdlib.h> 
#include <string.h> 
#include <stdlib.h> 

char **extendArray(char **oldArray, int oldLen, int newLen){ 
    char **newptr = malloc(newLen * sizeof (char*)); //updated here. No changed for result 
    // if(newptr == NULL){ 
    // perror("Failed to allocate"); 
    // exit(1); 
    // } 
    memcpy(newptr, oldArray, oldLen); 
    free(oldArray); 
    return newptr; 
} 

Моя основная функция кода в дальнейшем.

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

int main(int argc, char**argv){ 
    FILE *filename; 
    size_t len = 0; 
    char *line = NULL; 
    int index; 
    int countLine = 0, i = 0; 
    char** new_array = malloc(16 * sizeof(char*)); 
    char** extended_array; 
    char* current_line; 

    for(index = 1; index < argc; index++){ //loop for all files name you input in the command line 
     filename = fopen(argv[index], "r"); 
     if(filename == NULL){ 
      printf("The file '%s' did not exist.\n", argv[index]); 
     }else{ 
      while(getline(&line, &len, filename)!=EOF){ 
       current_line = strdup(line); 
       if(new_array == NULL){ 
        perror("Failed to allocate"); 
        exit(1); 
       } 
       if(i<16){ 
        new_array[i] = current_line; 
       } 
       else{ 
        if(i==16){ //actually index 17 
         extended_array = extendArray(new_array, i, countLine); 
         extended_array[i] = current_line; 
        }else{ 
         printf("aaa?\n"); 
         extended_array = extendArray(extended_array, i, countLine); 
         extended_array[i] = current_line; 
        } 
       } 
       i++; 
       countLine++; 
      } 
      //mprintf("%s\n", extended_array[0]); 
      //mprintf("-->m%d\n", countLine); 
      printf("here start\n"); 
      int j; 
      for(j = 0; j < countLine; j++){ 
       printf("%s\n", extended_array[j]); 
      } 
      //print line result after end of file 
      printf("The file '%s' had %d lines.\n", argv[index], countLine); 
     } 
    } 
} 

Я просто потеряли ....

+3

Вы не выделяете достаточное количество памяти. Аргумент 'malloc' - это количество байтов, но вы передаете количество элементов –

+0

@ M.M. Вы имеете в виду« malloc »в utitlityFunction.c? Я попытался ... Но результат не изменился ... Так что сбив с толку то, что он только напечатал первые два числа и последнее число ... – HxH

+0

Не следует ли использовать 'countLine' вместо' i' в качестве индекса массива ? – alk

ответ

0

Я думаю, ваша проблема исходит от тетсра. void * memcpy (void * destination, const void * source, size_t num); Вам нужно memcpy каждый случай вашей вкладки вместо char ** в один звонок. попробуйте

Смежные вопросы