Modifier un caractère * par pointeur dans une fonction donne un plantage

Question

Je ne suis pas familier avec C et j'essaye de trouver une allocation de mémoire dynamique pour la lecture d'un fichier. Au moins, je pense que c'est ce que je fais.

Quoi qu'il en soit, ce code fonctionne:

int readfromfile(FILE *filepointer) 
{ 
    size_t size = 2; 
    char *str = (char *) malloc(sizeof(char)); 
    int character = 0; 
    size_t counter = 0; 
    while((character = fgetc(filepointer)) != EOF) 
    { 
     str = (char *) realloc(str, size); 
     str[counter] = (char) character; 
     size ++; 
     counter++; 

    } 
    str[counter] = '\0'; 
    printf("+%s+", str); 
    free(str); 
    return 0; 
} 

Et ce code ne fonctionne pas:

int main() 
{ 
    char *str = (char *) malloc(sizeof(char)); 
    ... 
    readfromfile(ifpointer, &str); 
} 

int readfromfile(FILE *filepointer, char **str) 
{ 
    size_t size = 2; 
    int character = 0; 
    size_t counter = 0; 
    while((character = fgetc(filepointer)) != EOF) 
    { 
     *str = (char *) realloc(*str, size); 
     *str[counter] = (char) character; 
     size ++; 
     counter++; 
    } 
    str[counter] = '\0'; 
    printf("+%s+", *str); 
    free(str); 
    return 0; 
} 

Je ne comprends pas pourquoi, parce que ce que je comprends que je envoie un pointeur vers l'emplacement le tableau char à la fonction et l'accès aux données à chaque fois. Les compilateurs ne montrent aucun message d'erreur, ils ne font que passer une boucle et la deuxième boucle se bloque après le realloc à chaque fois. Le caractère affecté à la première valeur est également garbage. J'ai passé un certain temps à essayer de faire fonctionner cela et j'ai fait beaucoup de recherche, donc je m'excuse si j'ai manqué une solution, mais je suis vraiment bloqué à ce stade.

Answer 1

vous obtenez un accident parce que

*str[counter] = (char) character; 

est le même que

*(str[counter]) = (char) character; 

par opposition à

(*str)[counter] = (char) character; 

qui est en fait ce que vous vouliez. Lisez Operator Precedence on Wikipedia. Vous trouverez que [] a plus de priorité que le * (opérateur de déréférencement).

En outre, la distribution ici, ainsi que dans les appels à realloc et malloc, est inutile. N'oubliez pas de vérifier la valeur de retour de realloc, malloc etc pour voir s'ils ont réussi à allouer de la mémoire.

Maintenant, vous avez un autre problème: free(str); dans le deuxième code doit être free(*str);. Notez qu'après que *str a été libéré de la fonction, vous n'êtes pas censé lire ou écrire dans cet emplacement de mémoire de main car il est maintenant devenu invalide pour que vous fassiez une falsification.

Answer 2

dans votre int readfromfile(FILE *filepointer, char **str) le paramètre char **str est en fait le même que char *str[], ce qui signifie **str attend un tableau de pointeurs char. mais vous passez à ce char *str qui est juste un tableau de char

lorsque vous utilisez ReadFromFile (...) vous devriez le faire de cette façon (quelque chose comme ...):

char *str[2] = {"some char array", "another char array"}; 
readfromfile(ifpointer, str); 

ou :

char *a = "this char array"; 
char **str = &a[0]; 
readfromfile(ifpointer, str); 

vous obtiendrez l'idée ...