Question about File Input

In the program I'm writing I'm having a bit of a problem. The program is reading in a number of strings from an input file and placing them into a character array, malloc-ing the memory, then analyzing them in a number of different ways. However, I can't read in the strings before I declare the character array, and that's where I encounter my problem. Now I know each string will be less than one 'line' long i.e. less than 80 characters or so. The question is, how would I setup a while loop to determine the number of strings? Here's the input file, but obviously it's going to need to work with any number of input files.

Code:

palindromes
String manipulation and recursion
Otto
Command-line arguments
Anna
file handling and dynamic memory allocation
Able was I ere I saw Elba
ctype.h
Madam, I'm Adam.
Use of menus: Integer vs. Character input
Array of strings?
Looking for EOF?
Was it a cat I saw?
A man, a plan, a cat, a canal: Panama?

It's a palindrome project, hence some of the odd strings. Basically, I'm trying to figure out how to setup a loop that would determine I need an array of size 14 after reading that data file? Any tips appreciated!

The best way would be to maintain some dynamic data structure that grows accordingly. The "simplest" of these is a dynamic array. When you begin to reach the maximum size, realloc the array. Though I would rather use a linked list as I find them less finicky than realloc.

As an alternative, you could always read the file twice. Once to find out how many strings are there and again to save them to a new dynamically allocated array.

Right, that was my plan. I was going to read it and then just rewind. I just wasn't sure what to setup as the condition of my while loop to have it stop at the end of the input file.

Edit: anyone have any suggestions on how I can do that?

The two loops would be identical except the first wouldn't save the line and the second would:

Code:

int n;
char buf[81];
char (*lines)[81];

/* fopen fin and such */
n = 0;
while ( fgets ( buf, sizeof buf, fin ) != NULL ) {
  n++;
}
lines = malloc ( n * sizeof *lines );
rewind ( fin );
n = 0;
while ( fgets ( buf, sizeof buf, fin ) != NULL ) {
  strcpy ( lines[n++], buf );
}

Beautiful. Thanks a bunch!

Okay, here's my problem now. In my function FillArray, the array is filled with the strings from the input file. Then, just for testing purposes, I have it print those strings out. That function works perfectly. However, when I open the MenuOne function and attempt to display the strings to the user, as I'm supposed to do for the program, I get some strange results.

Here's the code, anyone have any ideas?

Code:

void FillArray ( FILE *data, char *strArray )
{
   char buf[81];
   int counter = 0;

   rewind ( data );

   while ( fgets ( buf, 81, data ) != NULL )
   {
      strcpy ( &strArray[counter], buf );
      printf( "\n%s", &strArray[counter] );
      counter++;
   }

   MenuOne ( counter, strArray );
}

Then:

Code:

/* Controls the first level menu                 */
/* name: MenuOne                                 */
/* inputs: Number of strings, array of strings   */
/* returned values: none                         */
void MenuOne ( int numStrings, char *strArray )
{
   int i;

   for ( i = 0; i < numStrings; i++ )
   {
      printf( "%d: %s\n", i + 1, &strArray[i] );
   }
}

Is strArray a single string, or an array of character pointers? If the latter, then use it correctly, instead of the botched way you're doing it:

Code:

void MenuOne( int numStrings, char *sArray[] )
{
    ...
        printf(" %d: %s\n", i+1, sArray[i] );
...

Your code has a single character array. Not an array of character pointers.

Quzah.

That's what I'm having trouble with. How would I do that? In the FillArray function I'm printing it in basically the same exact way as I am trying in the MenuOne function. Also, I understand that arrays of strings are two dimensional functions, but when printing the whole string, what would I have to do with the second number. I.e. strArray[2] would obviously be the third string in the array, but wouldn't strArray[2][5] just be the 6th character in that string? So are you saying I'd have to print it one character at a time?

>char *strArray
Not an array of strings unless you're doing something creative. Perhaps

Code:

char **strArray

would work better for you. But it depends on how strArray was originally declared.

strArray was declared as :

Code:

char **strArray = NULL;

It is supposed to be an array of strings. I probably don't have to tell you this is my first time messing with those. I've used two dimensional arrays, but always of a constant size. I.e. I knew the size when writing the program.

Basically, I'm having problems with the array going between functions. I.e. in the FillArray function it's working properly. I'm filling each element of the array one by one in the for loop, and prints it out afterwards. However, as soon as I leave that loop, it's not acting normally. I.e. initially it prints as such :

Code:

palindromes
String manipulation and recursion
Otto
Command-line arguments
Anna
file handling and dynamic memory allocation
Able was I ere I saw Elba
ctype.h
Madam, I'm Adam.
Use of menus: Integer vs. Character input
Array of strings?
Looking for EOF?
Was it a cat I saw?
A man, a plan, a cat, a canal: Panama?

But when I print it in the exact same manner in the next function, MenuOne it prints as such:

Code:

 pSOCAfAcMUALWA man, a plan, a cat, a canal: Panama?
SOCAfAcMUALWA man, a plan, a cat, a canal: Panama?
OCAfAcMUALWA man, a plan, a cat, a canal: Panama?
CAfAcMUALWA man, a plan, a cat, a canal: Panama?
AfAcMUALWA man, a plan, a cat, a canal: Panama?
fAcMUALWA man, a plan, a cat, a canal: Panama?
AcMUALWA man, a plan, a cat, a canal: Panama?
cMUALWA man, a plan, a cat, a canal: Panama?
MUALWA man, a plan, a cat, a canal: Panama?
UALWA man, a plan, a cat, a canal: Panama?
ALWA man, a plan, a cat, a canal: Panama?
LWA man, a plan, a cat, a canal: Panama?
WA man, a plan, a cat, a canal: Panama?
A man, a plan, a cat, a canal: Panama?

My function call for MenuOne is :

Code:

MenuOne ( counter, strArray );

And its prototype is as follows, and it's printed out as such :

Code:

void MenuOne ( int numStrings, char *strArray )

 for ( i = 0; i < numStrings; i++ )
      {
         printf( " %s\n", &strArray[i] );
      }

Code:

void MenuOne ( int numStrings, char *strArray )

See if you can spot the problem. If you can't, reread the above posts.

Quzah.

I'm sorry, I'm not trying to sound redundant. I'm just new to trying to work with an array of strings of a dynamic size. I can't seem to find an online resource, anything in my text book, etc. to explain it. It's the syntax I'm having a problem with, I pretty much understand the logic. I just need to know how to declare it, pass it to functions, and a way of being able to index it for analysis.

I just need to know how to declare it, pass it to functions, and a way of being able to index it for analysis.

Did you even read my last post? If you had, and followed it's instructions, you would have reread my first post and Prelude's last post. Either way works.

Quzah.