Thread: struct + pointer help..

hmm..so i have this struct where i want to hold some user data and a value so i know if a control is enabled or disabled. then i want to have a pointer, so that i can cut down on code when changing between two sets of data. here's the code i have..

Code:

struct player {
       int one[2];
       int two[2];
       int three[2];
       int four[2];
       int five[2];
       int six[2];
       int toak[2];
       int foak[2];
       int fh[2];
       int ss[2];
       int ls[2];
       int yahtzee[2];
       int chance[2];
       };

struct player *plyr;
struct player pone, ptwo;
.....
/* test */
void ChangePlayer(HWND hwnd)
{
     int i = 0;
     
     for (i = 500; i <= 512; i++)
     {
         SetDlgItemInt(hwnd, i, 0, FALSE);
     }
      EnableDisable(hwnd, FALSE, 500, 512);
     
     if (whosTurn == 1)
     {
         plyr = pone;
     }
     else if (whosTurn == 2)
     {
          plyr = ptwo;
     }
     else
     {
         MessageBox(0, "I have no idea who's turn it is..", "ERROR", MB_OK | MB_ICONEXCLAMATION);
     }
     
     SetDlgItemInt(hwnd, 600, plyr.one[0], FALSE);
     if (plyr.one[1] == 1) EnableDisable(hwnd, TRUE, 500, 0);
}

but at plyr=pone i get an error, "incompatible types in assignment".
could someone please tell me the error in my ways and how to get around this? i've tried googling some different combinations of pointer(s) and struct(s) but to no avail.

also, is there a way i could initialize each variable in the first position to 0 and in the second to one "automatically"? such that pone.one[0] == 0 && pone.one[1] == 1 ? like with char array's you can just go = {0}.?
thanks :]

Code:

player pone = { {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1} };
player ptwo = pone;

You need braces around arrays in structures when initializing their members. Also goes for structures which contain structures.


Quzah.

so uh...idk if i'm bad at this game or what, but i get errors.. lol

Code:

struct player *plyr;
struct player pone = { {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1} };
struct player ptwo = pone; // <-- line 44
----------
if (whosTurn == 1)
     {
         plyr = &pone;
     }
     else if (whosTurn == 2)
     {
          plyr = &ptwo;
     }
     else
     {
         MessageBox(0, "I have no idea who's turn it is..", "ERROR", MB_OK | MB_ICONEXCLAMATION);
     }
     
         SetDlgItemInt(hwnd, 600, plyr.one[0], FALSE);
         if (plyr.one[1] == 1) EnableDisable(hwnd, TRUE, 500, 0);
......

errors

yahtzee.c:44: error: initializer element is not constant

yahtzee.c: In function `ChangePlayer':
yahtzee.c:350: error: request for member `one' in something not a structure or union
yahtzee.c:351: error: request for member `one' in something not a structure or union

the problem is..? :] thx guys

Code:

typedef struct player
{
    int one[2];
    int two[2];
    int three[2];
    int four[2];
    int five[2];
    int six[2];
    int toak[2];
    int foak[2];
    int fh[2];
    int ss[2];
    int ls[2];
    int yahtzee[2];
    int chance[2];
} player;

int main( void )
{
    player *plyr;
    player pone = { {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1} };
    player ptwo = pone;
    
    return 0;
}

With exception to warnings about unused variables, this compiles without warning or error.

Code:

SetDlgItemInt(hwnd, 600, plyr.one[0], FALSE);

plyr is a pointer. As such, you use the arrow operator, not the dot operator, to access its members.


Quzah.

Code:

struct player pone = { {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1}, {0,1} };
This is an initialisation, which you can do OUTSIDE of functions, so long as all the members are constants.

struct player ptwo = pone; // <-- line 44
Just copy all the {} things

heh..sweet.
it compiles now :]

time for some testing..