user2500861
user2500861

Reputation:

How to assign char * to character array?

I have following code:

int main(){

    char sentence[] = "my name is john";
    int i=0;
    char ch[50];
    for (char* word = strtok(sentence," "); word != NULL; word = strtok(NULL, " "))
    {
        // put word into array
        //  *ch=word;
        ch[i]=word;
        printf("%s \n",ch[i]);
        i++;

        //Above commeted part does not work, how to put word into character array ch
    }
    return 0;
}

I am getting error: error: invalid conversion from ‘char*’ to ‘char’ [-fpermissive] I want to store each word into array, can someone help?

Upvotes: 4

Views: 2031

Answers (1)

alk
alk

Reputation: 70981

To store a whole set of words you need an array of words, or at least an array of pointers pointing to a word each.

The OP's ch is an array of characters and not an array of pointers to characters.

A possible approach would be:

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

#define WORDS_MAX (50)

int main(void)
{
  int result = EXIT_SUCCESS;

  char sentence[] = "my name is john";
  char * ch[WORDS_MAX] = {0}; /* This stores references to 50 words. */

  char * word = strtok(sentence, " "); /* Using the while construct, 
                                          keeps the program from running 
                                          into undefined behaviour (most 
                                          probably crashing) in case the 
                                          first call to strtok() would 
                                          return NULL. */
  size_t i = 0;
  while ((NULL != word) && (WORDS_MAX > i))
  {
    ch[i] = strdup(word); /* Creates a copy of the word found and stores 
                             it's address in ch[i]. This copy should 
                             be free()ed if not used any more. */
    if (NULL == ch[i]) 
    {
      perror("strdup() failed");
      result = EXIT_FAILURE;
      break;
    }

    printf("%s\n", ch[i]);
    i++;

    word = strtok(NULL, " ")
  }

  return result;
}

Upvotes: 7

Related Questions