Entlos funktion

Reply to Entlos funktion on Fri, 30 May 2003 19:51:00 GMT

Heinz — Fri, 30 May 2003 19:51:00 GMT

Ich haben folgens zusammen gebaut:

bool CHalfLifeSerialEditDlg::OnRadioCs() 
{
    int uebergabe = 0;

    CString CSKey$ = ReadKey(HKEY_CURRENT_USER,"Software\\Valve\\CounterStrike\\Settings","Key");
    CSKey$.Replace("-","");

    if (CSKey$ != "")
    {
        GetDlgItem(IDC_EDIT_KEYP1)->SetWindowText(CSKey$.Left(4));
        GetDlgItem(IDC_EDIT_KEYP2)->SetWindowText(CSKey$.Mid(4,5));
        GetDlgItem(IDC_EDIT_KEYP3)->SetWindowText(CSKey$.Right(4));

        uebergabe = 1;
    }
    else
    {
        Checkbool();

        uebergabe = 0;
    }
    return (uebergabe);
}

bool CHalfLifeSerialEditDlg::OnRadioHl() 
{
    int uebergabe = 0;

    CString HLKey$ = ReadKey(HKEY_CURRENT_USER,"Software\\Valve\\Half-Life\\Settings","Key");
    HLKey$.Replace("-","");

    if (HLKey$ != "")
    {
        MessageBox("HLKEY DA");

        GetDlgItem(IDC_EDIT_KEYP1)->SetWindowText(HLKey$.Left(4));
        GetDlgItem(IDC_EDIT_KEYP2)->SetWindowText(HLKey$.Mid(4,5));
        GetDlgItem(IDC_EDIT_KEYP3)->SetWindowText(HLKey$.Right(4));

        uebergabe = 1;
    }
    else
    {
        Checkbool();

        uebergabe = 0;
    }

    return (uebergabe);

void CHalfLifeSerialEditDlg::Checkbool()
{
    if (OnRadioHl() == 0 || OnRadioCs() == 0)
    {
        GetDlgItem(IDC_STATIC_NAVI)->SetWindowText("Es wurde weder ein CS noch ein HL CD-Key gefunden.\nAls einzutragendes Spiel wirt CS gewählt.");

        CButton *pRadiobuttonHL = (CButton*) GetDlgItem(IDC_RADIO_HL);
        pRadiobuttonHL->SetCheck(0);

        CButton *pRadiobuttonCS = (CButton*) GetDlgItem(IDC_RADIO_CS);
        pRadiobuttonCS->SetCheck(1);

        OnRadioCs();
    }
    else if (OnRadioHl() == 1 || OnRadioCs() == 1)
    {
        GetDlgItem(IDC_STATIC_NAVI)->SetWindowText("Es wurde ein CS und ein HL CD-Key gefunden.\nAls einzutragendes Spiel wirt CS gewählt.");

        CButton *pRadiobuttonHL = (CButton*) GetDlgItem(IDC_RADIO_HL);
        pRadiobuttonHL->SetCheck(0);

        CButton *pRadiobuttonCS = (CButton*) GetDlgItem(IDC_RADIO_CS);
        pRadiobuttonCS->SetCheck(1);

        OnRadioCs();
    }
    else if (OnRadioHl() == 0 || OnRadioCs() == 1)
    {
        GetDlgItem(IDC_STATIC_NAVI)->SetWindowText("Es wurde nur ein CS CD-Key gefunden.\nAls einzutragendes Spiel wirt CS gewählt.");

        CButton *pRadiobuttonHL = (CButton*) GetDlgItem(IDC_RADIO_HL);
        pRadiobuttonHL->SetCheck(0);

        CButton *pRadiobuttonCS = (CButton*) GetDlgItem(IDC_RADIO_CS);
        pRadiobuttonCS->SetCheck(1);

        OnRadioCs();

    }
    else if (OnRadioHl() == 1 || OnRadioCs() == 0)
    {
        GetDlgItem(IDC_STATIC_NAVI)->SetWindowText("Es wurde nur ein HL CD-Key gefunden.\nAls einzutragendes Spiel wirt HL gewählt.");

        CButton *pRadiobuttonHL = (CButton*) GetDlgItem(IDC_RADIO_HL);
        pRadiobuttonHL->SetCheck(1);

        CButton *pRadiobuttonCS = (CButton*) GetDlgItem(IDC_RADIO_CS);
        pRadiobuttonCS->SetCheck(0);

        OnRadioHl();

    }
    else
    {
        GetDlgItem(IDC_STATIC_NAVI)->SetWindowText("Das ist ein Fehler der garnicht möglich sein kann. Mh muss Memory fehler sein.");
    }
}
}

SO das aluft jetzt entlos lange wenn keiner der beiden keys vorhanden ist. Das soll er im prinzib nur 1 mal machen.

Reply to Entlos funktion on Fri, 30 May 2003 20:15:00 GMT

phlox81 — Fri, 30 May 2003 20:15:00 GMT

Ente gut, alles gut.

Reply to Entlos funktion on Fri, 30 May 2003 21:43:00 GMT

Heinz — Fri, 30 May 2003 21:43:00 GMT

Das hilft mir nich grade weiter.

Reply to Entlos funktion on Sun, 01 Jun 2003 13:22:00 GMT

Walli — Sun, 01 Jun 2003 13:22:00 GMT

Tipp: Schau dir mal deine Klammerung an:

return (uebergabe);
// <- Hmmm, fehlt hier nicht was???
void CHalfLifeSerialEditDlg::Checkbool()

Reply to Entlos funktion on Sun, 01 Jun 2003 14:32:00 GMT

nukleuz — Sun, 01 Jun 2003 14:32:00 GMT

genau dort fehlt die geschlossen klammer;)) dann sollte es laufen

Reply to Entlos funktion on Mon, 02 Jun 2003 03:08:00 GMT

Heinz — Mon, 02 Jun 2003 03:08:00 GMT

Die } ist im Programm vorhanden habe ich nur beim Kopieren vergessen.

Reply to Entlos funktion on Mon, 02 Jun 2003 06:20:00 GMT

RenéG — Mon, 02 Jun 2003 06:20:00 GMT

Oha, also ein bissel Logik wäre hier schon angebracht!

-> OnRadioCs()::Key=="" -> Checkbool()::Key=="" -> OnRadioCs()

Möglicherweise sind noch mehr solcher logischer Fehler drin!

Reply to Entlos funktion on Mon, 02 Jun 2003 13:46:00 GMT

Heinz — Mon, 02 Jun 2003 13:46:00 GMT

Ich habe das jetzt mit einem zähler gemacht dasmit er das nur 1 mal durch geht. Aber irgent wie geht er nich mal da rein wo er auswählen soll welche check er setzten soll. Darum poste ich mal alles:

long CHalfLifeSerialEditDlg::checkreg(HKEY pfad,CString dir,CString keyreg)
{
    long uebergabe;

    // -1 = pfad oder dir ist falsch
    //  0 = nur key ist falsch
    //  1 = alles richtig

    char checkreg[256];
    DWORD dwCount = sizeof(checkreg);

    CRegKey key;
    if(key.Open( pfad, dir) == ERROR_FILE_NOT_FOUND)
    {
        uebergabe = -1;
    }
    else
    {
            if(key.QueryValue(checkreg, keyreg, &dwCount) == ERROR_FILE_NOT_FOUND)
            {
                uebergabe = 0;
            }
            else
            {
                uebergabe = 1;
            }
    }
    key.Close();

    return (uebergabe);
}

CString CHalfLifeSerialEditDlg::ReadKey(HKEY pfad,CString dir,CString keyreg)
{
    CString uebergabe = "";

    // 0 = Pfad wurde nicht gefunden

    long checkregs = checkreg(pfad, dir, keyreg);

    if (checkregs == 1)
    {
            char checkreg[256];
            DWORD dwCount = sizeof(checkreg);

            CRegKey key;
            key.Open( pfad, dir);
            key.QueryValue(checkreg, keyreg, &dwCount);
            key.Close();

            uebergabe = checkreg;
    }

    return (uebergabe);
}

bool CHalfLifeSerialEditDlg::OnRadioCs() 
{
    int uebergabe = 0;

    CString CSKey$ = ReadKey(HKEY_CURRENT_USER,"Software\\Valve\\CounterStrike\\Settings","Key");
    CSKey$.Replace("-","");

    if (CSKey$ != "")
    {
        GetDlgItem(IDC_EDIT_KEYP1)->SetWindowText(CSKey$.Left(4));
        GetDlgItem(IDC_EDIT_KEYP2)->SetWindowText(CSKey$.Mid(4,5));
        GetDlgItem(IDC_EDIT_KEYP3)->SetWindowText(CSKey$.Right(4));

        uebergabe = 1;
    }
    else
    {
        uebergabe = 0;

        Checkbool();
    }
    return (uebergabe);
}

bool CHalfLifeSerialEditDlg::OnRadioHl() 
{
    int uebergabe = 0;

    CString HLKey$ = ReadKey(HKEY_CURRENT_USER,"Software\\Valve\\Half-Life\\Settings","Key");
    HLKey$.Replace("-","");

    if (HLKey$ != "")
    {
        GetDlgItem(IDC_EDIT_KEYP1)->SetWindowText(HLKey$.Left(4));
        GetDlgItem(IDC_EDIT_KEYP2)->SetWindowText(HLKey$.Mid(4,5));
        GetDlgItem(IDC_EDIT_KEYP3)->SetWindowText(HLKey$.Right(4));

        uebergabe = 1;
    }
    else
    {
        uebergabe = 0;

        Checkbool();
    }

    return (uebergabe);
}
void CHalfLifeSerialEditDlg::Checkbool()
{
    if (runcheck == 0)
    {
        if (OnRadioHl() == 0 || OnRadioCs() == 0)
        {
            GetDlgItem(IDC_STATIC_NAVI)->SetWindowText("Es wurde weder ein CS noch ein HL CD-Key gefunden.Als einzutragendes Spiel wirt CS gewählt.");

            CButton *pRadiobuttonHL = (CButton*) GetDlgItem(IDC_RADIO_HL);
            pRadiobuttonHL->SetCheck(0);

            CButton *pRadiobuttonCS = (CButton*) GetDlgItem(IDC_RADIO_CS);
            pRadiobuttonCS->SetCheck(1);

            MessageBox ("Na");

            OnRadioCs();
        }
        else if (OnRadioHl() == 1 || OnRadioCs() == 1)
        {
            GetDlgItem(IDC_STATIC_NAVI)->SetWindowText("Es wurde ein CS und ein HL CD-Key gefunden.\nAls einzutragendes Spiel wirt CS gewählt.");

            CButton *pRadiobuttonHL = (CButton*) GetDlgItem(IDC_RADIO_HL);
            pRadiobuttonHL->SetCheck(0);

            CButton *pRadiobuttonCS = (CButton*) GetDlgItem(IDC_RADIO_CS);
            pRadiobuttonCS->SetCheck(1);

            OnRadioCs();
        }
        else if (OnRadioHl() == 0 || OnRadioCs() == 1)
        {
            GetDlgItem(IDC_STATIC_NAVI)->SetWindowText("Es wurde nur ein CS CD-Key gefunden.\nAls einzutragendes Spiel wirt CS gewählt.");

            CButton *pRadiobuttonHL = (CButton*) GetDlgItem(IDC_RADIO_HL);
            pRadiobuttonHL->SetCheck(0);

            CButton *pRadiobuttonCS = (CButton*) GetDlgItem(IDC_RADIO_CS);
            pRadiobuttonCS->SetCheck(1);

            OnRadioCs();

        }
        else if (OnRadioHl() == 1 || OnRadioCs() == 0)
        {
            GetDlgItem(IDC_STATIC_NAVI)->SetWindowText("Es wurde nur ein HL CD-Key gefunden.\nAls einzutragendes Spiel wirt HL gewählt.");

            CButton *pRadiobuttonHL = (CButton*) GetDlgItem(IDC_RADIO_HL);
            pRadiobuttonHL->SetCheck(1);

            CButton *pRadiobuttonCS = (CButton*) GetDlgItem(IDC_RADIO_CS);
            pRadiobuttonCS->SetCheck(0);

            OnRadioHl();

        }
        else
        {
            GetDlgItem(IDC_STATIC_NAVI)->SetWindowText("Das ist ein Fehler der garnicht möglich sein kann. Mh muss Memory fehler sein.");
        }
    }

    runcheck = 1;
}

Reply to Entlos funktion on Mon, 02 Jun 2003 20:42:00 GMT

-bla- — Mon, 02 Jun 2003 20:42:00 GMT

En(t)los oder doch eher en(d)los.....

Reply to Entlos funktion on Mon, 02 Jun 2003 20:54:00 GMT

Heinz — Mon, 02 Jun 2003 20:54:00 GMT

ja ok du hast vollkommen recht. Aber habt ihr den keine Idee zu meinen Problem?

Reply to Entlos funktion on Mon, 02 Jun 2003 20:59:00 GMT

-bla- — Mon, 02 Jun 2003 20:59:00 GMT

wo bleibst denn hängen - so laut debugger ?

Reply to Entlos funktion on Mon, 02 Jun 2003 21:55:00 GMT

Heinz — Mon, 02 Jun 2003 21:55:00 GMT

Lauf alles einwandfrei durch. Nur geht er nich darein wo er soll.

Ich lese ja folgender werte aus:

HKEY_CURRENT_USER,"Software\\Valve\\CounterStrike\\Settings","Key"
und
HKEY_CURRENT_USER,"Software\\Valve\\Half-Life\\Settings","Key"

Dies beiden Keys sind 100% nich auf meinem Rechner vorhanden. Demnach müsten OnRadioCs() und OnRadioHl() beide eine 0 zurück geben. Das heißt das der RadioButton IDC_RADIO_CS markiert würd und eine Ausgabe von "Na" als test gemacht wirt. Und nich zu vergessen IDC_STATIC_NAVI mit der Info füllt. Das erfolgt aber net. Das is meine Problem. Das is mein Program der rest is ja nur Deklaraion usw.

Reply to Entlos funktion on Tue, 03 Jun 2003 04:25:00 GMT

MFK — Tue, 03 Jun 2003 04:25:00 GMT

Ist dir klar, dass diese beiden:

if (OnRadioHl() == 0 || OnRadioCs() == 0)
else if (OnRadioHl() == 1 || OnRadioCs() == 1)

nicht nur alle möglichen Fälle abdecken, sondern sich sogar überschneiden?

Ich würde auch empfehlen, OnRadioHl und OnRadioCs einmal am Anfang von Checkbool aufzurufen, und die Werte in zwei bool-Variablen zu speichern.

Reply to Entlos funktion on Tue, 03 Jun 2003 09:23:00 GMT

Heinz — Tue, 03 Jun 2003 09:23:00 GMT

ich habe nich nur 2 möglichkeiten

ic habe

0 0
1 1
0 1
und
1 0

Das sind meiner Meinung nach alle möglich keiten. Oder was meinst du?

Reply to Entlos funktion on Tue, 03 Jun 2003 09:28:00 GMT

MFK — Tue, 03 Jun 2003 09:28:00 GMT

Original erstellt von :
Das sind meiner Meinung nach alle möglich keiten. Oder was meinst du?

Das ist richtig. Aber das erste if deckt bereits drei davon ab.
Du schreibst ja OnRadioHl() == 0 || OnRadioCs() == 0.

Das trifft zu auf:
0 0
0 1
1 0

Reply to Entlos funktion on Tue, 03 Jun 2003 10:30:00 GMT

Heinz — Tue, 03 Jun 2003 10:30:00 GMT

Stimmt das müssem überall && hin stadt || aber das geht immer noch nicht. Das selbe Problem.

Reply to Entlos funktion on Tue, 03 Jun 2003 11:15:00 GMT

Hepi — Tue, 03 Jun 2003 11:15:00 GMT

Setz Dir doch mal ein paar Breakpoints und lies die Inhalte deiner Variablen dann aus.... dann siehst Du doch warum gewisse Punkte nicht angesprungen werden!

...du weißt hoffentlich, wie man mit einem Debugger solchen Problemen auf den Grund geht, oder?

Scheinbar nicht...

Reply to Entlos funktion on Tue, 03 Jun 2003 11:19:00 GMT

Walli — Tue, 03 Jun 2003 11:19:00 GMT

Vielleicht sollten wir mal ein kleines Debugger-Tutorial oder wenigstens ein paar Links dazu in die FAQ aufnehmen... Naja, FAQ ist vielleicht ne blöde Idee weil man das nur mit Text nicht so einfach erklären kann.

Reply to Entlos funktion on Tue, 03 Jun 2003 11:40:00 GMT

junix — Tue, 03 Jun 2003 11:40:00 GMT

Die Erkenntnisse des Artikels der hier vorgestellt wird, sind zwar vordergründig für den Borland C++ Builder, lassen sich aber auch bestimmt auf MSVC übertragen... Die Mittel sind die Selben (und in MSVC meines Erachtens sogar z.T. bequemer).

-junix