asked on

Spin control... yet again

Me again,
Something is going wrong... refer to the history of the question...
I inserted this into OnInitDialog...
OnVScroll(SB_THUMBPOSITION,m_spin1,NULL)
It compiles and crashes...oops!
When I debug, it perfroms that line and goes to OnVScroll and gets hit somewhere around if(bla bla bla)
HELP PLEEZ... I am now getting desperate
Thanx

rose3377

ASKER

Does this problem have something to do with the fact that the dialog is not yet displayed and all the DDV,DDX stuff?

Another thing...
If I overlook the inital display problem and I continue, when I press up arrow, it increments in 0.1 which is great.
If I type in 12 and press up arrow, it becomes 1.3
How do I correct this....?

tsauy

Yes, you need to wait until the window is ready to display. You can put OnVScroll() in OnPaint(). If you don't want to reload OnVScroll() afterwards, which means you only want to run once when the window starts, you need a flag to disable it, such as:
void YouClass::OnPaint()
{
.
.
static bool RunOnceFlag=false;
if(!RunOnceFlag) {
OnVScroll();
RunOnceFlag=true;
}
}

Answer to second question: You could use integer and then devided by 10.

rose3377

ASKER

Refer to the history of the question in titled Spin Control again
I placed that code in OnPaint. The program gets to OnVScroll...

CString strValue;
strValue.Format("%3.1f", (double) nPos / 10.0);
***((CSpinButtonCtrl*) pScrollBar)->GetBuddy()
->SetWindowText(strValue);

and crashes on the line with stars...

rose3377

ASKER

See Comment Above...

jrmcg

Yours:

***((CSpinButtonCtrl*) pScrollBar)->GetBuddy()
->SetWindowText(strValue);

There should be parenthesis around the GetBuddy function. Remember, it is returnin a CWnd*.

Try:::
(((CSpinButtonCtrl*)pScrollBar)->GetBuddy())->SetWindowText(strValue);

Does that do it?

ASKER CERTIFIED SOLUTION

mikeblas

membership

This solution is only available to members.

To access this solution, you must be a member of Experts Exchange.

Start Free Trial

mikeblas

Calling a notification function directly, like tsauy suggests to do with OnVScroll() directly from OnPaint(), is completely sick and wrong. If you're unlucky, it will actually work and you'll be fooled until your users upgrade to a new verison of windows and it breaks. Nominally, it'll cause a crash.

Don't do it; it's wrong.

B ekiM