Splitting a string into tokens

[unregistered]

Hi,
I've been trying to get this to work properly...maybe you guys can help me out here.
I'm trying to take a string and put each word into an array.
Ex: This prog is great
var1 = This
var2 = prog
and so on.
Heres the code I have to date:
----------------------
/* Seperate the words from the input string */
for (x = 0; x < strlen(command_entry); x ++) {
if (strcmp(command_entry[x], " ") == 0){ /* Found a space */
while (y < x) {
command_entry_sep[total_commands][y] = command_entry[x][y];
}
total_commands += 1;
y = x; /* May have to add one here to avoid space */
}
}
----------------------
both y and total_commands were initialyzed to zero.(earlier in the code) and are still 0.
This keeps core dumping. *sigh* Any help would be great!
Thanks in advance,
Chris

[bigtamscot]

Hi Hope this helps

#include <stdio.h>
#include <string.h>

int main ()
{
char str[]= "This prog is great", *p;
char split[4][6];

int i, j;
p = str;
for(i = 0; i < strlen(str) - 1; i++)
{
       j = 0;
      while(*p != ' ' && *p != '\0')
      {
	split[i][j] = *p;
	 p++;
	 j++;
      }
      if(*p == '\0') break;
      p++;
      split[i][j]='\0';
}
split[i][j]='\0';
printf("%s\n", str);
for(i = 0; i < 4; i++)
     printf("%s ", split[i]);
}

This prints out

This prog is great
This prog is great

[Stoned_Coder]

seems like a lot of code to do something that there is a function provided to do. That function is strtok(). Look at salems code.The first time it is called it is passed the address of the string to be tokenised. To keep tokenising the same string the next time you call strtok you pass a NULL.This tells strtok() to keep tokenising the string that it has already started. It is possible to change the delimiter as well if necessary between the calls.

[bigtamscot]

well pointed out stoned coder, although I have never used strtok before, having compiled salems I can well see this would be preferred option, looking up how strtok works at the moment.

[Unregistered]

Thanks Salem...You're code worked like a charm!
Cheers,
Chris