Reading in sourcelines from a file

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;
  char ws[] = " \t";
  FILE *pFile;

  if((pFile = fopen(filename,"r")) != NULL)
  {
      fgets(sourcelist, 81, pFile);
      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!

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

Quote:

---

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

Quote:

---

Originally Posted by williamsonj

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

---

Then you should call fgets on that LINE:

Code:

---


int i= 3; /* read line 3*/
fgets(sourcelist[i],81,stdin);

---

Also, how many lines is sourcelist supposed to contain? Is memory allocated statically or dynamically for these lines?

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.