Discussion :

[yanyin]

Bonjour à tous


Mon code est fonctionnel. Mais je cherche à expliquer son comportement
Voici le code:
-------------------------------

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
char*p="test ok1 ok2";
char*ch1,ch,lex;
char*tab[3];
ch1=malloc((strlen(p)+1)*sizeof(char)));
strcpy(ch1,s);
while (*ch1 && (lex=strpbrk(ch1," "))){
 
         ch=(char*)malloc(((lex-tmp)+1)*sizeof(char));/* mon problème est là*/
        strncpy(ch,ch1,(lex-ch1))[tok-ch1+1]='\0';			
         tab[i++]=ch;        
         ch1=lex+1;
         tab[i]='\0';
 
}

------------------------------
Le but est de découper slon des espaces et pointer tab[i] sur "le token"
Quand j'affiche tab[i] j'ai le résulatat attendu ( tab[0]=test et tab[1]=ok1).
Ma question:
Normalement quand je passe à la deuxième itération , je m'attends à ce que
le contenu de "ch" qui contient déja le token "test" soit écrasé. Donc l'affichage doit être seulement : tab[0]=ok1. Mais l'affichage est:
tab[0]=test
tab[1]=ok1
Ch n'est pas ecrasé? c'est bizarre?

Merci d'avance

[droggo]

Gia,

Bonjour à tous


Mon code est fonctionnel. Mais je cherche à expliquer son comportement
Voici le code:
-------------------------------

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
char*p="test ok1 ok2";
char*ch1,ch,lex;
char*tab[3];
ch1=malloc((strlen(p)+1)*sizeof(char)));
strcpy(ch1,s);
while (*ch1 && (lex=strpbrk(ch1," "))){
 
         ch=(char*)malloc(((lex-tmp)+1)*sizeof(char));/* mon problème est là*/
        strncpy(ch,ch1,(lex-ch1))[tok-ch1+1]='\0';			
         tab[i++]=ch;        
         ch1=lex+1;
         tab[i]='\0';
 
}

------------------------------
Le but est de découper slon des espaces et pointer tab[i] sur "le token"
Quand j'affiche tab[i] j'ai le résulatat attendu ( tab[0]=test et tab[1]=ok1).
Ma question:
Normalement quand je passe à la deuxième itération , je m'attends à ce que
le contenu de "ch" qui contient déja le token "test" soit écrasé. Donc l'affichage doit être seulement : tab[0]=ok1. Mais l'affichage est:
tab[0]=test
tab[1]=ok1
Ch n'est pas ecrasé? c'est bizarre?

Merci d'avance

C'est surtout bien faux :

ch et lex sont de simples caractères, et tu les utilises comme des pointeurs.

[dot-_-net]

Code :

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

char*p="test ok1 ok2";

Tu n’as pas fait une allocation ici

Code :

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

char*ch1,*ch,*lex;

Code :

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

while (*ch1 && (lex=strpbrk(ch1," ")))

Peux tu nous expliquer ce qu’elle fait cette boucle ?

[yanyin]

Code :

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

char*p="test ok1 ok2";

Tu n’as pas fait une allocation ici

Code :

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

char*ch1,*ch,*lex;

Code :

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

while (*ch1 && (lex=strpbrk(ch1," ")))

Peux tu nous expliquer ce qu’elle fait cette boucle ?

Pour l'allocation : une petite erreur à glisser . Strcpy(ch1,p) et non strcpy(ch1,s)
Pour le while:
Tanque ce n'est pas '\0' et il y'a une occurence (le retour de strpbrk !=NULL) :
1- allouer espace mémoire suffisant (pouvant contenir le mot avant *lex)
2-copier ce mot et ajouter '\0'
3- affectation
4-déplacer le pointeur
5- reprendre de nouveau si c'est vrai

[yanyin]

Gia,

C'est surtout bien faux :

ch et lex sont de simples caractères, et tu les utilises comme des pointeurs.

Vous pouvez me donner plus de détails

merci

[snake264]

Si tu veux définir les variables lex et ch en tant que pointeur il faut que tu mette "*" devant.

[yanyin]

Si tu veux définir les variables lex et ch en tant que pointeur il faut que tu mette "*" devant.

Oui t'as raison .Mais dans mon vrai code sont des pointeurs.

je précise encore que ce bout de code est fonctionnel. Il m'affiche ce que je veux voir. Ma question c'est pourqoui le contenue de ch1 n'est pas écrasé? une fois que je passe à la 2ième itération?

[snake264]

Comment peuvent-ils être considéré comme des pointeurs alors que tu les déclare comme simple variable ?
Poste nous ton code complet au moins compilable qu'on puisse voir ce que tu cherche à faire

[Emmanuel Delahaye]

Oui t'as raison .Mais dans mon vrai code sont des pointeurs.

je précise encore que ce bout de code est fonctionnel. Il m'affiche ce que je veux voir. Ma question c'est pourqoui le contenue de ch1 n'est pas écrasé? une fois que je passe à la 2ième itération?

Ce code est faux (manque le dernier champ) et inutilement complexe. Ceci fonctionne :

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
 
#include<stdio.h>
#include<stdlib.h>
#include<string.h>
 
int main (void)
{
   /* non modifiable */
   char const *s = "test ok1 ok2";
 
   /* prévoir un pointeur de plus pour le NULL de fin */
   char *tab[4];
 
   /* faire une copie modifiable de la chaine (utilise malloc() */
   char *sdup = strdup (s);     /* POSIX.1 */
 
/* Parcourir la chaine en remplaçant le séparateur par un 0,
   noter l'adresse de la chaine dans le tableau. */
   char *p = sdup;
   if (sdup != NULL)
   {
      {
         int i = 0;
         tab[i] = p;
         i++;
 
         while ((p = strchr (p, ' ')) != NULL)
         {
            *p = 0;
            p++;
            tab[i] = p;
            i++;
         }
         tab[i] = NULL;
      }
 
/* afficher */
      {
         int i = 0;
 
         while (tab[i])
         {
            fprintf (stdout, "tab[%d]= %s\n", i, tab[i]);
            i++;
         }
      }
 
      /* libérer */
      free (sdup), sdup = NULL;
   }
   /* supprimer les alias */
   {
      int i = 0;
 
      while (tab[i])
      {
         tab[i] = NULL;
         i++;
      }
   }
 
   return 0;
}

Code :

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

1
2
3
4
5
6
 
tab[0]= test
tab[1]= ok1
tab[2]= ok2
 
Press ENTER to continue.

[droggo]

Hia,

Oui t'as raison .Mais dans mon vrai code sont des pointeurs.

Tu sais ce qu'est copier/coller ?