enum Konstanten

>Was habe ich nicht beachtet?

enums sind keine Strukturen.

Remo H. schrieb:
> MCP2515_FILTER_EXTENDED(SWC_RH_SWButtons.CAN_ID)

Einfach nur:
MCP2515_FILTER_EXTENDED(CAN_ID)

Ein enum definiert einfach nur eine Reihe "globalen" Konstanten.

In C++ kannst du "enum class" statt "enum" schreiben und dann:

MCP2515_FILTER_EXTENDED(SWC_RH_SWButtons::CAN_ID),

meinst du sowas? macht wahrscheinlich kein Sinn aber was solls.

typedef struct
{
   int CAN_ID;
   char DataByte;
   char NoKEY;
   char Bit_KEY_Rev;
   char Bit_KEY_Fwd;
   char Bit_KEY_VolDn;
   char Bit_KEY_VolUp;
   char Bit_KEY_Yes;
   char Bit_KEY_No;
}SRSB;

SRSB SWC_RH_SWButtons =
{
    0x131726C,
    7,
    0x3F,
    0,
    1,
    2,
    3,
    4,
    5
};

MCP2515_FILTER_EXTENDED(SWC_RH_SWButtons.CAN_ID),    // Filter 0
MCP2515_FILTER_EXTENDED(SWC_RH_SWButtons.CAN_ID),    // Filter 1

viel Glück.
Gruß
Laspalmas

in C++ wäre es:
MCP2515_FILTER_EXTENDED(SWC_RH_SWButtons::CAN_ID),    // Filter 0

Nur leider sind enums in Konstanten kacke in C, namespaces (Gruppierung) 
gibt es erst in C++.

Nimm  SWC_RH_SWButtons als Präfix, also

SWC_RH_SWButtons_CAN_ID  = 0x131726C
SWC_RH_SWButtons_DataByte    = 7,
...

so ein enum ist ja auch nicht dazu gedacht Konstanten für eine 
kunterbunte Struktur mit verschiedenen Feldern zu halten.
Typisches Beispiel ist ja eine Variable 'Ampelfarben' die dann 'rot, 
gelb, grün' sein kann. In C kann man das mit einem enum so lesbar machen 
das man sofort erkennt was der Autor möchte und ein smarter Editor kann 
die passenden Möglichkeiten bei einer Zuweisung anbieten.
In C ist das aber kein echter Typ und Ampelfarbe=99 oder =Katze ist für 
den Compiler auch ok. In C++ wurde das stufenweise verbessert und 
typsicher gemacht.
Für dein Beispiel wären also je Feldtyp eigene enums angebracht, eines 
für die CAN IDs und eines für die verschiedenen Funktionsbits.

dagegen spricht nichts solange man die 'teuren' Features wie Exceptions 
und RTTI weglässt. Aber das Thema ist eine ewige Diskussion hier.
Trotzdem sind enum in C auch brauchbar weil sie den Code eben besser 
lesbar machen und Intellisense weiss was es dir anbieten kann.

Und die CAN_ID kann ja sicher eine von mehreren sein. Dann würde ich das 
enum nutzen den IDs schönere Namen zu geben, also sowas wie 
'ID_SEND_BUTTONSTATE' oder sowas. Im enum kann man den Wert dieser IDs 
setzen, das war in deinem Beispiel ja auch schon drin. Nur eben ein enum 
alleine für IDs und nicht alles in einem.

Defines sind aber reine Konvention und ermöglichen höchstens sowas wie 
'#define CAN_ID_xy', da kann Intellisense nicht helfen. Bei enums sind 
die passenden Möglichkeiten eben in einer Gruppe aufgezählt, das ist 
schon etwas anderes.

dann wirfst du wohl immer noch Strukturen und enums durcheinander. Ein 
enum hat nix mit einer Struktur zu tun.

1

typedef enum  {

2

  ID1 = 0x12345678,

3

  ID2,

4

  ID3

5

} CAN_ID;

6

7

typedef enum {

8

  cat,

9

  dog

10

} ANIMAL;

11

12

13

void foo() {

14

  CAN_ID id;

15

16

  id = ID1;

17

  id = CAN_ID::ID1; // nur in cpp

18

  id = 99;

19

  id = dog;

20

}

in C ist obiges ok, in cpp (gcc c++11) bekommt man Fehler:
error: invalid conversion from 'int' to 'CAN_ID' [-fpermissive]
error: cannot convert 'ANIMAL' to 'CAN_ID' in assignment

das ist doch schon Grund genug den cpp zu benutzen.

Das Problem bei Dir ist doch, dass die konstanten so überhaupt nicht 
gruppiert werden sollten. C bewährt dich vor einem gefährlichen Weg, 
der in C++ mit Guruvoodoo zwar möglich, aber katastrophal wäre.

Du solltest Dich mit Typisierung vertraut machen.

Die ID, Anzahl der Bytes und bestimmte Bedeutungen der nutzbytes in ein 
konstantenfeld zu quetschen ist wirklich schräg, weil sie thematisch 
nichts miteinander zu tun haben.

Rolf M. schrieb:
> 1) definiert eigentlich keine echte Konstante, sondern nur eine
> Variable, die man nicht ändern soll.

und wie sieht es mit der Speicherbelegung aus? ;)

Da kommen jetzt solche schrecklichen Gedanken auf:

Patrick C. schrieb:
> benutze ich #define, oder bei strings um
> memory zu besparen

:(

> Was spricht eigentlich gegen C++?
> Wenn ich solche sachen nutzen möchte?

Nichts. Das Arduino Framework macht uns vor, dass es durchaus Sinn 
machen kann, nur Teile von C++ zu verwenden.