finding the end of a line of a file.

[mgracecar]

I'm reading in a file line by line. Each line has a name, and a random number of classes.

for example

Jerry,CSE1310,CSE1320,CSE1325
Bob,CSE1105,CSE1310
Barry,CSE1104,CSE1105,CSE1310,CSE1320

These are the names and the classes they have already taken.
I'm trying to tread this file and store in a 2d array of strings. The first column will be the names and the following columns are the classes taken. My question is, since the number of classes is not definite, how do I create a loop to do this until the newline?

Here's what I have so far, but it's not working. I think my second while loop is where I'm messing up. I think it's a never ending loop because I'm not changing buffer. Just wondering how to basically say while it's not the end of the line, keep using strtok on the commas and put in the next column.

     FILE *fp;     
     char buffer[100];
     int line_count = 0;
     int class_count = 0;
     char  *temp;
     char **names_and_classes_taken = malloc (sizeof(char*));
     
     if ( (fp = fopen("hw11-data.csv", "r" )) == NULL )
     {
     printf("Couldn't open file\n");
     exit(1); 
     }
       
     while(fgets(buffer, sizeof(buffer), fp))
     {
         while ( strcmp(buffer, " \n") != 0)                
          {
               /* if class count is 0, use the buffer, else if the count is 
                  not 0, use the same buffer */
               if(class_count == 0)
                 temp = strtok(buffer, ",");
               else
                 temp = strtok(NULL, ",");
                 
              names_and_classes_taken[class_count] = malloc(strlen(temp));
              strcpy(names_and_classes_taken[class_count], temp);
          }       
          class_count = 0;        
          line_count++;
          names_and_classes_taken = realloc(names_and_classes_taken, (line_count+1)* sizeof(char*));    
     }
fclose(fp);

[Salem]

Your loop should be

for ( temp = strtok(buffer, ",") ; temp != NULL ; temp = strtok(NULL, ",") )

> names_and_classes_taken[class_count] = malloc(strlen(temp));
You need strlen(temp)+1

And your realloc needs to be inside the loop as well.

[mgracecar]

ahh, i never thought of the loop that way. Much easier than what I was trying to do. So I no longer need my if-else statements either.
Quick question though...Why does the realloc need to be inside that loop? Wouldn't the realloc of names_and_classes_taken just be in the while loop? Because isn't that realloc just to allocate memory for the next row?

for example something like this?

          while(fgets(buffer, sizeof(buffer), fp))  
   {
         for(temp = strtok(buffer, ","); temp != NULL; temp = strtok(NULL,","))              
          {          
              names_and_classes_taken[class_count] = malloc(strlen(temp)+ 1);
              strcpy(names_and_classes_taken[class_count], temp);     
              class_count++;
          }       
          class_count = 0;        
          line_count++;
          names_and_classes_taken = realloc(names_and_classes_taken, (line_count+1)* sizeof(char*));    
     }
fclose(fp);

[Salem]

At the moment, class_count is a constant inside the loop, and you keep trashing the same entry with
names_and_classes_taken[class_count] = malloc(strlen(temp)+ 1);

[mgracecar]

alright I'm going to fool around with it some more. Thanks for the help