Stocker un .txt dans un char**

**Arnau6** · 10/11/2012, 18h24

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** · 10/11/2012, 18h47

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** · 10/11/2012, 19h11

Envoyé par Neckara

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.

**Arnau6** · 10/11/2012, 19h15

Envoyé par 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.

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

**Neckara** · 10/11/2012, 19h26

Envoyé par Kirilenko

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** · 10/11/2012, 20h00

Envoyé par Neckara

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** · 10/11/2012, 20h13

Envoyé par Arnau6

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** · 10/11/2012, 20h28

Une fois la ligne 24 remplacé par : strcpy(elementsToFind[numLine-1],temp);
La fonction plante des qu'il rencontre cette ligne modifiée alors que je compile sans erreur. C'est la même chose avec strncpy

Invité · 10/11/2012, 20h45

Bonsoir,

C'est normal, il faut allouer l'espace où tu vas effectuer ta copie avec malloc().

**Arnau6** · 10/11/2012, 21h33

Envoyé par Winjerome

Bonsoir,

C'est normal, il faut allouer l'espace où tu vas effectuer ta copie avec malloc().

Maintenant j'ai ce code :

Code :

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

1
2
3
4
5
6
7
8
9
 if(strrchr(temp,'\n')) //all lines except the last one
                {
                    temp[strlen(temp)-1] = '\0';
                    elementsToFind[numLine-1] = (char *) malloc(strlen(temp) * sizeof(char));
                    strcpy(elementsToFind[numLine-1],temp);
                    printf("TEST %d : %s\n",numLine,elementsToFind[numLine-1]);
                    numLine ++;
 
                }

Avec ce warning au niveau du malloc : 'elementsToFind' may be used uninitialized in this function [-Wuninitialized]|
De plus le programme plante sur le malloc. Je ne suis pas encore a l'aise avec les malloc et les pointeurs. Pouvez vous me dire ou je fais fausse route ?

Merci

**Neckara** · 10/11/2012, 21h56

Code :

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

elementsToFind[numLine-1] = (char *) malloc(strlen(temp) * sizeof(char));

Il faut aussi allouer l'espace pour le '\0' final donc :

Code :

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

(char *) malloc(strlen(temp)+1);

De plus la définition de sizeof est que sizeof(char) = 1.

**Arnau6** · 10/11/2012, 22h02

Le +1 ne change rien.

J'ai changé mon code, Maintenant j'arrive a stocker tout les lignes dans un char[numLine][512] mais je n'arrive pas a le retourner correctement en tant que char**. Aucune erreur est retourné mais quand je veux afficher le contenu d'un char** elementsToFind = fOption();
avec des printf, le programme plante

Voici mon nouveau code

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
49
50
char** fOption()
{
    int numLine = 1;
    char count[512];
 
        char* values_include = "c:\\test.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
        {
            while(fgets(count, 512, fp))
            {
                numLine ++;
            }
        }
        fclose(fp);
 
 
        char temp[numLine-1][512];
        fp = fopen(values_include, "r");
        numLine = 1;
            while(fgets(temp[numLine-1], 512, fp))
            {
                if(strrchr(temp[numLine-1],'\n')) //all lines except the last one
                {
 
                    temp[numLine-1][strlen(temp[numLine-1])-1] = '\0';
                }
                //printf("%s\n",temp[numLine-1]);
                numLine ++;
            }
        fclose(fp);
 
 
        int i;
        for(i=1; i< numLine-1; i++)
            printf("==>%s\n",temp[i]);
 
        char** isReturn = NULL;
        isReturn = (char**)temp;
        return isReturn;
}

Stocker un .txt dans un char**

C

Discussions similaires

Partager

Partager