typedef struct führt zu unknown type name

[Kostjan K. (Gast)]

Hi,

ich habe versucht ein Demoprogramm was 2006 programmiert wurde mit Atmel 
Studio 6.1 nachzubilden und kriege vom Compiler die folgende 
Fehlermeldung: unknown type name 'CANMessage'

Ich habe es ohne typedef probiert. Oder in dem ich dieses struct vor den 
relevanten Funktionen deklariert habe (und auch definiert). Jetzt bin 
ich ziemlich ratlos. Kann mir Anfänger jemand erklären, woran sich der 
Compiler genau stört und wie ich das lösen kann?

Anbei die relevanten Code-Ausschnitte:

void can_send_message(CANMessage *p_message)
{
    uint8_t length = p_message->length;
   
    // ID einstellen
    mcp2515_write_register(TXB0SIDH, (uint8_t) (p_message->id>>3));
    mcp2515_write_register(TXB0SIDL, (uint8_t) (p_message->id<<5));
   
    // Ist die Nachricht ein "Remote Transmit Request"
    if (p_message->rtr)
    {
        /* Eine RTR Nachricht hat zwar eine Laenge,
           aber keine Daten */
       
        // Nachrichten Laenge + RTR einstellen
        mcp2515_write_register(TXB0DLC, (1<<RTR) | length);
    }
    else
    {
        // Nachrichten Laenge einstellen
        mcp2515_write_register(TXB0DLC, length);
       
        // Daten
        for (uint8_t i=0;i<length;i++) {
            mcp2515_write_register(TXB0D0 + i, p_message->data[i]);
        }
    }
   
    // CAN Nachricht verschicken
    PORT_CS &= ~(1<<P_CS);
    spi_putc(SPI_RTS | 0x01);
    PORT_CS |= (1<<P_CS);
}



int main(void)
{
  mcp2515_init();
  
typedef struct{
    uint16_t  id;
    uint8_t   rtr;
    uint8_t   length;
    uint8_t   data[8];
} CANMessage;

// Neue Nachricht erzeugen
CANMessage message;

// Daten eintragen
message.id = 0x0123;
message.rtr = 0;
message.length = 2;
message.data[0] = 0x04;
message.data[1] = 0xf3;


// Nachricht verschicken
    while(1)
    {
        can_send_message(&message);
    _delay_ms(5);
    }
}

[Peter II (Gast)]

teste mal so

struct CANMessageS {
    uint16_t  id;
    uint8_t   rtr;
    uint8_t   length;
    uint8_t   data[8];
};

typedef CANMessageS CANMessageT;


und jetzt kannst du CANMessageT als variablen Type verwenden.

CANMessageT message;

[Jim M. (turboj)]

Du benutzt den Typ bevor er definiert wurde:

void can_send_message(CANMessage *p_message)
//...
int main(void)
{
  mcp2515_init();
  
typedef struct{
    uint16_t  id;
    uint8_t   rtr;
    uint8_t   length;
    uint8_t   data[8];
} CANMessage;
//...
}

Hier ist der Typ außerdem nur innerhalb von "main()" definiert. Besorge 
Dir ein gutes Buch über die "C" Programmiersprache und arbeite es durch 
- das ist ein Anfängerfehler.

[Nase (Gast)]

Den Typ CanMessage vielleicht vor can_send_message() bekanntmachen?

[Kostjan K. (Gast)]

Peter II schrieb:
> teste mal so
>
> struct CANMessageS {
>     uint16_t  id;
>     uint8_t   rtr;
>     uint8_t   length;
>     uint8_t   data[8];
> };
>
> typedef CANMessageS CANMessageT;
>
>
> und jetzt kannst du CANMessageT als variablen Type verwenden.
>
> CANMessageT message;


Das führt bei mir zu einer weiteren "unknown type 
CANMessageT"-Fehlermeldung. Und vier Fehlermeldungen mit diesem Inhalt: 
request for member 'id' in something not a structure or union

Nase schrieb:
> Den Typ CanMessage vielleicht vor can_send_message() bekanntmachen?

Hab ich wie oben erwähnt bereits probiert. Wie muss so eine 
Bekanntmachung denn aussehen?
Ich habe probiert:
struct CANMessage;
struct CANMessage message;
Dann noch mit eckigen Klammern, sogar die ganze Definition vom struct 
habe ich vor der relevanten Funktion und der main() probiert. Das hat 
die eine Fehlermeldung aber nicht gelöst, sondern sogar weitere 
Fehlermeldungen hervorgerufen.

Ich habe den Code bloß übernommen und laut Kommentatoren soll er bereits 
funktionsfähig sein.

[Johann L. (gjlayde)]

Peter II schrieb:
> teste mal so
>
> struct CANMessageS {
>     uint16_t  id;
>     uint8_t   rtr;
>     uint8_t   length;
>     uint8_t   data[8];
> };
>
> typedef CANMessageS CANMessageT;
>
> und jetzt kannst du CANMessageT als variablen Type verwenden.
>
> CANMessageT message;

Warum schreibst du so'n Käse? Der Ursprüngliche Typedef war korrekt, 
vermutlich steht der nur an der falschen Stelle.

Wenn überhaupt dann

typedef struct CANMessageS CANMessageT;

aber weil das Original in der Beziehung bereits korrekt ist braucht's 
hier echt keine Nebelkerzen!

[Peter II (Gast)]

Johann L. schrieb:
> Warum schreibst du so'n Käse? Der Ursprüngliche Typedef war korrekt,
> vermutlich steht der nur an der falschen Stelle.

er hat aber geschrieben, das er es schon "oben" hingeschrieben hat.

> Wenn überhaupt dann
> typedef struct CANMessageS CANMessageT;

ja ok, war ein fehler.

[Rufus Τ. F. (rufus)]

Peter II schrieb:
> er hat aber geschrieben, das er es schon "oben" hingeschrieben hat.

Hat er nicht, er hat nur einen Versuch einer Vorwärtsdeklaration dort 
untergebracht.

Das typedef hat nichts innerhalb einer Funktion verloren, und es 
gehört vollständig --d.h. nicht als "Vorwärtsdeklaration"-- vor die 
erste Stelle, an der es genutzt wird.

Beim oben geposteten Codefragment also ganz an den Anfang, nocht vor 
die Funktion can_send_message.

Die Struktur, die mit dem typedef genutzt wird, kann eine namenlose 
Struktur bleiben,

typedef struct
{
  uint16_t  id;
  uint8_t   rtr;
  uint8_t   length;
  uint8_t   data[8];
} CANMessage;

ist also zulässig.

Soll aus irgendwelchen Gründen die Struktur zuätzlich ohne typedef 
genutzt werden, dann kann ihr noch ein Name verpasst werden:

typedef struct CANMessage_struct
{
  uint16_t  id;
  uint8_t   rtr;
  uint8_t   length;
  uint8_t   data[8];
} CANMessage;

*Aber:* Das muss alles vor dem allerersten Gebrauch geschehen, also 
vor der Deklaration und erst recht der Definition von Funktionen oder 
Variablen des Typs.