hab ein Problem mit folgendem Kode:

// test.h
namespace myNS {
	typedef unsigned int	myEnum;

	extern const myEnum ME_TIME;
	extern const myEnum ME_ALPHA;
	// ..
}

// anwendung.cpp
namespace andererNS {

	// ..
	using namespace myNS;
	switch (/*... */)
	{
	case ME_TIME:
		// ..
	case ME_ALPHA:
		// ..
	}
	// ..
}

und dazu folgenden Kompilerfehler:

f:\Quellkode\CPP\Tritratrallala\BlendingDemo\PatternDialog.cpp(42): error C2051: case-Ausdruck ist keine Konstante
f:\Quellkode\CPP\Tritratrallala\BlendingDemo\PatternDialog.cpp(43): error C2051: case-Ausdruck ist keine Konstante
f:\Quellkode\CPP\Tritratrallala\BlendingDemo\PatternDialog.cpp(44): error C2051: case-Ausdruck ist keine Konstante
usw ...

Der Fehler ist mir unerklärlich, da die case-Variable eine Konstante ist.

Was habe ich übersehen?

Danke,
don_basto.

Gruß Caipi

test.h:

namespace myNS
{
	typedef unsigned int myEnum;

	extern const myEnum ME_TIME;
    extern const myEnum ME_ALPHA;

	unsigned int getType(int type);
}

test.cpp:

#include "test.h"

namespace myNS
{
	const myEnum ME_TIME	= 0x0001;
    const myEnum ME_ALPHA	= 0X0002;

	unsigned int getType(int type)
	{
		if (type == 1)	return ME_TIME;
		else			return ME_ALPHA;
	}
}

anwendung.h:

namespace andererNS
{
	class myApp
	{
	public:
		void doSomething();
	};
}

anwendung.cpp:

#include "test.h"
#include "anwendung.h"

namespace andererNS
{
	void myApp::doSomething()
	{
		using namespace myNS;
		switch (getType(1))
		{
		case ME_TIME:
			break;
		case ME_ALPHA:
			break;
		}
	}
}

Und die passenden Fehler:

f:\Quellkode\CPP\Tritratrallala\SwitchTest\anwendung.cpp(15): warning C4060: switch-Anweisung enthält weder 'case'- noch 'default'-Marken
f:\Quellkode\CPP\Tritratrallala\SwitchTest\anwendung.cpp(13): error C2051: case-Ausdruck ist keine Konstante
f:\Quellkode\CPP\Tritratrallala\SwitchTest\anwendung.cpp(11): error C2051: case-Ausdruck ist keine Konstante

Vllt. hilft's ja weiter ..

In anwendung.cpp müssen die Variablen ME_TIME und ME_ALPHA zur compile-Zeit konstant sein, was aber nicht möglich ist, da sie als extern const deklariert sind und somit erst durch den Linker, der nach dem Compiler kommt, den Speicherbereich zugewiesen bekommen.

Vergl. z.B. auch den folgenden Code:

// main.cpp

// Externes Linkage (extern ist hier Pflicht!). Siehe dazu auch weiter unten.
extern const unsigned int global_var1 = 1;
extern const unsigned int global_var2 = 2;

void foo(unsigned int);

int main()
{
    cout << global_var1 << endl;
    foo(1);
    cout << endl;
}

// test.cpp

// extern-Deklaration. (Siehe dazu ebenfalls weiter unten).
extern const unsigned int global_var1;
extern const unsigned int global_var2;

void foo(unsigned int value)
{
    switch(value)
    {
        case global_var1: // ist zur compile-Zeit nicht konstant, da externes Linkage.
            cout << "global_var1"; 
            break;
        case global_var2: // dito.
            cout << "global_var2";
            break;
    }
}

Desweiteren enthält dein geposteter Code-Snippet einen weiteren Fehler: Globale (bzw. im namespace) definierte und mit dem Zugriffspezifizierer const versehene Variablen haben in C++ internes (und nicht externes) Linkage. Willst du Globale, konstante Variablen mit externem Linkage definieren, schreibst du:

extern const T var = ...;

(Wobei T der Typ und var der Bezeichner).

Gruß Caipi

Also dann werden die Konstanten nicht wahrgenommen, weil sie in einer anderen Übersetzungseinheit (test.cpp) definiert werden und somit beim Übersetzen von anwendung.cpp nicht bekannt sind, _obwohl_ über die Variablen bekannt ist, dass sie konstant sind (Deklaration in test.h). Das Verhalten ist nicht ganz intuitiv.
Die Lösung ist jetzt, die Definitionen von test.cpp in test.h zu verschieben, wenn 's auch nicht sehr schön ist.

Caipi schrieb:

Desweiteren enthält dein geposteter Code-Snippet einen weiteren Fehler: Globale (bzw. im namespace) definierte und mit dem Zugriffspezifizierer const versehene Variablen haben in C++ internes (und nicht externes) Linkage. Willst du Globale, konstante Variablen mit externem Linkage definieren, schreibst du:

extern const T var = ...;

(Wobei T der Typ und var der Bezeichner).

Was meinst du damit?
Wenn ich extern verwende, werde ich es ja garantiert woanders definieren und wenn ich's gleich definieren kann/will, brauch ich das Schlüsselwort extern nicht. Die Begriffe externes/internes Linkage muss ich erstmal nachschlagen :).

Grüsse,
don_basto.

so wie du es deklariert hast sind ME_TIME und ME_ALPHA variablen vom typ myEnum. und variablen sind natürlich nicht konstant. int-konstanten deklariert man in c++ am besten mit enum {}:

namespace myNS {
    enum myEnum
    {
      ME_TIME  = 1,
      ME_ALPHA = 2
    };
    // ..
}

(PS: Wieso ist es dir so wichtig, daß deine Konstanten EXTERNE Bindung bekommen?)

const myEnum ME_TIME = 1;
const myEnum ME_ALPHA = 2;

Hab gelesen, dass man grudnsätzlich keine Variablen in den Headern definieren sollte, wegen Mehrdeutigkeiten beim Linken.

Konstanten ohne explizite extern-Angabe bekommen automatisch interne Linkage, also bekommt der Linker sie überhaupt nicht zu sehen. Problematisch wird es nur mit "echten" Variablen, die doppelt dekalriert werden.

case expression not constant

was wiederum ok ist. Da hat wohl jemand bei der Übersetzung nicht aufgepasst und

case expression not a constant

gelesen.