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]

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

[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

[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

[yanyin]

Mon 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
15
16
17
18
19
20
21
22
23
24
25
 #include<stdio.h>
#include<stdlib.h>
#include<string.h>
 
int main(){
int i=0;
char*p="test ok1 ok2";
char*ch1,*ch,*lex;
char*tab[3];
ch1=malloc((strlen(p)+1)*sizeof(char));
strcpy(ch1,p);
while (*ch1 && (lex=strpbrk(ch1," "))){
         ch=(char*)malloc(((lex-ch1)+1)*sizeof(char));/* mon problème est là*/
			   strncpy(ch,ch1,(lex-ch1))[lex-ch1+1]='\0';			
         tab[i++]=ch;        
         ch1=lex+1;
         tab[i]='\0';
}
i=0;
while(tab[i]){
fprintf(stdout,"tab[%d]= %s\n",i,tab[i]);
i++;
}
return 0;
}

Attendez vous à un affichage comme ça :
tab[0]=test
tab[1]=ok1

Merci

[yanyin]

Et dans ce fichier , je pose ma question avec 2 shéma
Merci

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

[diogene]

Ma question c'est pourqoui le contenue de ch1 n'est pas écrasé? une fois que je passe à la 2ième itération?

Après avoir alloué de la mémoire par le malloc, copié "test" dans cette zone et rangé son adresse dans tab[0], le contenu de ch1 change pour pointer sur la suite de la chaîne "test ok1 ok2".
La deuxième itération par de la nouvelle position dans la chaîne (correspondant au caractère o de ok1). Le malloc alloue une nouvelle zone mémoire dans laquelle tu places la chaîne "ok1" et sauvegarde l'adresse de cette zone dans tab[1]. Puis la valeur de ch1 change pour pointer sur le o de "ok2".

Il est normal que tab[0] pointe sur "test" et tab[1] sur "ok1".

[droggo]

Hia,

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

Tu sais ce qu'est copier/coller ?

[yanyin]

Ce n'est que ma première participation . Je m'excuse pour les erreurs qui ont glissé


Mr Delahaye ,Votre code donne des warning à la compilation

Code :

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

1
2
3
4
5
 
gcc -Wall -pedantic -posix -o n.o n.c
n.c: In function ‘main’:
n.c:14: attention : implicit declaration of function ‘strdup’
n.c:14: attention : incompatible implicit declaration of built-in function ‘strdup’

Il fait un bon découpage et beaucoup plus simple que le mien. Mais je ne veux pas afficher la chaine "OK2" et encore plus je cherche uniquement à comprendre pourqoui le contenue de ch(et pas ch1) n'est pas ecrasé.

Diogene: si on raisonne sur ch1,c'est vrai .Son contenue ne doit pas bouger
Par contre ch doit changer à la 2ième itération. En effet tab pointe sur ch et ne sait rien sur ch1. Et si nous tournons ce programme à la main nous devrions avoir(à mon avis) tab[0]=oK1 et tab[1]=ok1puisque tab pointe sur ch et pas sur ch1
Mon shéma ci haut lui manque un autre pointeur (tab[1]) qui devra pointer encore une fois sur ch.

[Emmanuel Delahaye]

Mr Delahaye ,Votre code donne des warning à la compilation

Code :

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

1
2
3
4
5
 
gcc -Wall -pedantic -posix -o n.o n.c
n.c: In function ‘main’:
n.c:14: attention : implicit declaration of function ‘strdup’
n.c:14: attention : incompatible implicit declaration of built-in function ‘strdup’

Retire -pedantic, il inhibe la déclaration de strdup().

(et 'Emmanuel', suffira...)

[diogene]

nous devrions avoir(à mon avis) tab[0]=oK1 et tab[1]=ok1puisque tab pointe sur ch et pas sur ch1

Non, parce que lors du deuxième
ch = malloc(....),
une nouvelle adresse d'un nouveau bloc de mémoire a été placée dans ch et qu'une nouvelle chaîne
a été copiée dans ce nouveau bloc.
strncpy(ch,.....
tab ne pointe pas sur ch. Le tableau tab contient les adresses successives obtenues par les malloc

[yanyin]

tab ne pointe pas sur ch. Le tableau tab contient les adresses successives obtenues par les malloc

Donc si je libère la mémoire (free(ch)) le contenue de tab ne devra pas bouger puisque l'adresse est déja mémorisee dans tab[i] correspondant et ne sera effacé que par une affectation (tab[i]=NULL) .N'est pas?

Pour tester ,j'ai ajouté (free(ch)) à la fin de la boucle while et j'ai eu:
tab[0]=
tab[1]=

[diogene]

Donc si je libère la mémoire (free(ch)) le contenue de tab ne devra pas bouger

Non.
Par contre, les adresses contenues dans tab sont des adresses de zones mémoires devenues invalides (libérées par free) et tab contient alors l'adresse de zones détruites. Si on essaie d'afficher les chaînes dont l'adresse est alors dans tab, on va à la catastrophe puisque ces chaînes sont détruites.
Comment savoir si les adresses placées dans tab sont des adresses mémoires invalides ? Placer le contenu de tab à NULL permet de signifier que ces adresses ne sont pas des adresses d'objets.

[yanyin]

Merci diogene.
Tes réponses m'ont convaicu.

[Emmanuel Delahaye]

Merci diogene.
Tes réponses m'ont convaicu.

Le problème n'est pas d'être convaincu. Il ne s'agit pas ici d'opinion. La question est "est-ce que tu as compris ?" .