Anfängerproblem C

[Tobi84 (Gast)]

Hallo,

mir ist nicht ganz klar, warum das nicht funktioniert:


volatile uint8_t pwm_0 = 100;
volatile uint8_t pwm_1 = 100;
volatile uint8_t pwm_2 = 0;
volatile uint8_t pwm_3 = 100;
volatile uint8_t pwm_4 = 100;
volatile uint8_t pwm_5 = 100;
volatile uint8_t pwm_6 = 100;
volatile uint8_t pwm_7 = 100;

volatile uint8_t 
pwm_setting[8]={pwm_0,pwm_1,pwm_2,pwm_3,pwm_4,pwm_5,pwm_6,pwm_7};



Kann mir von euch einer sagen, was ich falsch mache?

[Sven P. (Gast)]

Tobi84 schrieb:
> Kann mir von euch einer sagen, was ich falsch mache?
Unter anderem:
* unvollständige Quelltexte posten
* die Forenregeln nicht lesen (da steht was von C-Code formatieren)
* keine Fehlermeldungen angeben

Ansonsten: Bei mir lässt sich dein Quelltextschnippel anstandslos 
compilieren (gcc und clang).

[Tobi84 (Gast)]

Oh sorry für das posting oben!

anbei nochmal richtig.

die Fehlermeldungen sind:

../main.c:32: error: initializer element is not constant

../main.c:32: error: (near initialization for 'pwm_setting[0]')

../main.c:32: error: initializer element is not constant

../main.c:32: error: (near initialization for 'pwm_setting[1]')

[Thomas W. (thomas100)]

In ANSI-C ist es glaub ich nicht zulässig, Arrays mit Variablen zu 
initialisieren. Mach mal deine pwm_X von "volatile" zu "const" oder 
trage die Werte direkt in die Feldinitialisierung ein.

[Klaus W. (mfgkw)]

Steht doch da: der Wert, mit dem du initalisierst, müsste konstant
sein. Tatsächlich ist es aber eine Variable.

Daß Sven es kann, liegt daran, daß er besser ist, oder daß der gcc
da nicht so pingelig ist in der Voreinstellung.

[Thomas W. (thomas100)]

Sven compiliert vermutlich mit den GNU-Erweiterungen. Und mit denen 
gehts wohl.

[Tobi84 (Gast)]

Ja, aber in der "main" geht es auch nicht.



../main.c:440: error: expected expression before '{' token

[holger (Gast)]

>  uint8_t pwm_0 = 100;

>  pwm_setting[8]={pwm_0,pwm_1,pwm_2,pwm_3,pwm_4,pwm_5,pwm_6,pwm_7};

So ein Schwachsinn;) pwm_0 wird nie wieder benutzt.

Also:

  pwm_setting[8]={100,pwm_1,pwm_2,pwm_3,pwm_4,pwm_5,pwm_6,pwm_7};

Restliche pwm_x genauso entsorgen.

[Tobi84 (Gast)]

Ich möchte aber die einzelnen Werte verändern!

[avr (Gast)]

Tobi84 schrieb:

> volatile uint8_t pwm_0 = 100;

Wenn es nur der Übersicht dient und nicht als Variable
benutzt wird (wie Holger schrieb) besser:

#define  pwm_0     100

Dann wird bei

pwm_setting[8]={pwm_0,pwm_1,pwm_2,pwm_3,pwm_4,pwm_5,pwm_6,pwm_7};

pwm_0 direkt durch die 100 ersetzt.

avr

[Tobi84 (Gast)]

OK, danke für die Hilfe. Ich habs hinbekommen.

kaum mach mans richtig, schon gehts ;-)


DANKE

[us73 (Gast)]

Wann einzeln verändern ?
Zur Compilezeit oder zur Laufzeit ?

Zur Compilezeit, dann nimm ein Makro, dann ists konstant und der 
Compiler zufrieden.
so: #define pwm_0 100

Zur Laufzeit: dann greif auf das entspr. Element im array mit Hilfe des 
Indexes zu und verändere es .
so: pwm_setting[0] = 100;
so: pwm_setting[1] = 80;
so: pwm_setting[i] = 23;

Vielleicht noch mehr Informationen, was Du mit der Variablen machen 
willst ???

[Sven P. (Gast)]

Klaus Wachtler schrieb:
> Steht doch da: der Wert, mit dem du initalisierst, müsste konstant
> sein. Tatsächlich ist es aber eine Variable.
>
> Daß Sven es kann, liegt daran, daß er besser ist, oder daß der gcc
> da nicht so pingelig ist in der Voreinstellung.

Nö, ich kompilierte strikt ISO-C99-konform. Dass es bei mir 
funktioniert, liegt ganz einfach daran: Da der OT keinen Kontext 
angegeben hat, hab ich mir halt einen ausgedacht. Wenn man Variablen und 
den Vektor lokal macht, funktioniert das ganz unspektakulär...

[Klaus W. (mfgkw)]

sagte ich doch: die einen können es, die anderen nicht :-)