mfc fejl?

Er tallet -842150451, eller?

Det er svært at gætte hvor fejlen er ud fra det beskrevne, noget kunne tyde på at variablen bliver overskrevet.

Post noget mere kode.

tallet starter ihvertfald med -8xxxxxxxx....
koden jeg bruger til at få variablens værdi og kalder funktionen er her
CKoeb dlg;
   
    UpdateData(true);
    dlg.DoModal();
    m_nKoebsAntal = 0;
    m_nKoebsAntal = dlg.m_nKoebsAntal;
    int nIndex = m_lbVarer.GetCurSel();
    if (nIndex != LB_ERR)
    {
    m_lbVarer.GetText(nIndex , m_strValg);
    }
    UpdateData(false);
    if(m_strValg != "")
    {
        IndtegnEgneVarer();
    }
}

void CButikDlg::IndtegnEgneVarer()
{
UpdateData(true);
if(m_strValg == m_strKniv)
{
    MessageBox(_T("Kniv"));
    m_nKoebtAntalKniv = m_nKoebtAntalKniv + m_nKoebsAntal;
}
sprintf(m_chAntalKniv, "%d", m_nKoebtAntalKniv);
MessageBox(m_chAntalKniv);
}

jeg har fundet ud af at fejlen nok ligger i sætningen:
m_nKoebtAntalKniv = m_nKoebtAntalKniv + m_nKoebsAntal;
da hvis jeg undlader den og konvertere m_nKoebsAntal istedet for m_nKoebtAntalKniv giver det den rigtige værdi i messageboxen? hvad gør jeg galt i den sætning eller i den kode blok?

Du har:
    CKoeb dlg;
    ...
    m_nKoebsAntal = 0;
    m_nKoebsAntal = dlg.m_nKoebsAntal;

Hvorfor sætter du m_nKoebsAntal to gange?

Er m_nKoebtAntalKniv blevet sat til at være noget inden linien med:
m_nKoebtAntalKniv = m_nKoebtAntalKniv + m_nKoebsAntal;

Jeg går ud fra at m_nKoebsAntal er deb samme son den m_nKoebsAntal du har ovenfor.

m_nKoebtAntalKniv = m_nKoebtAntalKniv + m_nKoebsAntal;
Er det samme som:
m_nKoebtAntalKniv += m_nKoebsAntal;

Member variable der ikke er blevet initialiseret vil blive sat til 0xCDCDCDCD = -842150451 af "kompileren" i debug builds.

heh.. havde innitialiseret m_nKoebtAntalKniv længere oppe.. have bare gjort det på en underlig måde... men virker nu... takker...

Et par kommentarer:

Generelt bør du ikke gemme værdier som let kan beregnes
Det ser du til at du har 3 members, m_nKoebtAntalKniv, m_nKoebtAntalKniv og m_nKoebsAntal hvor den første er summen af de 2 andre.
I stedet kunne du lave en simpel funktion:
GetKoebtAntalKniv() { return m_nKoebtAntalKniv + m_nKoebsAntal; };
Det kan nemlig give problemer hvis du opdaterer én variabel, men ikke de andre.


Hvilken type er m_chAntalKniv?
Er det blot en char som du skriver, eller er det et char array?

Du kan iøvrigt bruge CString's Format-metode istedet for sprintf, altså:
sprintf(m_chAntalKniv, "%d", m_nKoebtAntalKniv);
kan erstattes med:
m_strAntalKniv.Format("%d", m_nKoebtAntalKniv);
eller m_strAntalKniv.Format("%d", m_GetKoebtAntalKniv());
Hvor m_strAntalKniv er en CString.


Bare et par kommentarer, held og lykke med det...

lukker

hvis nogle mener de fortjente point skal i være velkomne til at kontakte mig sleizer@msn.com