Variable lässt sich nicht beschreiben

[Lukas W. (lukas_we)]

Hallo,

Ich habe ein Problem damit eine Variable du beschreiben.
Ich versuche einen String über die Serielle Schnittstelle über einen 
Recieve Interrupt einzulesen, was soweit funktioniert. Danach versuche 
ich den String mit hilfe der atoi() Funktion in einen Int Wert 
umzuwandeln welcher dann auch gleich in eine Global deklarierte Variable 
geschrieben werden soll. Doch wenn ich dann in der Main-Loop versuche 
diesen Wert abzurufen hat die Variable immer noch ihren initialen Wert. 
Woran könnte das liegen?

Mit Freundlichen Grüßen.
Lukas W.

Hier noch ein paar Infos was ich dazu verwende:
Mikrocontroller: Atmega328P
Programmierumgebung: Atmel Studio 7
Programmieradapter: AVR Dragon


Code für die Umwandlung in den String.

ISR(USART0_RX_vect)
{
  
  static int counter = 0;
  char income = UDR0;
  
  if(income != '\r')
  {
    if(((income >= '0') & (income <= '9')) || income == ',')
    {
      recv_string[counter] = income;
      counter++;
    }
    else
    {
      printf("Wrong character (only numbers and commas)");
    }
  }
  else
  {
    recv_string[counter] = '\r';
    conv_cmd_string();
  }
}

Code für die Umwandlung des Strings und Speichern in die Globale 
Variable.
cmd_value und val_value sind die global deklarierten Variablen.

void conv_cmd_string(void)
{
  int cmd_val = 0;
  int counter = 0;
  int cmd_counter = 0;
  int val_counter = 0;
  int x = 0;
  int y = 0;
  
  if(cmd_val == 0)
  {
    for(cmd_counter = 0;recv_string[counter] != ',';counter++)
    {
      cmd_buffer[cmd_counter] = recv_string[counter];
      cmd_counter++;
    }
    counter++;
    cmd_val = 1;
  }
  
  if(cmd_val == 1)
  {
    for(val_counter = 0;recv_string[counter] != '\r';counter++)
    {
      val_buffer[val_counter] = recv_string[counter];
      val_counter++;
    }
  }
   cmd_value = atoi(cmd_buffer);
   val_value = atoi(val_buffer);
}

[Einer (Gast)]

atoi erwartet einen nullterminierten String

Du solltest die Umwandlung nicht in der ISR machen.
Sondern in der ISR ein Flag setzen und die Auswertung in der mainloop.

[M. Н. (Gast)]

Wie ist die globale Variable deklariert?

Ist sie volatile?

also so:

volatile int val_value;

[Ralf (Gast)]

Sind cmd_value und val_value als volatile deklariert?

[Axel S. (a-za-z0-9)]

Lukas W. schrieb:
> Ich habe ein Problem damit eine Variable du beschreiben.
> Ich versuche einen String über die Serielle Schnittstelle über einen
> Recieve Interrupt einzulesen

Scheint so, als würde da ein volatile in deinem Quelltext fehlen.

https://de.wikipedia.org/wiki/Volatile_(Informatik)

[Lukas W. (lukas_we)]

Danke für die schnellen Antworten.

Der Fehler ist behoben oder besser gesagt beide Fehler sind behoben es 
war eine Kombination aus dem fehlenden volatile und das ich die 
Konvertierung im Interrupt aufgerufen habe.

[Dirk B. (dirkb2)]

Einer schrieb:
> atoi erwartet einen nullterminierten String

atoi hört aber beim ersten Zeichen, das nicht zum int passt auf. Das 
wäre hier das '\r' (das man aber besser durch die 0 ersetzt)

Lukas W. schrieb:
> Der Fehler ... das ich die
> Konvertierung im Interrupt aufgerufen habe.

Das war schlechtes Design, hat mit dem Fehler hier aber nichts zu tun.

[Andreas B. (bitverdreher)]

Einer schrieb:
> Du solltest die Umwandlung nicht in der ISR machen.

Genausowenig wie printf.

[Dirk B. (dirkb2)]

Der counter in der ISR sollte auch mal wieder auf 0 gesetzt werden.

[Walter S. (avatar)]

Lukas W. schrieb:
> int cmd_val = 0;
> ....
>   if(cmd_val == 0)

das ist auch etwas seltsam, wenn du das wirklich so willst kannst du dir 
auch das if sparen

[Andreas B. (bitverdreher)]

Walter S. schrieb:
> Lukas W. schrieb:
>> int cmd_val = 0;
>> ....
>>   if(cmd_val == 0)
>
> das ist auch etwas seltsam, wenn du das wirklich so willst kannst du dir
> auch das if sparen

Mit Verlaub, aber verwechselst DU da nicht etwas?

[Nop (Gast)]

Andreas B. schrieb:

> Mit Verlaub, aber verwechselst DU da nicht etwas?

Wenn man eine lokale Variable mit 0 initialisiert, dann hat sie auch den 
Wert 0 - da braucht man keine if-Abfrage mehr drauf zu machen. Die ist 
nämlich immer erfüllt.

[Oliver S. (oliverso)]

Nop schrieb:
> Wenn man eine lokale Variable mit 0 initialisiert, dann hat sie auch den
> Wert 0 - da braucht man keine if-Abfrage mehr drauf zu machen. Die ist
> nämlich immer erfüllt.

Ein static an passender Stelle würde da Wunder wqirken.

Oliver

[Einer K. (Gast)]

Oliver S. schrieb:
> Ein static an passender Stelle würde da Wunder wqirken.

Ein überflüssiges static ist ein böses static.
static ist gegen Wiedereintrittsfähigkeit.

Funktionen sollten bei identischen Parametern immer das gleiche tun.
So fordert es das "Prinzip der geringsten Verwunderung" ein.
static macht da einen Strich durch.

In der Not frisst der Teufel Fliegen und der Programmierer baut 
stattdessen lokale static Variablen in Funktionen rein.

[Andreas B. (bitverdreher)]

Nop schrieb:
> Andreas B. schrieb:
>
>> Mit Verlaub, aber verwechselst DU da nicht etwas?
>
> Wenn man eine lokale Variable mit 0 initialisiert, dann hat sie auch den
> Wert 0 - da braucht man keine if-Abfrage mehr drauf zu machen. Die ist
> nämlich immer erfüllt.

Gerade gesehen, alles zurück. Da ist ja in der Prozedur.