Pointeurs, Structures etc....

**kameha** · 25/11/2005, 11h26

Salut !!

je dois ecrire un programme en c pour le boulot, et je me heurte au probleme suivants:

j'ai une structure que voici:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
typedef struct {
  char *pszWorkDir;
  char *pszTempDir;
  char *pszEntreeDir;
  char *pszSortieDir;
  char *pszArchiveDir;
  char *pszRejetDir;
  char *pszErreurDir;
  char *pszAnomalieDir;
  char *pszSeparateur;
} ParametresILAB;

une fonction dont voici la signature:

Code :

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

short lireMotCle(char *pszFichierIni, char *pszSection, char *pszMotCle, char *pszResult);

dans mon code, j'appelle la fonction suivante:

Code :

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

1
2
3
4
5
6
7
8
9
.
..
...
ParametresILAB pstParametres;
sRetour = initialiserParametres(pszNomFichier, &pstParametres);
printf("%s\r\n", pstParametres.pszTempDir);
.
..
...

avec:

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
short initialiserParametres (char *pszFileIni, ParametresILAB *pstParametres){
 
  short sRetour = CR_OK;
  memset ((void *)pstParametres, (int) NULL, sizeof (ParametresILAB));
 
  pstParametre->pszWorkDir = (char *) getenv("OCTALE_WORK");
  sRetour = lireMotCle(pszFileIni, SECTION, "REP_TMP", (char *)&pstParametre->pszTempDir);
  printf("%s\r\n", &pstParametre->pszTempDir);
  sRetour = lireMotCle(pszFileIni, SECTION, "REP_ENTRE", (char *)&pstParametre->pszEntreeDir);    
  printf("%s\r\n", &pstParametre->pszEntreeDir);
  sRetour = lireMotCle(pszFileIni, SECTION, "REP_SORTIE", (char *)&pstParametre->pszSortieDir);
  printf("%s\r\n", &pstParametre->pszSortieDir);
  sRetour = lireMotCle(pszFileIni, SECTION, "REP_ARCHIVE", (char *)&pstParametre->pszArchiveDir);
  printf("%s\r\n", &pstParametre->pszArchiveDir);
  sRetour = lireMotCle(pszFileIni, SECTION, "REP_REJET", (char *)&pstParametre->pszRejetDir);
  printf("%s\r\n", &pstParametre->pszrejetDir);
  sRetour = lireMotCle(pszFileIni, SECTION, "REP_ERREUR", (char *)&pstParametre->pszErreurDir);
  printf("%s\r\n", &pstParametre->pszErreurDir);
  sRetour = lireMotCle(pszFileIni, SECTION, "REP_ANOMALIE", (char *)&pstParametre->pszAnomalieDir);
  printf("%s\r\n", &pstParametre->pszAnomalieDir);
  sRetour = lireMotCle(pszFileIni, SECTION, "SEPARATEUR", (char *)&pstParametre->pszSeparateur);
  printf("%s\r\n", &pstParametre->pszSeparateur);
 
  return (sRetour);
}

Voici le resultat afficher:

Code :

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

1
2
3
4
5
6
7
8
9
TMP
ENTREE
SORTIE
ARCHIVE
REJET
ERREUR
ANOMALIE
;
Memory fault

pourquoi !!???

merci pour votre aide !!

@+
Nico

**Emmanuel Delahaye** · 25/11/2005, 11h40

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
typedef struct {
  char *pszWorkDir;
  char *pszTempDir;
  char *pszEntreeDir;
  char *pszSortieDir;
  char *pszArchiveDir;
  char *pszRejetDir;
  char *pszErreurDir;
  char *pszAnomalieDir;
  char *pszSeparateur;
} ParametresILAB;

Belle collection de pointeurs non initialisés. Voyons la suite...

une fonction dont voici la signature:

Code :

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

short lireMotCle(char *pszFichierIni, char *pszSection, char *pszMotCle, char *pszResult);

Pourquoi 'short' ?

dans mon code, j'appelle la fonction suivante:

Code :

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

1
2
3
4
 
ParametresILAB pstParametres;
sRetour = initialiserParametres(pszNomFichier, &pstParametres);
printf("%s\r\n", pstParametres.pszTempDir);

avec:

Code :

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

1
2
3
4
5
 
short initialiserParametres (char *pszFileIni, ParametresILAB *pstParametres){
 
  short sRetour = CR_OK;
  memset ((void *)pstParametres, (int) NULL, sizeof (ParametresILAB));

C'est tordu ! C'est quoi tous ces castss. T'es sûr que tu compiles en C ? L'extension du fichier source est bien .c et non ..cpp ni .C ?

Code :

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

  memset (pstParametres, 0, sizeof *pstParametres);

Code :

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

1
2
 
  pstParametre->pszWorkDir = (char *) getenv("OCTALE_WORK");

Pourquoi pas. Il faut juste garder en mémoire que le pointeur peut être NULL. Il suffit que la variable n'existe pas.

Code :

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

1
2
3
 
  sRetour = lireMotCle(pszFileIni, SECTION, "REP_TMP", (char *)&pstParametre->pszTempDir);
  printf("%s\r\n", &pstParametre->pszTempDir);

Quelle est l'interface de lireMotCle() ? Pourquoi & ? J'aurais mis

Code :

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

  printf("%s\r\n", pstParametre->pszTempDir);

Et puis, là aussi, le pointeur peut être NULL. Tu ne testes pas le code retour (sRetour).

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
 
  sRetour = lireMotCle(pszFileIni, SECTION, "REP_ENTRE", (char *)&pstParametre->pszEntreeDir);    
  printf("%s\r\n", &pstParametre->pszEntreeDir);
  sRetour = lireMotCle(pszFileIni, SECTION, "REP_SORTIE", (char *)&pstParametre->pszSortieDir);
  printf("%s\r\n", &pstParametre->pszSortieDir);
  sRetour = lireMotCle(pszFileIni, SECTION, "REP_ARCHIVE", (char *)&pstParametre->pszArchiveDir);
  printf("%s\r\n", &pstParametre->pszArchiveDir);
  sRetour = lireMotCle(pszFileIni, SECTION, "REP_REJET", (char *)&pstParametre->pszRejetDir);
  printf("%s\r\n", &pstParametre->pszrejetDir);
  sRetour = lireMotCle(pszFileIni, SECTION, "REP_ERREUR", (char *)&pstParametre->pszErreurDir);
  printf("%s\r\n", &pstParametre->pszErreurDir);
  sRetour = lireMotCle(pszFileIni, SECTION, "REP_ANOMALIE", (char *)&pstParametre->pszAnomalieDir);
  printf("%s\r\n", &pstParametre->pszAnomalieDir);
  sRetour = lireMotCle(pszFileIni, SECTION, "SEPARATEUR", (char *)&pstParametre->pszSeparateur);
  printf("%s\r\n", &pstParametre->pszSeparateur);
 
  return (sRetour);
}

Voici le resultat afficher:
TMP
ENTREE
SORTIE
ARCHIVE
REJET
ERREUR
ANOMALIE
;
Memory fault

pourquoi !!???

Comportement indéfini ? Pointeur NULL ?

**kameha** · 25/11/2005, 14h57

Envoyé par Emmanuel Delahaye

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
 
typedef struct {
  char *pszWorkDir;
  char *pszTempDir;
  char *pszEntreeDir;
  char *pszSortieDir;
  char *pszArchiveDir;
  char *pszRejetDir;
  char *pszErreurDir;
  char *pszAnomalieDir;
  char *pszSeparateur;
} ParametresILAB;

Belle collection de pointeurs non initialisés. Voyons la suite...

-> genre j'ai le droit de faire ?????????
typedef struct {
char *pszWorkDir = NULL;
char *pszTempDir = NULL;
char *pszEntreeDir = NULL;
char *pszSortieDir = NULL;
char *pszArchiveDir = NULL;
char *pszRejetDir = NULL;
char *pszErreurDir = NULL;
char *pszAnomalieDir = NULL;
char *pszSeparateur = NULL;
} ParametresILAB;

Envoyé par Emmanuel Delahaye

une fonction dont voici la signature:

Code :

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

short lireMotCle(char *pszFichierIni, char *pszSection, char *pszMotCle, char *pszResult);

Pourquoi 'short' ?

-> bah je sais pas, c'est une fonction déjà écrite, alors c'est du short qu'elle renvoie

Envoyé par Emmanuel Delahaye

dans mon code, j'appelle la fonction suivante:

Code :

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

1
2
3
4
 
ParametresILAB pstParametres;
sRetour = initialiserParametres(pszNomFichier, &pstParametres);
printf("%s\r\n", pstParametres.pszTempDir);

avec:

Code :

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

1
2
3
4
5
 
short initialiserParametres (char *pszFileIni, ParametresILAB *pstParametres){
 
  short sRetour = CR_OK;
  memset ((void *)pstParametres, (int) NULL, sizeof (ParametresILAB));

C'est tordu ! C'est quoi tous ces castss. T'es sûr que tu compiles en C ?

-> bah si je cast pas, il me met des warning sur le type des pointeurs...

Envoyé par Emmanuel Delahaye

L'extension du fichier source est bien .c et non ..cpp ni .C ?

oui, c'est .c

Envoyé par Emmanuel Delahaye

Code :

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

  memset (pstParametres, 0, sizeof *pstParametres);

Code :

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

1
2
 
  pstParametre->pszWorkDir = (char *) getenv("OCTALE_WORK");

Pourquoi pas. Il faut juste garder en mémoire que le pointeur peut être NULL. Il suffit que la variable n'existe pas.

-> explications ??

Envoyé par Emmanuel Delahaye

Code :

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

1
2
3
 
  sRetour = lireMotCle(pszFileIni, SECTION, "REP_TMP", (char *)&pstParametre->pszTempDir);
  printf("%s\r\n", &pstParametre->pszTempDir);

Quelle est l'interface de lireMotCle() ?

-> interface ?? ou signature ??

Envoyé par Emmanuel Delahaye

Pourquoi & ? J'aurais mis

Code :

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

  printf("%s\r\n", pstParametre->pszTempDir);

Et puis, là aussi, le pointeur peut être NULL.

Tu ne testes pas le code retour (sRetour).

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
[/quote]
 
-> pas besoin pour le moment de tester le code retour...
 
-> bah si je met pas &, j'ai un Memory fault
 
[quote="Emmanuel Delahaye"]  sRetour = lireMotCle(pszFileIni, SECTION, "REP_ENTRE", (char *)&pstParametre->pszEntreeDir);    
  printf("%s\r\n", &pstParametre->pszEntreeDir);
  sRetour = lireMotCle(pszFileIni, SECTION, "REP_SORTIE", (char *)&pstParametre->pszSortieDir);
  printf("%s\r\n", &pstParametre->pszSortieDir);
  sRetour = lireMotCle(pszFileIni, SECTION, "REP_ARCHIVE", (char *)&pstParametre->pszArchiveDir);
  printf("%s\r\n", &pstParametre->pszArchiveDir);
  sRetour = lireMotCle(pszFileIni, SECTION, "REP_REJET", (char *)&pstParametre->pszRejetDir);
  printf("%s\r\n", &pstParametre->pszrejetDir);
  sRetour = lireMotCle(pszFileIni, SECTION, "REP_ERREUR", (char *)&pstParametre->pszErreurDir);
  printf("%s\r\n", &pstParametre->pszErreurDir);
  sRetour = lireMotCle(pszFileIni, SECTION, "REP_ANOMALIE", (char *)&pstParametre->pszAnomalieDir);
  printf("%s\r\n", &pstParametre->pszAnomalieDir);
  sRetour = lireMotCle(pszFileIni, SECTION, "SEPARATEUR", (char *)&pstParametre->pszSeparateur);
  printf("%s\r\n", &pstParametre->pszSeparateur);
 
  return (sRetour);
}

Voici le resultat afficher:
TMP
ENTREE
SORTIE
ARCHIVE
REJET
ERREUR
ANOMALIE
;
Memory fault

pourquoi !!???

Comportement indéfini ? Pointeur NULL ?

petite précision, je développe a distance (via une session telnet) sur une machine UNIX.

@+

**Emmanuel Delahaye** · 25/11/2005, 15h56

Envoyé par kameha

Envoyé par Emmanuel Delahaye

Belle collection de pointeurs non initialisés. Voyons la suite...

-> genre j'ai le droit de faire ?????????
typedef struct {
char *pszWorkDir = NULL;
char *pszTempDir = NULL;
char *pszEntreeDir = NULL;
char *pszSortieDir = NULL;
char *pszArchiveDir = NULL;
char *pszRejetDir = NULL;
char *pszErreurDir = NULL;
char *pszAnomalieDir = NULL;
char *pszSeparateur = NULL;
} ParametresILAB;

Non. Simplement, on s'attend quelque part à voir ces pointeurs de faire initialiser. C'est tout. Simple reflexe de celui qui lit ce genre de code.

Envoyé par kameha

Envoyé par Emmanuel Delahaye

C'est tordu ! C'est quoi tous ces casts. T'es sûr que tu compiles en C ?

-> bah si je cast pas, il me met des warning sur le type des pointeurs...

Envoyé par kameha

Envoyé par Emmanuel Delahaye

L'extension du fichier source est bien .c et non ..cpp ni .C ?

oui, c'est .c

Alors il y a un problème. Quel est ton compilateur ?

Envoyé par kameha

Envoyé par Emmanuel Delahaye

Envoyé par kameha

Code :

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

1
2
 
  pstParametre->pszWorkDir = (char *) getenv("OCTALE_WORK");

Pourquoi pas. Il faut juste garder en mémoire que le pointeur peut être NULL. Il suffit que la variable n'existe pas.

-> explications ??

La variable d'environnement OCTALE_WORK n'est pas forcément définie. getenv() peut très bien retourner NULL.

Envoyé par kameha

Envoyé par Emmanuel Delahaye

Envoyé par kameha

Code :

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

1
2
3
 
  sRetour = lireMotCle(pszFileIni, SECTION, "REP_TMP", (char *)&pstParametre->pszTempDir);
  printf("%s\r\n", &pstParametre->pszTempDir);

Quelle est l'interface de lireMotCle() ? Je me réponds tout seul, (et je me réveille, par la même occasion...)

Code :

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

short lireMotCle(char *pszFichierIni, char *pszSection, char *pszMotCle, char *pszResult);

Pourquoi & ? J'aurais mis

Code :

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

  printf("%s\r\n", pstParametre->pszTempDir);

Et puis, là aussi, le pointeur peut être NULL.

Tu ne testes pas le code retour (sRetour).

-> pas besoin pour le moment de tester le code retour...

Ah ? Et si la donnée du fichier n'existe pas ? La fonction met quoi dans la variable ?

Envoyé par kameha

-> bah si je met pas &, j'ai un Memory fault

L'erreur est ailleurs...

A mon avis, il faut appeler la fonction comme ça :

Code :

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

1
2
3
4
5
6
7
8
9
 
   sRetour = lireMotCle (pszFileIni
                                  , SECTION
                                  , "REP_TMP"
                                  , pstParametre->pszTempDir);
   if (sRetour == CR_OK)
   {
      printf ("%s\n", pstParametre->pszTempDir);
   }

Avec

Code :

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

1
2
3
4
5
6
 
typedef struct {
  char *pszWorkDir;
  char pszTempDir[256];
<...>
} ParametresILAB;

J'ajoute que l'interface (la signature) de la fonction lireMotCle() est faible. En effet il n'y a pas de paramètre indiquand la taille de la chaine de destination. J'ai mis (au pif) une taille de 256 bytes, mais il y a toujours un risque de débordement. C'est pas bon du tout.

Qui a écrit cette fonction que je lui botte les fesses ?

PS : la notation hongroise, à la Microsoft, ça me sort par les yeux...

**Médinoc** · 25/11/2005, 22h48

Plutôt que 256, je dirais plutôt 260 (MAX_PATH)

Et pour la notation hongroise, bien utilisée, elle a ses avantages... (Surtout pour lire le code d'un autre, et parfois même son propre code quand on n'a pas bossé dessus depuis un moment)

**red210** · 26/11/2005, 02h31

compile avec -W -Wall -ansi -pedantic -Werror...

Si ca compile deja tu auras beaucoup plus de chance que ca fonctionne...

Ensuite comme t'as dis kameha moi je pencherai avant toute chose sur les pointeurs non initialises...

**Médinoc** · 26/11/2005, 10h10

Ajoute -Wextra et -Wwrite-strings qui sont souvent utiles...

**Emmanuel Delahaye** · 26/11/2005, 11h54

Envoyé par Médinoc

Plutôt que 256, je dirais plutôt 260 (MAX_PATH)

MAX_PATH, ok. C'est standard ?

Et pour la notation hongroise, bien utilisée, elle a ses avantages... (Surtout pour lire le code d'un autre, et parfois même son propre code quand on n'a pas bossé dessus depuis un moment)

Comme ça, oui

http://emmanuel-delahaye.developpez....ge.htm#nommage

mais pas 'à-la-Microsoft'. Illisible, inmaintenable...