[C++] AfxMessageBox keuze maken

[SmidsY]

Ik ben bezig met een MFC programma in Visual C++. Ik wil dat als de gebruiker op een knop drukt een bericht krijgt met een keuze (ok en cancel) kiest hij ok, dan wordt de reset uitgevoerd, cancel, dan gebeurt er niets.
Ik heb m.b.v. msdn (http://msdn.microsoft.com...ml/_mfc_afxmessagebox.asp en aanverwante pagina's) het volgende stuk code geschreven,

C++:

void CArtikelView::OnBnClickedResetRecord()
{
    AfxMessageBox("Reset Database View!?",MB_ICONWARNING|MB_OKCANCEL);
    if (IDOK){
    m_pSet->Close(); 
    m_pSet->m_strFilter = ""; 
    m_pSet->Open();
    UpdateData(FALSE);
    }
    else (IDCANCEL);
}

Wat ik wil gebeurt, echter het maakt niet uit welke keuze je maakt (ok of cancel) hij voert toch de reset code uit. Wat doe ik fout!?

[Verwijderd]

je vergelijkt hier alleen de constantes IDOK en IDCANCEL, niet de uitkomst van AfxMessageBox, IDOK is blijkbaar ongelijk aan nul (zo gedefinieerd ), dus je if is altijd waar.

[ Voor 35% gewijzigd door Verwijderd op 06-05-2004 12:16 . Reden: verbeteren ]

[whoami]

code:

if( AfxMessageBox( "....." ) == IDOK )

Ik begrijp niet hoe je er toe komt om zo te testen:

code:

if( IDOK )

Met wat moet IDOK vergeleken worden dan?

[ Voor 60% gewijzigd door whoami op 06-05-2004 12:18 ]

[SmidsY]

whoami schreef op 06 mei 2004 @ 12:16:

code:

if( AfxMessageBox( "....." ) == IDOK )

Danku voor deze pointer, hiermee lukt het wel, vreemd dat ze dat niet als voorbeeld hebben gedaan in de msdn. De werkende code is nu alsvolgt,

C++:

void CArtikelView::OnBnClickedResetRecord()
{
    if( AfxMessageBox("Reset Database View!?",MB_ICONWARNING|MB_OKCANCEL) == IDOK ){
        m_pSet->Close(); 
        m_pSet->m_strFilter = ""; 
        m_pSet->Open();
        UpdateData(FALSE);
    }
    else;
}

[whoami]

Er staat toch in de MSDN wat de returnwaarde van die functie is, en je weet toch hoe je op return-waarde moet checken?

[SmidsY]

whoami schreef op 06 mei 2004 @ 12:32:
Er staat toch in de MSDN wat de returnwaarde van die functie is, en je weet toch hoe je op return-waarde moet checken?

Ja dat klopt, echter dat had ik geinterpeteerd op de manier zoals ik de eerste code had gepost, niet de correcte methode dus

[curry684]

Mjah dit is basic debuggen, pak een C++ tutorial en copy/paste de code goed uit MSDN