Discussion :

[kameha]

Salut à tous,

j'ai le programme suivant:

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
typedef struct {
        char *pszCheminCommun;
        char *pszCheminHome;
} ParametresLocaux;
 
 
int main(......){
 
   int iRet = 0;
   ParametresLocaux stParametre;
 
   iRet = foo( &stParametre );
 
}
 
int foo( ParametresLocaux *pstParametres){
   char szTest[]="TATA|TITI|TOTO";
 
   memset ((void *)pstParametre, (int) NULL, sizeof (ParametresLocaux));
 
   // La ligne suivante (A) ne fonctionne pas
   pstParametre->pszCheminCommun = strtok( szTest, "|" );
 
   // La ligne suivante (B) ne fonctionne pas
   strcpy( pstParametre->pszCheminCommun, strtok( szTest, "|" ) );
 
   // La ligne suivante (C) fonctionne
   pstParametre->pszCheminHome = "Toto";
 
   return 0;
 
}

je ne comprend pas pourquoi les lignes A et B ne fonctionnent pas, alors que la ligne C fonctionne ???


d'avance merci

[JulienT]

t'es sur que c'est pas ton

strtok( szTest, "|" )

qui mer..?

[Skyounet]

t'es sur que c'est pas ton

strtok( szTest, "|" )

qui mer..?

Mouais verifie si le retour n'est pas egal a NULL

[kameha]

t'es sur que c'est pas ton

strtok( szTest, "|" )

qui mer..?

Mouais verifie si le retour n'est pas egal a NULL

bah comment il pourrait être NULL alors que dans szTest j'ai "TATA|TITI|TOTO" séparé par des "|" ???

[Skyounet]

t'es sur que c'est pas ton

strtok( szTest, "|" )

qui mer..?

Mouais verifie si le retour n'est pas egal a NULL

bah comment il pourrait être NULL alors que dans szTest j'ai "TATA|TITI|TOTO" séparé par des "|" ???

bah je vois bien mais on ne sait jamais pour moi le code est correct alors maintenant p-e que l'appel a strtok echou...

[Skyounet]

Une question toute bête

Code :

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

1
2
3
4
5
6
7
 
 
int foo( ParametresLocaux *pstParametres){
[...]
   // La ligne suivante (A) ne fonctionne pas
   pstParametre->pszCheminCommun = strtok( szTest, "|" );
[...]

Le parametre c'est pstParametres

et apres tu utilises
pstParametre

Faute de frappe ou erreur tout court?

[Mokhtar BEN MESSAOUD]

(B) ne fonctionnera pas car tu copie une chaine dans pszCheminCommun qui n'est pas alloué

[Mokhtar BEN MESSAOUD]

(A) ne fonctionnera pas car tu mets dans pszCheminCommun
l'adresse d'une variable locale qui sera detruite a la sortie de la fonction
foo

[Mokhtar BEN MESSAOUD]

voici le code corrige

la variable szTest doit etre declaree comme globale

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
#include <stdlib.h>
#include <string.h>
 
 
typedef struct { 
        char *pszCheminCommun; 
        char *pszCheminHome; 
} ParametresLocaux; 
 
 char   szTest[]="TATA|TITI|TOTO";   
int main(){ 
 
   int iRet = 0; 
   ParametresLocaux stParametre; 
 
   iRet = foo( &stParametre ); 
 
   printf ("pszCheminCommun = %s\n",stParametre.pszCheminCommun); 
   printf (" pszCheminHome= %s\n",stParametre.pszCheminHome  ); 
 
 
} 
 
int foo( ParametresLocaux *pstParametres){ 
 
   memset ((void *)pstParametres, (int) NULL, sizeof (ParametresLocaux)); 
 
   // La ligne suivante (A) ne fonctionne pas 
   pstParametres->pszCheminCommun = strtok( (char *)szTest, "|" ); 
 
 
   // La ligne suivante (C) fonctionne 
   pstParametres->pszCheminHome = "Toto"; 
 
   return 0; 
 
}

[kameha]

Une question toute bête

Code :

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

1
2
3
4
5
6
7
 
 
int foo( ParametresLocaux *pstParametres){
[...]
   // La ligne suivante (A) ne fonctionne pas
   pstParametre->pszCheminCommun = strtok( szTest, "|" );
[...]

Le parametre c'est pstParametres

et apres tu utilises
pstParametre

Faute de frappe ou erreur tout court?

Faute de frappe...désolé..

[reggae]

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
 
typedef struct {
        char *pszCheminCommun;
        char *pszCheminHome;
} ParametresLocaux;
 
 
int main(......){
 
   int iRet = 0;
   ParametresLocaux stParametre;
 
   iRet = foo( &stParametre );
 
}
 
int foo( ParametresLocaux *pstParametres){
   char szTest[]="TATA|TITI|TOTO";
 
   memset ((void *)pstParametre, (int) NULL, sizeof (ParametresLocaux));
 
   // La ligne suivante (A) ne fonctionne pas
   pstParametre->pszCheminCommun = strtok( szTest, "|" );
 
   // La ligne suivante (B) ne fonctionne pas
   strcpy( pstParametre->pszCheminCommun, strtok( szTest, "|" ) );
 
   // La ligne suivante (C) fonctionne
   pstParametre->pszCheminHome = "Toto";
 
   return 0;
 
}

Il ne manque pas le prototype de "foo"?

[Mokhtar BEN MESSAOUD]

Une autre solution est de declarer la variable locale szTest statique (static)

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
 
#include <stdlib.h>
#include <stdio.h>
#include <string.h>
 
 
typedef struct { 
        char *pszCheminCommun; 
        char *pszCheminHome; 
} ParametresLocaux; 
 
static int foo( ParametresLocaux *pstParametres);
// char   szTest[]="TATA|TITI|TOTO";   
int main(){ 
 
   int iRet = 0; 
   ParametresLocaux stParametre; 
 
   iRet = foo( &stParametre ); 
 
   printf ("pszCheminCommun = %s\n",stParametre.pszCheminCommun); 
   printf (" pszCheminHome= %s\n",stParametre.pszCheminHome  ); 
 
   return EXIT_SUCCESS;
} 
 
int foo( ParametresLocaux *pstParametres){ 
 
 static char   szTest[]="TATA|TITI|TOTO";   
   memset ((void *)pstParametres, (int) NULL, sizeof (ParametresLocaux)); 
 
   // La ligne suivante (A) ne fonctionne pas 
   pstParametres->pszCheminCommun = strtok( (char *)szTest, "|" ); 
 
 
   // La ligne suivante (C) fonctionne 
   pstParametres->pszCheminHome = "Toto"; 
 
   return 0; 
 
}

[kameha]

merci pour vos réponses, ca a fonctionné...