Question about struct

If I was trying to pass the struct "stats" to the function sortStats, I need a parameter type. I tried struct but it doesn't like that as it seems to think I'm trying to create a new struct or something.

Code:

---

int sortStats(int numPlayers, ?? stats)

---

Code:

---

typedef struct
{
      int ind;
      char player[25];
      char team[4];
      char pos[2];
      int gamesPlayed;
      int goals;
      int assists;
      int points;
      int plusMinus;
      int penaltyMins;
      int ppGoals;
      int shGoals;
      int gwGoals;
      int otGoals;
      int shots;
      float shotPercent;
      char timeOnIce[6];
      float shiftsPerGame;
      float faceOffWinPercent;
} stats;

int sortStats(int numPlayers, stats)
{
    int mostGoals = 0;
    int mostGoalsInd = 0;
    int i = 0;
    for (i=0;i<numPlayers;i++)
    {
        if (myPlayers[i].goals > mostGoals)
        {
            mostGoals = myPlayers[i].goals;
            mostGoalsInd = myPlayers[i].ind;
        }
    }
    //printf("&#37;s scored the most goals this season with %d.\n", myPlayers[mostGoalsInd], mostGoals);

    return 0;
}

int main(int argc, char *argv[])
{
  int ind, gamesPlayed, goals, assists, points, plusMinus, penaltyMins, ppGoals, shGoals, gwGoals, otGoals, shots;
  float shotPercent, shiftsPerGame, faceOffWinPercent;
  char player[50];
  char team[4];
  char pos[2];
  char timeOnIce[10];
  int strPlayer = 0;
  int strTimeOnIce = 0;
  int numPlayers;
  char myStr[] = "nhl0708.txt";
  char myPlayers[1000];
  numPlayers = getNumPlayers(myStr);
  getLength(numPlayers, myStr);
  loadData(numPlayers, myStr);
  sortStats(numPlayers, stats);
  system("PAUSE");
  return 0;
}

---

This is where I'm at now...

A parameter needs a type, and then a name.
You have the type - "stats". Then all you need is to name the parameter.

In case you are confused, you declare a type with typedef. You declare the struct to be called stats. So stats in a new user-defined type. So if you wanted an array of int you would do this:

Code:

---

int sortStats(int numPlayers, int name[size])

---

if you want an array of stats you do this:

Code:

---

int sortStats(int numPlayers, stats name[size])

---

Hah ok. Fixed, and another error in main now when calling sortStats...almost got to the last line!

Code:

---

sortStats(numPlayers, stats);

---

syntax error before stats

*edit*
I was confused for a minute, but thought about it and figured it out, and then remembered the previous error when I was passing a single array. Now to this point...

stats is still a type, not an object. You need to pass your array.

Quote:

---

Originally Posted by C_ntua

In case you are confused, you declare a type with typedef. You declare the struct to be called stats.

---

That is what the OP did.

Quote:

---

Originally Posted by mikeman

Hah ok. Fixed, and another error in main now when calling sortStats...almost got to the last line!

Code:

---

sortStats(numPlayers, stats);

---

syntax error before stats

*edit*
I was confused for a minute, but thought about it and figured it out, and then remembered the previous error when I was passing a single array. Now to this point...

---

stats is type, not a variable.
You named your struct stats. You did not create an instance of stats.

Ok, so the name is myPlayers.

When I put that in, I get a warning saying [Warning] passing arg 2 of `sortStats' from incompatible pointer type

sortStats is expecting a stats struct, yet you pass a myPlayers array of type char[1000].
Not the same type, and probably not what you want to pass to sortStats, either, unless you want it to sort Players, in which case sortStats takes the wrong type of argument.

A bit confused now. I get what you're saying, I'm not sure where to go from here though. I've tried a few different things and gotten other errors. It seems to me like it should be struct, but I don't think that's proper--and it doesn't like it! Maybe even stats, since that is a type...but it doesn't like that either.

First step: create a struct that you need.
Optional step: rename it from struct X to X.
Second step: create an instance of the struct:
Third step: create function to pass struct to.
Fourth step: pass instance to function.

1) typedef struct { /* ... */ } X;
2) int main() { X x; }
3) void foo(X* x)
4) int main() { X x; foo(&x); }

Does that help, then?

So you need to have a line of code like so:

Code:

---

stats MyStats[1000];

---

(or however many) -- that's what gives you an array of stats to deal with. I would think loadData would want to have that array too, but that code was up there somewhere I missed it.

Well I think I've done all of that except for correctly passing the instance to the function.

I created the struct
I believe I created an instance of the struct within loadData

Code:

---

stats myPlayers[numPlayers];

---

I created the function to pass it to with the sortStats function.

So I need to pass an instance to sortStats...which makes me think I want myPlayers[i]

each myPlayers (myPlayers[0] myPlayers[1] .... myPlayers[852]) is an instance, no?

Yes, they are all instances.

Quote:

---

Originally Posted by mikeman

Well I think I've done all of that except for correctly passing the instance to the function.

I created the struct
I believe I created an instance of the struct within loadData

Code:

---

stats myPlayers[numPlayers];

---

---

Okay, but that doesn't count, because that goes away as soon as the loadData function finishes -- so there's no possible way sortStats can see it. It needs to exist in main. (You can create it in main and then pass it into loadData to be filled, and then into sortStats to be sorted.)

Quote:

---

Originally Posted by tabstop

Okay, but that doesn't count, because that goes away as soon as the loadData function finishes -- so there's no possible way sortStats can see it. It needs to exist in main. (You can create it in main and then pass it into loadData to be filled, and then into sortStats to be sorted.)

---

I understand it doesn't exist in main. I feel like Elysia was trying to lead me to figuring out how to do this without essentially running loadData in main. My only thought is to do something along the lines of

Code:

---

myPlayers = loadData(numPlayers, myStr)

---

But that doesn't work.