Hallo
Ich habe ein kleines Problem mit einem Cpp-Code
und finde den Fehler nicht.
Den Code habe ich im Anhang dazu gepackt.
Ich möchte dazu sagen, dass ich kein C-Profi bin, sondern noch lerne.
Folgendes:
Beim compilieren mit der Gnu-Compiler-Collection erhalte ich folgende
Fehlermeldung:
include/statemachine.cpp:26:6: error: ‘stateFlags’ was not declared in
this scope
if (stateFlags & (1<<FLAG_CHANGED))
^
Was ich nicht verstehe ist, dass die anderen Variablen, die genauso
deklariert wurden funktionieren.
Der Code ist einfach aufgebaut.
Ich möchte eine kleine Klasse schreiben namens Statemachine.
Diese hat 3 Routinen:
- setState(newState) zum setzen eines neuen Zustandes
- getState() zum Abfragen des Zustandes
- changed() zum prüfen, ob der Zustand sich geändert hat
Dafür gibt es eine entsprechende Header-Datei und die zugehörige
Code-Datei. Das ganze rufe ich in mein.cpp zum Testen auf.
Zum kompilieren habe ich eine makefile dabei.
Kann mir jemand den Fehler sagen, warum nur die Variable stateFlags
angeblich nicht deklariert sein soll, während die anderen Variablen
(stateCurrend und stateOld) tatellos verarbeitet werden?
Hier mal die Klassendefinition:
#ifndef _statemachine_h
#define _statemachine_h
#include <stdint.h>
#include <stdio.h>
//#include <string.h>
class Statemachine
{
public:
Statemachine(); // Constructor
void setState(uint8_t newState);
uint8_t getState();
bool changed();
private:
// internal variables
uint8_t stateCurrent; // Current State
uint8_t stateOld; // Previous State
uint8_t stateFlags; // Flags
// Definitions stateFlags
#define FLAG_CHANGED 0 // Flag State Changed
};
#endif // _statemachine_h
Wäre toll, wenn ihr mir helfen könnt.
Danke.
Micha
Gast
#5327632
Hallo! changed() ist nicht als Methode der Klasse definert, daher hat diese Routine keinen Zugriff auf die member. Es müsste in statemachine.cpp bool Statemachine::changed() statt bool changed() heissen.
Ojeh, ich vermute das war ein typischer Anfängerfehler. Jetzt funktioniert es, vielen Dank! Micha
Micha R. schrieb: > // Definitions stateFlags > #define FLAG_CHANGED 0 // Flag State Changed Was ist das denn? Ein #define innerhalb einer Klassendeklaration? Das ist unübersichtlich, und vor allem bezieht es sich nicht auf die Klasse, es gilt überall.
Wie gesagt, ich bin da Anfänger. Vielleicht sollte anstatt '#define' besser 'const' verwendet werden. Wie definieren und an welcher stelle müsste ich das dann, wenn die Konstanten privat nur in der Klasse Gültigkeit haben, definieren? Micha
Micha R. schrieb: > Vielleicht sollte anstatt '#define' besser 'const' verwendet werden. Genau so. Sofern sich dem noch weitere Werte hinzugesellen, könntest Du auch ein enum verwenden. > Wie definieren und an welcher stelle müsste ich das dann, An genau der bisherigen. Auch bei einem enum wäre das nicht anders, das gehört dann ebenfalls zur Klasse. Und damit ist auch das Namespace-Problem von enums in C umgangen, denn das enum gehört zur Klasse.
Wenn ich in der Klassendefinition im Block private schreibe:
private:
// internal variables
uint8_t stateCurrent; // Current State
uint8_t stateOld; // Previous State
uint8_t stateFlags; // Flags
// Definitions stateFlags
const static uint8_t FLAG_CHANGED = 0; // Flag State Changed
lässt es sich kcompilieren und FLAG_CHANGED ist in main.cpp ungültig.
So soll es sein.
Aber ohne static gibts Comilier-Fehler.
Muss static in dem Fall sein und
belegt das keinen Speicher (1 Byte)?
Wie wäre es, wenn ich die Definition des Flags
so mache
#define FLAG_CHANGED 0
aber die Definition in Statemachine.cpp, also nicht im Header ablege?
Das hat sich auch compilieren lassen und im main.cpp war das Flag
unbekannt. Also richtig.
Wäre das eine option?
Gast
#5327805
Möglichkeiten: - static - In Initialisierungsliste des Konstruktors initialisieren - enum verwenden https://stackoverflow.com/questions/14495536/how-to-initialize-const-member-variable-in-a-class > #define FLAG_CHANGED 0 Nein, das wäre Käse (und FLAG_CHANGED keine Klassenkonstante).
Gast
#5327815
BTW: Welche Compilerversion bzw. welche Flags benutzt du eigentlich?
Einfach "const" sollte natürlich ab C++11 funktioneren:
class Statemachine
{
const uint8_t FLAG_CHANGED = 0;
};
Danke für die vielen Hinweise In denke, "static const uint8_t FLAG_CHANGED=0;" belegt vermutlich ein Byte Speicher. ich habs jetzt mit enum gelöst.
1 | |
2 | |
3 | |
4 | |
5 | |
6 | |
7 | |
8 | |
9 | |
10 | |
11 | |
12 | |
13 | |