Thread: Newbie: having a problem with my string function.

I have a feeling that string is a reserved word, meaning you cannot use it for a string.

Surely the best way to go would be to use 2 strings, and pointers. That way you could increment the pointer to the next letter in the string, check the letter against the delimiter, IF it is the same, just increment the pointer without copying anything accross, IF IT ISN'T the same, copy the letter accross to the second string.

>I have a feeling that string is a reserved word, meaning you cannot use it for a string.
Nope, unless its in the global namespace where anything starting with str is reserved. The name strsplit is reserved because anything in the string.h header may add stuff.

>To help me actually learn C
Just my opinion, but it looks like you're learning C99. I'd recommend not doing that because C99 isn't used much at all and you need to know how it's done in C89 because there's bunches and bunches of C89 code drifting around. Anyway, there's a simpler way to do it using pointers.

Code:

Mon Oct 20 9:00:06am
sangut: ~/misc
> <c.c less
/**This is in a file named mystring.c**/
char *strsplit(char *string, char delimiter) {
  char *s;
  char *p;

  s = p = string;
  while (*s != '\0') { /* '\0' is the end of the string */
    if (*s != delimiter) /* Is it the delimiter? */
      *p++ = *s; /* No? Copy it to the final location */
    s++; /* Go to the next one */
  }
  *p = '\0'; /* Close the string */

  return string;
}

int main ( ) {
  char string[] = "Hello, there!";
  char remove = 'e';

  printf("The string was |%s|\n", strsplit(string, remove));

  return 0;
}

But your code has a single distinct problem:

>char output[newStringLen];
This is a variable local to the strsplit function. When you return, the memory for output is released and it'll probably have garbage in it. Those are probably your funny characters. If you want a variable to stick around between functions you can do one of several things: pass a buffer as an argument to strsplit, dynamically allocate anonymous memory with malloc/calloc, declare the string as static within strsplit, make it global. By the way, that's the order that you should try those suggestions too. They get progressively worse.