Discussion :

[acik59]

bonjour a tous
alors voila j'ai cree une fonction split *

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
int split(char s[],char* delimiteur)
{
 
int i=0;
char* p = strtok (s, delimiteur); 
char* tab[20];
   while (p != NULL) 
  { 
        tab[i]=p;
        p = strtok (NULL, delimiteur); 
        i++;
  } 
return(tab[20]);
}

Lors de la compil j'ai un warning

return transforme un pointeur en entier sans transtypage

alors voila si quelqu'un pouvait m'aider ce serait genial
merci d'avance!!!!!!!!

[stephl]

Que voulez-vous que split() retourne?

[acik59]

alors voila un exemple
split( "je;veux;manger" , ";" );

return(je veux manger)
dans un tableau
avec tab[0]=je
tab[1]=veux
tab[2]=manger

voila ce que j'aimerais qu'elle retourne ma fonction split

[stephl]

Vous avez déclaré votre tableau comme variable locale à la fonction split(). Il sera donc détruit lorsque la fonction terminera. Il faut utiliser l'allocation dynamique (malloc() et autres) ou bien passer le tableau en paramètre.

[Franck.H]

alors voila un exemple
split( "je;veux;manger" , ";" );

return(je veux manger)
dans un tableau
avec tab[0]=je
tab[1]=veux
tab[2]=manger

voila ce que j'aimerais qu'elle retourne ma fonction split

Alors pourquoi lui faire retourner un valeur de type int ?

[acik59]

alors voila la nouvelle solution :

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
char **decoupe_chaine(char *s, const char *ct)
{
   char **tab = NULL;
 
   if (s && ct)
   {
      int i;
      char *cs = NULL;
      size_t size = 1;
 
/* (1) */
      for (i = 0; (cs = strtok (s, ct)); i++)
      {
         if (size <= i + 1)
         {
            void *tmp = NULL;
 
/* (2) */
            size <<= 1;
            tmp = realloc (tab, sizeof (*tab) * size);
            if (tmp)
            {
               tab = tmp;
            }
            else
            {
               fprintf (stderr, "Memoire insuffisante\n");
               free (tab);
               tab = NULL;
               exit (EXIT_FAILURE);
            }
         }
/* (3) */
         tab[i] = cs;
         s = NULL;
      }
      tab[i] = NULL;
   }
   return tab;
}

qu'est ce que vous en penser!!!!!!!!

[souviron34]

Qu'est-ce que ça avance de faire :

Code :

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

1
2
 
  if (s && ct)

et pas

Code :

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

1
2
 
  if ( (s != NULL) && (ct != NULL) )

??

Deuxièmememt, strtok prend la chaîne la première fois, mais après il prend NULL...

Donc

Code :

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

1
2
 
      for (i = 0; (cs = strtok (s, ct)); i++)

ne marchera pas.

De plus, le mettre comme ça ne te permet pas de tester si il est nul, et est inutile..

Code :

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

1
2
3
4
5
6
7
8
 
    cs = strtok (s, ct) ;
    while ( cs !=  NULL )
      {
         ....
 
       cs = strtok (NULL, ct) ;
      }

Enfin strtok modifie la chaîne d'entrée.... qui devra donc inutilisable après...

[Franck.H]

Ta fonction tu peux complètement la recommencer Je te propose cette fonction:

Code :

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

char ** split (const char * str, const char * delim);

La fonction renvoie un pointeur sur pointeur sur char en vue bien sûr de créer un tableau dynamique de chaîne de caractères, surtout si tu ne connais pas par avance le nombre de mot que tu aura au final !

L'argument str est la chaîne dont il est préférable de faire une copie à l'intérieur de ta fonction car strtok modifi la chaîne sur laquelle il travail ! Tu pourras donc pourquoi pas en même temps créer une fonction de type strdup

L'argument delim bin c'est le délimiteur pour découper ta chaîne !