Vfrdtyky

I'm trying to parse through a line of a CSV file and pull out the desired value that I want. However, my function is cutting off the last character in the string, and I can't figure out why. I thought it has something to do with where I'm assigning the null terminator, but changing that didn't help. Any help would be appreciated. Thanks!

char* findKey(char lineBuffer, int columnNumber ){

    char tempArray[strlen(lineBuffer)+2];

    int commasCounted = 0;

    int i =0;



    for(i = 0; i < strlen(lineBuffer) - 1; i++){

        if (commasCounted == columnNumber){

            commasCounted = i;

            break;

        }



        if (lineBuffer[i] == '"'){

            i++;

            while(lineBuffer[i] && lineBuffer[i] != '"'){

                i++;

            }

        }



        if (lineBuffer[i] == ','){

            commasCounted++;

        }

    }



    if(lineBuffer[commasCounted] == ','){

        tempArray[0] = '0';

        tempArray[1] = '0';

        tempArray[2] = '0';

        tempArray[3] = '0';

        tempArray[4] = '';

    }else{

        int j = 0;

        for(i = commasCounted; i < strlen(lineBuffer) - 1; i++){

            if(lineBuffer[i] == '"'){

                i++;

                while(lineBuffer[i] && lineBuffer[i] != '"'){

                    tempArray[j] = lineBuffer[i];

                    i++;

                    j++;

                }

                break;

            }else if(lineBuffer[i] == ','){

                break;

            }else

                tempArray[j] = lineBuffer[i];

                j++;

        }

        tempArray[j] = '';

    }



    char* tempString = strtok(tempArray, "n");

    //printf("tempString before returning in findKey: %sn", tempString); //testing

    return tempString;

}

The CSV file can have certain columns wrapped in quotes, which is why there are some checks for quotes in there. This gets passed the string to check, and the column that holds the information. So, for example:
Passing in this for lineBuffer:

30,beforeyoustartedgackinyouusedtohaveabrainbutnowyoudontgeteventhesimpliesofthingsdrawalittlepictureevenusemyhandstrytoexplainbutyoujustdontunderstandoh

And a 1 for columnNumber

Results in this being returned:

beforeyoustartedgackinyouusedtohaveabrainbutnowyoudontgeteventhesimpliesofthingsdrawalittlepictureevenusemyhandstrytoexplainbutyoujustdontunderstando

You're never copying the last character of the line because you have i < strlen(lineBuffer) - 1 in the for loop condition. Change that to i < strlen(lineBuffer).

You also need to make a copy of the string before returning, you can't return a local array in C:

return strdup(tempString);

This also means that the caller needs to free this string when they're done with it, since strdup() allocates memory dynamically. Alternatively, you could use malloc() to allocate tempArray() in the first place, instead of declaring a local array.