Utilisation de sscanf

Version imprimable

13/12/2005, 19h51
Gryzzly

Utilisation de sscanf

Salut,

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

Code:

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
13/12/2005, 19h54
Trap D

Cherche dans les sources ou la FAQ, il y a un tuto sur scanf, (et donc sscanf).
13/12/2005, 20h01
Luther13

Voir plutôt du coté de la fonction strtok
13/12/2005, 20h04
fearyourself

Citation:

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!!
13/12/2005, 20h06
Gryzzly

Citation:

Envoyé par Luther13

Voir plutôt du coté de la fonction strtok

Utilisé sscanf m'est imposé
13/12/2005, 20h08
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.
13/12/2005, 20h09
sorry60

man sscanf (sous linux ou alors sur google)
13/12/2005, 21h51
Emmanuel Delahaye

Citation:

Envoyé par 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.

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

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:

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;
}

14/12/2005, 09h25
Emmanuel Delahaye

Citation:

Envoyé par 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 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...
14/12/2005, 09h48
Gryzzly

Citation:

Envoyé par Emmanuel Delahaye

Citation:

Envoyé par 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
14/12/2005, 09h50
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

Citation:

Envoyé par 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

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

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
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;
}

14/12/2005, 11h41
fearyourself
Une étude des pointeurs serait nécessaire...

Ceci ne marchera jamais:
Code:

1 2 3 4 char*tab; if(chaine(entree,tab))
Avec comme prototype:

Code:

int chaine(char *ligne,char *tab)

Et dans la fonction...
Code:

1 2 tab = (char*)malloc(sizeof(char)*nb_virgules);
Si je te donne le programme:
Code:

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...
14/12/2005, 11h53
Luther13
Déjà, dans ton compteur de virgules, lorsque tu croise la première virgule, tu n'implément plus la position ligne:
Code:

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.

Citation:

Envoyé par Gryzzly

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:

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; }

Citation:

Envoyé par Luther13

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

Code:

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:

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;
}

14/12/2005, 12h29
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:

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

1 2 tab[i++] = *ligne;
La ligne:
Code:

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:

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:

1 2 printf("%c",tab[0]);
Ce test est inutile, le test dans le for le gère parfaitement
Code:

1 2 if(nb_virgules >1 )
Jc

Citation:

Envoyé par Gryzzly

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:

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

Une façon compliquée d'écrire

Code:

lettre = *ligne;
Manque un 0 en fin de chaine...

Manque une libération.

Code:

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:

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;
}

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

Pourquoi ne pas faire simplement:

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
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;
}