Thread: Well, what do you think of my program?

Not bad for a first windows program.

But;
Why do you need 8 x 6000b buffers? (bit of over kill)
Use one large one (for validation) and then use lstrlen() to test if the user has entered a string that is too long to fit. Tell them to re-enter if they have.

Try an array of strings for the input and an array for the HWNDs. Use hash defines for the edits and then you can refrence both arrays with these defines.

Code:

#define  NUM_EDITS    8
#define  SURNAME      0
#define  FIRSTNAME    1
//ect
char     sArray[NUM_EDITS][64];
char     sBigBuffer[255];
HWND     hWndArray[NUM_EDITS];

for(i=0;i<NUM_EDITS;i++)
{
    GetWindowText(hWndArray[i],sBigBuffer,255);
    if(lstrlen(sBigBuffer)>64)
    {
         //do error
    }
    else sprintf(sArray[i],"%*s",64,sBigBuffer);
}

Get some dynaminc memory allocation in there to hold the data structure. Increase size each time the user saves.

I am working on the TAB from one box to another right now. I will post the new one with TAB when I'm done.

You need to set the style for TABSTOP when you create the buttons.

Also you have given all the buttons the same ID number (1). They should all be different.
Try a structure to hold the HWND, ID, text and other details of the button. Then you can use a loop to create them.
Which is why GetDlgItemText() won't work for you.