Discussion :

[Gryzzly]

Salut,

j'aimerais avoir une aide concernant ceci:
A partir d'une chaine de caractère,par exemple:

Code :

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

char *ligne = "    x   ,    y    ,z     "

il faut que j'insère les caractères lus dans un tableau char *tab en utilisant sscanf.
Il faut également le nombre de lettre insérées.

Merci d'avance

[Trap D]

Cherche dans les sources ou la FAQ, il y a un tuto sur scanf, (et donc sscanf).

[Luther13]

Voir plutôt du coté de la fonction strtok

[fearyourself]

Voir plutôt du coté de la fonction strtok

Cette fonction est déclaŕée dans les pages manuels comme une fonction à ne pas utiliser... Personnes averties seulement!!

[Gryzzly]

Voir plutôt du coté de la fonction strtok

Utilisé sscanf m'est imposé

[Trap D]

Oui, à cause du manque de réentrance mais la je ne pense pas que ce soit le cas ici.
Le problème est que strtok utilise un buffer interne static et donc il peut y avoir mélange en cas d'utilisation simultanée par deux threads distincts d'un même programme.
D'autre part, strtok modifie la chaîne passée en argument, celà peut poser des problèmes.

[sorry60]

man sscanf (sous linux ou alors sur google)

[Emmanuel Delahaye]

Oui, à cause du manque de réentrance mais la je ne pense pas que ce soit le cas ici.
Le problème est que strtok utilise un buffer interne static et donc il peut y avoir mélange en cas d'utilisation simultanée par deux threads distincts d'un même programme.
D'autre part, strtok modifie la chaîne passée en argument, celà peut poser des problèmes.

Plus simplemement, on se sais pas faire d'appels imbriqués de strtok()

Par exemple

a=1 b=2 c=3

Première découpe en
a=1
b=2
c=3

ensuite (ok ici, c'est trivial)

découpe imbriquée en:
a
=
1

b
=
2

c
=
3

strtok() ne sait pas faire ça 'à la volée', ou alors il faut copier la chaine (strdup())...

[Gryzzly]

Bonjour,

voici le programme que j'ai écrit concernant la question que j'avais posé mais à la compilation,le programme m'affiche une ligne vide ?

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
#include<stdio.h>
#include<ctype.h>
#include<stdlib.h>
 
int chaine(char *ligne,char *tab)
{
  char lettre;
  int nb_lettres = 0;  
 
  while(*ligne == ' ') ligne ++;
 
  while(*ligne != '\0')
    {
      if(!(isalpha(*ligne))) return 1;
 
      sscanf(ligne,"%c",&lettre);
      ligne++;
      nb_lettres ++;
      while(*ligne == ' ' || *ligne == ',') ligne ++;
    }     
  tab = (char*)malloc(sizeof(char)*nb_lettres);
  if(tab == NULL) return 1;
 
  return 0;
}
 
 
 
int main(void)
{
  char *entree = " x       ,          z    ";
  char*tab;  
 
  if(chaine(entree,tab))
    {
      printf("erreur \n");
      return 1;
    }
 
  printf("%s\n",tab);
 
  return 0;
}

[Emmanuel Delahaye]

voici le programme que j'ai écrit concernant la question que j'avais posé mais à la compilation,le programme m'affiche une ligne vide ?

Tu veux dire à l'exécution.
Déjà, chez moi, ça crash parce que tu passes une valeur indéterminée à 'chaine' (tab est un pointeur non initialisé). Il se passe donc n'importe quoi...

Ensuite, je n'ai pas réussi à déterminer ce que doit faire la fonction chaine(). Peut être qu'un meilleur nom serait plus parlant... (action, verbe...). En tout cas, elle cherche à modifier un paramètre, ce qui est probablement une erreur de conception...

[Gryzzly]

voici le programme que j'ai écrit concernant la question que j'avais posé mais à la compilation,le programme m'affiche une ligne vide ?

Tu veux dire à l'exécution.
Déjà, chez moi, ça crash parceque tu passes une valeur indéterminée à 'chaine' (tab est un pointeur non initialisé). Il se passe donc n'importe quoi...

Ensuite, je n'ai pas réussi à déterminer ce que doit faire la fonction chaine(). Peut être qu'un meilleur nom serait plus parlant... (action, verbe...)

la fonction chaine à pour but de récupérer les lettres lues et de les mettres dans tab

[Luther13]

L'idée est bonne mais tu ne remplis pas finalement le tableau lorsque tu recupère la lettre.
Je proposerais de compter d'abord le nombre de virgules dans chaine afin de connaitre d'avance la taille du tableau à allouer. puis après d'après d'effectuer le traiteemnt de remplissage

[Gryzzly]

L'idée est bonne mais tu ne remplis pas finalement le tableau lorsque tu recupère la lettre.
Je proposerais de compter d'abord le nombre de virgules dans chaine afin de connaitre d'avance la taille du tableau à allouer. puis après d'après d'effectuer le traiteemnt de remplissage

Lorsque j'execute le programme,l'affichage n'arrive pas.
Il doit y avoir un problème mais lequel?

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
#include<stdio.h>
#include<ctype.h>
#include<stdlib.h>
 
 
int compteVirgules(char *ligne)
{
  int compteur = 0;
 
  while(*ligne != '\0')
    {
 
      if(*ligne == ',') compteur ++;
 
      else ligne ++;
    }
  return (compteur + 1); 
}
 
 
int chaine(char *ligne,char *tab)
{
  char lettre; 
  int nb_virgules = compteVirgules(ligne);
  int i=0;  
 
  while(*ligne == ' ') ligne ++;
 
  tab = (char*)malloc(sizeof(char)*nb_virgules);
  if(tab == NULL) return 1;
 
 
  while(*ligne != '\0')
    {
      if(!(isalpha(*ligne))) return 1;
 
      sscanf(ligne,"%c",&lettre);
      ligne ++;
      tab[i++] = lettre;
 
      while(*ligne == ' ' || *ligne == ',') ligne ++;
    }     
  return 0;
}
 
 
 
int main(void)
{
  char entree[] = " x       ,          z    ";
  char*tab;  
 
  if(chaine(entree,tab))
    {
      printf("erreur \n");
      return 1;
    }
 
  printf("%s\n",tab);
 
  return 0;
}

[fearyourself]

Une étude des pointeurs serait nécessaire...

Ceci ne marchera jamais:

Code :

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

1
2
3
4
 
  char*tab; 
 
  if(chaine(entree,tab))

Avec comme prototype:

Code :

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

int chaine(char *ligne,char *tab)

Et dans la fonction...

Code :

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

1
2
 
  tab = (char*)malloc(sizeof(char)*nb_virgules);

Si je te donne le programme:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
 
void f(int a)
{
a=2;
}
 
int main()
{
int a;
f(a);
printf("%d\n",a);
return 0;
}

Qu'affiche ce programme?

Ton programme fait la même chose, il passe la valeur du pointeur tab et espère par un miracle que la fonction pourra la changer...

[Luther13]

Déjà, dans ton compteur de virgules, lorsque tu croise la première virgule, tu n'implément plus la position ligne:

Code :

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

1
2
3
4
5
if(*ligne == ',') compteur ++;
 
      /* le else n'est pas necessaire */
/*else   */
		  ligne ++;

Ensuite, l'allocation de memoire ne doit pas se faire dans la fonction chaine, ni la recher du nombre de virgules.

[Emmanuel Delahaye]

la fonction chaine à pour but de récupérer les lettres lues et de les mettres dans tab

Définit le critère 'lettre'.

Si tab est un tableau qui doit être vu de l'appelant, le plus simple est de l'allouer et de retourner son adresse

Je conseille ce prototype :

char *chaine (char const *ligne);

et cette utilisation :

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
 
int main (void)
{
   char const entree[] = " x       ,          z    ";
   char *tab = chaine (entree);
 
   if (tab != NULL)
   {
      printf ("%s\n", tab);
      free (tab), tab = NULL;
   }
   else
   {
      printf ("erreur \n");
   }
 
   return 0;
}

[Gryzzly]

Déjà, dans ton compteur de virgules, lorsque tu croise la première virgule, tu n'implément plus la position ligne:

Code :

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

1
2
3
4
5
if(*ligne == ',') compteur ++;
 
      /* le else n'est pas necessaire */
/*else   */
		  ligne ++;

Ensuite, l'allocation de memoire ne doit pas se faire dans la fonction chaine, ni la recher du nombre de virgules.

Merci de toutes vos remarques.
Voila le programme corrige et qui fonctionne.
Maitenant,je voudrais savoir si le code peut etre améliorer

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
64
65
66
67
68
69
70
71
72
 
#include<stdio.h>
#include<ctype.h>
#include<stdlib.h>
 
 
int compteVirgules(char *ligne)
{
  int compteur = 0;
 
  while(*ligne != '\0')
    {
 
      if(*ligne == ',') compteur ++;
      ligne ++;
    }
  return (compteur + 1); 
}
 
 
int chaine(char *ligne,char *tab)
{
  char lettre; 
  int i=0;  
 
  while(*ligne == ' ') ligne ++;
 
 
  while(*ligne != '\0')
    {
      if(!(isalpha(*ligne))) return 1;
 
      sscanf(ligne,"%c",&lettre);
      ligne ++;
      tab[i++] = lettre;
 
      while(*ligne == ' ' || *ligne == ',') ligne ++;
    }     
  return 0;
}
 
 
 
int main(void)
{
  int i;
  char entree[] = " x       ,          z    ";
  char*tab;  
 
  int nb_virgules = compteVirgules(entree);
 
  tab = (char*)malloc(sizeof(char)*nb_virgules);
  if(tab == NULL) return 1;
 
 
  if(chaine(entree,tab))
    {
      printf("erreur \n");
      return 1;
    }
 
  printf("%c",tab[0]);
 
  if(nb_virgules >1 )
    {
      for(i=1 ;i< nb_virgules; i++)
	printf(",%c",tab[i]);
    }
  printf("\n");
 
  return 0;
}

[fearyourself]

Voici quelques remarques:

Dans compteVirgules...

Initialise compteur à 1 et modifie ton return...

---------------------------

Dans chaine...

Ton test
Voir la remarque sur le main sur le problème des noms de variable...

Code :

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

1
2
3
4
 
      sscanf(ligne,"%c",&lettre);
      ligne ++;
      tab[i++] = lettre;

Pourrait être:

Code :

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

1
2
 
   tab[i++] = *ligne;

La ligne:

Code :

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

1
2
 
ligne++;

sera gérée par ton while...

---------------------------

Dans ton main...

A cause de ton allocation de tab, tes variables ne peuvent être qu'une seule lettre...

Code :

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

1
2
 
tab = (char*)malloc(sizeof(char)*nb_virgules);

Tu ne testes pas le cas où il n'y a pas d'arguments donc cet endroit est problématique...

Code :

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

1
2
 
printf("%c",tab[0]);

Ce test est inutile, le test dans le for le gère parfaitement

Code :

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

1
2
 
if(nb_virgules >1 )

Jc

[Emmanuel Delahaye]

Voila le programme corrige et qui fonctionne.

Non, il ne fonctionne pas.

• malloc() : cast inutile. sizeof(char) vaut 1 par définition.
  
  http://emmanuel-delahaye.developpez....tes.htm#malloc
  
• La fonction chaine n'a pas de paramètre 'taille' pour sa chaine de destination. aucun contrôle de débordement n'est donc possible. On ne peut pas sécuriser le code. Une instrumentation du code montre que la chaine allouée est évidemment trop petite... Il y a donc débordement et comportement indéfini. Tu cherches à faire quoi exactement ?
  

• Code :

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

     sscanf (ligne, "%c", &lettre);

  Une façon compliquée d'écrire

  Code :

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

  lettre = *ligne;

• Manque un 0 en fin de chaine...
• Manque une libération.
  

  Code :

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

  1
  2
  3
   
  SYSALLOC Err: Not-matched list:
  SYSALLOC Bloc 003D2488 (2 byte) malloc'ed at line 60 of '../main.c' not freed

Ceci fonctionne de façon sûre (mais je ne sais toujours pas ce que tu veux faire...).

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
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
 
#include<stdio.h>
#include<ctype.h>
#include<stdlib.h>
 
/* instrumentation */
#include<assert.h>
#include<string.h>
 
static int compteLettres (char const *ligne)
{
   int compteur = 0;
 
   while (*ligne != 0)
   {
      if (*ligne != ' ')
         compteur++;
      ligne++;
   }
   return compteur;
}
 
static int chaine (char const *ligne, char *tab, size_t size)
{
   size_t i = 0;
 
   while (*ligne == ' ')
      ligne++;
 
   while (*ligne != '\0')
   {
      if (!(isalpha (*ligne)))
         return 1;
      {
         int lettre = *ligne;
         ligne++;
 
         assert (i < size - 1);
         tab[i] = lettre;
      }
      i++;
 
      while (*ligne == ' ' || *ligne == ',')
         ligne++;
   }
 
   tab[i] = 0;
   return 0;
}
 
int main (void)
{
   char const entree[] = " x       ,          z    ";
 
   int nb_lettres = compteLettres (entree);
 
   if (nb_lettres != 0)
   {
      size_t const size = nb_lettres + 1;
      char *tab = malloc (size);
 
      if (tab != NULL)
      {
         memset (tab, '?', size);
 
         if (chaine (entree, tab, size))
         {
            printf ("erreur \n");
            return 1;
         }
 
         if (nb_lettres > 1)
         {
            printf ("'%s'\n", tab);
         }
 
         free (tab), tab = NULL;
      }
   }
   return 0;
}

[fearyourself]

Je viens de voir que tout ce que tu veux c'est enlever les espaces en fait...

Pourquoi ne pas faire simplement:

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
#include<stdio.h>
#include<ctype.h>
#include<stdlib.h>
#include<string.h>
 
/*
 Fonction qui retourne une copie de la chaine de caracteres passee en argument sans les espaces...
*/
 
char* enleve_espaces(const char *s)
{
int idx,cpidx;
 
int size = strlen(s);  //Taille de la chaine
char *res;
 
//Si la chaine est vide
if(!size)
   return NULL;
 
//Allocation
res = malloc(size);
if(res==NULL)
	return NULL; //Si erreur
 
for(idx=0,cpidx=0;idx<size;idx++)
	{
	if( (isalpha(s[idx])) || (s[idx]==','))  //Si c'est une lettre ou une virgule
		{
		res[cpidx] = s[idx];
		cpidx++;
		}
	else if( s[idx] != ' ') //Sinon cela peut encore etre une virgule, mais sinon c'est une erreur!
		{
		//Sinon erreur
		free(res);
		return NULL;  //On rend NULL
		}
	}
 
//Le '\0' à la fin
res[cpidx] = '\0';
return res;
}
 
int main (void)
{
   char const entree[] = " x  ,  y      ,          z    ";
   char *test;
 
   test = enleve_espaces(entree);
 
   if(test)
	   {
	   printf("%s\n",test);
	   free(test);
	   }
 
   return 0;
}