Classen in C++

[EWeiss]

Sorry ja!
Ich weis das ist ein Delphi Forum aber vielleicht kann mir trotzdem jemand helfen.
Ich arbeite an einer C++ Anwendung und verstehe die Erstellung der Klassen anscheinend nicht so richtig.
Ich verwende immer wieder gleich auftretende Funktionen die ich gerne in eine Klasse einsetzen würde
mein Problem ist aber das ich dann Probleme mit dem Linker und dem Zugriff auf dieser Klasse habe.

Code:

char *sTextMenu[MAX_MENU_STRINGS] = 
{"Setting",                //0
"Language",                //1
"Mode",                    //2
"no effect",               //3
"life",                    //4
"static color",            //5
"color changer",           //6
"color changer LR",        //7
"channel assignment",      //8
"Info",                    //9
"Exit",                    //10
};

CString sMenuText[MAX_MENU_STRINGS];
char szFileINI[MAX_PATH];
char szCurrentDir[MAX_PATH];

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);

Code:

   // Read Buffer from IniFile
   sprintf(szTemp, "%s\\%s.lng\0", szCurrentDir, szLang);
   for (i = 0; i < MAX_MENU_STRINGS; i++)
   {
      sprintf(szParam, "%d\0", i);
      GetPrivateProfileString("Menu", szParam, sTextMenu[i], Buffer, 512, szTemp);
      sMenuText[i] = Buffer;
      sMenuText[i].Replace("\\t", "\t");
      sMenuText[i].Replace("\\n", "\n");
   }

Wie kann ich jetzt das alles in eine Klasse unterbringen?
Und von allen anderen drauf zugreifen ohne besagte Linker error.

Gruss

[Namenloser]

Wie sieht deine Klasse aus und was genau sagt der Linker?

[EWeiss]

Zitat von Namenloser:

Wie sieht deine Klasse aus und was genau sagt der Linker?

Language.h

Code:

#pragma once

class Language
{
public:
   Language(void);
   virtual ~Language(void);
};

Language.cpp

Code:

#include "stdafx.h"
#include "Language.h"
#include "atlstr.h"

Language::Language(void)
{
}


Language::~Language(void)
{
}

Ich habe noch keine bzw. mich aber vorher schon dran versucht.
Der Linker meldet dann das die Deklaration unter Verwendung wären.

Normaler weise sollte es doch möglich sein eine Language.h zu erstellen auf der ich dann aus allen Dialogen zugreifen kann, der Übersetzung wegen halt.
Da kommen aber extrem viele Linker Fehler Meldungen.

EDIT:
Hab mal eine Default Klasse addiert.

gruss

[EWeiss]

Habe es nochmal versucht
Language.cpp

Code:

#include "stdafx.h"
#include "Language.h"

CLanguage::CLanguage(void)
{
}


CLanguage::~CLanguage(void)
{
}

char CLanguage::GetCurrentDir() 
{
   int i = GetCurrentDirectory(255, (char*)szCurrentDir);
   if (szCurrentDir[i - 1] == '\\')
     szCurrentDir[i - 1] = 0;

   return szCurrentDir[i];
}

Language.h

Code:

#pragma once

#include "atlstr.h"

#define MAX_CHANNELASSIGNMENT_STRINGS 6

char *sTextChannelAssignment[MAX_CHANNELASSIGNMENT_STRINGS] = 
{"Off",                    //0
"Top",                     //1
"Right",                   //2
"Bottom",                  //3
"Left",                    //4
"Combined",                //5
};

class CLanguage
{
private:

public:
   static CString sText[MAX_CHANNELASSIGNMENT_STRINGS];
  static char szCurrentDir[MAX_PATH];
  static char szFileINI[MAX_PATH];
  static char szLang[MAX_PATH], szTemp[512], Buffer[1024], szParam[32];   

   static char GetCurrentDir();

   CLanguage(void);
   virtual ~CLanguage(void);

};

Wenn ich jetzt die Language.h zu meiner EditChannelAssignment.cpp addiere bekomme ich beim kompilieren
Jede menge unresolved external und andere diverse Fehler.
Mir fällt nicht ein was da falsch sein sollte.

Code:

sprintf(szFileINI, "%s\\Language.ini\0", CLanguage::GetCurrentDir());

Dieser Aufruf sollte mir dann eigentlich die Directory zurückgeben
Aber wie wenn sich der Quelltext nicht mal compilieren lässt.

Fehlt mir noch irgendein include?
Ohne die beiden Dateien funktioniert alles es muss also mit dem einbinden dieser zu tun haben.

PS:
Warum die Variablen jetzt alle static sein müssen obwohl puplic deklariert versteh ich jetzt auch nicht.

gruss

[Namenloser]

Wie kompilierst du?

Ich hatte auch mal so ein Problem und es lag einfach daran, dass die .cpp-Datei niemals kompiliert wurde. Die .h-Dateien werden ja immer automatisch kompiliert, weil sie ja inkludiert werden, aber der Compiler erkennt nicht automatisch, dass die .cpp-Datei dazugehört. Deshalb muss die extra im Makefile angegeben werden. Vielleicht fehlt sie dort.

[EWeiss]

Zitat von Namenloser:

Wie kompilierst du?

Ich hatte auch mal so ein Problem und es lag einfach daran, dass die .cpp-Datei niemals kompiliert wurde. Die .h-Dateien werden ja immer automatisch kompiliert, weil sie ja inkludiert werden, aber der Compiler erkennt nicht automatisch, dass die .cpp-Datei dazugehört. Deshalb muss die extra im Makefile angegeben werden. Vielleicht fehlt sie dort.

Die frage wäre dann ist das nun C oder C++.
Ich verwende Visual Studio 2012 wüsste jetzt nicht ob ich da jetzt ein MakeFile anlegen muss, glaube aber nicht.
IN VS ist sie im Projekt eingebunden müsste also erkannt werden.

Werde nochmal einen versuch machen und dann alle Fehler posten.

gruss

[Namenloser]

Es macht eigentlich keinen Unterschied, ob C oder C++, das Kompilieren läuft bei beiden nach dem gleichen Prinzip ab. Wenn du eine IDE verwendest, dann legt die wahrscheinlich das Makefile automatisch für dich an.

[EWeiss]

Das sind die Fehler

Zitat:

Error 9 error LNK2005: "char * * sTextChannelAssignment" (?sTextChannelAssignment@@3PAPADA) already defined in Language.obj C:\Program Files (x86)\Microsoft Visual Studio\VB98\Sourcemein\AtmoLight Projekt\AtmoWin\AtmoEditChannelAssignment.obj AtmoWinA
Error 12 error LNK2001: unresolved external symbol "public: static class ATL::CStringT<char,class ATL::StrTraitATL<char,class ATL::ChTraitsCRT<char> > > * CLanguage::sText" (?sText@CLanguage@@2PAV?$CStringT@DV?$StrTraitATL@ DV?$ChTraitsCRT@D@ATL@@@ATL@@@ATL@@A) C:\Program Files (x86)\Microsoft Visual Studio\VB98\Sourcemein\AtmoLight Projekt\AtmoWin\AtmoEditChannelAssignment.obj AtmoWinA
Error 15 error LNK2001: unresolved external symbol "public: static char * CLanguage::szTemp" (?szTemp@CLanguage@@2PADA) C:\Program Files (x86)\Microsoft Visual Studio\VB98\Sourcemein\AtmoLight Projekt\AtmoWin\AtmoEditChannelAssignment.obj AtmoWinA
Error 17 error LNK2001: unresolved external symbol "public: static char * CLanguage::szParam" (?szParam@CLanguage@@2PADA) C:\Program Files (x86)\Microsoft Visual Studio\VB98\Sourcemein\AtmoLight Projekt\AtmoWin\AtmoEditChannelAssignment.obj AtmoWinA
Error 14 error LNK2001: unresolved external symbol "public: static char * CLanguage::szLang" (?szLang@CLanguage@@2PADA) C:\Program Files (x86)\Microsoft Visual Studio\VB98\Sourcemein\AtmoLight Projekt\AtmoWin\AtmoEditChannelAssignment.obj AtmoWinA
Error 13 error LNK2001: unresolved external symbol "public: static char * CLanguage::szFileINI" (?szFileINI@CLanguage@@2PADA) C:\Program Files (x86)\Microsoft Visual Studio\VB98\Sourcemein\AtmoLight Projekt\AtmoWin\AtmoEditChannelAssignment.obj AtmoWinA
Error 10 error LNK2001: unresolved external symbol "public: static char * CLanguage::szCurrentDir" (?szCurrentDir@CLanguage@@2PADA) C:\Program Files (x86)\Microsoft Visual Studio\VB98\Sourcemein\AtmoLight Projekt\AtmoWin\Language.obj AtmoWinA
Error 11 error LNK2001: unresolved external symbol "public: static char * CLanguage::szCurrentDir" (?szCurrentDir@CLanguage@@2PADA) C:\Program Files (x86)\Microsoft Visual Studio\VB98\Sourcemein\AtmoLight Projekt\AtmoWin\AtmoEditChannelAssignment.obj AtmoWinA
Error 16 error LNK2001: unresolved external symbol "public: static char * CLanguage::Buffer" (?Buffer@CLanguage@@2PADA) C:\Program Files (x86)\Microsoft Visual Studio\VB98\Sourcemein\AtmoLight Projekt\AtmoWin\AtmoEditChannelAssignment.obj AtmoWinA
Error 18 error LNK1120: 7 unresolved externals C:\Program Files (x86)\Microsoft Visual Studio\VB98\Sourcemein\AtmoLight Projekt\AtmoWin\Release\AtmoWinA.exe 1 1 AtmoWinA

Der Witz dabei sTextChannelAssignment already defined in Language.obj
Wurde nirgends anders definiert

gruss

[Namenloser]

Zitat von EWeiss:

Das sind die Fehler

Zitat:

Error 9 error LNK2005: "char * * sTextChannelAssignment" (?sTextChannelAssignment@@3PAPADA) already defined in Language.obj C:\Program Files (x86)\Microsoft Visual Studio\VB98\Sourcemein\AtmoLight Projekt\AtmoWin\AtmoEditChannelAssignment.obj AtmoWinA

Also ich würde mich zunächst mal um den ersten Fehler kümmern:

Offensichtlich ist die Variable sTextChannelAssignment schon woanders definiert. Du musst bei C++ (und C) beachten, dass die Dateien dort keine "Units" sind wie bei Delphi. Das heißt, es kann immer nur genau eine globale Variable mit einem bestimmten Namen im ganzen Programm (nicht bloß pro Datei) geben (das gleiche gilt für Konstanten, Klassennamen, Funktionsnamen etc.).

Die anderen Fehler sind möglicherweise Folgefehler, die daraus resultieren, dass wegen diesem Konflikt die .obj-Datei nicht gelinkt werden kann.

[EWeiss]

sTextChannelAssignment ist garantiert nicht doppelt deklariert.

Wenn ich auf sTextChannelAssignment gehe und mir die Definition davon aufzeigen lasse geht er direkt in die Language.h
auf

Code:

char *sTextChannelAssignment[MAX_CHANNELASSIGNMENT_STRINGS] = 
{"Off",                    //0
"Top",                     //1
"Right",                   //2
"Bottom",                  //3
"Left",                    //4
"Combined",                //5
};

Die ist garantiert nicht doppelt definiert.

gruss