Softwaretest mit Code Optimierung

[Siro]

Hallo Peter,
ich wollt mich nochmal bedanken, für die zusätzlichen Informationen.

Ich habe tatsächlich noch ein Problem gefunden. Ich habe eine "Inkontinenz" Inkonsistenz in meinem Code gefunden.
Im Header habe ich einen 32 Bit und in C-Code Modul einen 16 Bit Wert declariert/definiert. Das merkt der Compiler nicht und beim IAR-Compiler wird funktioneller Code erzeugt
der Gnu-Compiler hingegen erzeugt nicht funktionellen Code. Eindeutig meine Schuld , aber so unterschiedlich kann der erzeugte Code durch einen anderen Compiler halt werden.

Obwohl die Sourcen im Projekt zur Verfügung stehen, guckt der Compiler sie sich nicht an und damit gibt es auch kein Warning. Schade eigentlich.


Datei: FlowCalc.h
extern volatile U32 flowfactor;

Datei: FlowCalc.c
U16 flowfactor = 256;

mir fällt jetzt auch nichts ein, wie man solche Fehler vermeiden kann.

[botty]

Hallo Sisor,

kann es sein, dass Du FlowCalc.h nicht in FlowCalc.c inkludierst?

Also der GCC schreit normalerweise (oder spätestens bei -Wall) wenn eine Variable zuvor anders deklariert wurde, als sie dann später definiert wird.

Gruss
Chris

[Siro]

kann es sein, dass Du FlowCalc.h nicht in FlowCalc.c inkludierst?
Chris

Das kann nicht nur sein, sondern das ist so....

Wenn ich aus einer H-Datei nix brauche, includiere ich sie niemals. Sollte man das tun ?

Vielen Dank für den Hinweis Chris

[botty]

Was man immer macht, ist, dass man eine Include-Datei, in die zugehörige C-Datei inkludiert.
In Deinem Beispiel also FlowCalc.h in FlowCalc.c, eben genau um solche Fehler aufzudecken (Ich selbst habe den IAR-Compiler noch nie benutzt, aber auch der sollte dann meckern.).

Ausserdem schreibt man in eine H-Datei immer Guards, für den Fall das man aus Versehen sie mehrmals inkludiert - was ohne Guards wieder zu Fehlermeldungen führt.

Also fuer FlowCalc.h:

#ifndef _FlowCalc_h_
#define _FlowCalc_h_

/* Hier kommen jetzt alle defines, Deklarationen etc. */

#endif

[Siro]

Was man immer macht, ist, dass man eine Include-Datei, in die zugehörige C-Datei inkludiert.

Das habe ich bisher nur gemacht, wenn es wirklich nötig war und dann habe ich die entsprechenden "Guards" gesetzt.
Wuste garnicht, dass die so heissen.

Okay, dann werde ich meinen Stil ab jetzt ändern, damit der Compiler mögliche Konflikte auch aufdecken kann.
Man lernt nie aus. Ich danke Dir.

Siro

[Peter(TOO)]

Hallo Chris,

Das kann nicht nur sein, sondern das ist so....

Wenn ich aus einer H-Datei nix brauche, includiere ich sie niemals. Sollte man das tun ?

Du musst das so betrachten.

1. Man schreibt irgendein Modul, welches teilweise Funktionen bereit stellt, welche aus anderen Modulen benutzt werden können.
2. Man schreibt zu diesem Modul eine Header-Datei, welche alles enthält, was zum externen Aufruf benötigt wird. Also natürlich die Funktionen, aber auch öffentliche Konstanten und Variablen-Konstrukte.
3. Überall wo dann dieses Modul verwendet werden soll, wird die Header-Datei includiert.
4. Damit der Compiler auch überprüfen kann, dass Header und Modul zusammenpassen, includiert man diese auch im Modul!
5. Alle Funktionen welche in dem Modul nicht öffentlich sein sollen, declariert man als static, dann kann diese auch der Linker nicht finden und man darf die selben Namen auch in einem anderen Modul verwenden.

So wird der Code auch für andere übersichtlicher, besonders wenn man etwas anpassen muss:
Alle Funktionen und Variablen welche als static deklariert sind, kann man problemlos anpassen ohne auf Aufrufe aus anderen Modulen acht geben zu müssen.
Alles was öffentlich verwendet wird ist in der Header-Datei und der Compiler kann überprüfen ob es passt.

MfG Peter(TOO)