conflicting types for 'fonction'

Ohne Gewähr: muss evtl. im .h vor der Deklaration noch "extern" stehen.

.                                                . schrieb im Beitrag 
#5545444:
> Da sind sicher noch andere Fehlermeldungen.
> Weiß denn der Header wo "struct tval" definiert ist?

es gibt zwar andere Fehlermeldungen wie z.B.'val_printf' declared 
'static' but never defined.  oder 'struct tval' declared inside 
parameter list will not be visible outside of this declaration or 
definition; aber ich denke die sind an die erste meldung verbunden

Domi schrieb:
> lass mal das struct weg
>
> static void val_printf(tval *message_Id)

funktionniert leider nicht

50c schrieb:
> ...in diesem speziellen Fall oder allgemein?

In C ist extern bei Funktionsprototypen das implizite Gegenteil von 
static und kann daher immer wegfallen.

Dr. Sommer schrieb:
> Domi schrieb:
>> lass mal das struct weg
>>
>> static void val_printf(tval *message_Id)
>
> Dann geht's gar nicht in C. Witzig, wie hier wieder mit Halbwissen
> herumgeraten wird. Es fehlt jedenfalls mal der komplette Sourcecode,
> insbesondere der Header.

das ist der Coder


#define V  0
#define W       1
#define X   2
#define Y  3
#define Z  4


struct tval
{
  int typ;
  union
  {
    float v;
    float y;
    uint16_x;
    uint16_t y;
    uint32_t z;
  }val;

};
static void val_printf(struct tval *message_Id)
{
  switch(message_Id -> typ)
  {
  case V:
    printf("v = %.2f\n", message_Id -> val.v);
    break;
  case W:
    printf("x = %.2f\n", message_Id -> val.w);
    break;
  case X:
    printf("x= %i\n", message_Id -> val.x);
    break;
  case Y:
    printf("y = %i\n", message_Id -> val.y);
    break;
  case Z:
    printf("z = %i\n", message_Id -> val.z);
    break;
  }
}




das ist mein Header

#ifndef PROTOCOL_H_
#define PROTOCOL_H_


static void val_printf(struct tval *);


#endif /* PROTOCOL_H_ */

Kaiser K. schrieb:
> oder 'struct tval' declared inside
> parameter list will not be visible outside of this declaration or
> definition;

Dies besagt, dass die zuerst innerhalb der Parameter auftauchende 
Deklaration der Strukt eine andere Strukt ist als die global definierte.

Dies sind also zwei inkompatible Deklarationen:
  void f(struct s *); struct s; // zwei verschiedene structs
  struct s; void f(struct *);   // zweimal die gleiche struct

Dr. Sommer schrieb:
> An der Stelle ist "tval" nicht definiert. Daher kommt hier ein Fehler.
> Das "struct tval" muss im Header definiert werden.

Eine forward declaration als
  struct tval;
reicht bei einem Pointer darauf aus. Nur darf die erste Erwähnung davon 
nicht in einer Paramterdeklaration erfolgen.

50c schrieb:
> ...du musst den struct tval als Typ definieren (in der Header-Datei) und
> dann mit diesem Typ bei den Deklarationen arbeiten

Kann man machen, muss man aber nicht.

Dr. Sommer schrieb:

> Noch einmal die Frage:
> Warum eine "static" Funktion im Header deklarieren? Was soll das
> bezwecken?

also ohne diese Funktion zu deklarieren bekomme ich als fehlermeldung : 
undefined reference to 'val_printf'

Mr. OCD schrieb:
> tval.w ist nicht und ob in der Union 2 y möglich sind?
> "w" wird auch nie geprinted.

das ist nur schreibfehler

Schreib mal deinen Header so:

1

#ifndef PROTOCOL_H_

2

#define PROTOCOL_H_

3

4

struct tval

5

{

6

  int typ;

7

  union

8

  {

9

    float v;

10

    float y;

11

    uint16_x;

12

    uint16_t y;

13

    uint32_t z;

14

  }val;

15

};

16

17

static void val_printf(struct tval *);

18

19

#endif /* PROTOCOL_H_ */

merciless

static-forward-Deklarationen gehören in den Kopf der C-Datei und NICHT 
in den Header!

Lesen, was Dr. Sommer schreibt!

Dr. Sommer schrieb:

> Kann nicht sein. Höchstens wenn du sie nicht definierst.

definieren meinte ich sorry

Mach das 'static' weg, füge eine forward-deklaration ein und gut.

Dein Header sieht dann so aus:

1

#ifndef PROTOCOL_H_

2

#define PROTOCOL_H_

3

4

struct tval;

5

void val_printf(struct tval *);

6

7

#endif /* PROTOCOL_H_ */

Und in der C-Datei machst du das 'static' auch weg.

Wenn du eine Funktion im Header erwähnst, dann gehört sie zur 
öffentlichen Schnittstelle des Moduls und ist nicht static. Wenn sie 
nicht zur öffentlichen Schnittstelle gehört, dann ist sie static und 
steht nicht im Headerfile.

Als Anfänger, und so lange man die Funktion von static noch nicht 
verstanden hat, kann man i.d.R. auch ohne static gut leben.

W.S. schrieb:
> Das spart einem das Suchen, ob das nun was
> Externes oder bloß ne Vorwärtsdeklaration in derselben Quelle ist.

Das ist reine Geschmackssache - dem Compiler ist es egal.

Statt dem TO in seinem Lernprozess zu unterstützen, geht jetzt wieder 
die übliche "Korinthenkackerei" los....

Und das nur, weil Manche sich unbedingt als "C-Versteher" profilieren 
müssen.

W.S. schrieb:
> Aus Gründen der Einheitlichkeit, auch der visuellen Einheitlichkeit
> kommt bei mir

Daß Du einen sehr spezifischen C-Stil hast, ist bekannt.

"extern" vor Funktionsprototypen irritiert, da das sonst annähernd 
niemand* macht und somit damit versehener Code ... exotisch ist.


*) ich lese seit bald 30 Jahren C-Sourcen der verschiedensten Quellen

Rufus Τ. F. schrieb:
> W.S. schrieb:
>> Aus Gründen der Einheitlichkeit, auch der visuellen Einheitlichkeit
>> kommt bei mir
>
> Daß Du einen sehr spezifischen C-Stil hast, ist bekannt.
>
> "extern" vor Funktionsprototypen irritiert, da das sonst annähernd
> niemand* macht und somit damit versehener Code ... exotisch ist.
>
>
> *) ich lese seit bald 30 Jahren C-Sourcen der verschiedensten Quellen

+1

S. R. schrieb:
> Mach das 'static' weg, füge eine forward-deklaration ein und gut.
>
> Dein Header sieht dann so aus:
>

1

> #ifndef PROTOCOL_H_

2

> #define PROTOCOL_H_

3

> 

4

> struct tval;

5

> void val_printf(struct tval *);

6

> 

7

> #endif /* PROTOCOL_H_ */

8

>

>
 das hat geklappt... Danke.
ich hätte noch eine weitere Frage.

also beim schreiben des Programmes habe ich eine variable struct tval 
myval deklariert und die funktion mit val_printf(&myval) aufgerufen. 
Dies hat leier nicht funktionnert.

Dr. Sommer schrieb:
> Lass mich raten. Du hast immer noch "static" dran stehen. Warum? Weil du
> das irgendwo her kopiert hast?


static habe ich nicht mehr dran stehen. aber es funktioniert sowohl mit 
static als auch ohne. das einzige was mir gefehlt hat ist in header 
struct tval zu schreiben....

Arduino Fanboy D. schrieb:
> Gibts irgendeine Programmiersprachen, welche einen nicht bekloppt
> macht...?

Ja. Jene, die man nur verwendet, wenn man schon bekloppt ist.

Kaiser K. schrieb:
> Dies hat leier nicht funktionnert.

Fangen wir mal ganz klassisch an.

Was hast du gemacht? Code, bitte.
Was ist passiert?
Was hätte deiner Meinung nach passieren sollen?

Kaiser K. schrieb:
> Dr. Sommer schrieb:
>
>> Noch einmal die Frage:
>> Warum eine "static" Funktion im Header deklarieren? Was soll das
>> bezwecken?
>
> also ohne diese Funktion zu deklarieren bekomme ich als fehlermeldung :
> undefined reference to 'val_printf'

Die Frage war nicht, warum es deklariert ist, sondern warum static und 
gleichzeitig in einem Header. Das ergibt keinen Sinn. In einem Header 
steht die Deklaration, weil man die Funktion auch von einer anderen 
Übersetzungseinheit aus aufrufen können will. Static schreibt man davor, 
weil man das nicht will. Die Kombination aus beiden ist also unsinnig.

W.S. schrieb:
> Rufus Τ. F. schrieb:
>> In C ist extern bei Funktionsprototypen das implizite Gegenteil von
>> static und kann daher immer wegfallen.
>
> Kann aber MUSS aber nicht.

Man kann es auch hinschreiben, aber es macht keinerlei Unterschied für 
das Programm.

> Aus Gründen der Einheitlichkeit, auch der visuellen Einheitlichkeit
> kommt bei mir nicht nur bei externen Daten, sondern auch bei externen
> Funktionen das extern dran. Das spart einem das Suchen, ob das nun was
> Externes oder bloß ne Vorwärtsdeklaration in derselben Quelle ist.

Bei einer Vorwärtsdeklaration in der selben Quelle schreibt man eh 
static davor. Und eine Vorwärtsdeklaration externer Funktionen gehört 
nicht in die C-Datei, sondern in einen Header. Es besteht also keinerlei 
Gefahr, die Deklaration ohne das explizite extern als intern zu 
missverstehen.

W.S. schrieb:
> S. R. schrieb:
>> Wenn sie
>> nicht zur öffentlichen Schnittstelle gehört, dann ist sie static und
>> steht nicht im Headerfile.
>
> Stimmt so nicht ganz. Wenn sie nicht explizit als static deklariert
> wurde, kann man immer noch mit extern void val_printf(struct tval *);
> drauf zugreifen.

Deswegen macht man sie ja static, wenn man diesen Zugriff nicht will.

> C hat eben kein wirklich sauberes Modulkonzept, das muß man leider
> hinnehmen.

Hä? static unterbindet die Zugriffsmöglichkeiten von außen und hilft 
nicht nur gegen versehentliche Zugriffe, sondern sogar gegen mutwillige 
Sabotage.
Da gibt's also nichts hinzunehmen. Man muss es lediglich richtig machen.