C++ Button problem

Quote:

Original post by stevek2k ... Quote: hButton = CreateWindow("BUTTON", row[col], WS_VISIBLE|WS_CHILD|BS_PUSHBUTTON|BS_BITMAP,temp2, temp1, 135, 27, hwnd, 0, g_hInst, 0); The idea of this was to have the id as the value of the button, which would then be picked up by Quote: switch(HIWORD(wParam)) { case BN_CLICKED: { switch(LOWORD(wParam)) { .... default: { //ID PICKED UP HERE } ... Can anyone tell me if there is a way of finding the button value, or at least a more senisble way of doing this, the method i was working on was only thing i could come up with at the time and im sure theres a more efficent and less sloppy way of doing it.

What do you mean by "button value"?

You already have the button's resource id with "LOWORD(wParam)"... and for most WM_COMMAND notifications, the value of the button's HWND is just the LPARAM (cast to HWND)

Unfortunately this is just how Windows' messaging system delivers the button event to you.

You can use std::map<> to map an event to a functor

std::map<UINT, functor_ptr_t> events;

where "functor_ptr_t" is the type of your functor pointer... if you're talking about managing the "sloppyness" of the switch statement in WndProc()...

If the map is used inside a class, the map should be static... and you'd have to register some type of "destroy" mechanism to empty the map when the program exits (use atexit() to register a type void ( *)() function that will delete the pointers in the map)

....

If you're just talking about a different way to tell which button was pressed, use a different map that depends on the "LOWORD(wParam)", or resource ID of your button...

std::map<WORD, functor_ptr_t>  button_events;

You would set these mappings (buttons to events) in your set-up sequence in the application code... something like:

//pseudo-codebutton_events.insert(std::make_pair(IDC_BUTTON1, new functor_t(function ptr here...)));// probably not the safest way to create the functor; if the functor constructor fails, //it will place a null function pointer into the map

Some ideas for you anyway. Was this what you were asking anyway?


Chad

(reworded this a little so easier to understand)
Ok can I just start by saying i'm fairly new to C++ so be gentle.

So where to start, well, i'm attempting to develop a client that connects to a MySQL database and takes infomation of it, so far that has been fine and i've managed it. When you are on a particular window I want it to display various cutdown records, once again that has gone fine, on this particular page I want to give the user the option to view each record display in a seperate window with all details via the use of a button or something along those lines.

Ok i'll give you an example of what I mean, say we have a table which holds user infomation, on the first form only a list of the usernames will be displayed, when a button is clicked a seperate window with all the details about that user e.g. first name, surname, address etc.

Now this is were the issue arrises that I can't get my head around, basically using the user example, each user with have their own unique ID, which will be a number, say admin is 1, test 15, steve 135 etc. (much the same way forums work) how can I go about making the username on the first form clickable, I thought about adding a view button to the right of each username, but when you pass it through the

switch(HIWORD(wParam))
{
case BN_CLICKED:

there is no way of knowing which username is clicked, obviously I could assign each button with a ID and called it via

switch(LOWORD(wParam))
{
case ID_BUTTON:

but obviously since the users id's are quite random and numerous at times it would require great effort and sloppyness to do it this way (unless theres a method I do not currently know)

Now looking at this I decided prehaps it was possible to look at the value that was written on the button e.g.
(ascii inturuptation of button ;p)
------
| 15 | <- value = 15
------

e.g. (where row[col]==15)
hButton = CreateWindow("BUTTON", row[col], WS_VISIBLE|WS_CHILD|BS_PUSHBUTTON|BS_BITMAP,temp2, temp1, 135, 27, hwnd, 0, g_hInst, 0);

and place an image over it so you never seen the value, okay its not nice but it would do what i want it to do, but once again I have no idea who the WM_COMMAND would pick up the value.

Ok now onto idea 3.
Instead of displaying the usernames in a static text box or edit box display them in a list, each record of which is double clickable, once again I fall into the problem of knowing which record was actually clicked.

Needless to say this is driving me crazy and I cant find anything on Google that would help, any alternatives or solutions to the current problems would be much appriecated.

Hope this makes sense to someone, and hope someone can point me in right direction, and remember I'm learning from scratch as i'm doing this so dont make it to complicated.

ahh, sorry bout that verg, i tried reading what you had given me but couldnt really understand it that well, in my head it didnt assiocate to what i asked so guessed you had got the wrong end of the stick cus i'd explained it bad.

I'm gonna give what mr. anonymous said there a go as well, but after fiddling all night this is what i ended up with so far and seem to be making a little progress, first I decided to go with the listbox

so i created the listbox and then used

int index = SendMessage(list,LB_ADDSTRING,0,(LPARAM)(LPCSTR)tempQuery);

to add the string to it, also grabbing the index, then with that passed itemdata that would represent the user_id

SendMessage(list, LB_SETITEMDATA, index, user_id);

Ok this kinda seems to work, however the problem is all itemdata seems to be the last user_id entered, even with fiddling with certain parameters it either stays like that, all user_ids become 0 or it just uses the first user_id

Needless to say ive got a messagebox that is continously going to alert me of what itemdata is being passed, it just doesnt seem to want to hold the info.

Any idea whats wrong with this, as im eventually starting to get somewhere now i've chosen a path and got some documentation to help (all of which say what i have should work)

If i cant move foward on this ill have to definately have to get my head around the advice given here previous and go on with that as best i can

nevermind figured out the list bit :)

was being really stupid, was using

int indexIs = SendMessage(list,LB_GETITEMDATA,0,0);

so it never knew what index it was meant to be looking for so

int indexIs = SendMessage(list,LB_GETCURSEL,0,0);
int itemData = SendMessage(list,LB_GETITEMDATA,indexIs,0);

and it works fine, i was to busy looking at how it was being put in rather than how it was looked up *slaps forehead*

Thanks for help anyway, can really push foward now i know this