Problème de CRLF et de strcpy

**Otacon** · 16/04/2007, 17h18

Bonjour à tous,

Je développe en ce moment un programme qui accède à des fichiers de configuration grace à une fonction que j'ai trouvée et modifiée un peu et je rencontre un problème :

lorsque le caractère de fin de ligne est un simple LF (\n), pas de problèmes, tout ce passe bien
lorsque le caractère est un CR (\r), la fonction ne reconnait pas la fin de ligne et considère le fichier comme vide
lorsque c'est un CRLF (\r\n) la fonction rajoute un caractère NUL (\0) pour chaque ligne présente dans le fichier et ce à la fin.

Le code de cette fonction est :

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
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
int conf_set_str(const char file[], const char name[], const char value[])
{
    int done = 0, len = 0, found = 0;
    unsigned int i = 0;
    char *buf = NULL, *p = NULL;
    FILE *conf_file = NULL;
 
    if(!file || !name)
    {
        return -1;
    }
 
    if((conf_file = fopen(file, "r")) == NULL)
    {
        return -2;
    }
 
 
    fseek(conf_file, 0L, SEEK_END);
    len = ftell(conf_file);
    fseek(conf_file, 0L, SEEK_SET);
 
    if((p = buf = malloc(len)) == NULL)
    {
        return -3;
    }
 
 
 
    fread(buf, len, 1, conf_file);
 
    while(*p)
    {
        if(*p == '#')
        {
            while(*p != '\n')
            {
                p++, done++;
            }
 
            p++, done++;
            continue;
        }
        else if(*p == name[0])
        {
            for(found = 1, i = 0 ; i < strlen(name) ; i++, p++, done++)
            {
                if(*p != name[i])
                {
                    found = 0;
                    break;
                }
            }
 
            if(found)
            {
                while(isspace(*p))
                {
                    p++, done++;
                }
 
                done++;
 
                if(*p++ != '=')
                {
                    if(buf)
                    {
                        free(buf), buf = NULL;
                    }
 
                    return -4;
                }
 
                while(isspace(*p))
                {
                    p++, done++;
                }
 
                if(*p == '"')
                {
                    *p = '\0';
 
                    freopen(file, "w", conf_file);
 
                    if(fwrite(buf, done, 1, conf_file) != 1)
                    {
                        if(buf)
                        {
                            free(buf), buf = NULL;
                        }
 
                        return -5;
                    }
 
                    *p = '"';
                    fprintf(conf_file, "\"%s\"", value);
 
                    do
                    {
                        p++, done++;
                    }
                    while(*p != '"');
 
                    p++;
 
                    if(len - done > 0)
                    {    
                        if(fwrite( p, len - done, 1, conf_file ) != 1)
                        {
                            if(buf)
                            {
                                free(buf), buf = NULL;
                            }
 
                            return -5;
                        }
                    }
                }
                else
                {
                    if(buf)
                    {
                        free(buf), buf = NULL;
                    }
 
                    return -6;
                }
 
                fclose(conf_file);
 
                if(buf)
                {
                    free(buf), buf = NULL;
                }
 
                return 1;
            }
            else
            {
                while(*p != '\n' && *p != '\0')
                {
                    p++, done++;
                }
            }
        }
 
        while(*p != '\n')
        {
            p++, done++;
        }
 
        p++, done++;
    }
 
    fwrite(p, len - done, 1, conf_file);
    fclose(conf_file);
 
    if(buf)
    {
        free(buf), buf = NULL;
    }
 
    /* Create the parameter if it doesn't exist */
 
    if((conf_file = fopen(file, "a")) == NULL)
    {
        return -2;
    }
 
    fprintf( conf_file, "%s = \"%s\"\n", name, value );
    fclose(conf_file);
 
    return 1;
}

Je l'appelle comme cela :

Code :

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

1
2
3
4
5
6
7
 
int error = 0;
if((error = conf_set_str("nom_du_fichier.conf", "Name", "valeur")) != 1)
{
    fprintf(stderr, "Error while updating name : %i\n", error);
    return -2;
}

Le fichier de config. est sous cette forme :

Code :

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

1
2
# Nom 
Name = "Nom"

S'il y a des CRLF on aura :

Code :

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

1
2
# Nom 
Name = "Nom"\0

(une seule ligne avec le caractère de fin)

-----------------------

J'ai aussi un autre problème concernant une structure cette fois :
Celle-ci est définie comme cela :

Code :

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

1
2
3
4
5
6
typedef struct pet_t
{
    int health_points;
    char *name;
    (d'autres choses sans importance ...)
} pet_t;

J'ai une fonction pour l'initialiser :

Code :

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

1
2
3
4
5
6
void init_pet(pet_t *pet)
{
    pet->health_points = 0;
    pet->name = NULL;
    ...
}

Je fais mes appels comme cela :

Code :

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

1
2
pet_t pet;
init_pet(&pet);

Le problème a lieu lorsque j'essaie de me mettre une valeur à name. Si je fais pet.name = "Le nom"; je me retrouve avec une erreur du compilo : "warning: assignment discards qualifiers from pointer target type". J'ai donc essayé avec strcpy(pet.name, "Le nom"); mais je me retrouve avec un magnifique SIGSEGV et un monumental plantage.

Je vous remercie par avance de l'aide que vous pourrez m'apporter,

Cordialement
Otacon

**Médinoc** · 16/04/2007, 17h24

Tu dois allouer une place pour le nom. Soit un buffer de taille fixe dans ta structure, soit une zone mémoire allouée dynamiquement avec malloc() ou strdup()...

**souviron34** · 16/04/2007, 17h29

quant à la fonction pour trouver les mots-clés et leurs valeurs, quelle horreur .....

Pour un fichier aussi simple.. si c'est pas malheureux de rendre un code simple aussi complexe...

**Emmanuel Delahaye** · 16/04/2007, 18h31

Envoyé par Otacon

Je développe en ce moment un programme qui accède à des fichiers de configuration grace à une fonction que j'ai trouvée et modifiée un peu et je rencontre un problème :

lorsque le caractère de fin de ligne est un simple LF (\n), pas de problèmes, tout ce passe bien
lorsque le caractère est un CR (\r), la fonction ne reconnait pas la fin de ligne et considère le fichier comme vide
lorsque c'est un CRLF (\r\n) la fonction rajoute un caractère NUL (\0) pour chaque ligne présente dans le fichier et ce à la fin.

Il y a 3 façons de traiter ce problème, qui n'est pas un problème de C...

1 - Traitement en amont. Convertir manuellement (ou automatiquement, script...) les fichiers textes au format natifs du système avec les outils traditionnels dos2unix, unix2dos etc. Le programme n'a plus affaire qu'à des fichiers textes 'normaux' (donc traités en mode texte).

2 - Variante du 1 : pré-traitement dans l'application : reconnaitre les terminaisons et convertir le fichier en conséquence en appelant l'outil de conversion qui va bien avec system()...

3 - Traitement dans le programme en mode binaire. On fait l'hypothèse que le fin de ligne est une des 4 formes possibles :

CR
LF
CR/LF
LF/CR

après avoir reconnu la première forme, on se base sur celle-ci pour déterminer la fin de ligne dans le reste du programme.

**Otacon** · 16/04/2007, 19h02

Tout d'abord merci pour vos réponses

!

Envoyé par Médinoc

Tu dois allouer une place pour le nom. Soit un buffer de taille fixe dans ta structure, soit une zone mémoire allouée dynamiquement avec malloc() ou strdup()...

D'accord merci, il me semblait bien que j'oubliais quelque chose

Envoyé par souviron34

quant à la fonction pour trouver les mots-clés et leurs valeurs, quelle horreur .....

Pour un fichier aussi simple.. si c'est pas malheureux de rendre un code simple aussi complexe...

Désolé mais comme je l'ai dit la fonction n'est pas de moi j'ai juste viré quelques trucs bizarres mais sinon je ne l'ai pas codé

Envoyé par Emmanuel Delahaye

Il y a 3 façons de traiter ce problème, qui n'est pas un problème de C...

1 - Traitement en amont. Convertir manuellement (ou automatiquement, script...) les fichiers textes au format natifs du système avec les outils traditionnels dos2unix, unix2dos etc. Le programme n'a plus affaire qu'à des fichiers textes 'normaux' (donc traités en mode texte).

2 - Variante du 1 : pré-traitement dans l'application : reconnaitre les terminaisons et convertir le fichier en conséquence en appelant l'outil de conversion qui va bien avec system()...

3 - Traitement dans le programme en mode binaire. On fait l'hypothèse que le fin de ligne est une des 4 formes possibles :

* CR
* LF
* CR/LF
* LF/CR

après avoir reconnu la première forme, on se base sur celle-ci pour déterminer la fin de ligne dans le reste du programme.

Tu veux donc dire que
1) soit je passe par des programmes style sed, unixtodos ... pour faire cela
2) soit je l'ouvre en mode binaire avec rb (ou wb) et que je recherche la première forme du caractère de fin en considérant qu'ils sont tous pareils ?

EDIT : j'ai essayé en ouvrant le fichier en binaire et ça marche la première fois sans problème. Après ça me rajoute ça : un $, un SOH, un ACK, un NUL, un NUL ...

En tout cas merci encore de me depanner

Cordialement
Otacon

**souviron34** · 16/04/2007, 23h52

Je pense plus sérieusement que tu ferais mieux de réfléchor et dd'exposer clairement ton problème, ce serait nettement plus simple..

La fonction que tu as récupérée est complexe et "geek-like" pour un problème simple..

Que veux-tu faire exactement ?

Tu veux lire le fichier de config ?

En tirer les valeurs ? Eventuellement les remodifier et le réécrire c'est ça ?

Les valeurs sont toujours du texte ?

**Otacon** · 17/04/2007, 01h13

Envoyé par souviron34

Je pense plus sérieusement que tu ferais mieux de réfléchor et dd'exposer clairement ton problème, ce serait nettement plus simple..

La fonction que tu as récupérée est complexe et "geek-like" pour un problème simple..

Que veux-tu faire exactement ?

Tu veux lire le fichier de config ?

En tirer les valeurs ? Eventuellement les remodifier et le réécrire c'est ça ?

Les valeurs sont toujours du texte ?

En fait, j'aimerais simplement pouvoir lire un fichier et en tirer des valeurs et les modifier/créer comme tu l'as dit.
Les valeurs seraient soit du texte, soit des valeurs entière (signées) soit des booléens.

Le fichier aurait cette forme par exemple :

Code :

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

1
2
3
4
5
6
7
8
9
# Commentaire
# chaine de caractère entre ""
Nom = "Le nom"
 
# valeur entière signée
Age = 14
 
# booléen
Vivant = TRUE

Bien sûr, il se peu qu'il y ai plus de lignes ^^

Il me faudrait donc : une fonction pour obtenir la valeur associé à un élément et une autre pour mettre à jour/créer des valeurs.

Merci encore une fois par avance

Cordialement
Otacon

**femtosa** · 17/04/2007, 08h46

Envoyé par Otacon

Il me faudrait donc : une fonction pour obtenir la valeur associé à un élément et une autre pour mettre à jour/créer des valeurs.

Et alors, tu y a réfléchi ...

Commence par réfléchir à la façon dont tu vas t'y prendre, fais un algorithme et code-le ! Quelqu'un t'aidera dans une de ces étapes, mais personne les fera pour toi ... !

Si t'as besoin d'aide hésite pas, mais pose une question précise et non un "voilà ce dont j'ai besoin"

**souviron34** · 17/04/2007, 12h30

et deux petites indications supplémentaires :

en C il existe le type void (ou en pointeur void* ) qui peut accepter n'importe quel type...
dans un tel fichier de config, mettre une chaîne entre guillemets n'est pas nécessaire.... et provoque des opérations inutiles..

Exemple de ce qu'on peut faire :

messagesFile : Messages.dat
archiveDirectory : /home/data/images/
archiverCommand : tar cvf
hasSwitchBoard : True
backgroundImage_topLeftLat : -4.79
internationalNumber : 00

A toi de chercher un peu...

**Otacon** · 17/04/2007, 12h58

Bon j'ai une idée et je vais la poster histoire de voir si je ne fais pas n'importe quoi ^^

On ouvre le fichier
On le lit ligne par ligne
Si la ligne commence par un #, un ; ou un autre genre de commentaire (avec possibilité qu'il y ait des espaces devant) on la passe
On continue jusqu'à ce qu'on trouve l'entrée que l'on veut
On lit la ligne jusqu'après le = ou un : comme dans l'exemple de souviron34 et le(s) espace(s) consécutifs pour on met ça dans un void * ou je le renvoie par return (plutôt pour les codes d'erreurs nan ?)
On ferme le fichier quand on a la valeur
Si on ne la trouve on renvoie une jolie erreur

Voilà pour la fonction qui lit. Celle qui écrit/met à jour devrait pas être trop différente.

Merci encore pour l'aide

PS : j'ai le droit d'écrire quelque chose du genre

Code :

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

1
2
void bidule = 1;
void truc[] = "Salut"

???

EDIT : sur la wikipédia anglophone j'ai trouvé : The language does not allow the use of the void type in variable declarations. :S

Cordialement, Otacon

**crocodilex** · 17/04/2007, 13h30

Envoyé par Otacon

PS : j'ai le droit d'écrire quelque chose du genre

Code :

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

1
2
void bidule = 1;
void truc[] = "Salut"

???

Ceci n'a pas de sens en langage C.

**femtosa** · 17/04/2007, 13h36

Envoyé par Otacon

On ouvre le fichier

Jusque là, on est bon

Envoyé par Otacon

On le lit ligne par ligne
Si la ligne commence par un #, un ; ou un autre genre de commentaire (avec possibilité qu'il y ait des espaces devant) on la passe

Si j'étais toi je me casserai pas la tête, je n'autoriserai qu'un symbole pour les commentaires ... A toi de voir !

Envoyé par Otacon

On continue jusqu'à ce qu'on trouve l'entrée que l'on veut
On lit la ligne jusqu'après le = ou un : comme dans l'exemple de souviron34 et le(s) espace(s) consécutifs pour on met ça dans un void * ou je le renvoie par return (plutôt pour les codes d'erreurs nan ?)

Si je traduit, ça donne : "une fois que la ligne commence par l'identifiant recherché, on récupère la donnée associée qui sera retournée via le paramètre reçu qui sera du type void*" C'est juste ... ?

Envoyé par Otacon

On ferme le fichier quand on a la valeur
Si on ne la trouve on renvoie une jolie erreur

Voilà une façon de faire qui pourrait marcher ! A ton clavier

**crocodilex** · 17/04/2007, 13h37

Envoyé par souviron34

en C il existe le type void (ou en pointeur void* ) qui peut accepter n'importe quel type...

Je ne pense pas que ce soit juste ce que tu as écrit. Ou alors tu t'es mal exprimé

PS : Je n'ai pas compris pourquoi vous voulez utiliser un pointeur générique. Le but n'est-il pas d'extraire des chaines de caratères à partir d'un fichier ?

**Otacon** · 17/04/2007, 13h45

Envoyé par crocodilex

Ceci n'a pas de sens en langage C.

C'est ce que je pensais aussi ^^'

Au pire je stocke les valeurs dans un pointeur sur un char[] et pi j'utilise les fonctions de conversion selon ce qu'il contient nan ?

Envoyé par femtosa

Si je traduit, ça donne : "une fois que la ligne commence par l'identifiant recherché, on récupère la donnée associée qui sera retournée via le paramètre reçu qui sera du type void*" C'est juste ... ?

Euh ... oui ça doit être ça en plus simple ^^

**crocodilex** · 17/04/2007, 13h57

Envoyé par Otacon

C'est ce que je pensais aussi ^^'
Au pire je stocke les valeurs dans un pointeur sur un char[] et pi j'utilise les fonctions de conversion selon ce qu'il contient nan ?

Si tu connais les paramètres que tu as à extraire de ton fichier, il suffit de faire un strcmp() sur ce paramètre puis une conversion (du style strtol(), strtod(), etc..). Evidemment, je simplifie un peu, mais bon....

**Otacon** · 17/04/2007, 14h08

Oui bah écoutez ça me semble pas mal pour démarer.
Merci encore une fois.

**souviron34** · 17/04/2007, 14h14

Envoyé par crocodilex

PS : Je n'ai pas compris pourquoi vous voulez utiliser un pointeur générique. Le but n'est-il pas d'extraire des chaines de caratères à partir d'un fichier ?

NON

le PO a mentionné :

Les valeurs seraient soit du texte, soit des valeurs entière (signées) soit des booléens.

Et ensuite pour :

Envoyé par crocodilex

Je ne pense pas que ce soit juste ce que tu as écrit. Ou alors tu t'es mal exprimé

Exemple :

Code :

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

1
2
3
4
5
6
7
8
 
int Define_Resource_Element ( char *RootName, 
                              char *Name, char *Type, 
			      void *Adr, void *DefaultValue );
 
Boolean Checks_If_Resource ( char *line,  
                             char *param_name, char *AppliName, 
		             char *param_type, void *address );

**Otacon** · 17/04/2007, 14h40

Hum je pensais plutot à

Code :

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

1
2
int get_conf(const char filename[], const char id[], void* value)
int get_conf(const char filename[], const char id[], void value)

puis le strtol(...) ou dans le même genre

Mais apparement void value c'est pas bon :S

**femtosa** · 17/04/2007, 14h49

Code :

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

int get_conf(const char[] filename, const char[] id, void* value)

Ceci te suffit ... C'est dans le dernier paramètre que tu recevras la valeur contenu dans ton fichier de config ... !

**crocodilex** · 17/04/2007, 16h29

Envoyé par Otacon

Code :

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

1
2
int get_conf(const char[] filename, const char[] id, void* value)
int get_conf(const char[] filename, const char[] id, void value)

Ces 2 lignes ne sont pas du C.

Problème de CRLF et de strcpy

C

Discussions similaires

Partager

Partager