CODEWITHSUNDEEP
carrayscharacter-arrays
RestlessAPI
RestlessAPI

Reputation: 169

I am trying to assign a string value to an array of character arrays

I am trying to assign a string value (char *) to an element in an array of char arrays. My problem is, I can do the assignment, but it places the last read value into every element, instead of the specific element I wanted. My code looks as follows.

int   wordListIterator = 0;

int main(int argc, char **argv) {

    FILE *fp = fopen(argv[1], "r");

    int lineCount = 0;

    char line[80];
    while (fgets(line, sizeof(line), fp) != 0) {
    parseLine(line, strlen(line), lineCount);
    lineCount++;
    }
    fclose(fp);
} // end main

void parseLine(char * myLine, int stringLength, int myLineCount) {

    if (isspace(myLine[1])) {
        int i;
    for (i = 0; i < stringLength; i++) {
        Puzzle[myLineCount][i] = myLine[i];
    }

} else
if (isspace(myLine[1]) == 0) {
     Wordlist[wordListIterator] = myLine;
    wordListIterator++;
}
}

My output will have Wordlist[] be full of the last word read. This is for a word search puzzle. The Puzzle[][] assignment works great, so I do not understand where I am going wrong.

Upvotes: 0

Views: 65

Answers (1)

Daniel
Daniel

Reputation: 4549

You need to use strdup to copy your line, otherwise your going to overwrite the line every time, and every single one of your values is going to be a pointer to the same string. That string is going to contain the last input, so all of them are going to be pointing to the last input.

Your "puzzle" assignment works alright because you're copying the actual characters over one at a time.

Upvotes: 1

Related Questions