[sortie standard] problème d'affichage

**beekeep** · 22/02/2008, 16h25

Rebonjour à tous,

j'ai un problème d'affichage des sauts de ligne et des tabulations avec la fonction printf.

en sortie, j'obtiens une chaine comprenant des \n et \t textuellement au lieux d'avoir des sauts et tabulations.

je récupère les chaines à partir d'un fichier texte (fgets) et les stock dans des tableaux.

ensuite j'ai une fonction qui va chercher la chaine en fonction d'une clé et qui la retourne,
ce qui donne :

Code :

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

printf(getMessage("key"));

j'ai deux petites choses qui ne vont pas aussi:
- la chaine comprend toujours le saut de ligne ajouté à la fin par fgets
- j'ai un warning qui me dit "warning: improper pointer/integer combination: arg #1" alors que ma fonction retourne bien un char *

Merci d'avance

**homeostasie** · 22/02/2008, 16h53

Envoyé par beekeep

j'ai un problème d'affichage des sauts de ligne et des tabulations avec la fonction printf.

en sortie, j'obtiens une chaine comprenant des \n et \t textuellement au lieux d'avoir des sauts et tabulations.

je récupère les chaines à partir d'un fichier texte (fgets) et les stock dans des tableaux.

Peux tu montrer un extrait de ton fichier?

Envoyé par beekeep

la chaine comprend toujours le saut de ligne ajouté à la fin par fgets

A toi de mettre un caractère de fin de chaine à la place du saut de ligne.

Envoyé par =beekeep

j'ai un warning qui me dit "warning: improper pointer/integer combination: arg #1" alors que ma fonction retourne bien un char *

Peux tu montrer le code?

**beekeep** · 22/02/2008, 17h03

Envoyé par homeostasie

Peux tu montrer un extrait de ton fichier?

fichier texte classique (clé/valeur):

[Msg_INFO_Msg19]
\n \t\t%s for the program %d,

---

Envoyé par homeostasie

Peux tu montrer le code?

Code :

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

printf(getMessage("Msg_INFO_Msg19"));

Code :

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

1
2
3
4
5
6
char * getMessage(char * key)
{
  char ** tab = NULL;
  ..
  return tab[i];
}

ça t'aide ?

**diogene** · 22/02/2008, 17h15

Tu n'as pas de warning indiquant l'absence de prototype de getMessage au moment du printf ?

**Trap D** · 22/02/2008, 17h29

Ben tu retournes une adresse tab[i], on se sait pas comment est allouée cette adresse si elle est locale où si elle correspond à une adresse allouée par malloc, donc il faut plus de détails.

**homeostasie** · 22/02/2008, 17h31

Code :

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

1
2
3
4
5
6
char * getMessage(char * key)
{
  char ** tab = NULL;
  ..
  return tab[i];
}

Le prototype serait plutot:

Code :

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

1
2
3
4
5
6
char * getMessage(char const * key)
{
  char ** tab = NULL;
  ..
  return tab[i];
}

**beekeep** · 22/02/2008, 17h32

Envoyé par diogene

Tu n'as pas de warning indiquant l'absence de prototype de getMessage au moment du printf ?

non, il manque le prototype dans un .h c'est ça ?

Envoyé par Trap D

Ben tu retournes une adresse tab[i], on se sait pas comment est allouée cette adresse si elle est locale où si elle correspond à une adresse allouée par malloc, donc il faut plus de détails.

elle correspond à une adresse allouée avec malloc.

**Trap D** · 22/02/2008, 17h34

OK, et que copies-tu dedans.
Au fait, comment est alloué tab ?
Tu n'aurais pas des problèmes de fuite mémoire par hasard quelque part ?

**diogene** · 22/02/2008, 17h37

Envoyé par beekeep

non, il manque le prototype dans un .h c'est ça ?

Le prototype de getMessage est-il connu au moment du printf (getMessage définie avant le printf ou présence avant le printf d'un prototype)

**beekeep** · 22/02/2008, 17h41

Envoyé par Trap D

OK, et que copies-tu dedans.
Au fait, comment est alloué tab ?
Tu n'aurais pas des problèmes de fuite mémoire par hasard quelque part ?

tab est alloué comme dans la FAQ (voir le post : http://www.developpez.net/forums/showthread.php?t=496331 )

il n'y a pas de problème de mémoire, les chaines sont bien récupérés et tout (j'ai testé)

Envoyé par diogene

Le prototype de getMessage est-il connu au moment du printf ?

getMessage est définie dans un autre fichier

**diogene** · 22/02/2008, 17h48

Envoyé par beekeep

getMessage est définie dans un autre fichier

Ca ne répond pas à ma question.
Alors, getMessage at-il un prototype dans un .h qui est inclus au début du fichier .c où se trouve le printf en question ?

**Trap D** · 22/02/2008, 17h49

Oui donc, comme tab est une variible locale à la fonction, il y a bien fuite mémoire puisque celle-ci n'est pas désallouée avant la sortie de la fonction.
Il faut revoir ton code.

**beekeep** · 22/02/2008, 17h51

toutes les sources (même si il me parait pas que ce soit utile) : (3 fichiers différents)

Code :

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

1
2
3
4
5
6
7
8
9
/* Tableaux des messages */
char ** oMessagesError;
char ** oMessagesInfo;
char ** oMessagesGUI;
char ** oMessagesReport;
char ** oMessagesIdError;
char ** oMessagesIdInfo;
char ** oMessagesIdGUI;
char ** oMessagesIdReport;

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
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
241
242
243
244
245
246
247
248
249
250
251
252
253
int oInitMessages( void ) 
{
  FILE * pFileToRead = NULL;
  char line[512];
  int i = 0;
  int iCurErrorMsg = 0;
  int iCurInfoMsg = 0;
  int iCurGUIMsg = 0;
  int iCurReportMsg = 0;
  int iMsgType = 0;
 
  iNbErrorMsg = 0;
  iNbInfoMsg = 0;
  iNbGUIMsg = 0;
  iNbReportMsg = 0;
 
#ifdef LANG_FR
  printf("---> FR\n\n");
  pFileToRead = fopen("messagesFR.conf","r");
#else
  printf("---> ENG\n\n");
  pFileToRead = fopen("messagesENG.conf","r");
#endif
 
  if(!pFileToRead)
  {
    printf("IMPOSSIBLE D'ACCEDER AU FICHIER CONTENANT LES MESSAGES.\n");
    return 0;
  }
 
  while(fgets(line, sizeof(line), pFileToRead) != NULL)
  {
 
    if(strcmp(line,"\n") != NULL && line[0] != '#')
    {
      if(line[0] == '[')
      {
        if(strstr(line,"_ERROR_") != NULL)
        {
          iNbErrorMsg = iNbErrorMsg + 1;
        }else{
          if(strstr(line,"_INFO_") != NULL)
          {
            iNbInfoMsg = iNbInfoMsg + 1;
          }else{
            if(strstr(line,"_GUI_") != NULL)
            {
              iNbGUIMsg = iNbGUIMsg + 1;
            }else{
              if(strstr(line,"_REPORT_") != NULL)
              {
                iNbReportMsg = iNbReportMsg + 1;
              }
            }
          }
        }
      }
    }
  }
  oMessagesIdError = malloc(iNbErrorMsg * sizeof(char *));
  oMessagesError = malloc(iNbErrorMsg * sizeof(char *));
  for(i=0; i<= iNbErrorMsg; i++)
  {
    oMessagesIdError[i] = malloc( 512*sizeof(char) );
    oMessagesError[i] = malloc( 512*sizeof(char) );
  }
  oMessagesIdInfo = malloc(iNbInfoMsg * sizeof(char *));
  oMessagesInfo = malloc(iNbInfoMsg * sizeof(char *));
  for(i=0; i<= iNbInfoMsg; i++)
  {
    oMessagesIdInfo[i] = malloc( 512*sizeof(char));
    oMessagesInfo[i] = malloc( 512*sizeof(char));
  }
  oMessagesIdGUI = malloc(iNbGUIMsg * sizeof(char *));
  oMessagesGUI = malloc(iNbGUIMsg * sizeof(char *));
  for(i=0; i<= iNbGUIMsg; i++)
  {
    oMessagesIdGUI[i] = malloc( 512*sizeof(char));
    oMessagesGUI[i] = malloc( 512*sizeof(char));
  }
  oMessagesIdReport = malloc(iNbReportMsg * sizeof(char *));
  oMessagesReport = malloc(iNbReportMsg * sizeof(char *));
  for(i=0; i<= iNbReportMsg; i++)
  {
    oMessagesIdReport[i] = malloc( 512*sizeof(char));
    oMessagesReport[i] = malloc( 512*sizeof(char));
  }
 
  if(oMessagesError == NULL || oMessagesInfo == NULL ||
     oMessagesGUI == NULL || oMessagesReport == NULL ||
     oMessagesIdError == NULL || oMessagesIdInfo == NULL ||
     oMessagesIdGUI == NULL || oMessagesIdReport == NULL)
  {
    if(oMessagesIdError != NULL)
      free(oMessagesIdError);
 
    if(oMessagesIdGUI != NULL)
      free(oMessagesIdGUI);
 
    if(oMessagesIdReport != NULL)
      free(oMessagesIdReport);
 
    if(oMessagesIdInfo != NULL)
      free(oMessagesIdInfo);
 
    if(oMessagesError != NULL)
      free(oMessagesError);
 
    if(oMessagesGUI != NULL)
      free(oMessagesGUI);
 
    if(oMessagesReport != NULL)
      free(oMessagesReport);
 
    if(oMessagesInfo != NULL)
      free(oMessagesInfo);
 
    printf("IMPOSSIBLE D'ALLOUER LA MEMOIRE.\n");
    return 0;
  }
 
#ifdef LANG_FR
  printf("---> FR\n\n");
  pFileToRead = fopen("messagesFR.conf","r");
#else
  printf("---> ENG\n\n");
  pFileToRead = fopen("messagesENG.conf","r");
#endif
 
  if(!pFileToRead)
  {
    printf("IMPOSSIBLE D'ACCEDER AU FICHIER CONTENANT LES MESSAGES.\n");
    return 0;
  }
 
  while(fgets(line, sizeof(line), pFileToRead) != NULL)
  {
 
    if(strcmp(line,"\n") != NULL && line[0] != '#')
    {
      if(line[0] == '[')
      {
        if(strstr(line,"_ERROR_") != NULL)
        {          
          strcpy(oMessagesIdError[iCurErrorMsg], line);
          /*
          printf("id: %i\n", iCurErrorMsg);
          printf("key: %s\n", oMessagesIdError[iCurErrorMsg]);*/
 
          iCurErrorMsg++;
          iMsgType = 1;
        }else{
          if(strstr(line,"_INFO_") != NULL)
          {
            strcpy(oMessagesIdInfo[iCurInfoMsg], line);
            iCurInfoMsg++;
            iMsgType = 2;
          }else{
            if(strstr(line,"_GUI_") != NULL)
            {
              strcpy(oMessagesIdGUI[iCurGUIMsg], line);
              iCurGUIMsg++;
              iMsgType = 3;
            }else{
              if(strstr(line,"_REPORT_") != NULL)
              {
                strcpy(oMessagesIdReport[iCurReportMsg], line);
                iCurReportMsg++;
                iMsgType = 4;
              }
            }
          }
        }
      }else{
        char *p = strrchr (line, '\n');
        if (p)
        {
            *p = 0;
        }
        if(iMsgType == 1)
        {
          strcpy(oMessagesError[iCurErrorMsg-1], line);
        }else{
          if(iMsgType == 2)
          {
            strcpy(oMessagesInfo[iCurInfoMsg-1], line);
          }else{
            if(iMsgType == 3)
            {
              strcpy(oMessagesGUI[iCurGUIMsg-1], line);
            }else{
              if(iMsgType == 4)
              {
                strcpy(oMessagesReport[iCurReportMsg-1], line);
              }
            }
          }
        }
      }
    }
  }
 
  return 1;
}
 
char * getMessage(char * key)
{
  char ** pTabId = NULL;
  char ** pTabVal = NULL;
  int i;
  int iNbMsg;
 
  if(strstr(key,"_ERROR_") != NULL)
  {
    pTabId = oMessagesIdError;
    pTabVal = oMessagesError;
    iNbMsg = iNbErrorMsg;
  }else{
    if(strstr(key,"_INFO_") != NULL)
    {
      pTabId = oMessagesIdInfo;
      pTabVal = oMessagesInfo;
      iNbMsg = iNbInfoMsg;
    }else{
      if(strstr(key,"_GUI_") != NULL)
      {
        pTabId = oMessagesIdGUI;
        pTabVal = oMessagesGUI;
        iNbMsg = iNbGUIMsg;
      }else{
        if(strstr(key,"_REPORT_") != NULL)
        {
          pTabId = oMessagesIdReport;
          pTabVal = oMessagesReport;
          iNbMsg = iNbReportMsg;
        }
      }
    }
  }
 
  if(pTabId != NULL && pTabVal != NULL)
  {
    for(i=0;i<iNbMsg;i++)
    {
      if(strstr(pTabId[i], key) != NULL)
      {
        return pTabVal[i];
      }
    }
  }
 
  return "message not found";
}

Code main :

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

1
2
if(oInitMessages())   {     printf(getMessage("oMsgErrBilanDetRes_INFO_Msg18"));
  }

**beekeep** · 22/02/2008, 17h53

Envoyé par diogene

getMessage at-il un prototype dans un .h qui est inclus au début du fichier .c où se trouve le printf en question ?

non.

Envoyé par Trap D

Oui donc, comme tab est une variible locale à la fonction, il y a bien fuite mémoire puisque celle-ci n'est pas désallouée avant la sortie de la fonction.
Il faut revoir ton code.

en fait, tab pointe juste sur un autre tableau global lui. (voir les sources > fct getMessage)

**Emmanuel Delahaye** · 22/02/2008, 18h05

Envoyé par beekeep

Code :

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

printf(getMessage("key"));

On ne fait jamais ça. On passe toujours par la chaine de formatage :

Code :

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

printf("%s", getMessage("key"));

sinon, certains caractères pourraient être interprétés de travers par printf().

**diogene** · 22/02/2008, 18h06

Il faut déclarer ce prototype, sinon, la fonction getMessage, inconnue au moment de son utilisation, va être supposée par le compilateur comme retournant un int -> d'où le warning. Ton compilateur ne doit pas être bien réglé car il aurait du te signaler cela.

Maintenant, pour l'anomalie de fonctionnement, ça va être coton parce que le code est bien touffu.

**beekeep** · 22/02/2008, 18h08

Envoyé par Emmanuel Delahaye

On ne fait jamais ça. On passe toujours par la chaine de formatage :

Code :

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

printf("%s", getMessage("key"));

sinon, certains caractères pourraient être interprétés de travers par printf().

oui, mais les messages retournés par getMessage contiennent aussi des %s, %d ..

en faisant comme ça je pourrai passer les variables à afficher comme avant ?

**beekeep** · 22/02/2008, 18h11

Envoyé par diogene

Il faut déclarer ce prototype, sinon, la fonction getMessage, inconnue au moment de son utilisation, va être supposée par le compilateur comme retournant un int -> d'où le warning. Ton compilateur ne doit pas être bien réglé car il aurait du te signaler cela.

Maintenant, pour l'anomalie de fonctionnement, ça va être coton parce que le code est bien touffu.

ok, merci beaucoup.

en fait, dans le code deux lignes suffisent à cerner le problème :

Code :

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

1
2
3
4
  while(fgets(line, sizeof(line), pFileToRead) != NULL)
  {
    printf("%s", line);
  }

si la ligne lue contient un \n, il s'affiche tel quel.

**Trap D** · 22/02/2008, 22h52

Envoyé par beekeep

oui, mais les messages retournés par getMessage contiennent aussi des %s, %d ..

en faisant comme ça je pourrai passer les variables à afficher comme avant ?

Dans ce cas, il me semble qu'il faut faire

Code :

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

printf(getMessage("key"), <les arguments nécessaires>);

si tu sais à chaque fois les arguments à envoyer.

**beekeep** · 23/02/2008, 15h40

Salut et merci,

Envoyé par Trap D

Dans ce cas, il me semble qu'il faut faire

Code :

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

printf(getMessage("key"), <les arguments nécessaires>);

si tu sais à chaque fois les arguments à envoyer.

oui, j'avais testé cette fonctionnalité qui marche très bien.

il faut vraiment que je trouve une solution pour le reste.

[sortie standard] problème d'affichage

C

Discussions similaires

Partager

Partager