Pointeurs, Structures etc....

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:

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:

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

dans mon code, j'appelle la fonction suivante:

Code:

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

avec:

Code:

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:

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

pourquoi !!???

merci pour votre aide !!

@+
Nico

Re: Pointeurs, Structures etc....

Citation:
Code:

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...

Citation:

une fonction dont voici la signature:

Code:

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

Pourquoi 'short' ?
Citation:
dans mon code, j'appelle la fonction suivante:
Code:

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

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:

memset (pstParametres, 0, sizeof *pstParametres);
Citation:
Code:

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.
Citation:
Code:

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:

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

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

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 ?

Re: Pointeurs, Structures etc....

Citation:
Envoyé par Emmanuel Delahaye
Citation:
Code:

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;

Citation:

Envoyé par Emmanuel Delahaye

Citation:

une fonction dont voici la signature:

Code:

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
Citation:
Envoyé par Emmanuel Delahaye
Citation:
dans mon code, j'appelle la fonction suivante:
Code:

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

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...

Citation:

Envoyé par Emmanuel Delahaye

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

oui, c'est .c
Citation:
Envoyé par Emmanuel Delahaye
Code:

memset (pstParametres, 0, sizeof *pstParametres);
Citation:
Code:

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 ??
Citation:
Envoyé par Emmanuel Delahaye
Citation:
Code:

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 ??
Citation:
Envoyé par Emmanuel Delahaye
Pourquoi & ? J'aurais mis

Code:

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

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

Tu ne testes pas le code retour (sRetour).
Citation:
Code:

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.

@+

Re: Pointeurs, Structures etc....

Citation:

Envoyé par kameha

Citation:

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.

Citation:

Envoyé par kameha

Citation:

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...

Citation:

Envoyé par kameha

Citation:

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 ?
Citation:
Envoyé par kameha
Citation:
Envoyé par Emmanuel Delahaye
Citation:
Envoyé par kameha
Code:

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.
Citation:
Envoyé par kameha
Citation:
Envoyé par Emmanuel Delahaye
Citation:
Envoyé par kameha
Code:

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:

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

Pourquoi & ? J'aurais mis

Code:

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 ?

Citation:

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:

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:

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 ?

http://mymarkup.net/mt/images/buttkicking.jpg

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

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)

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...

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

Citation:

Envoyé par Médinoc

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

Citation:

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...

Citation:

Envoyé par Emmanuel Delahaye

MAX_PATH, ok. C'est standard ?

Sous DOS/Windows, oui, ailleurs, Ça n'a pas trop l'air... :(

Code:

grep MAX_PATH /usr/include/*

Ne renvoit rien sous FreeBSD...

De plus, est-ce qu'il y a une limite sous nux?

Bonjour,

Citation:

Envoyé par Médinoc

Citation:

Envoyé par Emmanuel Delahaye

MAX_PATH, ok. C'est standard ?

Sous DOS/Windows, oui, ailleurs, Ça n'a pas trop l'air... :(

Code:

grep MAX_PATH /usr/include/*

Ne renvoit rien sous FreeBSD...

De plus, est-ce qu'il y a une limite sous nux?

FILENAME_MAX est standard! Inclure <stdio.h> ;)