Thread: Can someone please help me with my program? I am new to C programming.

okay

Adak any ideas?

I tried changing my code a bit....This is as close as I got but i am still far from where I need to be

Code:

for(n=1; n <= ttl_golfers; n++){
    for(i = 0; i < 18; i++) {
      fscanf(ifp, "%d", &tries[n][i]);
      score[n][i] = tries[n][i]- parscore[i];
    }
    printf("Golfer#%d's score:%d\n\n", n, score[n][j]);
    }

I'm touching it up, now.

This is a substantial touch up, so be sure and copy it completely!

I did not check the scores for accuracy.

Code:

/*

Please someone help, I am completely all out of ideas....

Can someone just help me understand how to make the output of my program output like this:

Golfer #1: 1 1 3 4 2 3 4 2 5 -2 1 -2 -3(upto 18 holes)
Golfer #2:.......................... (upto 18 holes)
...

*/
#include <stdio.h>

int main() {
    FILE *ifp;
    int ttl_golfers, totalscore, i, n;
    int tries [9][18];
    int score [9][18];
    int parscore[18];

    ifp = fopen("golf.txt","rt");

    fscanf(ifp, "%d\n", &ttl_golfers);
    printf("%d golfers played on this golf course:\n\n", ttl_golfers);

    printf("\n\n\nPar is: ");
    for (i=0; i < 18; i++)  {
        fscanf(ifp, "%d ", &parscore[i]);
        printf("%d ", parscore[i]);
    }
    printf("\n\n              Scores Compared with Par\n");
    for(i = 0; i < 80; i++)
      putchar(196);
    printf(" Player  Hole: ");
    printf("1  2  3  4  5  6  7  8  9 10 11 12 13 14 15 16 17 18\n");
    for(i = 0; i < 80; i++)
      putchar(196);
    for(n=0; n < ttl_golfers; n++){
      if(n) putchar('\n');
      printf("Golfer#%d's:  ", n+1);
      for(i = 0; i < 18; i++) {
        fscanf(ifp, "%d", &tries[n][i]);
        score[n][i] = tries[n][i]- parscore[i];
        printf("%3d",  score[n][i]);
      }
    }
    putchar('\n');

    fclose(ifp);

    system("PAUSE");
     
    return 0;

}

Originally Posted by matthayzon89

What exactly is a buffer?

I do not think anyone has actually answered this directly, but you can check the FOLDOC entry for buffer. With respect to nonpuz's statement, it just means "an array of characters".

For an alternative definition, see buffer

Matt, You could use a 1-d array of structs (though each struct would have an array inside).
Or I guess you could use a 1-d array of length 18xN (This is really what a 2-d array is, but the compiler simplifies the syntax for you).

The array of structs is probably the best route to go. Structs help to keep your data organized and reduce the possibility of making mistakes such as transposing the array indices (eg writing [j][i] instead of [i][j]).

Originally Posted by matthayzon89

I was wondering, I talked to one of my friends and they said i would be able to use two 1d arrays for this problem if i want to, is that really possible? and is my method probably the best way to go? why? I would just like to understand when it is smart to use 1d vs. 2d arrays [coming from people who have experiences with them]

For what you've done so far (and no more), you could get by with just one 1D array of 18 elements, for the par scores. Everything else can be generated just as it's needed. The problem with that "minimal" type of program design, is that you're stuffed if you ever want to do anything more with the data. Like go back and correct a score - that would be a major pita, with this kind of design.

Moving up, you could use just one 2D array and have the rows organized:

row 1 is for the par scores,
row 2 for the tries of each golfer
row 3 is the 'real' scores of each golfer

In choosing your data structures, use the one that fits best, that makes it really clear what you're doing and, of course, is also efficient. In your case, the design is very clear, and obviously efficient.

In the case of interleaving rows I mention just above, the efficiency is good, but the clarity takes a step backward. If you wanted to look at the scores for golfer #7, would you know right off the bat, what row to look at?

If you said "14", right away, then it might be good for you to use it. If not, I wouldn't use it. The run time of this program is very short, and most of that, depends on the time it takes to read data from the file. So any change in data structure for better efficiency, is going to be a waste of time. If you want a very slight increase in speed, then get a faster hard drive.

The keyboarder entering the data - there's the real bottleneck.