Thread: Sorting/comparing arrays from inputs.

Greetings fellow programmers. I'm just starting out as a programmer in Microsoft Visual C++ Express and i've run into a stump with a more or less simple program.
The aim is to open an input file, read the file and store it, then alphabetically sort and display them.
My programming is sound for the alphabetically sorting and displaying, but once I changed my code from manually inputing the names to trying to read them from an input file, things went horribly wrong. Here is my code so far: any help would be greatly appreciated.

Code:

Code:

#include
"stdafx.h"

#include
"string.h"

#define
 STRSIZE 21

#define
 MAXBAT 16


int
 alpha_first(char *list[], int min_sub, int max_sub);

void
 select_sort_str(char *list[], int n);


int
 main(void)

{

    
char batters[MAXBAT][STRSIZE];

    
char batman[STRSIZE];

    
char *alpha[MAXBAT];

    
int num_bat = 0;

    
int i = 0;

    
FILE *inpa;

    
int inpa_status;


inpa = fopen("H:\\GENG1003\\input2a.dat", "r");


inpa_status = fscanf(inpa, "%s", &batman);

num_bat = 16;


    
while (inpa_status != EOF){

        printf("\nPlayer:\n%s\n", batman);

        inpa_status = fscanf (inpa, "%s", &batman);

        strcpy(batman[STRSIZE],batters[MAXBAT]);

    }


    
for (i = 0; i < num_bat; ++i){

        alpha[i] = batters[i];

    }

    select_sort_str(alpha, num_bat);

    

    printf("\n\n%-30s%5c%-30s\n\n", "Application Order", ' ', "Alphabetical Order");

    
for (i = 0; i < num_bat; ++i){

        printf("%-30s%5c%-30s\n", batters[i], ' ', alpha[i]);

    }

    fclose(inpa);


    
return 0;

}

This is supposed to be C or C++? If it is really supposed to be C, I will move this thread accordingly.

Anyway, look closely:

Code:

inpa_status = fscanf(inpa, "%s", batters);

batters is an array of array of char. You really want to pass a pointer to char. Therefore, you need to provide it with an array of char, which will then be converted to a pointer to its first element, e.g.,

Code:

inpa_status = fscanf(inpa, "%s", batters[0]);

Since you will be doing this in a loop, you need a variable to keep track of which array of char you are reading into.

Originally Posted by siege

As for your question of it being C or C++...I'm actually unsure of that. I'm using a C++ compiler, but i'm still very much an amateur.

I am not asking what it is. I am asking what it is supposed to be. If you don't know what it is supposed to be, then you should decide. Otherwise, it is as if you are trying to learn Chinese in a Japanese language school: someone unfamiliar with both languages might think they are the same, but they are not.

Okay, i'll try coding it that way and see what happenes, thank you Whiteflags.

Ha! Yes, I see what you're getting at Laser. Based off what i've learned and seen, i'm going to assume 'C' and not 'C++'.

EDIT:

Yes, definatly 'C' programming language. Sorry for the confusion.