Classen in C++

[EWeiss]

Wäre klasse wenn ich das teil noch in eine function bekomme
Hab ich es einmal verstanden sollte das kein Problem mehr sein

Code:

   // Find available Language Files
    void *hSearch;
    WIN32_FIND_DATA wfd;
    char szFile[MAX_PATH];
    hSearch = FindFirstFile("*.lng", &wfd);
    GetPrivateProfileString("Common", "Language", "English", szLang, 256, szFileINI);

    nCurrentLanguage = MENUID_FIRST_LANGUAGE;
    AppendMenu(m_hLanguageSubMenu, MF_STRING, MENUID_FIRST_LANGUAGE, "&English");
    nLanguages = 1;

    if (hSearch != INVALID_HANDLE_VALUE)
    {
       do
       {
          strcpy(szFile, wfd.cFileName);
          szFile[strlen(szFile) - 4] = 0;
          if (!strcmp(szFile, "English"))
             continue;

          if (!strcmp(szFile, szLang))
             nCurrentLanguage = MENUID_FIRST_LANGUAGE + nLanguages;
          AppendMenu(m_hLanguageSubMenu, MF_STRING, MENUID_FIRST_LANGUAGE + nLanguages, szFile);
          nLanguages++;
       }
       while (FindNextFile(hSearch, &wfd));
    }
    FindClose(hSearch);

Sollte ein Array der Language Files zurück gegeben werden.
Funktioniert ist aber zur zeit nicht als function ausgelegt.

Sollte Teil der language.h sein

gruss

[BUG]

Zitat von EWeiss:

Sollte ein Array der Language Files zurück gegeben werden.
Funktioniert ist aber zur zeit nicht als function ausgelegt.

Soll das eine Funktion werden oder eine Methode von CLanguage?
Ich könnte mir vorstellen, das szLang die Klassenvariable von CLanguage ist. Aber wo kommt m_hLanguageSubMenu her? Das gleiche gilt für nLanguages und nCurrentLanguage.

Zitat von EWeiss:

Sollte Teil der language.h sein

Warum? Soll das eine inline-Funktion werden? Ansonsten gibt es kaum einen guten Grund, warum du die Funktion im Header definieren solltest.


BTW: Wenn du weiter mit C++ arbeiten willst, solltest du dir die Standard-Library angucken. Nahe an den Betriebssystemfunktionen mögen Character-Buffer noch sinnvoll sein, ansonsten ist die Klasse std::string um Welten angenehmer. Auch die Container-Klassen sind ziemlich nützlich.

[EWeiss]

Zitat von BUG:

Zitat von EWeiss:

Sollte ein Array der Language Files zurück gegeben werden.
Funktioniert ist aber zur zeit nicht als function ausgelegt.

Soll das eine Funktion werden oder eine Methode von CLanguage?
Ich könnte mir vorstellen, das szLang die Klassenvariable von CLanguage ist. Aber wo kommt m_hLanguageSubMenu her? Das gleiche gilt für nLanguages und nCurrentLanguage.

Zitat von EWeiss:

Sollte Teil der language.h sein

Warum? Soll das eine inline-Funktion werden? Ansonsten gibt es kaum einen guten Grund, warum du die Funktion im Header definieren solltest.


BTW: Wenn du weiter mit C++ arbeiten willst, solltest du dir die Standard-Library angucken. Nahe an den Betriebssystemfunktionen mögen Character-Buffer noch sinnvoll sein, ansonsten ist die Klasse std::string um Welten angenehmer. Auch die Container-Klassen sind ziemlich nützlich.

hLanguageSubMenu.. JA hatte ich nicht bedacht.
Denke muss die abfrage dann wohl in der aktuellen Funktion belassen.
Ist aber nicht weiter tragisch da ich diese nur einmal aufrufe um im Ordner die Sprachdateien zu finden.

Man will es immer besonders gut machen und schlechten code will ich nicht gerade abliefern
Danke für die zusätzlichen Infos werden bestimmt mal nützlich sein.

Ach so.. Danke nochmal im nachhinein für mein Geburtstags Geschenk!
Die Classe läuft jetzt so wie sie soll.

gruss