Discussion :

[Arnau6]

Bonjour,

J'ai crée une fonction qui devrai retourner le contenu d'un .txt dans un char**.

Le soucis que je rencontre c'est que ma fonction retourne un char** avec comme valeur la dernière ligne du fichier .txt pour chaque "cellule" alors que chaque "cellule" devraient retourner une ligne différente. Autrement dis, à chaque fois que le char** reçoit un nouveau char* (une ligne du fichier .txt), il écrase tout les char* précédent avec la valeur en cour.

Il doit surement il y avoir un aspect dans l'utilisation des char** que j'ai mal compris. Pourriez vous m'aider.

Voici ma fonction, elle affiche chaque ligne une fois stockées dans le char** puis affiche son contenu une fois que tout le texte à été parcouru, les saisies se font a la ligne 24 et 30. Mon erreur se trouve surement ici

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
char** fOption()
{
        char* elementsToFind[512] ;
        char* values_include = "c:\\test2.txt";
        FILE *fp;
        fp = fopen(values_include, "r");
 
        if( fp == NULL )
        {
            printf("f, unable to open the file : %s\n",values_include);
            exit (0);
        }
 
        else
        {
            int numLine = 1;
            char temp [512]= "";
            while(fgets(temp, 512, fp))
            {
 
                if(strrchr(temp,'\n')) //all lines except the last one
                {
                    temp[strlen(temp)-1] = '\0';
                    elementsToFind[numLine-1] = temp;
                    printf("TEST %d : %s\n",numLine,elementsToFind[numLine-1]);
                    numLine ++;
                }
                else //last line
                {
                    elementsToFind[numLine-1] = temp;
                    printf("TEST %d : %s\n",numLine,elementsToFind[numLine-1]);
                    elementsToFind[numLine] = NULL;
                }
            }
        }
        fclose(fp);
 
        int iterator = 0;
        printf("------\ncharr** Final : \n");
        while(elementsToFind[iterator] != NULL)
        {
            printf("==>%s\n",elementsToFind[iterator]);
            iterator++;
        }
        char** tempElementsToFind = elementsToFind;
        return tempElementsToFind;
 
}

[Neckara]

Bonjour,

Pour copier une chaîne de caractère, il faut utiliser strcpy ou strncpy (plus sûr).

Ensuite

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
        char** tempElementsToFind = elementsToFind;
        return tempElementsToFind;

Pourquoi ne pas directement faire :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

return ElementsToFind;

?

N'oublie pas que le pointeur retourné va pointé vers une variable locale donc à la fin de ta fonction, elle seront détruite et tu risques des erreurs de segmentations.

Il faut alors utiliser malloc et free.

[Kirilenko]

Pour copier une chaîne de caractère, il faut utiliser strcpy ou strncpy (plus sûr).

strncpy n'a pas été réalisé pour la copie de chaînes de caractère. Ce n'est PAS une version plus sûre de strcpy.

[Neckara]

strncpy n'a pas été réalisé pour la copie de chaînes de caractère. Ce n'est PAS une version plus sûre de strcpy.

Attention, je n'ai pas dit que strncpy était une version de strcpy mais qu'utiliser strncpy est plus sûr que strcpy.
Il faut bien évidement ne pas oublier que si la chaîne source est trop grande, le '\0' final ne sera pas ajouté.

Quelle erreur t'indique le compilateur?
J'ai testé mais j'ai juste un Warning pour dire que la fonction retourne un pointeur sur une variable locale.

[Arnau6]

Quelle erreur t'indique le compilateur?

Il m'indique le même warning que toi. Mais ne tenez pas compte de ceci, l erreur se fait des le printf soit avant le return.

Lors de la boucle du printf ligne 42, seulement la dernière ligne de mon txt est affiché X fois, X étant le nombre de ligne de mon fichier

[Neckara]

Il m'indique le même warning que toi. Mais ne tenez pas compte de ceci, l erreur se fait des le printf soit avant le return.

Lors de la boucle du printf ligne 42, seulement la dernière ligne de mon txt est affiché X fois, X étant le nombre de ligne de mon fichier

La réponse t'as déjà été donnée, il faut utiliser une fonction pour copier des chaînes de caractères et pas seulement faire une affectation avec temp.

En effet, tu va juste copier l'adresse vers le premier caractère de temp or à chaque tour de boucle, tu modifies temp et comme toutes les lignes pointent vers le premier caractère de temp, toutes les lignes affichent la même chose.

[Arnau6]

Bonjour,

Pour copier une chaîne de caractère, il faut utiliser strcpy ou strncpy (plus sûr).

Ensuite

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
        char** tempElementsToFind = elementsToFind;
        return tempElementsToFind;

Pourquoi ne pas directement faire :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

return ElementsToFind;

?

N'oublie pas que le pointeur retourné va pointé vers une variable locale donc à la fin de ta fonction, elle seront détruite et tu risques des erreurs de segmentations.

Il faut alors utiliser malloc et free.

je retourne pas ElementsToFind; parce que c est un char *[512] et si j'essaye on m indique une erreur.

Mais mon erreur ne se localise pas au niveau de ma copie comme tu le suggère puisque ce que j affiche avec printf est l élément d'origine et pas ce que je retourne