eigener Variablentyp

Dirk Gast 25.03.2009 14:44 #1205926

Lesenswert?

•

Hallo

ich programmiere gerade die Routinen für einen AD9833 DDS 
(Signalerzeuger) Baustein.

Dabei möchte ich eine Prozedur schreiben die in etwa folgenden Prototyp 
hat:

void dds_SetWaveForm(dds_WV WaveForm);

dds_WV soll letztenendes nur ein uint8_t sein alledings möchte ich das 
man keine Zahl direkt reinschreiben darf sondern nur:
dds_WF_Sin
dds_WF_Tri
dds_WF_Rect

die dann wie folgt definiert sind:
#define dds_WF_Sin 0x00;
#define dds_WF_Tri 0x02;
#define dds_WF_Rect 0x28;


Ich weiß ich könnte jetzt auch z.B. dds_SetWaveFrom(dds_WF_Sin) machen 
wenn ich WaveFrom (s. Prototyp) als uint8_t angebe. Aber ich will halt 
verhindern, dass man aus versehen eine (falsche) Zahl reinschreiben 
kann. Und nur die drei o.A. angaben zulassen -> sonst fehler: 
inkompatible typen
Wie kann ich das in avr-gcc anstellen?

Vielen Dank

Dirk

Beitrag melden Bearbeiten Thread verschieben Thread sperren Anmeldepflicht aktivieren Anpinnen Thread löschen Thread mit anderem zusammenführen Markierten Text zitieren Antwort Antwort mit Zitat

Karl H. (kbuchegg) 25.03.2009 14:51 #1205943

Lesenswert?

•

▲
▼

Dirk wrote:

> Wie kann ich das in avr-gcc anstellen?

Die schlechte Nachricht ist: gar nicht

Die etwas bessere Nachricht ist: mit einem enum

enum WaveForms { dds_WF_Sin, dds_WF_Tri, dds_WF_Rect };
void dds_SetWaveForm( enum WaveForms form );
int main()
  SetWaveForm( dds_WF_Sin );


allerdings kann dich C nicht davor schützen, dass sich ein Programmierer 
einen ungültigen Wert zusammencastet.

int main()
  SetWaveForm( 5 );                     // <- das gibt einen Fehler
  SetWaveForm( (enum WaveForms)5 );     // <- aber das geht durch den Compiler


Zumindest hast du so beim Funktionsprototypen dokumentiert, dass du nur 
bestimmte Werte haben möchtest und welche das sind.

Beitrag melden Bearbeiten Löschen Markierten Text zitieren Antwort Antwort mit Zitat

Johannes M. (johnny-m) 25.03.2009 14:54 #1205948

Lesenswert?

•

▲
▼

Dirk wrote:
> #define dds_WF_Sin 0x00;
> #define dds_WF_Tri 0x02;
> #define dds_WF_Rect 0x28;
Das ist syntaktisch falsch, ein #define wird (wie alle 
Präprozesoranweisungen) nicht mit einem Semikolon abgeschlossen. Ich 
hoffe, in Deinem Programm machst Du das nicht so...

Beitrag melden Bearbeiten Löschen Markierten Text zitieren Antwort Antwort mit Zitat

Daniel Gast 25.03.2009 14:57 #1205960

Lesenswert?

•

▲
▼

Wenn ich es richtig verstanden habe: Kannst du in deiner Funktion eine 
Switch case oda if-else Abfrage machen:

z.b:


void dds_SetWaveForm(dds_WV WaveForm)
{

   switch(WavForm)
   {
       case dds_WF_Sin:
                 ......break;
       case dds_WF_Tri:
                 ......break;
       case dds_WF_Rect:
                 ......break;
       default: Error break;


    }



}
Oder du kannst es gleich überprüfen bevor du deine Variable der Funktion 
übergibst, ob sie gültig ist.

Beitrag melden Bearbeiten Löschen Markierten Text zitieren Antwort Antwort mit Zitat

P. S. Gast 25.03.2009 15:18 #1206007

Lesenswert?

•

▲
▼

Karl heinz Buchegger wrote:
> Dirk wrote:

>> Wie kann ich das in avr-gcc anstellen?
> Die schlechte Nachricht ist: gar nicht

Wieso "gar nicht"? Du erklaerst doch danach, wie es geht?

> allerdings kann dich C nicht davor schützen, dass sich ein Programmierer
> einen ungültigen Wert zusammencastet.

Was dann ja kein Versehen mehr waere, sondern Absicht und gegebenenfalls 
Bloedheit...

Beitrag melden Bearbeiten Löschen Markierten Text zitieren Antwort Antwort mit Zitat

P. S. Gast 25.03.2009 15:20 #1206011

Lesenswert?

•

▲
▼

Johannes M. wrote:
> Dirk wrote:

> Das ist syntaktisch falsch, ein #define wird (wie alle
> Präprozesoranweisungen) nicht mit einem Semikolon abgeschlossen. Ich
> hoffe, in Deinem Programm machst Du das nicht so...

Syntaktisch ist das durchaus richtig und manchmal auch sinnvoll. Man 
darf nur nicht vergessen, dass das Semikolon Teil der Ersetzung ist und 
dann eben auch an der Stelle landet, wo das define verwendet wird...

Beitrag melden Bearbeiten Löschen Markierten Text zitieren Antwort Antwort mit Zitat

Karl H. (kbuchegg) 25.03.2009 15:21 #1206012

Lesenswert?

•

▲
▼

Peter Stegemann wrote:
> Karl heinz Buchegger wrote:
>> Dirk wrote:
>
>>> Wie kann ich das in avr-gcc anstellen?
>> Die schlechte Nachricht ist: gar nicht
>
> Wieso "gar nicht"? Du erklaerst doch danach, wie es geht?

Ich zeig ihm aber auch, wie man bewusst diesen vermeintlichen Schutz 
aushebeln kann.
In C ist das so. In anderen Sprachen nicht. Dort sind Aufzählungstypen 
restriktiver.

Beitrag melden Bearbeiten Löschen Markierten Text zitieren Antwort Antwort mit Zitat

Karl H. (kbuchegg) 25.03.2009 15:24 #1206019

Lesenswert?

•

▲
▼

Peter Stegemann wrote:

> Was dann ja kein Versehen mehr waere, sondern Absicht und gegebenenfalls
> Bloedheit...

Wenn du wüstest, was ich schon alles an "unabsichtlichen casts" gesehen 
habe. Da gibt der Compiler eine Fehlermeldung, dass der Datentyp nicht 
passt. Schreibt noch dazu, welcher Datentyp eigentlich erwartet wird und 
schon steht ein entsprechender Cast dort.

Gut: fällt alles in die Kategorie 'Blödheit'. Nichts desto trotz würde 
man sich gegen so manchen Cast als Programmierer gerne schützen können 
:-)

Beitrag melden Bearbeiten Löschen Markierten Text zitieren Antwort Antwort mit Zitat

Dirk Gast 25.03.2009 15:37 #1206049

Lesenswert?

•

▲
▼

@Johannes:
hab ich dann auch ziemlich schnell gemerkt ;)

Genau so wie Karl Heinz es vorschlägt wollte ich es haben!
Das man wenn man unbedingt will da was flasches casten kann ist nicht so 
schlimm, da ist man dann selber schuld!

Allerdings funktioniert das bei mir noch nicht so ganz...
ich habe einen enum:

enum WaveForms { dds_WF_Sin=0x00, dds_WF_Tri=0x02, dds_WF_Rect=0x28 };


und einen Prototypen in der datei dds.h

void dds_SetWaveForm(enum WaveForms WaveForm);

in der dds.c steht dann

void dds_SetWaveForm(enum WaveForms WaveForm){
  send16 s;
  s.i16=0;
  if (dds_Stat.FSel) s.i16|=dds_CRB_FSELECT;
  if (dds_Stat.PSel) s.i16|=dds_CRB_PSELECT;
  s.i16|=(dds_CRB_B28|WaveForm);
  dds_Stat.WvFrm=WaveForm;
  dds_CS_En();
  spi_Write(s.i8h);    //Send Hi Byte
  spi_Write(s.i8l);    //Send Lo Byte
  dds_CS_Dis();


In der main include ich die dds.h und rufe folgendes auf:

dds_SetWaveForm(5);


Das gibt allerdings leider KEINEN Compilerfehler :(
Was mache ich falsch?
Funktionieren an sich tut das Prog nur bekomme ich halt keinen Fehler..

Beitrag melden Bearbeiten Löschen Markierten Text zitieren Antwort Antwort mit Zitat

Daniel Gast 25.03.2009 15:49 #1206075

Lesenswert?

•

▲
▼

Weil der Compiler einen enum-Typ als Integerwert annimmt also kann jeder 
Wert gültig sein! Überprüfen ob er gültig ist bei Werten musst du es so 
machen wie ich es beschrieben habe!

Beitrag melden Bearbeiten Löschen Markierten Text zitieren Antwort Antwort mit Zitat

Daniel Gast 25.03.2009 15:51 #1206078

Lesenswert?

•

▲
▼

Egal ob Casten oda nicht in C ist und bleibt es ein Integerwert!!

Beitrag melden Bearbeiten Löschen Markierten Text zitieren Antwort Antwort mit Zitat

Detlev T. (detlevt) 25.03.2009 15:52 #1206082

Lesenswert?

•

▲
▼

Hallo Dirk,

mit fällt da nur so seltsame Sachen ein wie per typedef ein struct 
"waveform" zu definieren, das nur ein uint8_t-Element enthält, und 
entsprechende konstante Variablen damit zu definieren.

typedef struct{uint8_t wert;} waveform;
const waveform dds_WF_Sin = {0x00};
const waveform dds_WF_Tri = {0x02};
const waveform dds_WF_Rect = {0x28};

Schön ist das aber nicht.

Wäre es nicht wirklich besser, die Nutzer auf die Verwendung der 
#define-Konstanten zu verpflichten und zur Sicherheit einen Test zur 
Laufzeit auf gültige Werte zu nutzen?

Gruß, DetlevT

Beitrag melden Bearbeiten Löschen Markierten Text zitieren Antwort Antwort mit Zitat

Karl H. (kbuchegg) 25.03.2009 16:02 #1206096

Lesenswert?

•

▲
▼

Dirk wrote:

> In der main include ich die dds.h und rufe folgendes auf:
>

> dds_SetWaveForm(5);

>
> Das gibt allerdings leider KEINEN Compilerfehler :(

Shit.
Habs extra noch ausprobiert. Hab aber übersehen, dass mein Compiler als 
C++ Compiler arbeitet.

Beitrag melden Bearbeiten Löschen Markierten Text zitieren Antwort Antwort mit Zitat

P. S. Gast 25.03.2009 16:04 #1206099

Lesenswert?

•

▲
▼

Karl heinz Buchegger wrote:

> Shit.
> Habs extra noch ausprobiert. Hab aber übersehen, dass mein Compiler als
> C++ Compiler arbeitet.

Einer der vielen guten Gruende, auch C mit dem C++-Compiler zu 
entwickeln ;-)

Beitrag melden Bearbeiten Löschen Markierten Text zitieren Antwort Antwort mit Zitat

Dirk Gast 25.03.2009 16:28 #1206144

Lesenswert?

•

▲
▼

grml grml
Hmm naja gut kann man nichts machen. Ursprünglich habe ich  mal Pascal 
gelernt da wäre das sicherlich gegangen.. Naja C halt..

Wie dem auch sei, EUCH allen trotzdem ganz vielen Dank!

Grüße
Dirk H.

Beitrag melden Bearbeiten Löschen Markierten Text zitieren Antwort Antwort mit Zitat

Sven P. Gast 25.03.2009 16:40 #1206170

Lesenswert?

•

▲
▼

Vorsicht übrigens noch mit der 'enum' -- die braucht beim AVR-GCC gerne 
mal einen ganzen Integer, was meistens Speicherverschwendung ist :-)

Aber mal ehrlich: Wenn du nicht gerade ein weltbewegendes API 
programmierst, sollte man doch in der Lage sein, die paar Aufrufe 
korrekt hinzuschreiben :-)

Beitrag melden Bearbeiten Löschen Markierten Text zitieren Antwort Antwort mit Zitat

eigener Variablentyp

Antwort schreiben