Discussion :

[Ganondorf]

bonsoir à tous,

je me suis créer une petite fonction split pour éclater une chaine et mettre chaque element dans un tableau:

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
 
char **split(char *buf, char delim)
{
	unsigned int idx = 0;
	char *list[MAXPARA];
 
	while (*buf)
	{
		while (*buf == delim) *buf++ = 0;
		/* dans le cas ou l'argument commence par un ':' on le supprime */
		if (*buf == ':')
		{
			list[idx++] = buf + 1;
			break;
		}
		/* on recupere l'argument */
		list[idx++] = buf;
		while (*buf && *buf != ' ') ++buf;
	}
	list[idx] = NULL;
	return list;
}

cette dernière me retourne le tableau, mon probleme se situe au moment de recuperer sa valeur:

Code :

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

1
2
3
4
5
 
// buffer contient une chaine de caracteres
char *tab[MAXPARA];
 
tab = split (buffer, ' ');

j'ai l'erreur 'error: incompatible types in assignment'
j'ai encore un peu de mal avec les pointeurs j'imagine que la valeur de retour du split n'est pas compatible avec ma declaration du tableau mais je n'en suis pas sur et je ne sais comment faire autrement

[diogene]

j'ai encore un peu de mal avec les pointeurs j'imagine que la valeur de retour du split n'est pas compatible avec ma declaration du tableau mais je n'en suis pas sur et je ne sais comment faire autrement

C'est plus fondamental que ça : tab est un tableau (de pointeur sur char) et on ne peut jamais écrire "tableau = quelque chose ;"

Il faut passer le tableau tab en argument de split() :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
void split(char *buf, char delim, char** list)
{
	unsigned int idx = 0;
	while (*buf)
	{
....
	}
	list[idx] = NULL;
}
....
char *tab[MAXPARA];
 
split (buffer, ' ',tab);

Remarque :

Code :

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

while (*buf == delim) *buf++ = 0;

Pourquoi modifier la chaine de départ ?

[Ganondorf]

merci pour ton aide, oui en effet je modifie la chaine c'est pas terrible, il faudrait mieux que je stock la chaine dans un autre pointeur pour la parcourir

[diogene]

merci pour ton aide, oui en effet je modifie la chaine c'est pas terrible, il faudrait mieux que je stock la chaine dans un autre pointeur pour la parcourir

Non, tu peux sans problèmes utiliser le pointeur buf

Mais pourquoi ne pas simplement faire

Code :

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

while (*buf == delim) buf++;

?

[Ganondorf]

je marquai mes positions pour le découpage pour que chaque element se termine par le caractere de fin de chaine

[diogene]

je marquai mes positions pour le découpage pour que chaque element se termine par le caractere de fin de chaine

Ok.
Dans ce cas, pourquoi ne pas utiliser strtok() ?

[Ganondorf]

strtok modifie son 1er argument donc au final je modifie ma chaine également, de plus j'ai souvent entendu dire qu'il fallait éviter cette fonction

si vous voyez une autre manière plus efficace pour la fonction split je suis preneur