adding a space to a string

how do i add a space to the end of a string?


this function takes the users first, middle, and last name. then it puts them all together in the string result. the output shows no spaces between the first, middle and last names. how do i add teh space.

i've tried doing something like...

result[ i ] = result[ i + ' ' ] ;

but it has no effect. when i put that inside the for loop, it displayed these weird characters.

Code:

---

void concat( char result[], char first[], char middle[], char last[] )
{
        int i, j, k ;                                                        /* Loop counters */

        for ( i = 0 ; first[ i ] != NULL ; i++ )                /* Copy first to the */
                result[ i ] = first[ i ] ;                                /* string result */

        for ( j = 0 ; middle[ j ] != NULL ; j++ )                /* Copy middle to the end of string */
                result[ i + j ] = middle[ j ] ;                                /* result from where we left off. */

        for ( k = 0 ; last[ k ] != NULL ; k++ )                        /* Copy last to the end of string */
                result[ i + j + k ] = last[ k ] ;                        /* result from where we left off. */

        result[ i + j + k ] = '\0' ;                                        /* Add the null byte to the end of the string */
}

---

Your biggest problem is fooling around with multiple indices when you could more easily reuse one variable for that purpose:

Code:

---

void concat ( char result[], char first[], char middle[], char last[] )
{
  int i, j = 0;

  for ( i = 0; first[i] != '\0'; i++ )
    result[j++] = first[i];

  result[j++] = ' ';

  for ( i = 0; middle[i] != '\0'; i++ )
    result[j++] = middle[i];

  result[j++] = ' ';

  for ( i = 0; last[i] != '\0'; i++ )
    result[j++] = last[i];

  result[j++] = '\0';
}

---

>first[ i ] != NULL
This is generally a bad idea, the NULL macro may not be the same as the nul character ('\0') so you really shouldn't use them interchangeably.

This function could be simplified greatly by using the standard library as well:

Code:

---

void concat ( char result[], char first[], char middle[], char last[] )
{
  sprintf ( result, "%s %s %s", first, middle, last );
}

---

-Prelude