Problem mit dem Auslagern von Funktionen

[Marcel F. (marcellobs)]

Hallo zusammen,

ich bin grade dabei mich mit dem Umgang von Funktionen vertraut zu 
machen. Um den Code kürzer halten zu können und ggf. Funktionen in 
anderen Programmen wieder verwenden zu können, wollte ich diese in einer 
externen c-datei auslagern.
Dafür sollte eine einfache Funktion "berechne", die zwei Werte 
multipliziert, ausgelagert werden.
Mein Vorgehen:

Main:

#include <avr/io.h>
#include "berechne.h"

int main(void)
{
    uint8_t Ergebnis = 0, a = 12, b = 2;
    while (1) 
    {
    berechne(&Ergebnis, a, b);
    }
}

.c-Datei:

void berechne(uint8_t *p_Ergebnis, uint8_t x, uint8_t y){
  *p_Ergebnis = x*y;
}

.h-Datei:

#ifndef BERECHNE_H_
#define BERECHNE_H_

void berechne(uint8_t *p_Ergebnis, uint8_t x, uint8_t y);

#endif /* BERECHNE_H_ */

Im Anhang habe ich Screenshots von der Fehlermeldung und dem Code 
hinterlegt. Hat irgendwer eine Idee woran es liegen könnte?

[quotendepp (Gast)]

lies mal die fehlermeldung(en). die sind meistens sehr gesprächig und 
sagen (hier) sehr genau wo das problem liegt.

tipp: es fehlen includes

[Marcel F. (marcellobs)]

Super danke, jetzt läuft es.

[Stefan F. (Gast)]

Nächstes mal Quelltext bitte als Text posten, nicht als Screenshot. 
Screenshots sind meistens unvollständig und man kann den Text nicht 
heraus kopieren um das Problem nachzuvollziehen.

Es wurden auch schon Anhänge und ZIP Dateien erfunden.

[Hannes (Gast)]

@Stefanus

Lies den ersten Post erst komplett, bevor Du Dich echauffierst - der TO 
hat den Quelltext mit gepostet, Kollege!

[Stefan F. (Gast)]

Hannes schrieb:
> Lies den ersten Post erst komplett, bevor Du Dich echauffierst - der TO
> hat den Quelltext mit gepostet, Kollege!

Du hast wohl Recht. Das ist so wenig, dass ich dachte, es seien nur 
Fragmente vom Ganzen.

[nicht Gast (Gast)]

Stefan ⛄ F. schrieb:
> Du hast wohl Recht. Das ist so wenig, dass ich dachte, es seien nur
> Fragmente vom Ganzen.

Aber Hauptsache erst mal wieder den Zeigefinger gehoben haben.

[nicht Gast (Gast)]

Ansonsten noch für den TO, in den Headern reichen die Datentypen der 
Funktion, die Namen brauchen da nicht rein.

void berechne(uint8_t*, uint8_t, uint8_t);

Hätte also auch gereicht, nur muss eben der Datentyp uint8_t bekannt 
sein, also inttypes.h oder stdint.h je nach sonstigem Bedarf in den 
Header mit rein.

[Stefan F. (Gast)]

nicht Gast schrieb:
> Ansonsten noch für den TO, in den Headern reichen die Datentypen der
> Funktion, die Namen brauchen da nicht rein.

Allerdings bringe ich die Dokumentation immer in den Header Dateien 
unter und da gehören dann sprechende Namen für die Parameter dazu.

[Falk B. (falk)]

nicht Gast schrieb:
> Ansonsten noch für den TO, in den Headern reichen die Datentypen der
> Funktion, die Namen brauchen da nicht rein.void berechne(uint8_t*,
> uint8_t, uint8_t);

Für den Compiler mag das reichen, für den Menschen, welcher die 
Funktionen benutzen will eher nicht! Schon gar nicht, wenn andere Leute 
sie irgendwann mal benutzen sollen.

Wer exzessiven Minimalismus praktizieren will darf gern Maschinencode 
schreiben, wahlweise auch BO8 Assembler ;-)

https://www.mikrocontroller.net/articles/8bit-CPU:_bo8