Thread: Reading in sourcelines from a file

sourcelist is declared as an array of strings from what I gather and fgets takes a string as the first parameter.

Right, I'm trying to save each line as a string in my array, each index in the array holds a line from the file

Originally Posted by williamsonj

Hey there, I'm trying to read in sourcelines from a file and save them to an array. I want to skip any lines that contain only whitespace first though. Here's what I've got so far but I get an incompatible pointer type error. I always have issues with pointers lol

Code:

/* Pre-Processor Directives */
#include <stdio.h>
#include <string.h>
#include "cop3601.h"

extern char sourcelist[][81];

int getsource(char filename[])
{
   int lines = 0;
/* ws needs a size. "\t" looks like the char '\t' but is actually a string with two chars:
\t\0. EVERY string MUST have an end of string char appended to it, or it's just a
bunch of letters - and not a string.
*/

   char ws[] = " \t";   //give it a size of 2, at least
   FILE *pFile;

   if((pFile = fopen(filename,"r")) != NULL)
   {
       You need a while loop to start here, with fgets(... etc), inside the testing loop

      fgets(sourcelist, 81, pFile); //you need an index number for which row: sourcelist[i]
      strtok(sourcelist, "\n");
   }
   else
      hprintf("\nFile %s does not exist.\n\n");

   fclose(pFile);

   return lines;
}

I'm sure it's something really simple that I'm screwing up, but any ideas are helpful!

hprintf(...etc.), needs to be just printf(...etc).

Good looking code, though.

hprintf is a function included in the cop3601.h file

sourcelist will be set in a driving class and will most likely be done dynamically

Ok, well the incompatible pointer type is because you are passing sourcelist (i.e. char**) to fgets instead of sourcelist[i] (i.e char*).

OK, thanks, I'll try making those changes. I'm not 100% sure how the sourcelist array size gets set so I'll have to figure that out first.