Prototypen anlegen mit struct

Man macht in einer H-Datei keine Variablendeklarationen.
Oder allenfalls (da gibt es aber Meinungsverschiedenheiten) mittels 
einem Conditional define für "extern".

Da Du die H-Datei in main.c und httpd.c einfügst werden die Variablen 
natürlich doppelt deklariert.
Ändere das und alles wird gut.

Also das ist im relais.c - file:

1

uint16_t RelaisSchalten( struct Relais* theRelais)  

2

{

3

4

  AusgangSetBit( theRelais->bitToSet );

5

  AusgangUnsetBit( theRelais->bitToClear ); 

6

  AusgangOut();

7

8

  theRelais->setCycles++;

9

10

11

  if (theRelais->setCycles >= 200)

12

    {

13

    AusgangUnsetBit( theRelais->bitToSet ); 

14

    AusgangUnsetBit( theRelais->bitToClear ); 

15

    AusgangOut();

16

17

    theRelais->setCycles = 0;

18

    return true;

19

    }  

20

21

  return false;

22

}

Und das im relais.h file :

struct Relais {
   uint16_t   bitToSet;
   uint16_t   bitToClear;
   uint16_t   setCycles;     // Zaehlt die 0-Durchgänge bis zum 
Abschalten
};


struct Relais   K1_Ein   = { 1, 0, 0 };
struct Relais   K1_Aus   = { 0, 1, 1 };

struct Relais   K2_Ein   = { 3, 2, 2 };
struct Relais   K2_Aus   = { 2, 3, 3 };

struct Relais   K3_Ein   = { 5, 4, 4 };
struct Relais   K3_Aus   = { 4, 5, 5 };

struct Relais   K4_Ein   = { 7, 6, 6 };
struct Relais   K4_Aus   = { 6, 7, 7 };

struct Relais   K5_Ein   = { 9, 8, 8 };
struct Relais   K5_Aus   = { 8, 9, 9 };

struct Relais   K6_Ein   = { 11, 10, 10 };
struct Relais   K6_Aus   = { 10, 11, 11 };

struct Relais   K7_Ein   = { 13, 12, 12 };
struct Relais   K7_Aus   = { 12, 13, 13 };

struct Relais   K8_Ein   = { 15, 14, 14 };
struct Relais   K8_Aus   = { 14, 15, 15 };

//Prototyp

uint16_t RelaisSchalten( struct Relais* theRelais)  ;
[/c]

Hmm schrieb:
> Man macht in einer H-Datei keine Variablendeklarationen.
> Oder allenfalls (da gibt es aber Meinungsverschiedenheiten) mittels
> einem Conditional define für "extern".

Also Variablendeklarationen nur im c.File sozusagen?!

Stefan Ernst hat mal was recht. Holt den Kalender. :-)

Ja. Ist richtig. Es handelt sich um Definitonen. Mein Fehler.

Stefan Ernst schrieb:
> Hmm schrieb:
>> Man macht in einer H-Datei keine Variablendeklarationen.
>
> Doch, genau da gehören sie hin. Aber eben keine Definitionen, wie die
> oben.

Definition heisst:
Dem Compiler sagen, dass er dieses etwas anlegen soll. So stehts im 
Buch. Das dient dann doch als "Prototyp". Ich bin zu blöd dafür, btte 
klär mich noch nen bischen mehr auf...

Also das ist für mich die Definition:

 uint16_t RelaisSchalten( struct Relais* theRelais)  ;

>Definition heisst:
>Dem Compiler sagen, dass er dieses etwas anlegen soll. So stehts im
Buch.
Ja. Richtig. In Bezug auf Variablen und Funktionen.

>Das dient dann doch als "Prototyp".

Definitionen "können" Deklarationen sozusagen "ersetzen", wenn sie vor 
der ersten Verwendung stehen.

Aber unter einem "Prototyp" versteht man eine Deklaration. Insofern ist 
ein Prototyp, also eine Deklaration nicht identisch mit einer 
Definition.

> Also das ist für mich die Definition:
> uint16_t RelaisSchalten( struct Relais* theRelais)  ;

Nein. Das ist eine Deklaration.

Puhh...ich kriege es nicht hin.

Ich habe z.B. eine einfache Funktion im .c file:

1

//Min Max begrenzen

2

uint16_t MinMax (uint16_t Input, uint16_t Min , uint16_t Max)

3

{

4

            if (Input <= Min)

5

                Input = Min;  

6

7

            if (Input >= Max)

8

                Input = Max;

9

10

      return Input;

11

12

}

In der .h
[c
uint16_t MinMax (uint16_t Input, uint16_t Min , uint16_t Max);
[/c]

und alles läuft. Warum nicht das mit der Struktur als Übergabewert?

Wie muss ich's denn machen?

Das was ich oben schon geschrieben habe unter: 
Beitrag "Re: Prototypen anlegen mit struct"
Nur habe ich anstelle von Deklaration, das Wort Definition geschrieben.

D.h. Du hast mehrfache Definitionen von Variablen. Das ist das Problem. 
Nicht das struct.

Das ist doch genau das, was die Fehlermeldungen sagen:

httpd.o:(.data.K1_Aus+0x0): multiple definition of `K1_Aus'
main.o:(.data.K1_Aus+0x0): first defined here


"multiple definition" heisst: "mehrfache Definition".

Hmm schrieb:
> Das ist doch genau das, was die Fehlermeldungen sagen:
>
> httpd.o:(.data.K1_Aus+0x0): multiple definition of `K1_Aus'
> main.o:(.data.K1_Aus+0x0): first defined here
>
>
> "multiple definition" heisst: "mehrfache Definition".

Wo soll ich dann das hin packen?

1

struct Relais {

2

   uint16_t   bitToSet; 

3

   uint16_t   bitToClear;

4

   uint16_t   setCycles;     // Zaehlt die 0-Durchgänge bis zum Abschalten

5

};

6

7

struct Relais   K1_Ein   = { 1, 0, 0 };

8

struct Relais   K1_Aus   = { 0, 1, 1 };

9

10

struct Relais   K2_Ein   = { 3, 2, 2 };

11

struct Relais   K2_Aus   = { 2, 3, 3 };

12

13

....

>> "multiple definition" heisst: "mehrfache Definition".
>
> Wo soll ich dann das hin packen?
>


Das musst doch in die H-Datei. Ok, dadurch dsas die H datei woanders 
auch noch eingebunden war gab es multiple definitions. Aber wie muss ich 
es denn machen?

Die einfachste Anfängermöglichkeit ist folgende:

1. Du kopierst die Variablendefinitionen aus der H-Datei in main.c Damit 
werden eben dort die Definitionen gemacht.

2. Du setzt das Wort "extern" vor die Variablendefinitionen in der 
h-Datei. Damit werden das Deklarationen. Sie werden also beim include in 
main.c und httpd.c nicht mehr doppelt definiert.

Denke daran die Initialisierungen aus der H-Datei zu löschen.

In main.c siehts so aus:

1

extern struct Relais   K1_Ein   = { 1, 0, 0 };

2

extern struct Relais   K1_Aus   = { 0, 1, 1 };

3

4

extern struct Relais   K2_Ein   = { 3, 2, 2 };

5

extern struct Relais   K2_Aus   = { 2, 3, 3 };

6

7

.....

in der anderen .h steht das noch:

1

struct Relais {

2

   uint16_t   bitToSet; 

3

   uint16_t   bitToClear;

4

   uint16_t   setCycles;     // Zaehlt die 0-Durchgänge bis zum Abschalten

5

};

in dem anderen .c file steht das:

1

uint16_t RelaisSchalten( struct Relais* theRelais)  

2

{

3

4

  AusgangSetBit( theRelais->bitToSet );

5

  AusgangUnsetBit( theRelais->bitToClear ); 

6

  AusgangOut();

7

8

  theRelais->setCycles++;

9

10

11

  if (theRelais->setCycles >= 200)

12

    {

13

    AusgangUnsetBit( theRelais->bitToSet ); 

14

    AusgangUnsetBit( theRelais->bitToClear ); 

15

    AusgangOut();

16

17

    theRelais->setCycles = 0;

18

    return true;

19

    }  

20

21

  return false;

22

}

dan kommt diese Meldung:
Compiling: main.c
avr-gcc -c -mmcu=atmega1284p -I. -Os -funsigned-char 
-funsigned-bitfields -fpack-struct -fshort-enums -Wall 
-Wstrict-prototypes -Wa,-adhlns=main.lst  -fno-inline-small-functions 
-fno-split-wide-types -fno-move-loop-invariants -mcall-prologues 
-ffunction-sections -fdata-sections -Wl,--gc-sections -Wl,--relax 
-std=gnu99 main.c -o main.o
main.c:89: warning: 'K1_Ein' initialized and declared 'extern'
main.c:90: warning: 'K1_Aus' initialized and declared 'extern'
main.c:92: warning: 'K2_Ein' initialized and declared 'extern'
main.c:93: warning: 'K2_Aus' initialized and declared 'extern'

Georg G. schrieb:
> Das folgende Beispiel compiliert ohne Fehler. Du darfst selbst suchen,
> was du falsch gemacht hast (einfach nur jeden Leerraum, jedes Zeichen
> ansehen)
>
> #include <avr/io.h>
> #include <avr/interrupt.h>


Danke ich begehe mich auf die Suche!

Ja. Schön. Aber was soll das bringen, das Du das erstmal in einer 
anderen Variante falsch machst? :-)

Warum nicht einfach so wie hier 
Beitrag "Re: Prototypen anlegen mit struct" beschrieben?

Georg G. schrieb:
> Der Hauptfehler ist ein fehlendes Leerzeichen... aber mehrfach
> systematisch.

?????????????? Wo ?

Der macht hier nur den Troll. Einfach ignorieren.
Da fehlen keine Leerzeichen.

>>Hmm schrieb:
>> Man macht in einer H-Datei keine Variablendeklarationen.
>Dafür sind Header doch da?
>Du meinst Variablendefinitonen?

@ Johann L.
Ich würde sogar sagen es handelt sich um Variablendefinitionen. Dieser 
Hmm ist ein absoluter Idiot.

Ich bin auch komplett verwirrt ;-(

>Ich bin auch komplett verwirrt ;-(

Meine Güte.
Du bist hier dran: Beitrag "Re: Prototypen anlegen mit struct"

LOL.

Klaus Wachtler schrieb:

>nicht ganz - nur die erste geht durch den Compiler, der Rest nicht, weil
>mehrfach definiert

Ach Mensch! Das sagst Du jetzt? Was mache ich jetzt mit der Tüte 
Leerzeichen?
Au fein, ich weiss. Zusammen mit Zucker und Himbeeren in den Quark 
rühren.

<8 ??? Wie stehts denn bei Dir. Schon was geändert? Oder beim 
Abendessen?

Ich guck jetzt "Stalker". Das passt jetzt.

Was? Wie? Nein, nein. Andrej Tarkowskij!

Wer? Ach der! Nein, nein. Mit Alexander Kajdanowski.

Ach Rufus! Spielen wir wieder Schultze&Schulze? :-)
Siehe: Beitrag "Re: Prototypen anlegen mit struct"

@ Rufus

>... aber danach so viel ... unsubstantiiertes...
Hust. Räusper. Also wie Du die Auslassungszeichen setzt! Unglaublich.